linux-kernel - Re: [PATCH 1/1] spi: dt-bindings: fsl-dspi: add dmas and dma-names properties

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: Fri, 28 Jun 2024 01:13:42 +0300
From: Vladimir Oltean <olteanv@...il.com>
To: Frank Li <Frank.li@....com>
Cc: Mark Brown <broonie@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	"open list:FREESCALE DSPI DRIVER" <linux-spi@...r.kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>,
	open list <linux-kernel@...r.kernel.org>, imx@...ts.linux.dev
Subject: Re: [PATCH 1/1] spi: dt-bindings: fsl-dspi: add dmas and dma-names
 properties

On Thu, Jun 27, 2024 at 06:07:08PM -0400, Frank Li wrote:
> On Fri, Jun 28, 2024 at 12:53:38AM +0300, Vladimir Oltean wrote:
> > On Thu, Jun 27, 2024 at 04:33:08PM -0400, Frank Li wrote:
> > > Add dmas and dma-names properties because dspi support dma transfer.
> > > Fix below warnings:
> > > arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var1.dtb: spi@...0000: Unevaluated properties are not allowed ('dma-names', 'dmas', 'little-endian' were unexpected)
> > >         from schema $id: http://devicetree.org/schemas/spi/fsl,dspi.yaml#
> > > 
> > > Signed-off-by: Frank Li <Frank.Li@....com>
> > > ---
> > 
> > For the contents:
> > 
> > Reviewed-by: Vladimir Oltean <olteanv@...il.com>
> > 
> > and FWIW, I noticed this as well, but didn't want to put yet another
> > roadblock in the conversion of a binding which was way overdue.
> > I was planning to send a patch as well, once the binding was merged.
> > 
> > But... to whose tree is this targeted? I only got a notification
> > from Shawn for patch 3/3 (the device tree change), but none for the
> > acceptance of the other 2 patches.
> 
> This patch base on the below patch:
> https://lore.kernel.org/imx/20240624-ls_qspi-v4-0-3d1c6f5005bf@nxp.com/T/#t
> 
> Sorry, I miss understand 3 patches already picked. Actually only pick 3rd
> one.
> 
> Frank
> 

Yup. I guess it's best to resend a new version of the (reduced) series
to the SPI tree now, with this patch rolled up into the dt-schema
conversion, and the review tags preserved.

But you might want to wait at least 24 hours since you sent this patch,
to allow people a chance to respond.