linux-kernel - Re: [PATCH v3 0/6] Introduce ASPEED AST27XX BMC SoC

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5d91b7a7169e6db27b2493213d0df2b77699a49e.camel@codeconstruct.com.au>
Date: Fri, 13 Dec 2024 15:37:52 +1030
From: Andrew Jeffery <andrew@...econstruct.com.au>
To: Kevin Chen <kevin_chen@...eedtech.com>, robh@...nel.org,
 krzk+dt@...nel.org,  conor+dt@...nel.org, joel@....id.au,
 tglx@...utronix.de, catalin.marinas@....com,  will@...nel.org,
 arnd@...db.de, olof@...om.net, quic_bjorande@...cinc.com, 
 geert+renesas@...der.be, dmitry.baryshkov@...aro.org,
 konradybcio@...nel.org,  neil.armstrong@...aro.org,
 johan+linaro@...nel.org, devicetree@...r.kernel.org, 
 linux-arm-kernel@...ts.infradead.org, linux-aspeed@...ts.ozlabs.org, 
 linux-kernel@...r.kernel.org, soc@...ts.linux.dev
Subject: Re: [PATCH v3 0/6] Introduce ASPEED AST27XX BMC SoC

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