On Sun, Jan 10, 2021 at 8:52 PM Henry Chen [off-list ref] wrote:

Hi Rob,

On Fri, 2021-01-08 at 20:30 -0700, Rob Herring wrote:

quoted

On Fri, 08 Jan 2021 15:48:03 +0800, Henry Chen wrote:

quoted

Document the binding for enabling dvfsrc on MediaTek SoC.

Signed-off-by: Henry Chen <redacted>
---
 .../devicetree/bindings/soc/mediatek/dvfsrc.yaml   | 67 ++++++++++++++++++++++
 1 file changed, 67 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/soc/mediatek/dvfsrc.yaml

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
Documentation/devicetree/bindings/soc/mediatek/dvfsrc.example.dts:19:18: fatal error: dt-bindings/interconnect/mtk,mt8183-emi.h: No such file or directory
   19 |         #include <dt-bindings/interconnect/mtk,mt8183-emi.h>
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Sorry, because this header file is added on "[V7,07/13] dt-bindings:
interconnect: add MT8183 interconnect dt-bindings".
Should I change the order of the patchset (let the yaml pathc behind the
header) to fixed that ?

DT headers should be part of the binding schema patch. (Or at least come first).

Rob

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel