[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <175678731287.878052.15508784585002236677.robh@kernel.org>
Date: Tue, 02 Sep 2025 08:18:18 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Ryan Chen <ryan_chen@...eedtech.com>
Cc: soc@...ts.linux.dev, Conor Dooley <conor+dt@...nel.org>,
dkodihalli@...dia.com, Arnd Bergmann <arnd@...db.de>,
Andrew Jeffery <andrew@...econstruct.com.au>,
Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
nfraprado@...labora.com, linux-arm-kernel@...ts.infradead.org,
Will Deacon <will@...nel.org>, Nishanth Menon <nm@...com>,
Yuxiao Zhang <yuxiaozhang@...gle.com>, Joel Stanley <joel@....id.au>,
Eric Biggers <ebiggers@...gle.com>, Taniya Das <quic_tdas@...cinc.com>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, spuranik@...dia.com,
Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>,
linux-kernel@...r.kernel.org,
Bjorn Andersson <bjorn.andersson@....qualcomm.com>,
linux-aspeed@...ts.ozlabs.org, Rom Lemarchand <romlem@...gle.com>,
Geert Uytterhoeven <geert@...ux-m68k.org>, Mo Elbadry <elbadrym@...gle.com>,
leohu@...dia.com, Catalin Marinas <catalin.marinas@....com>,
devicetree@...r.kernel.org, William Kennington <wak@...gle.com>,
wthai@...dia.com
Subject: Re: [PATCH v5 0/5] Introduce ASPEED AST2700 BMC SoC
On Mon, 01 Sep 2025 11:13:06 +0800, Ryan Chen wrote:
> This introduces initial support for the Aspeed AST2700 SoC and the AST2700
> Evaluation Board (EVB) to the Linux kernel. The AST27XX is the 8th
> generation Baseboard Management Controller (BMC) SoC from Aspeed,
> featuring improved performance, enhanced security, and expanded I/O
> capabilities compared to previous generations.
>
> AST27XX SOC Family
> - https://www.aspeedtech.com/server_ast2700/
> - https://www.aspeedtech.com/server_ast2720/
> - https://www.aspeedtech.com/server_ast2750/
>
> Bindings Dependencies:
> - intc-ic: Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2700-intc.yaml
> - scu/silicon-id: Documentation/devicetree/bindings/mfd/aspeed,ast2x00-scu.yaml
> - gpio: Documentation/devicetree/bindings/gpio/aspeed,ast2400-gpio.yaml
> - mdio: Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
>
> v5:
> - modify ast27XX 7th generation description to 8th generation.
> - aspeed.yaml
> - modify missing blank line.
> - Kconfig.platforms
> - modify ast27XX 7th generation to 8th generation.
>
> v4:
> - make CHECK_DTBS=y arch/arm64/boot/dts/aspeed/ fix.
> - modify commit message remove itemlize.
> - remove modify aspeed,ast2700-intc.yaml patch.
> - aspeed.yaml
> - Add AST2700 board compatible.
> - aspeed-g7.dtsi
> - modify all size-cells from 1 to 2.
> - add serial aliases, gpio, mdio, uart0 ~ 14.
> - add firmware for optee, reserved memory for atf and optee.
> - modify cpu@0 to cpu0: cpu@0.
> - fix intc-ic for yaml dependency.
> - ast2700-evb.dts
> - update stdout-path = "serial12:115200n8";
>
> v3:
> - https://lore.kernel.org/all/20241212155237.848336-1-kevin_chen@aspeedtech.com/
> - Split clk and reset driver to other commits, which are in series of
> "Add support for AST2700 clk driver".
> - For BMC console by UART12, add uart12 using ASPEED INTC architecture.
>
> aspeed,ast2700-intc.yaml
> - Add minItems to 1 to fix the warning by "make dtbs_check W=1".
> - Add intc1 into example.
>
> Kconfig.platforms
> - Remove MACH_ASPEED_G7.
>
> Ryan Chen (5):
> dt-bindings: arm: aspeed: Add AST2700 board compatible
> arm64: Kconfig: Add Aspeed SoC family (ast27XX) Kconfig support
> arm64: dts: aspeed: Add initial AST2700 SoC device tree
> arm64: dts: aspeed: Add AST2700 Evaluation Board
> arm64: configs: Update defconfig for AST2700 platform support
>
> .../bindings/arm/aspeed/aspeed.yaml | 6 +
> arch/arm64/Kconfig.platforms | 6 +
> arch/arm64/boot/dts/Makefile | 1 +
> arch/arm64/boot/dts/aspeed/Makefile | 4 +
> arch/arm64/boot/dts/aspeed/aspeed-g7.dtsi | 452 ++++++++++++++++++
> arch/arm64/boot/dts/aspeed/ast2700-evb.dts | 22 +
> arch/arm64/configs/defconfig | 1 +
> 7 files changed, 492 insertions(+)
> create mode 100644 arch/arm64/boot/dts/aspeed/Makefile
> create mode 100644 arch/arm64/boot/dts/aspeed/aspeed-g7.dtsi
> create mode 100644 arch/arm64/boot/dts/aspeed/ast2700-evb.dts
>
> --
> 2.34.1
>
>
>
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
This patch series was applied (using b4) to base:
Base: attempting to guess base-commit...
Base: tags/v6.17-rc1 (exact match)
If this is not the correct base, please add 'base-commit' tag
(or use b4 which does this automatically)
New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/aspeed/' for 20250901031311.1247805-1-ryan_chen@...eedtech.com:
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...01b00 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...01b00 (aspeed,ast2700-intc-ic): interrupts: [[0, 192, 4], [0, 193, 4], [0, 194, 4], [0, 195, 4], [0, 196, 4], [0, 197, 4], [0, 198, 4], [0, 199, 4], [0, 200, 4], [0, 201, 4]] is too long
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: syscon@...02000 (aspeed,ast2700-scu0): #size-cells: 1 was expected
from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: syscon@...02000 (aspeed,ast2700-scu1): compatible: ['aspeed,ast2700-scu1'] is too short
from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: syscon@...02000 (aspeed,ast2700-scu1): 'ranges' is a required property
from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: syscon@...02000 (aspeed,ast2700-scu1): '#address-cells' is a required property
from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: syscon@...02000 (aspeed,ast2700-scu1): '#size-cells' is a required property
from schema $id: http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18100 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18100 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 0]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18110 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18110 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 1]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18120 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18120 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 2]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18130 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18130 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 3]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18140 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18140 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 4]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18150 (aspeed,ast2700-intc-ic): #interrupt-cells: 2 was expected
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
arch/arm64/boot/dts/aspeed/ast2700-evb.dtb: interrupt-controller@...18150 (aspeed,ast2700-intc-ic): interrupts-extended: [[5, 5]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml#
Powered by blists - more mailing lists