Skip to content

Commit 77289a8

Browse files
miquelraynalbroonie
authored andcommitted
spi: dt-bindings: cdns,qspi-nor: Be more descriptive regarding what this controller is
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@bootlin.com> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Link: https://patch.msgid.link/20250319094651.1290509-2-miquel.raynal@bootlin.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent 48303ef commit 77289a8

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Documentation/devicetree/bindings/spi/cdns,qspi-nor.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
$id: http://devicetree.org/schemas/spi/cdns,qspi-nor.yaml#
55
$schema: http://devicetree.org/meta-schemas/core.yaml#
66

7-
title: Cadence Quad SPI controller
7+
title: Cadence Quad/Octal SPI controller
88

99
maintainers:
1010
- Vaishnav Achath <vaishnav.a@ti.com>
@@ -76,6 +76,9 @@ properties:
7676
- ti,am654-ospi
7777
- ti,k2g-qspi
7878
- xlnx,versal-ospi-1.0
79+
# The compatible is qspi-nor for historical reasons but such
80+
# controllers are meant to be used with flashes of all kinds,
81+
# ie. also NAND flashes, not only NOR flashes.
7982
- const: cdns,qspi-nor
8083
- const: cdns,qspi-nor
8184

0 commit comments

Comments
 (0)