From: Conor Dooley <conor@kernel.org> To: Hal Feng <hal.feng@starfivetech.com> Cc: Conor Dooley <conor.dooley@microchip.com>, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, Palmer Dabbelt <palmer@dabbelt.com>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Paul Walmsley <paul.walmsley@sifive.com>, Albert Ou <aou@eecs.berkeley.edu>, Ben Dooks <ben.dooks@sifive.com>, Daniel Lezcano <daniel.lezcano@linaro.org>, Thomas Gleixner <tglx@linutronix.de>, Marc Zyngier <maz@kernel.org>, Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Philipp Zabel <p.zabel@pengutronix.de>, Linus Walleij <linus.walleij@linaro.org>, Emil Renner Berthing <emil.renner.berthing@canonical.com>, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 6/7] riscv: dts: starfive: Add initial StarFive JH7110 device tree Date: Wed, 15 Feb 2023 07:59:27 +0000 [thread overview] Message-ID: <Y+yQ3+0lyo5OQr2z@spud> (raw) In-Reply-To: <Y+yM6HgAbDoWlu1G@spud> [-- Attachment #1: Type: text/plain, Size: 4327 bytes --] On Wed, Feb 15, 2023 at 07:42:32AM +0000, Conor Dooley wrote: > Hey Hal! > > On Wed, Feb 15, 2023 at 11:07:15AM +0800, Hal Feng wrote: > > On Thu, 2 Feb 2023 19:41:33 +0000, Conor Dooley wrote: > > > On Fri, Feb 03, 2023 at 02:56:41AM +0800, Hal Feng wrote: > > >> On Wed, 1 Feb 2023 08:21:05 +0000, Conor Dooley wrote: > > >> > On Wed, Feb 01, 2023 at 03:21:48PM +0800, Hal Feng wrote: > > >> >> On Wed, 28 Dec 2022 22:48:43 +0000, Conor Dooley wrote: > > >> >> > On Tue, Dec 20, 2022 at 09:12:46AM +0800, Hal Feng wrote: > > >> > > > >> >> >> +/ { > > >> >> >> + compatible = "starfive,jh7110"; > > >> >> >> + #address-cells = <2>; > > >> >> >> + #size-cells = <2>; > > >> >> >> + > > >> >> >> + cpus { > > >> >> >> + #address-cells = <1>; > > >> >> >> + #size-cells = <0>; > > >> >> >> + > > >> >> >> + S76_0: cpu@0 { > > >> >> >> + compatible = "sifive,u74-mc", "riscv"; > > >> >> > > > >> >> > The label here says S76 but the compatible says u74-mc. > > >> >> > > >> >> U74-MC has 5 cores including 1 * S7 core and 4 * U74 cores. > > >> >> > > >> >> > Which is correct? Your docs say S7 and S76, so I would imagine that it > > >> >> > is actually an S76? > > >> >> > > >> >> I found SiFive website [1] call it S76, but call it S7 in other places. > > >> >> So I misunderstood this. Considering the ISA difference you described > > >> >> as below, I think it's proper to change the label to "S7_0". > > >> > > > >> > I'm less worried about the label & more interested in the compatible. > > >> > hart0 is, as you say, not a u74. Should we not be adding a "sifive,s7" > > >> > compatible string to Documentation/devicetree/bindings/riscv/cpus.yaml > > >> > and using that here instead? > > >> > > >> First of all, it's my fault that I didn't check the revision of U74-MC > > >> manual, so most of my previous replies might not make sense. > > > > > > No that's fine. The manual stuff confused me too when I went looking > > > initially, and I still get get mixed up by the fact that there are > > > core-complex manuals but not core manuals. > > > > > >> If we add a new compatible string for S7, should we change the compatibles > > >> of hart1~3 to "sifive,u74" also? And then, there may be no point keeping some > > >> compatible strings of core complex like "sifive,u74-mc" and "sifive,u54-mc". > > >> I'm not sure about this. > > > > > > [...] > > > > > >> >> Yes, "RV64IMAC" is correct. The monitor core in U74-MC is a > > >> >> S7-series core, not S76. > > >> > > > >> > Cool, thanks. > > >> > > >> Now I think it might be another version of S76. > > > > > > The SiFive docs describe the u74-mc core complex, which AFAIU you have, > > > as being 1x S7 & 4x U7. > > > > > > I'd be happy with new binding for "sifive,s7" & then we use that here. > > > If you're sure it's S76, we can also use that. S76 is described, in what > > > docs I can see, as a core complex containing an S7, so S7 seems likely > > > to be correct? > > > > I will add a new binding for "sifive,s7" and modify the code as follows. > > > > S7_0: cpu@0 { > > compatible = "sifive,s7", "riscv"; > > ... > > riscv,isa = "rv64imac_zicsr_zba_zbb"; > > I'm not sure that I'd bother with the zicsr, it gets added automagically > by the Makefile if needed: Meh, I probably shouldn't have replied to this first thing in the morning as this comment of mine doesn't really make sense. I skipped the middle part of my point here... What I meant was that you can avoid zicsr & zifencei because when the binding was defined they were included in i. I meant to use the following as a kinda explanation of it depending on the version of the ISA spec & that we just assume that zicsr & zifencei are present. I suppose you can add them to the isa string if you like, dtbs_check shouldn't complain! > | # Newer binutils versions default to ISA spec version 20191213 which moves some > | # instructions from the I extension to the Zicsr and Zifencei extensions. > | toolchain-need-zicsr-zifencei := $(call cc-option-yn, -march=$(riscv-march-y)_zicsr_zifencei) > | riscv-march-$(toolchain-need-zicsr-zifencei) := $(riscv-march-y)_zicsr_zifencei > > Otherwise, thanks for the actual confirmation of zba/zbb! > > Thanks, > Conor. > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor@kernel.org> To: Hal Feng <hal.feng@starfivetech.com> Cc: Conor Dooley <conor.dooley@microchip.com>, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, Palmer Dabbelt <palmer@dabbelt.com>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Paul Walmsley <paul.walmsley@sifive.com>, Albert Ou <aou@eecs.berkeley.edu>, Ben Dooks <ben.dooks@sifive.com>, Daniel Lezcano <daniel.lezcano@linaro.org>, Thomas Gleixner <tglx@linutronix.de>, Marc Zyngier <maz@kernel.org>, Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Philipp Zabel <p.zabel@pengutronix.de>, Linus Walleij <linus.walleij@linaro.org>, Emil Renner Berthing <emil.renner.berthing@canonical.com>, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 6/7] riscv: dts: starfive: Add initial StarFive JH7110 device tree Date: Wed, 15 Feb 2023 07:59:27 +0000 [thread overview] Message-ID: <Y+yQ3+0lyo5OQr2z@spud> (raw) In-Reply-To: <Y+yM6HgAbDoWlu1G@spud> [-- Attachment #1.1: Type: text/plain, Size: 4327 bytes --] On Wed, Feb 15, 2023 at 07:42:32AM +0000, Conor Dooley wrote: > Hey Hal! > > On Wed, Feb 15, 2023 at 11:07:15AM +0800, Hal Feng wrote: > > On Thu, 2 Feb 2023 19:41:33 +0000, Conor Dooley wrote: > > > On Fri, Feb 03, 2023 at 02:56:41AM +0800, Hal Feng wrote: > > >> On Wed, 1 Feb 2023 08:21:05 +0000, Conor Dooley wrote: > > >> > On Wed, Feb 01, 2023 at 03:21:48PM +0800, Hal Feng wrote: > > >> >> On Wed, 28 Dec 2022 22:48:43 +0000, Conor Dooley wrote: > > >> >> > On Tue, Dec 20, 2022 at 09:12:46AM +0800, Hal Feng wrote: > > >> > > > >> >> >> +/ { > > >> >> >> + compatible = "starfive,jh7110"; > > >> >> >> + #address-cells = <2>; > > >> >> >> + #size-cells = <2>; > > >> >> >> + > > >> >> >> + cpus { > > >> >> >> + #address-cells = <1>; > > >> >> >> + #size-cells = <0>; > > >> >> >> + > > >> >> >> + S76_0: cpu@0 { > > >> >> >> + compatible = "sifive,u74-mc", "riscv"; > > >> >> > > > >> >> > The label here says S76 but the compatible says u74-mc. > > >> >> > > >> >> U74-MC has 5 cores including 1 * S7 core and 4 * U74 cores. > > >> >> > > >> >> > Which is correct? Your docs say S7 and S76, so I would imagine that it > > >> >> > is actually an S76? > > >> >> > > >> >> I found SiFive website [1] call it S76, but call it S7 in other places. > > >> >> So I misunderstood this. Considering the ISA difference you described > > >> >> as below, I think it's proper to change the label to "S7_0". > > >> > > > >> > I'm less worried about the label & more interested in the compatible. > > >> > hart0 is, as you say, not a u74. Should we not be adding a "sifive,s7" > > >> > compatible string to Documentation/devicetree/bindings/riscv/cpus.yaml > > >> > and using that here instead? > > >> > > >> First of all, it's my fault that I didn't check the revision of U74-MC > > >> manual, so most of my previous replies might not make sense. > > > > > > No that's fine. The manual stuff confused me too when I went looking > > > initially, and I still get get mixed up by the fact that there are > > > core-complex manuals but not core manuals. > > > > > >> If we add a new compatible string for S7, should we change the compatibles > > >> of hart1~3 to "sifive,u74" also? And then, there may be no point keeping some > > >> compatible strings of core complex like "sifive,u74-mc" and "sifive,u54-mc". > > >> I'm not sure about this. > > > > > > [...] > > > > > >> >> Yes, "RV64IMAC" is correct. The monitor core in U74-MC is a > > >> >> S7-series core, not S76. > > >> > > > >> > Cool, thanks. > > >> > > >> Now I think it might be another version of S76. > > > > > > The SiFive docs describe the u74-mc core complex, which AFAIU you have, > > > as being 1x S7 & 4x U7. > > > > > > I'd be happy with new binding for "sifive,s7" & then we use that here. > > > If you're sure it's S76, we can also use that. S76 is described, in what > > > docs I can see, as a core complex containing an S7, so S7 seems likely > > > to be correct? > > > > I will add a new binding for "sifive,s7" and modify the code as follows. > > > > S7_0: cpu@0 { > > compatible = "sifive,s7", "riscv"; > > ... > > riscv,isa = "rv64imac_zicsr_zba_zbb"; > > I'm not sure that I'd bother with the zicsr, it gets added automagically > by the Makefile if needed: Meh, I probably shouldn't have replied to this first thing in the morning as this comment of mine doesn't really make sense. I skipped the middle part of my point here... What I meant was that you can avoid zicsr & zifencei because when the binding was defined they were included in i. I meant to use the following as a kinda explanation of it depending on the version of the ISA spec & that we just assume that zicsr & zifencei are present. I suppose you can add them to the isa string if you like, dtbs_check shouldn't complain! > | # Newer binutils versions default to ISA spec version 20191213 which moves some > | # instructions from the I extension to the Zicsr and Zifencei extensions. > | toolchain-need-zicsr-zifencei := $(call cc-option-yn, -march=$(riscv-march-y)_zicsr_zifencei) > | riscv-march-$(toolchain-need-zicsr-zifencei) := $(riscv-march-y)_zicsr_zifencei > > Otherwise, thanks for the actual confirmation of zba/zbb! > > Thanks, > Conor. > [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 161 bytes --] _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2023-02-15 7:59 UTC|newest] Thread overview: 112+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-12-20 1:12 [PATCH v3 0/7] Basic device tree support for StarFive JH7110 RISC-V SoC Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 1:12 ` [PATCH v3 1/7] dt-bindings: riscv: Add StarFive JH7110 SoC and VisionFive 2 board Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 10:05 ` Krzysztof Kozlowski 2022-12-20 10:05 ` Krzysztof Kozlowski 2022-12-23 2:05 ` Hal Feng 2022-12-23 2:05 ` Hal Feng 2022-12-20 20:58 ` Conor Dooley 2022-12-20 20:58 ` Conor Dooley 2022-12-23 2:15 ` Hal Feng 2022-12-23 2:15 ` Hal Feng 2022-12-20 1:12 ` [PATCH v3 2/7] dt-bindings: timer: Add StarFive JH7110 clint Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 1:12 ` [PATCH v3 3/7] dt-bindings: interrupt-controller: Add StarFive JH7110 plic Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 1:12 ` [PATCH v3 4/7] dt-bindings: sifive,ccache0: Support StarFive JH7110 SoC Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 20:21 ` Rob Herring 2022-12-20 20:21 ` Rob Herring 2022-12-20 1:12 ` [PATCH v3 5/7] soc: sifive: ccache: Add StarFive JH7110 support Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 21:14 ` Conor Dooley 2022-12-20 21:14 ` Conor Dooley 2022-12-20 1:12 ` [PATCH v3 6/7] riscv: dts: starfive: Add initial StarFive JH7110 device tree Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 10:10 ` Krzysztof Kozlowski 2022-12-20 10:10 ` Krzysztof Kozlowski 2022-12-25 10:31 ` Hal Feng 2022-12-25 10:31 ` Hal Feng 2022-12-25 11:56 ` Krzysztof Kozlowski 2022-12-25 11:56 ` Krzysztof Kozlowski 2022-12-20 21:31 ` Conor Dooley 2022-12-20 21:31 ` Conor Dooley 2022-12-25 14:31 ` Hal Feng 2022-12-25 14:31 ` Hal Feng 2022-12-27 20:58 ` Conor Dooley 2022-12-27 20:58 ` Conor Dooley 2022-12-28 22:48 ` Conor Dooley 2022-12-28 22:48 ` Conor Dooley 2022-12-29 5:25 ` Icenowy Zheng 2022-12-29 5:25 ` Icenowy Zheng 2022-12-29 9:02 ` Conor Dooley 2022-12-29 9:02 ` Conor Dooley 2023-02-01 7:53 ` Hal Feng 2023-02-01 7:53 ` Hal Feng 2023-02-01 7:31 ` Hal Feng 2023-02-01 7:31 ` Hal Feng 2023-02-01 7:21 ` Hal Feng 2023-02-01 7:21 ` Hal Feng 2023-02-01 8:21 ` Conor Dooley 2023-02-01 8:21 ` Conor Dooley 2023-02-02 18:56 ` Hal Feng 2023-02-02 18:56 ` Hal Feng 2023-02-02 19:41 ` Conor Dooley 2023-02-02 19:41 ` Conor Dooley 2023-02-09 11:11 ` Conor Dooley 2023-02-09 11:11 ` Conor Dooley 2023-02-13 9:41 ` Hal Feng 2023-02-13 9:41 ` Hal Feng 2023-02-13 10:07 ` Conor Dooley 2023-02-13 10:07 ` Conor Dooley 2023-02-14 2:37 ` Hal Feng 2023-02-14 2:37 ` Hal Feng 2023-02-15 3:07 ` Hal Feng 2023-02-15 3:07 ` Hal Feng 2023-02-15 7:42 ` Conor Dooley 2023-02-15 7:42 ` Conor Dooley 2023-02-15 7:59 ` Conor Dooley [this message] 2023-02-15 7:59 ` Conor Dooley 2023-01-31 2:00 ` Hal Feng 2023-01-31 2:00 ` Hal Feng 2023-01-31 6:17 ` Conor Dooley 2023-01-31 6:17 ` Conor Dooley 2023-02-02 2:42 ` Hal Feng 2023-02-02 2:42 ` Hal Feng 2023-02-02 6:19 ` Conor Dooley 2023-02-02 6:19 ` Conor Dooley 2022-12-20 1:12 ` [PATCH v3 7/7] riscv: dts: starfive: Add StarFive JH7110 VisionFive 2 board " Hal Feng 2022-12-20 1:12 ` Hal Feng 2022-12-20 21:26 ` Conor Dooley 2022-12-20 21:26 ` Conor Dooley 2022-12-23 3:12 ` Hal Feng 2022-12-23 3:12 ` Hal Feng 2022-12-28 22:49 ` Conor Dooley 2022-12-28 22:49 ` Conor Dooley 2023-01-10 17:59 ` Conor Dooley 2023-01-10 17:59 ` Conor Dooley 2023-01-18 23:43 ` Conor Dooley 2023-01-18 23:43 ` Conor Dooley 2023-02-14 9:53 ` Emil Renner Berthing 2023-02-14 9:53 ` Emil Renner Berthing 2023-02-15 14:03 ` Hal Feng 2023-02-15 14:03 ` Hal Feng 2023-02-16 9:27 ` Emil Renner Berthing 2023-02-16 9:27 ` Emil Renner Berthing 2023-02-16 9:50 ` Conor Dooley 2023-02-16 9:50 ` Conor Dooley 2023-02-16 10:09 ` Conor Dooley 2023-02-16 10:09 ` Conor Dooley 2023-02-16 10:32 ` Emil Renner Berthing 2023-02-16 10:32 ` Emil Renner Berthing 2023-02-16 12:27 ` Hal Feng 2023-02-16 12:27 ` Hal Feng 2023-02-16 13:02 ` Conor Dooley 2023-02-16 13:02 ` Conor Dooley 2022-12-26 23:01 ` [PATCH v3 0/7] Basic device tree support for StarFive JH7110 RISC-V SoC Conor Dooley 2022-12-26 23:01 ` Conor Dooley 2022-12-27 7:58 ` Krzysztof Kozlowski 2022-12-27 7:58 ` Krzysztof Kozlowski 2022-12-27 14:20 ` Conor Dooley 2022-12-27 14:20 ` 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=Y+yQ3+0lyo5OQr2z@spud \ --to=conor@kernel.org \ --cc=aou@eecs.berkeley.edu \ --cc=ben.dooks@sifive.com \ --cc=conor.dooley@microchip.com \ --cc=daniel.lezcano@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=emil.renner.berthing@canonical.com \ --cc=hal.feng@starfivetech.com \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linus.walleij@linaro.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=maz@kernel.org \ --cc=mturquette@baylibre.com \ --cc=p.zabel@pengutronix.de \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=robh+dt@kernel.org \ --cc=sboyd@kernel.org \ --cc=tglx@linutronix.de \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.