[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<ZQ2PR01MB13079F7073193A2C64BD83F4E6CC2@ZQ2PR01MB1307.CHNPR01.prod.partner.outlook.cn>
Date: Wed, 12 Nov 2025 07:47:08 +0000
From: Hal Feng <hal.feng@...rfivetech.com>
To: Conor Dooley <conor@...nel.org>, Heinrich Schuchardt
<heinrich.schuchardt@...onical.com>
CC: E Shattow <e@...eshell.de>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>, "linux-riscv@...ts.infradead.org"
<linux-riscv@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, Conor Dooley <conor+dt@...nel.org>, Rob
Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Palmer
Dabbelt <palmer@...belt.com>, Paul Walmsley <pjw@...nel.org>, Albert Ou
<aou@...s.berkeley.edu>, "Rafael J . Wysocki" <rafael@...nel.org>, Viresh
Kumar <viresh.kumar@...aro.org>, Bjorn Helgaas <bhelgaas@...gle.com>, Lorenzo
Pieralisi <lpieralisi@...nel.org>, Krzysztof WilczyĆski
<kwilczynski@...nel.org>, Manivannan Sadhasivam <mani@...nel.org>, Emil
Renner Berthing <emil.renner.berthing@...onical.com>
Subject: RE: [PATCH v2 0/8] Add support for StarFive VisionFive 2 Lite board
> On 08.11.25 01:20, Conor Dooley wrote:
> On Fri, Nov 07, 2025 at 12:21:46PM +0100, Heinrich Schuchardt wrote:
> > On 11/7/25 12:11, E Shattow wrote:
> > >
> > >
> > > On 11/7/25 01:55, Hal Feng wrote:
> > > > VisionFive 2 Lite is a mini SBC based on the StarFive JH7110S
> > > > industrial SoC which can run at -40~85 degrees centigrade and up to
> 1.25GHz.
> > > >
> > > > Board features:
> > > > - JH7110S SoC
> > > > - 4/8 GiB LPDDR4 DRAM
> > > > - AXP15060 PMIC
> > > > - 40 pin GPIO header
> > > > - 1x USB 3.0 host port
> > > > - 3x USB 2.0 host port
> > > > - 1x M.2 M-Key (size: 2242)
> > > > - 1x MicroSD slot (optional non-removable 64GiB eMMC)
> > > > - 1x QSPI Flash
> > > > - 1x I2C EEPROM
> > > > - 1x 1Gbps Ethernet port
> > > > - SDIO-based Wi-Fi & UART-based Bluetooth
> > > > - 1x HDMI port
> > > > - 1x 2-lane DSI
> > > > - 1x 2-lane CSI
> > > >
> > > > VisionFive 2 Lite schematics:
> > > > https://doc-en.rvspace.org/VisionFive2Lite/PDF/VF2_LITE_V1.10_TF_2
> > > > 0250818_SCH.pdf VisionFive 2 Lite Quick Start Guide:
> > > > https://doc-en.rvspace.org/VisionFive2Lite/VisionFive2LiteQSG/inde
> > > > x.html More documents:
> > > > https://doc-en.rvspace.org/Doc_Center/visionfive_2_lite.html
> > > >
> > > > Changes since v1:
> > > > - Drop patch 1 because it is applied.
> > > > - Rename jh7110.dtsi to jh711x.dtsi.
> > > > - Move the content of jh7110-common.dtsi to the new file
> > > > jh711x-common.dtsi and move opp table to jh7110-common.dtsi.
> > > > patch 4:
> > > > - Move the uncommon nodes to jh7110-common.dtsi instead of board
> dts.
> > > > patch 5:
> > > > - Add jh7110s-common.dtsi and include it in jh7110s-starfive-visionfive-
> 2-lite.dtsi.
> > > >
> > > > Changes since RFC:
> > > > - Add jh7110s compatible to the generic cpufreq driver.
> > > > - Fix the dtbs_check error by adding the missing "enable-gpios" property
> > > > in jh7110 pcie dt-bindings.
> > > > - Rebase on the latest mainline.
> > > > - Add VisionFive 2 Lite eMMC board device tree and add a common
> board dtsi
> > > > for VisionFive 2 Lite variants.
> > > > - Add usb switch pin configuration (GPIO62).
> > > > - Improve the commit messages.
> > > >
> > > > History:
> > > > v1:
> > > > https://lore.kernel.org/all/20251016080054.12484-1-hal.feng@starfi
> > > > vetech.com/
> > > > RFC:
> > > > https://lore.kernel.org/all/20250821100930.71404-1-hal.feng@starfi
> > > > vetech.com/
> > > >
> > > > Hal Feng (8):
> > > > dt-bindings: PCI: starfive,jh7110-pcie: Add enable-gpios property
> > > > dt-bindings: riscv: Add StarFive JH7110S SoC and VisionFive 2 Lite
> > > > board
> > > > riscv: dts: starfive: Rename jh7110.dtsi to jh711x.dtsi
> > > > riscv: dts: starfive: Split jh7110-common.dtsi and move opp table to
> > > > it
> > > > riscv: dts: starfive: jh711x-common: Move out some nodes to jh7110
> > > > common dtsi
> > > > riscv: dts: starfive: Add common board dtsi for JH7110s and VisionFive
> > > > 2 Lite variants
> > > > riscv: dts: starfive: Add VisionFive 2 Lite board device tree
> > > > riscv: dts: starfive: Add VisionFive 2 Lite eMMC board device
> > > > tree
> > > >
> > > > .../bindings/pci/starfive,jh7110-pcie.yaml | 4 +
> > > > .../devicetree/bindings/riscv/starfive.yaml | 6 +
> > > > arch/riscv/boot/dts/starfive/Makefile | 3 +
> > > > .../boot/dts/starfive/jh7110-common.dtsi | 653 +----------------
> > > > .../boot/dts/starfive/jh7110s-common.dtsi | 27 +
> > > > ...h7110s-starfive-visionfive-2-lite-emmc.dts | 22 +
> > > > .../jh7110s-starfive-visionfive-2-lite.dts | 20 +
> > > > .../jh7110s-starfive-visionfive-2-lite.dtsi | 126 ++++
> > > > .../boot/dts/starfive/jh711x-common.dtsi | 656
> ++++++++++++++++++
> > > > .../dts/starfive/{jh7110.dtsi => jh711x.dtsi} | 16 -
> > > > 10 files changed, 879 insertions(+), 654 deletions(-)
> > > > create mode 100644 arch/riscv/boot/dts/starfive/jh7110s-
> common.dtsi
> > > > create mode 100644 arch/riscv/boot/dts/starfive/jh7110s-starfive-
> visionfive-2-lite-emmc.dts
> > > > create mode 100644 arch/riscv/boot/dts/starfive/jh7110s-starfive-
> visionfive-2-lite.dts
> > > > create mode 100644 arch/riscv/boot/dts/starfive/jh7110s-starfive-
> visionfive-2-lite.dtsi
> > > > create mode 100644 arch/riscv/boot/dts/starfive/jh711x-common.dtsi
> > > > rename arch/riscv/boot/dts/starfive/{jh7110.dtsi => jh711x.dtsi}
> > > > (99%)
> > > >
> > > >
> > > > base-commit: df5d79720b152e7ff058f11ed7e88d5b5c8d2a0c
> > >
> > > Small nit that "lite-emmc" is confusing together. In patches to
> > > U-Boot dev mailing list the EEPROM product id is demonstrated to be with
> "SL"
> > > suffix when compared to VisionFive 2 (JH7110) so I suggest avoid
> > > confusion in upstream and use for VisionFive 2 Lite (JH7110S) these
> > > compatible names:
> > >
> > > starfive,visionfive-2sl-lite
> > > starfive,visionfive-2sl-emmc
> > >
> > > Also filenames:
> > >
> > > jh7110s-starfive-visionfive-2sl-lite.dts
> > > jh7110s-starfive-visionfive-2sl.dtsi
> > > jh7110s-starfive-visionfive-2sl-emmc.dts
> > >
> > > What do you think?
> > >
> >
> > This is a serial number for the Lite board:
> > VF7110SL-2310-D002E000-xxxxxxxx
> >
> > Here E000 encodes that we have no eMMC.
> >
> > The S is part of 7110S which we already have in 'jh7110s'. And the L
> > is already decoded as 'lite' in this patch series. Duplicating this
> > information as 'sl' as you suggested provides no benefit.
> >
> > Let's just stick with Hal's suggestion.
>
> The marketing materials etc call it the visionfive 2 lite, for example on
> kickstarter: https://www.kickstarter.com/projects/starfive/visionfive-2-lite-
> unlock-risc-v-sbc-at-199
> I'm happy enough with what Hal has here as a result.
Hi, Conor,
Thanks for your review. VisionFive 2 Lite is the official name of this product,
so it is appropriate to use "starfive,visionfive-2-lite" as the compatible and
"jh7110s-starfive-visionfive-2-lite.dts" as the device tree filename.
Best regards,
Hal
Powered by blists - more mailing lists