linux-kernel - RE: [PATCH] spi: dt-bindings: cdns,qspi-nor: Update minItems/maxItems of resets for Cadence OSPI controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <IA0PR12MB7699ABC4D20D02AED632C012DCA02@IA0PR12MB7699.namprd12.prod.outlook.com>
Date: Fri, 28 Mar 2025 13:42:47 +0000
From: "Mahapatra, Amit Kumar" <amit.kumar-mahapatra@....com>
To: Krzysztof Kozlowski <krzk@...nel.org>, "broonie@...nel.org"
	<broonie@...nel.org>, "robh@...nel.org" <robh@...nel.org>,
	"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "conor+dt@...nel.org"
	<conor+dt@...nel.org>
CC: "linux-spi@...r.kernel.org" <linux-spi@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "git
 (AMD-Xilinx)" <git@....com>, "amitrkcian2002@...il.com"
	<amitrkcian2002@...il.com>
Subject: RE: [PATCH] spi: dt-bindings: cdns,qspi-nor: Update minItems/maxItems
 of resets for Cadence OSPI controller

[AMD Official Use Only - AMD Internal Distribution Only]

Hello Krzysztof,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@...nel.org>
> Sent: Wednesday, March 26, 2025 5:51 PM
> To: Mahapatra, Amit Kumar <amit.kumar-mahapatra@....com>;
> broonie@...nel.org; robh@...nel.org; krzk+dt@...nel.org; conor+dt@...nel.org
> Cc: linux-spi@...r.kernel.org; devicetree@...r.kernel.org; linux-
> kernel@...r.kernel.org; git (AMD-Xilinx) <git@....com>;
> amitrkcian2002@...il.com
> Subject: Re: [PATCH] spi: dt-bindings: cdns,qspi-nor: Update minItems/maxItems of
> resets for Cadence OSPI controller
>
> On 26/03/2025 12:37, Amit Kumar Mahapatra wrote:
> > The Cadence Octal SPI (OSPI) controller on AMD Versal SoCs requires
> > only one reset entry. To reflect this, the maxItems for "resets" and
> > "reset-names" has been set to 1 for AMD Versal SoCs, and the minItems
> > for these properties has also been updated to 1. Additionally, these
> > properties have been added to the required property list for Versal SoCs.
> >
> > Signed-off-by: Amit Kumar Mahapatra <amit.kumar-mahapatra@....com>
> > ---
> > BRANCH: mtd/next
> > ---
> >  .../devicetree/bindings/spi/cdns,qspi-nor.yaml     | 14 ++++++++++++--
> >  1 file changed, 12 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> > b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> > index d48ecd6cd5ad..cc94c59280a1 100644
> > --- a/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> > +++ b/Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml
> > @@ -17,8 +17,18 @@ allOf:
> >            contains:
> >              const: xlnx,versal-ospi-1.0
> >      then:
> > +      properties:
> > +        resets:
> > +          maxItems: 1
> > +
> > +        reset-names:
> > +          maxItems: 1
> > +          items:
> > +            enum: [ qspi ]
>
> Just list the items instead of these three lines.

Do you mean something like this?
        reset-names:
          items:
            - const: [ qspi ]
>
> >        required:
> >          - power-domains
> > +        - resets
> > +        - reset-names
>
> Why? That's an ABI break.

Went through the driver, and believe these properties do not need to be in
the required list. I will revert this change in my next series.
>
>
> >    - if:
> >        properties:
> >          compatible:
> > @@ -132,11 +142,11 @@ properties:
> >      maxItems: 1
> >
> >    resets:
> > -    minItems: 2
> > +    minItems: 1
>
> You need to update other variants now.

Are you referring to other SoCs when you mention other variants?

Regards,
Amit
>
> Best regards,
> Krzysztof