linux-kernel - RE: [EXT] Re: [PATCH 4/7] spi: cadence: Change dt-bindings documentation for Cadence XSPI 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] [thread-next>] [day] [month] [year] [list]

Date: Mon, 29 Apr 2024 15:13:13 +0000
From: Witold Sadowski <wsadowski@...vell.com>
To: Krzysztof Kozlowski <krzk@...nel.org>,
        "broonie@...nel.org"
	<broonie@...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>,
        "jpawar@...ence.com" <jpawar@...ence.com>,
        "pthombar@...ence.com"
	<pthombar@...ence.com>,
        "konrad@...ence.com" <konrad@...ence.com>,
        Wojciech
 Bartczak <wbartczak@...vell.com>,
        Wojciech Zmuda <wzmuda@...vell.com>
Subject: RE: [EXT] Re: [PATCH 4/7] spi: cadence: Change dt-bindings
 documentation for Cadence XSPI controller

> ----------------------------------------------------------------------
> On 19/12/2022 15:42, Witold Sadowski wrote:
> > Add parameter cdns,read-size.
> > Parameter is controlling SDMA read size length.
> 
> Use subject prefixes matching the subsystem (git log --oneline -- ...).
> 
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC.  It might happen, that command when run on an older
> kernel, gives you outdated entries.  Therefore please be sure you base
> your patches on recent Linux kernel.
> 
> >
> > Signed-off-by: Witold Sadowski <wsadowski@...vell.com>
> > ---
> >  Documentation/devicetree/bindings/spi/cdns,xspi.yaml | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/spi/cdns,xspi.yaml
> > b/Documentation/devicetree/bindings/spi/cdns,xspi.yaml
> > index f71a9c74e2ca..1274e3bf68e6 100644
> > --- a/Documentation/devicetree/bindings/spi/cdns,xspi.yaml
> > +++ b/Documentation/devicetree/bindings/spi/cdns,xspi.yaml
> > @@ -37,6 +37,10 @@ properties:
> >    interrupts:
> >      maxItems: 1
> >
> > +  cdns,read-size:
> > +    items:
> > +      - description: size of single SDMA read operation
> 
> Why is this a property of DT?

Removed, read size check will be based on matching.
The problem is internal bus connection to xSPI IP.

> 
> > +
> >  required:
> >    - compatible
> >    - reg
> > @@ -60,6 +64,7 @@ examples:
> >              reg-names = "io", "sdma", "aux";
> >              interrupts = <0 90 IRQ_TYPE_LEVEL_HIGH>;
> >              interrupt-parent = <&gic>;
> > +            cdns,read-size=<0>;
> 
> That's not DT coding style.
> 
> >
> >              flash@0 {
> >                  compatible = "jedec,spi-nor";
> 
> Best regards,
> Krzysztof

Regards
Witek