Re: [PATCH v12 12/38] dt-bindings: pwm: Add Cirrus EP93xx

Uwe Kleine-König
2024-09-09T11:41:04Z

[-- Attachment #1: Type: text/plain, Size: 1442 bytes --]

On Mon, Sep 09, 2024 at 04:45:07AM -0500, Rob Herring (Arm) wrote:
>
> On Mon, 09 Sep 2024 11:10:37 +0300, Nikita Shubin wrote:
> > Add YAML bindings for ep93xx SoC PWM.
> >
> > Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me>
> > Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > ---
> > .../devicetree/bindings/pwm/cirrus,ep9301-pwm.yaml | 53 ++++++++++++++++++++++
> > 1 file changed, 53 insertions(+)
> >
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dts:18:18: fatal error: dt-bindings/clock/cirrus,ep9301-syscon.h: No such file or directory
> 18 | #include <dt-bindings/clock/cirrus,ep9301-syscon.h>
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> compilation terminated.
> make[2]: *** [scripts/Makefile.lib:442: Documentation/devicetree/bindings/pwm/cirrus,ep9301-pwm.example.dtb] Error 1
> make[2]: *** Waiting for unfinished jobs....
> make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1432: dt_binding_check] Error 2
> make: *** [Makefile:224: __sub-make] Error 2

The file that was claimed to be missing is added in patch #6 of this
series. So I don't think this error is valid.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]