| Message ID | 20221107204355.31971-1-Sergey.Semin@baikalelectronics.ru |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | dt-bindings: opp-v2: Fix clock-latency-ns prop in example | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 8 lines checked |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 07-11-22, 23:43, Serge Semin wrote: > Accidentally discovered a hidden typo in the DT-nodes defined in the > opp-v2 bindings example. Instead of specifying the "clock-latency-ns" > property the DT-node has been created with the "lock-latency-ns" property > in it, which doesn't exist neither in opp-v2 nor in the base schemas. > Let's fix the name to having the "clock-" prefix as it was originally > implied and as the rest of the similar nodes has. > > Fixes: 94274f20f6bf ("dt-bindings: opp: Convert to DT schema") > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> Applied. Thanks.
diff --git a/Documentation/devicetree/bindings/opp/opp-v2.yaml b/Documentation/devicetree/bindings/opp/opp-v2.yaml index eaf8fba2c691..2f05920fce79 100644 --- a/Documentation/devicetree/bindings/opp/opp-v2.yaml +++ b/Documentation/devicetree/bindings/opp/opp-v2.yaml @@ -155,7 +155,7 @@ examples: opp-hz = /bits/ 64 <1200000000>; opp-microvolt = <1025000>; opp-microamp = <90000>; - lock-latency-ns = <290000>; + clock-latency-ns = <290000>; turbo-mode; }; };
Accidentally discovered a hidden typo in the DT-nodes defined in the opp-v2 bindings example. Instead of specifying the "clock-latency-ns" property the DT-node has been created with the "lock-latency-ns" property in it, which doesn't exist neither in opp-v2 nor in the base schemas. Let's fix the name to having the "clock-" prefix as it was originally implied and as the rest of the similar nodes has. Fixes: 94274f20f6bf ("dt-bindings: opp: Convert to DT schema") Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> --- Note unevaluatedProperties flag hasn't revealed the problem here most likely due to the property having the "-ns" suffix which implies the property being single-number. It is implicitly evaluated by the dt-schema core thus preventing the unevaluatedProperties schema to detect the inconsistency. --- Documentation/devicetree/bindings/opp/opp-v2.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)