On Wed, 10 Sept 2025 at 09:11, Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
> On 10/09/2025 01:39, Kuninori Morimoto wrote:
> >>> + chosen {
> >>> + bootargs = "ignore_loglevel rw maxcpus=1";
> >>
> >> Drop bootargs. Not suitable for mainline. I really do not understand why
> >> all users in mainline and other projects (not your development) must be
> >> limited to cpus=1.
> >
> > This is initial patch, and only 1 CPU is available for now.
> > The limitaion will be removed in the future, but not yet for now.
>
> You do not understand the problem - DTS describes the hardware, not your
> current stage of Linux support. These bootargs are wrong, I have been
> commenting on this since years.
Well, we can't post the full DTS describing the full hardware yet, as
that hasn't been written yet. E.g. lots of DT bindings for new or
changed components are still missing. So we have to fall back to the
customary incremental DTS writing...
If currently only one CPU can/must be used, the right way to handle
that is to drop all but the first CPU node from the .dtsi, and only
add the other CPU nodes later. This is what we've been doing for
other SoCs before, too.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
