Re: [PATCH v11 0/5] arm64: add Ambarella CV75 SoC support

[Brian Masney]

Hi Long / Krzysztof / Conor / Rob,

> Add the Ambarella CV75 SoC and EVK support: the board binding, the
> RCT clock binding and driver, ARCH_AMBARELLA, the devicetree and
> MAINTAINERS.
> 
> The clock ID header is added before the devicetree that includes it,
> so patch 2 (the clock binding) has to be applied before patch 4 (the
> devicetree).
> 
> The ambarella vendor prefix and the CV75 secure scratchpad syscon
> binding are already in linux-next. This series does not repeat
> either of them.
> 
> GPIO, UART and pinctrl bindings and drivers are separate series.
> This devicetree references those compatibles.
> 
> Tested on the CV75 EVK. The board boots. That boot used a local tree
> that also includes the GPIO, UART and pinctrl patches.
> 
> To: Arnd Bergmann <arnd@xxxxxxxx>
> To: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> To: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>
> To: soc@xxxxxxxxxxxxxxx
> To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> To: Michael Turquette <mturquette@xxxxxxxxxxxx>
> To: Stephen Boyd <sboyd@xxxxxxxxxx>
> Cc: Jerome Brunet <jbrunet@xxxxxxxxxxxx>
> Cc: linux-clk@xxxxxxxxxxxxxxx
> Cc: Brian Masney <masneyb@xxxxxxxxxx>
> Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Cc: Rob Herring <robh@xxxxxxxxxx>
> Cc: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> Cc: Conor Dooley <conor+dt@xxxxxxxxxx>
> Cc: Catalin Marinas <catalin.marinas@xxxxxxx>
> Cc: Will Deacon <will@xxxxxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: Long Zhao <longzhao@xxxxxxxxxxxxx>
> Cc: Long Zhao <zl020895@xxxxxxx>
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> 
> Signed-off-by: Long Zhao <longzhao@xxxxxxxxxxxxx>
> ---
> Changes in v11:
> - clk: use builtin_platform_driver(). The Kconfig symbol is bool. (Brian)
> - clk: drop CLK_IS_CRITICAL from the core PLL. It has no enable op. (Brian)
> - clk: drop the misleading (u64) cast in the div_u64() call. (Brian)
> - clk: add .suppress_bind_attrs = true to block sysfs unbind. (Brian)
> - clk: drop MODULE_AUTHOR. (Brian)
> - Link to v10:
>   https://patch.msgid.link/20261019-cv75-soc-v10-0-d386065e2dd5@xxxxxxxxxxxxx
> 
> ---
> Long Zhao (5):
>       dt-bindings: arm: add Ambarella CV75 platforms
>       dt-bindings: clock: add Ambarella CV75 RCT
>       clk: ambarella: add CV75 RCT clock controller
>       arm64: ambarella: add ARCH_AMBARELLA and CV75 EVK DT
>       MAINTAINERS: add ARM/AMBARELLA SoC support

The clk changes look good to me.

Before I start to merge things, and since I'm a new maintainer here, I
just want to make sure that I am doing this correctly, and have the
expected merge strategy for everyone involved.

- I create a new shared immutable branch based on v7.4-rc1 and I can put
  the first two dt-bindings patches in that. I merge this into clk-pile
  (feeds clk-next) with --no-ff. The DTS folks will merge this immutable
  branch as well.

- I can then merge patches 3 and 5 into clk-pile.

- The DTS folks will take patch 4.

Sound good?

Brian

-- 
Brian Masney <masneyb@xxxxxxxxxx>