linux-kernel - Re: [PATCH v6 5/5] riscv: dts: starfive: add DeepComputing FML13V01 board device tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAH1PCMZ8bxG6_DoWL43E639HJ+PX-ynCSpbXPKMkRA5tCByuuQ@mail.gmail.com>
Date: Mon, 28 Oct 2024 09:34:01 +0800
From: Guodong Xu <guodong@...cstar.com>
To: Emil Renner Berthing <emil.renner.berthing@...onical.com>
Cc: Conor Dooley <conor@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Emil Renner Berthing <kernel@...il.dk>, rafal@...ecki.pl, 
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Paul Walmsley <paul.walmsley@...ive.com>, 
	Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, 
	Neil Armstrong <neil.armstrong@...aro.org>, Heiko Stuebner <heiko.stuebner@...rry.de>, 
	Michael Zhu <michael.zhu@...rfivetech.com>, Drew Fustini <drew@...gleboard.org>, 
	Alexandru Stan <ams@...me.work>, Daniel Schaefer <dhs@...me.work>, 
	Sandie Cao <sandie.cao@...pcomputing.io>, Yuning Liang <yuning.liang@...pcomputing.io>, 
	Huiming Qiu <huiming.qiu@...pcomputing.io>, Alex Elder <elder@...cstar.com>, linux@...me.work, 
	devicetree@...r.kernel.org, linux-riscv@...ts.infradead.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 5/5] riscv: dts: starfive: add DeepComputing FML13V01
 board device tree

On Mon, Oct 28, 2024 at 12:42 AM Emil Renner Berthing
<emil.renner.berthing@...onical.com> wrote:
>
> Guodong Xu wrote:
> > From: Sandie Cao <sandie.cao@...pcomputing.io>
> >
> > The FML13V01 board from DeepComputing incorporates a StarFive JH7110 SoC.
> > It is a mainboard designed for the Framework Laptop 13 Chassis, which has
> > (Framework) SKU FRANHQ0001.
> >
> > The FML13V01 board features:
> > - StarFive JH7110 SoC
> > - LPDDR4 8GB
> > - eMMC 32GB or 128GB
> > - QSPI Flash
> > - MicroSD Slot
> > - PCIe-based Wi-Fi
> > - 4 USB-C Ports
> >  - Port 1: PD 3.0 (60W Max), USB 3.2 Gen 1, DP 1.4 (4K@...z/2.5K@...z)
> >  - Port 2: PD 3.0 (60W Max), USB 3.2 Gen 1
> >  - Port 3 & 4: USB 3.2 Gen 1
> >
> > Create the DTS file for the DeepComputing FML13V01 board. Based on
> > 'jh7110-common.dtsi', usb0 is enabled and is set to operate as a "host".
> >
> > Signed-off-by: Sandie Cao <sandie.cao@...pcomputing.io>
> > [elder@...cstar.com: revised the description, updated some nodes]
> > Signed-off-by: Alex Elder <elder@...cstar.com>
> > Signed-off-by: Guodong Xu <guodong@...cstar.com>
>
> Thanks! Will you be adding more stuff later like fx. the I2S sound chip?

Yes! Any device that has been tested and shown to work correctly can be
enabled / updated later.

>
> Reviewed-by: Emil Renner Berthing <emil.renner.berthing@...onical.com>
>

Thank you Emil.

> > ---
> > v6: Dropped all node status disabled code
> >     Enabled usb0 and set its operation mode to "host"
> > v5: No change
> > v4: Changed model string to "DeepComputing FML13V01"
> >     Changed dts filename and Makefile accordingly to reflect the change
> >     Updated device nodes status, and verified functional
> >     Revised the commit message
> > v3: Updated the commit message
> > v2: Changed the model and copmatible strings
> >     Updated the commit message with board features
> >
> >  arch/riscv/boot/dts/starfive/Makefile           |  1 +
> >  .../starfive/jh7110-deepcomputing-fml13v01.dts  | 17 +++++++++++++++++
> >  2 files changed, 18 insertions(+)
> >  create mode 100644 arch/riscv/boot/dts/starfive/jh7110-deepcomputing-fml13v01.dts
> >
> > diff --git a/arch/riscv/boot/dts/starfive/Makefile b/arch/riscv/boot/dts/starfive/Makefile
> > index 7a163a7d6ba3..b3bb12f78e7d 100644
> > --- a/arch/riscv/boot/dts/starfive/Makefile
> > +++ b/arch/riscv/boot/dts/starfive/Makefile
> > @@ -8,6 +8,7 @@ DTC_FLAGS_jh7110-starfive-visionfive-2-v1.3b := -@
> >  dtb-$(CONFIG_ARCH_STARFIVE) += jh7100-beaglev-starlight.dtb
> >  dtb-$(CONFIG_ARCH_STARFIVE) += jh7100-starfive-visionfive-v1.dtb
> >
> > +dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-deepcomputing-fml13v01.dtb
> >  dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-milkv-mars.dtb
> >  dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-pine64-star64.dtb
> >  dtb-$(CONFIG_ARCH_STARFIVE) += jh7110-starfive-visionfive-2-v1.2a.dtb
> > diff --git a/arch/riscv/boot/dts/starfive/jh7110-deepcomputing-fml13v01.dts b/arch/riscv/boot/dts/starfive/jh7110-deepcomputing-fml13v01.dts
> > new file mode 100644
> > index 000000000000..30b0715196b6
> > --- /dev/null
> > +++ b/arch/riscv/boot/dts/starfive/jh7110-deepcomputing-fml13v01.dts
> > @@ -0,0 +1,17 @@
> > +// SPDX-License-Identifier: GPL-2.0 OR MIT
> > +/*
> > + * Copyright (C) 2024 DeepComputing (HK) Limited
> > + */
> > +
> > +/dts-v1/;
> > +#include "jh7110-common.dtsi"
> > +
> > +/ {
> > +     model = "DeepComputing FML13V01";
> > +     compatible = "deepcomputing,fml13v01", "starfive,jh7110";
> > +};
> > +
> > +&usb0 {
> > +     dr_mode = "host";
> > +     status = "okay";
> > +};
> > --
> > 2.34.1
> >