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]

Message-ID: <20240627215338.qrry6zpkusw5nazw@skbuf>
Date: Fri, 28 Jun 2024 00:53:38 +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 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.