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

[Brian Masney]

Hi Arnd / krzk,

On Tue, Nov 10, 2026 at 04:11:52PM +0200, Andy Shevchenko wrote:
> On Tue, Nov 10, 2026 at 07:24:19AM -0500, Brian Masney wrote:
> > On Tue, Nov 10, 2026 at 09:46:29AM +0100, Arnd Bergmann wrote:
> > > On Tue, Nov 10, 2026, at 09:27, Krzysztof Kozlowski wrote:
> > > > On 10/11/2026 09:23, Arnd Bergmann wrote:
> 
> ...
> 
> > > > There is a build dependency - DTS needs clock binding
> > > > header - so it has to be second, unless the dependency
> > > > is decoupled.
> > > 
> > > Ok, I see. Let's go with Brian's suggestion then.
> > 
> > I see that the pinctrl driver was merged into v7.4-rc1 and also
> > lists ARCH_AMBARELLA. It doesn't cause any build warnings or issues, so
> > I'm fine with this too just as long as ARCH_AMBARELLA is going to appear
> > in the tree.
> 
> We have a bug about dangling / orphaned symbols in Kconfig. It's discouraged to
> have them. I dunno if we get the checker patch being applied (it was some
> activity a few months ago).

OK that makes sense.

So how about this:

- Have Long post a new version that splits out the creation of the new
  Kconfig symbol ARCH_AMBARELLA into it's own patch. I assume Arnd could
  provide an ACK on this?

- I create an immutable branch that contains that the two dt-binding
  patches, plus the ARCH_AMBARELLA Kconfig patch.

- Then I can merge the clk driver cleanly. The DTS folks can take the
  DTS changes.

If the DTS folks didn't want the ARCH_AMBARELLA in the immutable branch,
then we could do two immutable branches:
one for ARCH_AMBARELLA from SoC folks, and one from me for the
dt-bindings.

Alternatively, I could just provide an Acked-by for the clk driver, and
this could just go through the SoC tree.

Brian