public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
From: Inochi Amaoto <inochiama@gmail.com>
To: Guo Ren <guoren@kernel.org>, Inochi Amaoto <inochiama@gmail.com>
Cc: Rob Herring <robh@kernel.org>,
	 Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	 Paul Walmsley <pjw@kernel.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	 Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	 Chen Wang <unicorn_wang@outlook.com>,
	Han Gao <rabenda.cn@gmail.com>,
	 Nutty Liu <liujingqi@lanxincomputing.com>,
	Guodong Xu <guodong@riscstar.com>,
	 Xiaoguang Xing <xiaoguang.xing@sophgo.com>,
	devicetree@vger.kernel.org, linux-riscv@lists.infradead.org,
	 sophgo@lists.linux.dev, linux-kernel@vger.kernel.org,
	Yixun Lan <dlan@gentoo.org>,  Longbin Li <looong.bin@gmail.com>
Subject: Re: [PATCH 1/2] riscv: dts: sophgo: sg2044: use hex for CPU unit address
Date: Tue, 7 Apr 2026 10:36:34 +0800	[thread overview]
Message-ID: <adRtlyqIupCqvkeD@inochi.infowork> (raw)
In-Reply-To: <CAJF2gTTPt7UvFDRcA2EHfib1Yd0V8jjwAtqV8D2RCCGTn=RQ_g@mail.gmail.com>

On Tue, Apr 07, 2026 at 09:26:11AM +0800, Guo Ren wrote:
> On Tue, Apr 7, 2026 at 7:27 AM Inochi Amaoto <inochiama@gmail.com> wrote:
> >
> > Previous the CPU unit address cpu of sg2044 use decimal, it is
> > not following the general convention for unit addresses of the
> > OF. Convent the unit address to hex to resolve this problem.
> >
> > The introduces a small ABI break for the CPU id, but it should
> > affect nothing since there is no direct full-path reference to
> > these CPU nodes.
> >
> > Fixes: 967a94a92aaa ("riscv: dts: add initial Sophgo SG2042 SoC device tree")
> > Signed-off-by: Inochi Amaoto <inochiama@gmail.com>
> > Link: https://lore.kernel.org/devicetree-spec/00ddad5a-02f5-474e-af9c-11ce7716ddfc@iscas.ac.cn/
> 
> Remove the Link tag, and add:
> 
> Reviewed-by: Guo Ren <guoren@kernel.org>
> 

This link provides the motivation for this patch, I think it will be
fine for other to know why this patch is needed.

Regards,
Inochi

> > ---
> >  arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi | 236 ++++++++++----------
> >  1 file changed, 118 insertions(+), 118 deletions(-)
> >
> > diff --git a/arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi b/arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi
> > index 3135409c2149..f66a382c95bd 100644
> > --- a/arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi
> > +++ b/arch/riscv/boot/dts/sophgo/sg2044-cpus.dtsi
> > @@ -14,7 +14,7 @@ cpus {
> >
> >                 cpu0: cpu@0 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <0>;
> > +                       reg = <0x0>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -50,7 +50,7 @@ cpu0_intc: interrupt-controller {
> >
> >                 cpu1: cpu@1 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <1>;
> > +                       reg = <0x1>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -86,7 +86,7 @@ cpu1_intc: interrupt-controller {
> >
> >                 cpu2: cpu@2 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <2>;
> > +                       reg = <0x2>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -122,7 +122,7 @@ cpu2_intc: interrupt-controller {
> >
> >                 cpu3: cpu@3 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <3>;
> > +                       reg = <0x3>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -158,7 +158,7 @@ cpu3_intc: interrupt-controller {
> >
> >                 cpu4: cpu@4 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <4>;
> > +                       reg = <0x4>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -194,7 +194,7 @@ cpu4_intc: interrupt-controller {
> >
> >                 cpu5: cpu@5 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <5>;
> > +                       reg = <0x5>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -230,7 +230,7 @@ cpu5_intc: interrupt-controller {
> >
> >                 cpu6: cpu@6 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <6>;
> > +                       reg = <0x6>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -266,7 +266,7 @@ cpu6_intc: interrupt-controller {
> >
> >                 cpu7: cpu@7 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <7>;
> > +                       reg = <0x7>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -302,7 +302,7 @@ cpu7_intc: interrupt-controller {
> >
> >                 cpu8: cpu@8 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <8>;
> > +                       reg = <0x8>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -338,7 +338,7 @@ cpu8_intc: interrupt-controller {
> >
> >                 cpu9: cpu@9 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <9>;
> > +                       reg = <0x9>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -372,9 +372,9 @@ cpu9_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu10: cpu@10 {
> > +               cpu10: cpu@a {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <10>;
> > +                       reg = <0xa>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -408,9 +408,9 @@ cpu10_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu11: cpu@11 {
> > +               cpu11: cpu@b {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <11>;
> > +                       reg = <0xb>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -444,9 +444,9 @@ cpu11_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu12: cpu@12 {
> > +               cpu12: cpu@c {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <12>;
> > +                       reg = <0xc>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -480,9 +480,9 @@ cpu12_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu13: cpu@13 {
> > +               cpu13: cpu@d {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <13>;
> > +                       reg = <0xd>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -516,9 +516,9 @@ cpu13_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu14: cpu@14 {
> > +               cpu14: cpu@e {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <14>;
> > +                       reg = <0xe>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -552,9 +552,9 @@ cpu14_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu15: cpu@15 {
> > +               cpu15: cpu@f {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <15>;
> > +                       reg = <0xf>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -588,9 +588,9 @@ cpu15_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu16: cpu@16 {
> > +               cpu16: cpu@10 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <16>;
> > +                       reg = <0x10>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -624,9 +624,9 @@ cpu16_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu17: cpu@17 {
> > +               cpu17: cpu@11 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <17>;
> > +                       reg = <0x11>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -660,9 +660,9 @@ cpu17_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu18: cpu@18 {
> > +               cpu18: cpu@12 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <18>;
> > +                       reg = <0x12>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -696,9 +696,9 @@ cpu18_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu19: cpu@19 {
> > +               cpu19: cpu@13 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <19>;
> > +                       reg = <0x13>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -732,9 +732,9 @@ cpu19_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu20: cpu@20 {
> > +               cpu20: cpu@14 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <20>;
> > +                       reg = <0x14>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -768,9 +768,9 @@ cpu20_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu21: cpu@21 {
> > +               cpu21: cpu@15 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <21>;
> > +                       reg = <0x15>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -804,9 +804,9 @@ cpu21_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu22: cpu@22 {
> > +               cpu22: cpu@16 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <22>;
> > +                       reg = <0x16>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -840,9 +840,9 @@ cpu22_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu23: cpu@23 {
> > +               cpu23: cpu@17 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <23>;
> > +                       reg = <0x17>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -876,9 +876,9 @@ cpu23_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu24: cpu@24 {
> > +               cpu24: cpu@18 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <24>;
> > +                       reg = <0x18>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -912,9 +912,9 @@ cpu24_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu25: cpu@25 {
> > +               cpu25: cpu@19 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <25>;
> > +                       reg = <0x19>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -948,9 +948,9 @@ cpu25_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu26: cpu@26 {
> > +               cpu26: cpu@1a {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <26>;
> > +                       reg = <0x1a>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -984,9 +984,9 @@ cpu26_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu27: cpu@27 {
> > +               cpu27: cpu@1b {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <27>;
> > +                       reg = <0x1b>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1020,9 +1020,9 @@ cpu27_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu28: cpu@28 {
> > +               cpu28: cpu@1c {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <28>;
> > +                       reg = <0x1c>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1056,9 +1056,9 @@ cpu28_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu29: cpu@29 {
> > +               cpu29: cpu@1d {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <29>;
> > +                       reg = <0x1d>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1092,9 +1092,9 @@ cpu29_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu30: cpu@30 {
> > +               cpu30: cpu@1e {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <30>;
> > +                       reg = <0x1e>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1128,9 +1128,9 @@ cpu30_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu31: cpu@31 {
> > +               cpu31: cpu@1f {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <31>;
> > +                       reg = <0x1f>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1164,9 +1164,9 @@ cpu31_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu32: cpu@32 {
> > +               cpu32: cpu@20 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <32>;
> > +                       reg = <0x20>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1200,9 +1200,9 @@ cpu32_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu33: cpu@33 {
> > +               cpu33: cpu@21 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <33>;
> > +                       reg = <0x21>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1236,9 +1236,9 @@ cpu33_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu34: cpu@34 {
> > +               cpu34: cpu@22 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <34>;
> > +                       reg = <0x22>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1272,9 +1272,9 @@ cpu34_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu35: cpu@35 {
> > +               cpu35: cpu@23 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <35>;
> > +                       reg = <0x23>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1308,9 +1308,9 @@ cpu35_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu36: cpu@36 {
> > +               cpu36: cpu@24 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <36>;
> > +                       reg = <0x24>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1344,9 +1344,9 @@ cpu36_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu37: cpu@37 {
> > +               cpu37: cpu@25 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <37>;
> > +                       reg = <0x25>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1380,9 +1380,9 @@ cpu37_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu38: cpu@38 {
> > +               cpu38: cpu@26 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <38>;
> > +                       reg = <0x26>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1416,9 +1416,9 @@ cpu38_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu39: cpu@39 {
> > +               cpu39: cpu@27 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <39>;
> > +                       reg = <0x27>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1452,9 +1452,9 @@ cpu39_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu40: cpu@40 {
> > +               cpu40: cpu@28 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <40>;
> > +                       reg = <0x28>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1488,9 +1488,9 @@ cpu40_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu41: cpu@41 {
> > +               cpu41: cpu@29 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <41>;
> > +                       reg = <0x29>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1524,9 +1524,9 @@ cpu41_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu42: cpu@42 {
> > +               cpu42: cpu@2a {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <42>;
> > +                       reg = <0x2a>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1560,9 +1560,9 @@ cpu42_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu43: cpu@43 {
> > +               cpu43: cpu@2b {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <43>;
> > +                       reg = <0x2b>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1596,9 +1596,9 @@ cpu43_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu44: cpu@44 {
> > +               cpu44: cpu@2c {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <44>;
> > +                       reg = <0x2c>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1632,9 +1632,9 @@ cpu44_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu45: cpu@45 {
> > +               cpu45: cpu@2d {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <45>;
> > +                       reg = <0x2d>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1668,9 +1668,9 @@ cpu45_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu46: cpu@46 {
> > +               cpu46: cpu@2e {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <46>;
> > +                       reg = <0x2e>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1704,9 +1704,9 @@ cpu46_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu47: cpu@47 {
> > +               cpu47: cpu@2f {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <47>;
> > +                       reg = <0x2f>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1740,9 +1740,9 @@ cpu47_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu48: cpu@48 {
> > +               cpu48: cpu@30 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <48>;
> > +                       reg = <0x30>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1776,9 +1776,9 @@ cpu48_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu49: cpu@49 {
> > +               cpu49: cpu@31 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <49>;
> > +                       reg = <0x31>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1812,9 +1812,9 @@ cpu49_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu50: cpu@50 {
> > +               cpu50: cpu@32 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <50>;
> > +                       reg = <0x32>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1848,9 +1848,9 @@ cpu50_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu51: cpu@51 {
> > +               cpu51: cpu@33 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <51>;
> > +                       reg = <0x33>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1884,9 +1884,9 @@ cpu51_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu52: cpu@52 {
> > +               cpu52: cpu@34 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <52>;
> > +                       reg = <0x34>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1920,9 +1920,9 @@ cpu52_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu53: cpu@53 {
> > +               cpu53: cpu@35 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <53>;
> > +                       reg = <0x35>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1956,9 +1956,9 @@ cpu53_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu54: cpu@54 {
> > +               cpu54: cpu@36 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <54>;
> > +                       reg = <0x36>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -1992,9 +1992,9 @@ cpu54_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu55: cpu@55 {
> > +               cpu55: cpu@37 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <55>;
> > +                       reg = <0x37>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2028,9 +2028,9 @@ cpu55_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu56: cpu@56 {
> > +               cpu56: cpu@38 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <56>;
> > +                       reg = <0x38>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2064,9 +2064,9 @@ cpu56_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu57: cpu@57 {
> > +               cpu57: cpu@39 {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <57>;
> > +                       reg = <0x39>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2100,9 +2100,9 @@ cpu57_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu58: cpu@58 {
> > +               cpu58: cpu@3a {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <58>;
> > +                       reg = <0x3a>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2136,9 +2136,9 @@ cpu58_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu59: cpu@59 {
> > +               cpu59: cpu@3b {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <59>;
> > +                       reg = <0x3b>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2172,9 +2172,9 @@ cpu59_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu60: cpu@60 {
> > +               cpu60: cpu@3c {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <60>;
> > +                       reg = <0x3c>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2208,9 +2208,9 @@ cpu60_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu61: cpu@61 {
> > +               cpu61: cpu@3d {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <61>;
> > +                       reg = <0x3d>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2244,9 +2244,9 @@ cpu61_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu62: cpu@62 {
> > +               cpu62: cpu@3e {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <62>;
> > +                       reg = <0x3e>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > @@ -2280,9 +2280,9 @@ cpu62_intc: interrupt-controller {
> >                         };
> >                 };
> >
> > -               cpu63: cpu@63 {
> > +               cpu63: cpu@3f {
> >                         compatible = "thead,c920", "riscv";
> > -                       reg = <63>;
> > +                       reg = <0x3f>;
> >                         i-cache-block-size = <64>;
> >                         i-cache-size = <65536>;
> >                         i-cache-sets = <512>;
> > --
> > 2.53.0
> >
> 
> 
> -- 
> Best Regards
>  Guo Ren

  reply	other threads:[~2026-04-07  2:37 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-06 23:26 [PATCH 0/2] riscv: dts: sophgo: Fix CPU unit address for SG2042/SG2044 Inochi Amaoto
2026-04-06 23:26 ` [PATCH 1/2] riscv: dts: sophgo: sg2044: use hex for CPU unit address Inochi Amaoto
2026-04-07  1:26   ` Guo Ren
2026-04-07  2:36     ` Inochi Amaoto [this message]
2026-04-07  3:08       ` Guo Ren
2026-04-07  3:35         ` Inochi Amaoto
2026-04-06 23:26 ` [PATCH 2/2] riscv: dts: sophgo: sg2042: " Inochi Amaoto
2026-04-07  1:26   ` Guo Ren
2026-04-07 16:31   ` Conor Dooley

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=adRtlyqIupCqvkeD@inochi.infowork \
    --to=inochiama@gmail.com \
    --cc=alex@ghiti.fr \
    --cc=aou@eecs.berkeley.edu \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dlan@gentoo.org \
    --cc=guodong@riscstar.com \
    --cc=guoren@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=liujingqi@lanxincomputing.com \
    --cc=looong.bin@gmail.com \
    --cc=palmer@dabbelt.com \
    --cc=pjw@kernel.org \
    --cc=rabenda.cn@gmail.com \
    --cc=robh@kernel.org \
    --cc=sophgo@lists.linux.dev \
    --cc=unicorn_wang@outlook.com \
    --cc=xiaoguang.xing@sophgo.com \
    /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