Skip to content

Commit 50605d2

Browse files
miquelraynalbroonie
authored andcommitted
spi: dt-bindings: cdns,qspi-nor: Require some peripheral properties
There are 5 mandatory peripheral properties. They are described in a separate binding but not explicitly required. Make sure they are correctly marked required and update the example to reflect this. Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> Reviewed-by: Rob Herring (Arm) <robh@kernel.org> Link: https://patch.msgid.link/20250319094651.1290509-4-miquel.raynal@bootlin.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent 8b35d65 commit 50605d2

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

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

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,18 @@ properties:
146146
items:
147147
enum: [ qspi, qspi-ocp, rstc_ref ]
148148

149+
patternProperties:
150+
"^flash@[0-9a-f]+$":
151+
type: object
152+
$ref: cdns,qspi-nor-peripheral-props.yaml
153+
additionalProperties: true
154+
required:
155+
- cdns,read-delay
156+
- cdns,tshsl-ns
157+
- cdns,tsd2d-ns
158+
- cdns,tchsh-ns
159+
- cdns,tslch-ns
160+
149161
required:
150162
- compatible
151163
- reg
@@ -177,5 +189,10 @@ examples:
177189
flash@0 {
178190
compatible = "jedec,spi-nor";
179191
reg = <0x0>;
192+
cdns,read-delay = <4>;
193+
cdns,tshsl-ns = <60>;
194+
cdns,tsd2d-ns = <60>;
195+
cdns,tchsh-ns = <60>;
196+
cdns,tslch-ns = <60>;
180197
};
181198
};

0 commit comments

Comments
 (0)