Dear Rob, Thanks for your comments~ On Thu, 2021-11-11 at 08:57 -0600, Rob Herring wrote: > On Thu, 11 Nov 2021 15:12:13 +0800, Biao Huang wrote: > > Convert mediatek-dwmac to DT schema, and delete old mediatek- > > dwmac.txt. > > > > Signed-off-by: Biao Huang <biao.huang@xxxxxxxxxxxx> > > --- > > .../bindings/net/mediatek-dwmac.txt | 91 -------- > > .../bindings/net/mediatek-dwmac.yaml | 211 > > ++++++++++++++++++ > > 2 files changed, 211 insertions(+), 91 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/net/mediatek- > > dwmac.txt > > create mode 100644 Documentation/devicetree/bindings/net/mediatek- > > dwmac.yaml > > > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for > dtbs_check. > This will change in the future. > > Full log is available here: > https://patchwork.ozlabs.org/patch/1553803 > > > ethernet@1101c000: clock-names: ['axi', 'apb', 'mac_main', 'ptp_ref'] > is too short > arch/arm64/boot/dts/mediatek/mt2712-evb.dt.yaml > > ethernet@1101c000: clocks: [[27, 34], [27, 37], [6, 154], [6, 155]] > is too short > arch/arm64/boot/dts/mediatek/mt2712-evb.dt.yaml > > ethernet@1101c000: compatible: ['mediatek,mt2712-gmac'] does not > contain items matching the given schema > arch/arm64/boot/dts/mediatek/mt2712-evb.dt.yaml > > ethernet@1101c000: compatible: 'oneOf' conditional failed, one must > be fixed: > arch/arm64/boot/dts/mediatek/mt2712-evb.dt.yaml > Yes, I should add a dts related patch to fix this issue in next send.