[PATCH v3 1/3] arm64: dts: nuvoton: fix warning and nodes order

[Tomer Maimon]

Hi Krzysztof,

Thanks for your comments

On Sun, 19 Oct 2025 at 13:35, Krzysztof Kozlowski <krzk at kernel.org> wrote:
>
> On 25/09/2025 22:06, Tomer Maimon wrote:
> > Fix the warning in the gcr and timer nodes, and modify nodes order by
>
> What warning?
This warning that I got from Andrew mail
[I] 0 andrew at heihei ~/s/k/l/o/build.arm64.default ((00e2ab2e))> make
CHECK_DTBS=y nuvoton/nuvoton-npcm845-evb.dtb
  SCHEMA  Documentation/devicetree/bindings/processed-schema.json
  DTC [C] arch/arm64/boot/dts/nuvoton/nuvoton-npcm845-evb.dtb
/home/andrew/src/
kernel.org/linux/origin/build.arm64.default/arch/arm64/boot/dts/nuvoton/nuvoton-npcm845-evb.dtb:
/ (nuvoton,npcm845-evb): memory at 0: 'device_type' is a required property
        from schema $id: http://devicetree.org/schemas/memory.yaml#

*/home/andrew/src/kernel.org/linux/origin/build.arm64.default/arch/arm64/boot/dts/nuvoton/nuvoton-npcm845-evb.dtb
<http://kernel.org/linux/origin/build.arm64.default/arch/arm64/boot/dts/nuvoton/nuvoton-npcm845-evb.dtb>:
system-controller at f0800000 (nuvoton,npcm845-gcr): compatible:
['nuvoton,npcm845-gcr', 'syscon'] is too short        from schema $id:
http://devicetree.org/schemas/soc/nuvoton/nuvoton,npcm-gcr.yaml#
<http://devicetree.org/schemas/soc/nuvoton/nuvoton,npcm-gcr.yaml#>*
>
> > ascending unit address.
>
> Why are you combining multiple issues into one commit. No. Read
> submitting patches how to organize your work.
>
> >
> > Signed-off-by: Tomer Maimon <tmaimon77 at gmail.com>
> > ---
> >  .../dts/nuvoton/nuvoton-common-npcm8xx.dtsi   | 35 +++++++++----------
> >  1 file changed, 17 insertions(+), 18 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> > index 24133528b8e9..a43514f624c0 100644
> > --- a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> > +++ b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> > @@ -18,7 +18,7 @@ soc {
> >               ranges;
> >
> >               gcr: system-controller at f0800000 {
> > -                     compatible = "nuvoton,npcm845-gcr", "syscon";
> > +                     compatible = "nuvoton,npcm845-gcr", "syscon",
"simple-mfd";
>
> This makes no sense. No children here.
>
> >                       reg = <0x0 0xf0800000 0x0 0x1000>;
> >               };
> >
> Best regards,
> Krzysztof

Best regards,

Tomer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20251019/cfcd146e/attachment.htm>