Re: [PATCH 1/3] spi: dt-bindings: cdns,qspi-nor: Be more descriptive regarding what this controller is

[Krzysztof Kozlowski <krzk@xxxxxxxxxx>]

On Wed, Mar 05, 2025 at 09:01:31PM +0100, Miquel Raynal wrote:
> Despite being very common in commit logs, SPI NOR controllers simply do
> not exist. At least, they are not as specific as the name implies. There
> are SPI memory controllers which are indeed "specialized" and optimized
> for handling "memories", but most of them are just generic and accept
> almost any kind of opcode, address, dummy and data cycles, making them
> as suitable for NANDs than NORs.
> 
> Furthermore, this controller supports any kind of bus, from single to
> octal NAND, so make it clear.
> 
> Also add a comment to mention that the initial compatible naming is too
> specific (but obviously kept for backward compatibility reasons).
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Best regards,
Krzysztof