Re: [PATCH v2 6/8] spi: dt-bindings: cdns,xspi: add ADI SC846 compatible

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1256 bytes --]

On Thu, Nov 12, 2026 at 09:41:17AM +0000, Nuno Sá wrote:
> On Thu, Nov 12, 2026 at 09:32:36AM +0000, Nuno Sá wrote:
> > On Wed, Nov 11, 2026 at 05:58:09PM +0000, Conor Dooley wrote:
> > > On Wed, Nov 11, 2026 at 01:23:07PM +0000, Nuno Sá wrote:
> > > > The Analog Devices ADSP-SC846 SoC integrates the Cadence xSPI controller
> > > > with its own PHY timing configuration and with the support needed for
> > > > the auto command (ACMD) mode with the internal DMA, so it cannot be
> > > > described by the generic "cdns,xspi-nor" compatible. Add a SoC specific
> > > > compatible for it.
> > > 
> > > I assume it's the phy timing that means the cdns,xspi-nor compatible
> > > cannot be used as a fallback?
> > 
> > Exactly. Without proper phy settings we cannot probe the memory chips.
> > There are some works being done to support phy tuning but even when
> > that lands, not sure how generic it could be supported.
> > 
> > Hmm, I guess you want the above mentioned in the commit right? 
> 
> Nevermind, it's already in there.

ye I just wanted to clarify, because it wasn't entirely clear that this
made it incompatible since I'm not a domain expert.

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]