[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <176295563530.1637905.10146284162033916464.robh@kernel.org>
Date: Wed, 12 Nov 2025 08:26:53 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Ryan Chen <ryan_chen@...eedtech.com>
Cc: Conor Dooley <conor+dt@...nel.org>, nfraprado@...labora.com,
jk@...econstruct.com.au, linux-kernel@...r.kernel.org,
Joel Stanley <joel@....id.au>, devicetree@...r.kernel.org,
Taniya Das <quic_tdas@...cinc.com>, linux-aspeed@...ts.ozlabs.org,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Andrew Jeffery <andrew@...econstruct.com.au>, bmc-sw@...eedtech.com,
Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>,
Will Deacon <will@...nel.org>, Lee Jones <lee@...nel.org>,
Bjorn Andersson <bjorn.andersson@....qualcomm.com>,
Geert Uytterhoeven <geert@...ux-m68k.org>, Arnd Bergmann <arnd@...db.de>,
Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
linux-arm-kernel@...ts.infradead.org,
Catalin Marinas <catalin.marinas@....com>,
Eric Biggers <ebiggers@...nel.org>, Nishanth Menon <nm@...com>
Subject: Re: [PATCH v8 0/4] Introduce ASPEED AST27xx BMC SoC
On Wed, 12 Nov 2025 18:11:53 +0800, Ryan Chen wrote:
> This introduces initial support for the Aspeed AST27xx 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:
> check with "make CHECK_DTBS=y W=1 arch/arm64/boot/dts/aspeed/ dtbs"
>
> - scu/silicon-id: Documentation/devicetree/bindings/mfd/aspeed,ast2x00-scu.yaml
> - gpio: Documentation/devicetree/bindings/gpio/aspeed,ast2400-gpio.yaml
> Binding dependency patch:
> - intc: Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2700-intc.yaml
> - https://lore.kernel.org/all/20251030060155.2342604-2-ryan_chen@aspeedtech.com/
> (merged into the irq/drivers)
>
> v8:
> - Kconfig.platforms
> - update commit message and help description.
> - aspeed-g7-a35.dtsi,aspeed-g7-common.dtsi,ast2700-evb.dts
> - change license using a dual license of MIT + GPL2.0+
> - add company copyright.
> - merge original v7 patch(3/5) and (4/5) to 1 v8patch(3/4)
> - that can do build test with make CHECK_DTBS=y W=1 arch/arm64/boot/dts/aspeed/ dtbs
>
> v7:
> - remove aspeed,ast2x00-scu.yaml modify
> - separate aspeed-g7.dtsi to aspeed-g7-a35.dtsi and aspeed-g7-common.dtsi
> - move aliases to ast2700-evb.dts file
>
> v6:
> - rebased on v6.18-rc1
> - aspeed,ast2x00-scu.yaml
> - fixed dt-binding yaml issuse report.
>
> 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 (4):
> dt-bindings: arm: aspeed: Add AST2700 board compatible
> arm64: Kconfig: Add ASPEED SoC family Kconfig support
> arm64: dts: aspeed: Add initial AST27xx SoC device tree
> 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-a35.dtsi | 154 ++++++++
> .../boot/dts/aspeed/aspeed-g7-common.dtsi | 351 ++++++++++++++++++
> arch/arm64/boot/dts/aspeed/ast2700-evb.dts | 31 ++
> arch/arm64/configs/defconfig | 1 +
> 8 files changed, 554 insertions(+)
> create mode 100644 arch/arm64/boot/dts/aspeed/Makefile
> create mode 100644 arch/arm64/boot/dts/aspeed/aspeed-g7-a35.dtsi
> create mode 100644 arch/arm64/boot/dts/aspeed/aspeed-g7-common.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.18-rc3-15-gbe755eb2b021 (exact match)
Base: tags/v6.18-rc3-15-gbe755eb2b021 (use --merge-base to override)
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 20251112101157.2149169-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: 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: [[7, 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: [[7, 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: [[7, 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: [[7, 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: [[7, 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: [[7, 5]] is too short
from schema $id: http://devicetree.org/schemas/interrupt-controller/aspeed,ast2700-intc.yaml
Powered by blists - more mailing lists