| Message ID | 20200823161550.3981-7-krzk@kernel.org |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | [01/22] dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 10 lines checked |
The subject contains a "imx-pwm", presumably from the last patch. Sascha On Sun, Aug 23, 2020 at 06:15:35PM +0200, Krzysztof Kozlowski wrote: > DTSes with new i.MX 8M SoCs introduce their own compatibles so add them > to fix dtbs_check warnings like: > > arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: > compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm'] > From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml > > arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: > compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long > > arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: > compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > index cba3f83ccd5f..3d896173b3b0 100644 > --- a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > +++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > @@ -36,6 +36,10 @@ properties: > - fsl,imx6sx-uart > - fsl,imx6ul-uart > - fsl,imx7d-uart > + - fsl,imx8mm-uart > + - fsl,imx8mn-uart > + - fsl,imx8mp-uart > + - fsl,imx8mq-uart > - const: fsl,imx6q-uart > > reg: > -- > 2.17.1 > >
On Mon, Aug 24, 2020 at 07:46:01AM +0200, Sascha Hauer wrote: > > The subject contains a "imx-pwm", presumably from the last patch. Ah, copy-paste. Thanks for pointing this out. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml index cba3f83ccd5f..3d896173b3b0 100644 --- a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml +++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml @@ -36,6 +36,10 @@ properties: - fsl,imx6sx-uart - fsl,imx6ul-uart - fsl,imx7d-uart + - fsl,imx8mm-uart + - fsl,imx8mn-uart + - fsl,imx8mp-uart + - fsl,imx8mq-uart - const: fsl,imx6q-uart reg:
DTSes with new i.MX 8M SoCs introduce their own compatibles so add them to fix dtbs_check warnings like: arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: compatible:0: 'fsl,imx8mm-pwm' is not one of ['fsl,imx1-pwm', 'fsl,imx27-pwm'] From schema: Documentation/devicetree/bindings/pwm/imx-pwm.yaml arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: compatible: ['fsl,imx8mm-pwm', 'fsl,imx27-pwm'] is too long arch/arm64/boot/dts/freescale/imx8mm-evk.dt.yaml: pwm@30660000: compatible: Additional items are not allowed ('fsl,imx27-pwm' was unexpected) Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml | 4 ++++ 1 file changed, 4 insertions(+)