From: Icenowy Zheng <icenowy@aosc.io>
To: Jesse Taube <mr.bossman075@gmail.com>,
Andre Przywara <andre.przywara@arm.com>,
Maxime Ripard <mripard@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Samuel Holland <samuel@sholland.org>
Cc: Rob Herring <robh+dt@kernel.org>,
Mesih Kilinc <mesihkilinc@gmail.com>,
Giulio Benetti <giulio.benetti@benettiengineering.com>,
George Hilliard <thirtythreeforty@gmail.com>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev
Subject: Re: [PATCH 05/14] ARM: dts: suniv: F1C100: fix CPU node
Date: Tue, 08 Mar 2022 12:23:23 +0800 [thread overview]
Message-ID: <5edad86493b145761a7f156f830d84f3413091c9.camel@aosc.io> (raw)
In-Reply-To: <526e09f8-6006-643a-70c6-d9b85e6cdb75@gmail.com>
在 2022-03-07星期一的 21:44 -0500,Jesse Taube写道:
>
>
> On 3/7/22 09:34, Andre Przywara wrote:
> > The /cpu node in the f1c100s.dtsi is not spec compliant, it's
> > missing
> > the reg property, and the corresponding address and size cells
> > properties.
> >
> > Add them to make the bindings check pass.
> >
> > Signed-off-by: Andre Przywara <andre.przywara@arm.com>
> > ---
> > arch/arm/boot/dts/suniv-f1c100s.dtsi | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/suniv-f1c100s.dtsi
> > b/arch/arm/boot/dts/suniv-f1c100s.dtsi
> > index 922efd5e9457..43d342eaf661 100644
> > --- a/arch/arm/boot/dts/suniv-f1c100s.dtsi
> > +++ b/arch/arm/boot/dts/suniv-f1c100s.dtsi
> > @@ -29,9 +29,13 @@ osc32k: clk-32k {
> > };
> >
> > cpus {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > cpu {
> cpu@0
>
> also is memory node required?
It should be filled by U-Boot.
>
> Thanks,
> jesse
> > compatible = "arm,arm926ej-s";
> > device_type = "cpu";
> > + reg = <0x0>;
> > };
> > };
> >
next prev parent reply other threads:[~2022-03-08 4:23 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-07 14:34 [PATCH 00/14] ARM: suniv: dts: update Allwinner F1C100 Andre Przywara
2022-03-07 14:34 ` [PATCH 01/14] dt-bindings: watchdog: sunxi: fix F1C100s compatible Andre Przywara
2022-03-08 16:08 ` Rob Herring
2022-03-09 23:02 ` Guenter Roeck
2022-03-10 0:46 ` Samuel Holland
2022-03-14 17:39 ` Andre Przywara
2022-03-07 14:34 ` [PATCH 02/14] ARM: dts: suniv: F1C100: fix watchdog compatible Andre Przywara
2022-03-09 23:03 ` Guenter Roeck
2022-03-07 14:34 ` [PATCH 03/14] dt-bindings: arm: sunxi: document LicheePi Nano name Andre Przywara
2022-03-08 16:09 ` Rob Herring
2022-03-11 1:27 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 04/14] ARM: dts: suniv: F1C100: add clock and reset macros Andre Przywara
2022-03-11 1:30 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 05/14] ARM: dts: suniv: F1C100: fix CPU node Andre Przywara
2022-03-08 2:44 ` Jesse Taube
2022-03-08 4:23 ` Icenowy Zheng [this message]
2022-03-08 10:42 ` Andre Przywara
2022-03-07 14:34 ` [PATCH 06/14] ARM: dts: suniv: F1C100: fix timer node Andre Przywara
2022-03-11 2:19 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 07/14] dt-bindings: mmc: sunxi: add Allwinner F1c100s compatible Andre Przywara
2022-03-08 16:10 ` Rob Herring
2022-03-11 2:19 ` Samuel Holland
2022-03-11 15:41 ` Ulf Hansson
2022-03-07 14:34 ` [PATCH 08/14] ARM: dts: suniv: F1C100: add MMC controllers Andre Przywara
2022-03-11 2:19 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 09/14] ARM: dts: suniv: licheepi-nano: add microSD card Andre Przywara
2022-03-11 2:19 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 10/14] dt-bindings: spi: sunxi: document F1C100 controllers Andre Przywara
2022-03-08 16:10 ` Rob Herring
2022-03-11 2:19 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 11/14] ARM: dts: suniv: F1C100: add SPI support Andre Przywara
2022-03-11 2:19 ` Samuel Holland
2022-03-11 13:33 ` Andre Przywara
2022-03-07 14:34 ` [PATCH 12/14] ARM: dts: suniv: licheepi-nano: add SPI flash Andre Przywara
2022-03-11 2:20 ` Samuel Holland
2022-03-07 14:34 ` [PATCH 13/14] ARM: configs: sync multi_v5_defconfig from savedefconfig Andre Przywara
2022-03-08 9:38 ` Arnd Bergmann
2022-03-08 12:07 ` Andre Przywara
2022-03-08 13:33 ` Arnd Bergmann
2022-03-08 13:40 ` Arnd Bergmann
2022-03-08 14:30 ` Nicolas Ferre
2022-03-08 15:17 ` Arnd Bergmann
2022-03-10 10:33 ` Andre Przywara
2022-03-07 14:34 ` [PATCH 14/14] ARM: configs: multi_v5: Enable Allwinner F1C100 Andre Przywara
2022-03-07 18:03 ` [PATCH 00/14] ARM: suniv: dts: update " Jesse Taube
2022-03-07 18:22 ` Giulio Benetti
2022-03-11 1:38 ` Jesse Taube
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5edad86493b145761a7f156f830d84f3413091c9.camel@aosc.io \
--to=icenowy@aosc.io \
--cc=andre.przywara@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=giulio.benetti@benettiengineering.com \
--cc=jernej.skrabec@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-sunxi@lists.linux.dev \
--cc=mesihkilinc@gmail.com \
--cc=mr.bossman075@gmail.com \
--cc=mripard@kernel.org \
--cc=robh+dt@kernel.org \
--cc=samuel@sholland.org \
--cc=thirtythreeforty@gmail.com \
--cc=wens@csie.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).