On Fri, 2022-04-29 at 15:06 -0500, Rob Herring wrote: > On Fri, Apr 29, 2022 at 09:55:37AM +0800, Rex-BC Chen wrote: > > On Thu, 2022-04-28 at 15:33 -0500, Rob Herring wrote: > > > On Thu, 28 Apr 2022 21:37:50 +0800, Rex-BC Chen wrote: > > > > From: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx> > > > > > > > > Convert mediatek,dsi.txt to mediatek,dsi.yaml format > > > > > > > > Signed-off-by: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx> > > > > Signed-off-by: Rex-BC Chen <rex-bc.chen@xxxxxxxxxxxx> > > > > --- > > > > .../display/mediatek/mediatek,dsi.txt | 62 --------- > > > > .../display/mediatek/mediatek,dsi.yaml | 122 > > > > ++++++++++++++++++ > > > > 2 files changed, 122 insertions(+), 62 deletions(-) > > > > delete mode 100644 > > > > Documentation/devicetree/bindings/display/mediatek/mediatek,dsi > > > > .txt > > > > create mode 100644 > > > > Documentation/devicetree/bindings/display/mediatek/mediatek,dsi > > > > .yam > > > > l > > > > > > > > > > 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://urldefense.com/v3/__https://patchwork.ozlabs.org/patch/__;!!CTRNKA9wMg0ARbw!wKbRsUmeUS_4mtOwj1t30buVNEilHYYhsUmEd5MvZ7P9VyDXg6cikERof47mkwETQzFL$ > > > > > > > > > > > > dsi@1400c000: compatible: ['mediatek,mt7623-dsi', > > > 'mediatek,mt2701- > > > dsi'] is too long > > > arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dtb > > > arch/arm/boot/dts/mt7623n-rfb-emmc.dtb > > > > > > dsi@14014000: #address-cells:0:0: 2 was expected > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14- > > > sku2.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku6.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku7.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > > > sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dtb > > > > > > dsi@14014000: 'port' is a required property > > > arch/arm64/boot/dts/mediatek/mt8183-evb.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14- > > > sku2.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku6.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku7.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > > > sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dtb > > > > > > dsi@14014000: #size-cells:0:0: 2 was expected > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-cozmo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14- > > > sku2.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku6.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > > sku7.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > > > sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > > sku1.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dtb > > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dtb > > > > > > dsi@1401b000: 'port' is a required property > > > arch/arm64/boot/dts/mediatek/mt8173-elm.dtb > > > arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dtb > > > arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dtb > > > > > > > Hello Rob, > > > > Thanks for your comments. > > The purpose of this series is not to fix dts for previous SoCs. > > Therefore, if there is a chance, we could send another series to > > fix > > them. > > Conversions often find that the actual dts files vary a bit more > than > the binding doc said. You should look at the warnings and decide if > they > should be fixed or the schema relaxed. It's a judgement call. I have > no > idea if you did that already or not, so I send this out on > conversions. > The check runs automatically, but sending it I review briefly. > > Rob Hello Rob, Thanks for your explanation! In addition, do you have any suggestion for this conversion? BRs, Rex