| Message ID | 20230820175655.206723-1-aford173@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/3] ASoC: dt-bindings: fsl_easrc: Add support for imx8mp-easrc | expand |
On Sun, 20 Aug 2023 12:56:53 -0500, Adam Ford wrote: > The i.MX8MP appears to have the same easrc support as the Nano, so > add imx8mp as an option with a fallback to imx8mn. > > Signed-off-by: Adam Ford <aford173@gmail.com> > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/sound/fsl,easrc.example.dtb: easrc@300c0000: compatible: 'oneOf' conditional failed, one must be fixed: ['fsl,imx8mn-easrc'] is too short 'fsl,imx8mn-easrc' is not one of ['fsl,imx8mp-easrc'] from schema $id: http://devicetree.org/schemas/sound/fsl,easrc.yaml# doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230820175655.206723-1-aford173@gmail.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 20/08/2023 19:56, Adam Ford wrote: > The i.MX8MP appears to have the same easrc support as the Nano, so > add imx8mp as an option with a fallback to imx8mn. > > Signed-off-by: Adam Ford <aford173@gmail.com> > > diff --git a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > index bdde68a1059c..2d53b3b10f2c 100644 > --- a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > +++ b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > @@ -14,7 +14,11 @@ properties: > pattern: "^easrc@.*" > > compatible: > - const: fsl,imx8mn-easrc > + oneOf: > + - items: > + - enum: > + - fsl,imx8mp-easrc > + - const: fsl,imx8mn-easrc You need here also const for fsl,imx8mn-easrc, otherwise you do not allow it alone. Test it for fsl,imx8mn-easrc DTS - you will notice warnings. Best regards, Krzysztof
On Sun, Aug 20, 2023 at 3:33 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > On 20/08/2023 22:32, Krzysztof Kozlowski wrote: > > On 20/08/2023 19:56, Adam Ford wrote: > >> The i.MX8MP appears to have the same easrc support as the Nano, so > >> add imx8mp as an option with a fallback to imx8mn. > >> > >> Signed-off-by: Adam Ford <aford173@gmail.com> > >> > >> diff --git a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > >> index bdde68a1059c..2d53b3b10f2c 100644 > >> --- a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > >> +++ b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > >> @@ -14,7 +14,11 @@ properties: > >> pattern: "^easrc@.*" > >> > >> compatible: > >> - const: fsl,imx8mn-easrc > >> + oneOf: > >> + - items: > >> + - enum: > >> + - fsl,imx8mp-easrc > >> + - const: fsl,imx8mn-easrc > > > > You need here also const for fsl,imx8mn-easrc, otherwise you do not > > allow it alone. Test it for fsl,imx8mn-easrc DTS - you will notice warnings. > > Actually, I see now Rob's report... you did not have to test DTS even. > It was enough to test your change and this test was missing :(. Please > test your changes before sending. For what it's worth, I did run 'make dt_binding_check', but I didn't run it with the extra flags from Rob's e-mail. The tool didn't return any errors. adam > > Best regards, > Krzysztof >
diff --git a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml index bdde68a1059c..2d53b3b10f2c 100644 --- a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml +++ b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml @@ -14,7 +14,11 @@ properties: pattern: "^easrc@.*" compatible: - const: fsl,imx8mn-easrc + oneOf: + - items: + - enum: + - fsl,imx8mp-easrc + - const: fsl,imx8mn-easrc reg: maxItems: 1
The i.MX8MP appears to have the same easrc support as the Nano, so add imx8mp as an option with a fallback to imx8mn. Signed-off-by: Adam Ford <aford173@gmail.com>