Re: [PATCH 2/3] spi: dt-bindings: Add num-cs property for mpfs-spi

From: Prajna.Rajendrakumar
Date: Tue May 07 2024 - 03:58:35 EST


On Fri, 2024-05-03 at 16:46 +0200, Krzysztof Kozlowski wrote:
> [Some people who received this message don't often get email from
> krzk@xxxxxxxxxx. Learn why this is important at
> https://aka.ms/LearnAboutSenderIdentification ]
>
> EXTERNAL EMAIL: Do not click links or open attachments unless you
> know the content is safe
>
> On 03/05/2024 14:54, Prajna.Rajendrakumar@xxxxxxxxxxxxx wrote:
> > > > + - if:
> > > > + properties:
> > > > + compatible:
> > > > + contains:
> > > > + const: microchip,mpfs-spi
> > > > + not:
> > > > + required:
> > > > + - cs-gpios
> > >
> > > I don't understand what you are expressing here. Did you actually
> > > validate it that it achieves exactly what you want?
> >
> > Since the controller supports only one chip select, the num-cs
> > should
> > default to 1 and cannot exceed 1 unless GPIOs are used as chip
> > selects.
>
> That's not really the answer... or you want to say you did not test
> it?
>
I overlooked mentioning this in my previous reply. It's been thoroughly
validated.

> Best regards,
> Krzysztof
>
Best regards,
Prajna