| Message ID | 20230305225901.7119-2-stefan.wahren@i2se.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | ARM: dts: imx6ull: Fix dtbs_check warnings | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 05/03/2023 23:58, Stefan Wahren wrote: > Currently the dtbs_check for imx6 generates warnings like this: > > 'fsl,imx6sl-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp'] > ['fsl,imx6sl-dcp', 'fsl,imx28-dcp'] is too long > > or > > 'fsl,imx6ull-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp'] > ['fsl,imx6ull-dcp', 'fsl,imx28-dcp'] is too long > > So add them to the devicetree binding. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > --- > Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml > index 99be01539fcd..1695c4c58dc8 100644 > --- a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml > +++ b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml > @@ -11,9 +11,14 @@ maintainers: > > properties: > compatible: > - enum: > - - fsl,imx23-dcp > - - fsl,imx28-dcp > + oneOf: > + - const: fsl,imx23-dcp > + - const: fsl,imx28-dcp Keep these two as enum (so just indent under oneOf). > + - items: > + - enum: > + - fsl,imx6sl-dcp > + - fsl,imx6ull-dcp > + - const: fsl,imx28-dcp > > reg: > maxItems: 1 Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml index 99be01539fcd..1695c4c58dc8 100644 --- a/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml +++ b/Documentation/devicetree/bindings/crypto/fsl-dcp.yaml @@ -11,9 +11,14 @@ maintainers: properties: compatible: - enum: - - fsl,imx23-dcp - - fsl,imx28-dcp + oneOf: + - const: fsl,imx23-dcp + - const: fsl,imx28-dcp + - items: + - enum: + - fsl,imx6sl-dcp + - fsl,imx6ull-dcp + - const: fsl,imx28-dcp reg: maxItems: 1
Currently the dtbs_check for imx6 generates warnings like this: 'fsl,imx6sl-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp'] ['fsl,imx6sl-dcp', 'fsl,imx28-dcp'] is too long or 'fsl,imx6ull-dcp' is not one of ['fsl,imx23-dcp', 'fsl,imx28-dcp'] ['fsl,imx6ull-dcp', 'fsl,imx28-dcp'] is too long So add them to the devicetree binding. Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> --- Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-)