Re: [PATCH v3 02/13] spi: dt-bindings: cdns,qspi-nor: add PHY tuning pattern partition property

[Santhosh Kumar K]

On 01/06/26 17:17, Krzysztof Kozlowski wrote:
> On Mon, Jun 01, 2026 at 01:26:27PM +0200, Krzysztof Kozlowski wrote:
>   >>
> > > > This should be used in any of applicable examples.
> > >
> > > Please refer to the DT changes included in the repository linked in the
> > > cover letter, which should provide a clearer picture of how this
> > > property is intended to be used. [0]
> > >
> > > I'll also add a DT binding example in the next version to make the usage
> > > more explicit.
> >
> > Why do you new a new example? Why existing examples are not applicable?
>
> typo: "Why do you need a new example?"

No, I  wasn't planning to add a new example. I'll extend one of the
existing binding examples.

> > > > I also have doubts that this is per-device property. Your commit msg
> > > > suggests it is per controller.
> > >
> > > This is a per-device property. It is consumed by the controller driver
> > > only to locate and retrieve the offset of the PHY pattern partition
> >
> > So with two devices on a bus, you need two separate partitions for tuning?

Each SPI NOR flash device needs a partition to store PHY tuning pattern.

Regards,
Santhosh.

> Best regards,
> Krzysztof