mbox series

[v3,0/6] Introduce ASPEED AST27XX BMC SoC

Message ID 20241212155237.848336-1-kevin_chen@aspeedtech.com (mailing list archive)
Headers show
Series Introduce ASPEED AST27XX BMC SoC | expand

Message

Kevin Chen Dec. 12, 2024, 3:52 p.m. UTC
---
v3:
  - 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.

Kevin Chen (6):
  dt-bindings: interrupt-controller: Refine size/interrupt-cell usage.
  dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
  arm64: aspeed: Add support for ASPEED AST27XX BMC SoC
  arm64: dts: aspeed: Add initial AST27XX device tree
  arm64: dts: aspeed: Add initial AST2700 EVB device tree
  arm64: defconfig: Add ASPEED AST2700 family support

 .../bindings/arm/aspeed/aspeed.yaml           |   6 +
 .../aspeed,ast2700-intc.yaml                  |  60 ++++-
 MAINTAINERS                                   |   1 +
 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     | 236 ++++++++++++++++++
 arch/arm64/boot/dts/aspeed/ast2700-evb.dts    |  57 +++++
 arch/arm64/configs/defconfig                  |   1 +
 9 files changed, 359 insertions(+), 13 deletions(-)
 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

Comments

Andrew Jeffery Dec. 13, 2024, 5:07 a.m. UTC | #1
Hi Kevin,

On Thu, 2024-12-12 at 23:52 +0800, Kevin Chen wrote:
> ---
> v3:
>   - 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.
> 
> Kevin Chen (6):
>   dt-bindings: interrupt-controller: Refine size/interrupt-cell
> usage.
>   dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
>   arm64: aspeed: Add support for ASPEED AST27XX BMC SoC
>   arm64: dts: aspeed: Add initial AST27XX device tree
>   arm64: dts: aspeed: Add initial AST2700 EVB device tree
>   arm64: defconfig: Add ASPEED AST2700 family support

Do you mind sending a v4, because I received a confusing arrangement of
patches:

[PATCH v3 1/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
[PATCH v3 1/6] dt-bindings: interrupt-controller: Refine size/interrupt-cell usage.
[PATCH v3 2/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
[PATCH v3 2/6] dt-bindings: interrupt-controller: Fix the size-cells in ast2700-intc
...

Where the content of 

   [PATCH v3 1/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
   
and

   [PATCH v3 2/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
   
Differs, as does the content of

   [PATCH v3 1/6] dt-bindings: interrupt-controller: Refine size/interrupt-cell usage.
   
and

   [PATCH v3 2/6] dt-bindings: interrupt-controller: Fix the size-cells in ast2700-intc

Despite sounding like they might have the same intent

Andrew
Kevin Chen Dec. 16, 2024, 3:34 a.m. UTC | #2
Hi,

> > ---
> > v3:
> >   - 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.
> >
> > Kevin Chen (6):
> >   dt-bindings: interrupt-controller: Refine size/interrupt-cell usage.
> >   dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
> >   arm64: aspeed: Add support for ASPEED AST27XX BMC SoC
> >   arm64: dts: aspeed: Add initial AST27XX device tree
> >   arm64: dts: aspeed: Add initial AST2700 EVB device tree
> >   arm64: defconfig: Add ASPEED AST2700 family support
> 
> Do you mind sending a v4, because I received a confusing arrangement of
> patches:
OK. I put a mixed commits in our send-mail server. I will try to send v4 patches.
Thanks for your reminder.

> 
> [PATCH v3 1/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC [PATCH v3
> 1/6] dt-bindings: interrupt-controller: Refine size/interrupt-cell usage.
> [PATCH v3 2/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC [PATCH v3
> 2/6] dt-bindings: interrupt-controller: Fix the size-cells in ast2700-intc ...
> 
> Where the content of
> 
>    [PATCH v3 1/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
> 
> and
> 
>    [PATCH v3 2/6] dt-bindings: arm: aspeed: Add ASPEED AST27XX SoC
> 
> Differs, as does the content of
> 
>    [PATCH v3 1/6] dt-bindings: interrupt-controller: Refine size/interrupt-cell
> usage.
> 
> and
> 
>    [PATCH v3 2/6] dt-bindings: interrupt-controller: Fix the size-cells in
> ast2700-intc
> 
> Despite sounding like they might have the same intent
> 
> Andrew