linux-kernel - Re: [PATCH v3] dt-bindings: mtd: microchip,mchp23k256: convert to DT schema

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: <20260106-talented-hare-of-glee-fd1e63@quoll>
Date: Tue, 6 Jan 2026 08:49:15 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Akhila YS <akhilayalmati@...il.com>
Cc: Miquel Raynal <miquel.raynal@...tlin.com>, 
	Richard Weinberger <richard@....at>, Vignesh Raghavendra <vigneshr@...com>, 
	Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, 
	Conor Dooley <conor+dt@...nel.org>, linux-mtd@...ts.infradead.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] dt-bindings: mtd: microchip,mchp23k256: convert to DT
 schema

On Mon, Jan 05, 2026 at 03:08:00PM +0000, Akhila YS wrote:
> Convert Microchip 23K256 SPI SRAM MTD binding to YAML format.
> Changes during conversion:
> - Remove "address-cells" and "size-cells" from properties and
>   required sections as there is no child node for sram.

I looked at the driver and clearly see usage of partitions from DT, thus
the binding needed both above and the partitions coming from MTD schema,
so this is not really correct. Looks like you did not really try to
understand this binding and the driver.

Converting bindings is not trivial task and you must really understand
also what was incomplete in the binding.

Anyway, I don't want to deal with this more, so:

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@....qualcomm.com>

Best regards,
Krzysztof