From: Conor Dooley <conor.dooley@microchip.com> To: Emil Renner Berthing <emil.renner.berthing@canonical.com> Cc: Hal Feng <hal.feng@starfivetech.com>, <linux-riscv@lists.infradead.org>, <devicetree@vger.kernel.org>, Conor Dooley <conor@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>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v3 7/7] riscv: dts: starfive: Add StarFive JH7110 VisionFive 2 board device tree Date: Thu, 16 Feb 2023 09:50:37 +0000 [thread overview] Message-ID: <Y+38bT8cnahu19bw@wendy> (raw) In-Reply-To: <CAJM55Z-+Cz8d=YySRaFJSAffDfoZ4Madx322qCX100-nAcx+5Q@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 2282 bytes --] On Thu, Feb 16, 2023 at 10:27:17AM +0100, Emil Renner Berthing wrote: > On Wed, 15 Feb 2023 at 15:04, Hal Feng <hal.feng@starfivetech.com> wrote: > > On 2/14/2023 5:53 PM, Emil Renner Berthing wrote: > > > On Tue, 20 Dec 2022 at 02:12, Hal Feng <hal.feng@starfivetech.com> wrote: > > >> + model = "StarFive VisionFive 2 VB"; > > >> + compatible = "starfive,visionfive-2-vb", "starfive,jh7110"; > > > > > > Hi Hal, > > > > > > I just want to double check, is "VisionFive 2 VA" / "visoinfive-2-va" > > > and "VisionFive 2 VB" / "visionfive-2-vb" really what you want? I > > > still think having these names match what is printed on the silkscreen > > > makes it a lot easier for everybody. Even your own releases calls the > > > boards "v1.2A" and "v1.3B": > > > https://github.com/starfive-tech/VisionFive2/releases/ > > > > > > So I'd suggest > > > model = "StarFive VisionFive 2 v1.3B"; > > > compatible = "starfive,visionfive-2-v1.3b", "starfive,jh7110"; > > > > > > I haven't seen these "VA" and "VB" anywhere else, so if you don't want > > > the version numbers and can promise that there will be no incompatible > > > future revisions of the boards then maybe just drop the "V". Eg. > > > model = "StarFive VisionFive 2 B"; > > > compatible = "starfive,visionfive-2-b", "starfive,jh7110"; > > > > The version A board has reached the end of life. As far as I know, the > > version B board will not update also unless there are some important > > requirements and StarFive decides to update. Furthermore, it's too late > > to change the compatible as patch 1 was already accepted. Will it be > > easier to read if I modify it as below? > > > > model = "StarFive VisionFive 2 vB"; > > compatible = "starfive,visionfive-2-vb", "starfive,jh7110"; > > Oh, that's sad that the kernel will now end up calling the boards > something that's used nowhere else, even by StarFive :/ > But yeah, I guess vA and vB are a little easier to read. Nothing has been released with that name so AFAIU we can change it still. If you sort it out today/tomorrow I'll try get it to Arnd before the merge window opens... I might've jumped the gun a bit here, I thought that it'd been changed to what you (Emil) had suggested. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor.dooley@microchip.com> To: Emil Renner Berthing <emil.renner.berthing@canonical.com> Cc: Hal Feng <hal.feng@starfivetech.com>, <linux-riscv@lists.infradead.org>, <devicetree@vger.kernel.org>, Conor Dooley <conor@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>, <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v3 7/7] riscv: dts: starfive: Add StarFive JH7110 VisionFive 2 board device tree Date: Thu, 16 Feb 2023 09:50:37 +0000 [thread overview] Message-ID: <Y+38bT8cnahu19bw@wendy> (raw) In-Reply-To: <CAJM55Z-+Cz8d=YySRaFJSAffDfoZ4Madx322qCX100-nAcx+5Q@mail.gmail.com> [-- Attachment #1.1: Type: text/plain, Size: 2282 bytes --] On Thu, Feb 16, 2023 at 10:27:17AM +0100, Emil Renner Berthing wrote: > On Wed, 15 Feb 2023 at 15:04, Hal Feng <hal.feng@starfivetech.com> wrote: > > On 2/14/2023 5:53 PM, Emil Renner Berthing wrote: > > > On Tue, 20 Dec 2022 at 02:12, Hal Feng <hal.feng@starfivetech.com> wrote: > > >> + model = "StarFive VisionFive 2 VB"; > > >> + compatible = "starfive,visionfive-2-vb", "starfive,jh7110"; > > > > > > Hi Hal, > > > > > > I just want to double check, is "VisionFive 2 VA" / "visoinfive-2-va" > > > and "VisionFive 2 VB" / "visionfive-2-vb" really what you want? I > > > still think having these names match what is printed on the silkscreen > > > makes it a lot easier for everybody. Even your own releases calls the > > > boards "v1.2A" and "v1.3B": > > > https://github.com/starfive-tech/VisionFive2/releases/ > > > > > > So I'd suggest > > > model = "StarFive VisionFive 2 v1.3B"; > > > compatible = "starfive,visionfive-2-v1.3b", "starfive,jh7110"; > > > > > > I haven't seen these "VA" and "VB" anywhere else, so if you don't want > > > the version numbers and can promise that there will be no incompatible > > > future revisions of the boards then maybe just drop the "V". Eg. > > > model = "StarFive VisionFive 2 B"; > > > compatible = "starfive,visionfive-2-b", "starfive,jh7110"; > > > > The version A board has reached the end of life. As far as I know, the > > version B board will not update also unless there are some important > > requirements and StarFive decides to update. Furthermore, it's too late > > to change the compatible as patch 1 was already accepted. Will it be > > easier to read if I modify it as below? > > > > model = "StarFive VisionFive 2 vB"; > > compatible = "starfive,visionfive-2-vb", "starfive,jh7110"; > > Oh, that's sad that the kernel will now end up calling the boards > something that's used nowhere else, even by StarFive :/ > But yeah, I guess vA and vB are a little easier to read. Nothing has been released with that name so AFAIU we can change it still. If you sort it out today/tomorrow I'll try get it to Arnd before the merge window opens... I might've jumped the gun a bit here, I thought that it'd been changed to what you (Emil) had suggested. [-- 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-16 9:51 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 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 [this message] 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+38bT8cnahu19bw@wendy \ --to=conor.dooley@microchip.com \ --cc=aou@eecs.berkeley.edu \ --cc=ben.dooks@sifive.com \ --cc=conor@kernel.org \ --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.