[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <171958904483.3122797.18431731519773107175.robh@kernel.org>
Date: Fri, 28 Jun 2024 09:41:53 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Yixun Lan <dlan@...too.org>
Cc: Krzysztof Kozlowski <krzk+dt@...nel.org>,
Inochi Amaoto <inochiama@...look.com>, linux-riscv@...ts.infradead.org,
Samuel Holland <samuel.holland@...ive.com>,
Meng Zhang <zhangmeng.kevin@...cemit.com>, linux-kernel@...r.kernel.org,
Yangyu Chen <cyy@...self.name>, Daniel Lezcano <daniel.lezcano@...aro.org>,
Palmer Dabbelt <palmer@...belt.com>, Conor Dooley <conor+dt@...nel.org>,
Palmer Dabbelt <palmer@...ive.com>, devicetree@...r.kernel.org,
Thomas Gleixner <tglx@...utronix.de>, Conor Dooley <conor@...nel.org>,
linux-serial@...r.kernel.org, Lubomir Rintel <lkundrak@...sk>,
Paul Walmsley <paul.walmsley@...ive.com>, Jiri Slaby <jirislaby@...nel.org>,
Anup Patel <anup@...infault.org>, Albert Ou <aou@...s.berkeley.edu>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: Re: [PATCH v2 00/10] riscv: add initial support for SpacemiT K1
On Thu, 27 Jun 2024 15:31:14 +0000, Yixun Lan wrote:
> SpacemiT K1 is an ideal chip for some new extension such as RISC-V Vector
> 1.0 and Zicond evaluation now. Add initial support for it to allow more
> people to participate in building drivers to mainline for it.
>
> This kernel has been tested upon Banana Pi BPI-F3 board on vendor U-Boot
> bootflow generated by Armbian SDK[1] and patched OpenSBI[2] to enable
> Zicboz, which does not in the vendor dts on its U-Boot. Then successfully
> booted to busybox on initrd with this log[3].
>
> As previous discussion in patch v1[4], maintainer expect more basic drivers
> ready before really merging it, which would be fine. For other follow-up patches,
> that are clk, pinctrl/gpio, reset.. My current goal would target at a headless
> system including SD card, emmc, and ethernet.
>
> P.S: talked to Yangyu, I will help and take care of this patch series, thanks
> ---
> Changes in v2:
> - fix timebase-frequency according to current setting
> - add other uart dt nodes, fix input frequency
> - introduce new uart compatible for K1 SoC
> - add 'k1' prefix to bananapi-f3.dts
> - fix k1-clint compatible
> - fix some typos
> - Link to v1: https://lore.kernel.org/r/tencent_BC64B7B1876F5D10479BD19112F73F262505@qq.com
>
> Link: https://github.com/BPI-SINOVOIP/armbian-build/tree/v24.04.30 [1]
> Link: https://gist.github.com/cyyself/a07096e6e99c949ed13f8fa16d884402 [2]
> Link: https://gist.github.com/cyyself/a2201c01f5c8955a119641f97b7d0280 [3]
> Link: https://lore.kernel.org/r/20240618-hardwood-footrest-ab5ec5bce3cf@wendy [4]
>
> To: Rob Herring <robh@...nel.org>
> To: Krzysztof Kozlowski <krzk+dt@...nel.org>
> To: Conor Dooley <conor+dt@...nel.org>
> To: Conor Dooley <conor@...nel.org>
> To: Paul Walmsley <paul.walmsley@...ive.com>
> To: Palmer Dabbelt <palmer@...belt.com>
> To: Albert Ou <aou@...s.berkeley.edu>
> To: Daniel Lezcano <daniel.lezcano@...aro.org>
> To: Thomas Gleixner <tglx@...utronix.de>
> To: Samuel Holland <samuel.holland@...ive.com>
> To: Anup Patel <anup@...infault.org>
> To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> To: Jiri Slaby <jirislaby@...nel.org>
> To: Lubomir Rintel <lkundrak@...sk>
> Cc: devicetree@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> Cc: Palmer Dabbelt <palmer@...ive.com>
> Cc: linux-riscv@...ts.infradead.org
> Cc: linux-serial@...r.kernel.org
> Cc: Inochi Amaoto <inochiama@...look.com>
> Cc: Meng Zhang <zhangmeng.kevin@...cemit.com>
>
> Signed-off-by: Yangyu Chen <cyy@...self.name>
> Signed-off-by: Yixun Lan <dlan@...too.org>
>
> ---
> Yangyu Chen (9):
> dt-bindings: vendor-prefixes: add spacemit
> dt-bindings: riscv: Add SpacemiT X60 compatibles
> dt-bindings: riscv: add SpacemiT K1 bindings
> dt-bindings: timer: Add SpacemiT K1 CLINT
> dt-bindings: interrupt-controller: Add SpacemiT K1 PLIC
> riscv: add SpacemiT SOC family Kconfig support
> riscv: dts: add initial SpacemiT K1 SoC device tree
> riscv: dts: spacemit: add Banana Pi BPI-F3 board device tree
> riscv: defconfig: enable SpacemiT SoC
>
> Yixun Lan (1):
> dt-bindings: serial: 8250: Add SpacemiT K1 uart compatible
>
> .../interrupt-controller/sifive,plic-1.0.0.yaml | 5 +-
> Documentation/devicetree/bindings/riscv/cpus.yaml | 1 +
> .../devicetree/bindings/riscv/spacemit.yaml | 24 ++
> Documentation/devicetree/bindings/serial/8250.yaml | 4 +-
> .../devicetree/bindings/timer/sifive,clint.yaml | 1 +
> .../devicetree/bindings/vendor-prefixes.yaml | 2 +
> arch/riscv/Kconfig.socs | 5 +
> arch/riscv/boot/dts/Makefile | 1 +
> arch/riscv/boot/dts/spacemit/Makefile | 2 +
> arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dts | 19 ++
> arch/riscv/boot/dts/spacemit/k1.dtsi | 378 +++++++++++++++++++++
> arch/riscv/configs/defconfig | 1 +
> 12 files changed, 441 insertions(+), 2 deletions(-)
> ---
> base-commit: f2661062f16b2de5d7b6a5c42a9a5c96326b8454
> change-id: 20240626-k1-01-basic-dt-1aa31eeebcd2
>
> Best regards,
> --
> Yixun Lan <dlan@...too.org>
>
>
>
My bot found new DTB warnings on the .dts files added or changed in this
series.
Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.
If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:
pip3 install dtschema --upgrade
New warnings running 'make CHECK_DTBS=y spacemit/k1-bananapi-f3.dtb' for 20240627-k1-01-basic-dt-v2-0-cc06c7555f07@...too.org:
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17100: $nodename:0: 'uart@...17100' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17200: $nodename:0: 'uart@...17200' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17300: $nodename:0: 'uart@...17300' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17400: $nodename:0: 'uart@...17400' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17500: $nodename:0: 'uart@...17500' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17600: $nodename:0: 'uart@...17600' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17700: $nodename:0: 'uart@...17700' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dtb: uart@...17800: $nodename:0: 'uart@...17800' does not match '^serial(@.*)?$'
from schema $id: http://devicetree.org/schemas/serial/8250.yaml#
Powered by blists - more mailing lists