| Message ID | 20211203064027.14993-2-yong.wu@mediatek.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | MT8186 SMI SUPPORT | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 8 lines checked |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Fri, 03 Dec 2021 14:40:24 +0800, Yong Wu wrote: > If a platform's larb support gals, there will be some larbs have a one > more "gals" clock while the others still only need "apb"/"smi" clocks. > then the minItems is 2 and the maxItems is 3. > > Fixes: 27bb0e42855a ("dt-bindings: memory: mediatek: Convert SMI to DT schema") > Signed-off-by: Yong Wu <yong.wu@mediatek.com> > --- > .../bindings/memory-controllers/mediatek,smi-larb.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > 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/1563127 larb@14016000: 'mediatek,larb-id' is a required property arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml larb@14017000: clock-names: ['apb', 'smi'] is too short arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml larb@15001000: 'mediatek,larb-id' is a required property arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml larb@16010000: clock-names: ['apb', 'smi'] is too short arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml larb@16010000: 'mediatek,larb-id' is a required property arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml larb@17010000: clock-names: ['apb', 'smi'] is too short arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku1.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml
On Fri, 2021-12-03 at 17:34 -0600, Rob Herring wrote: > On Fri, 03 Dec 2021 14:40:24 +0800, Yong Wu wrote: > > If a platform's larb support gals, there will be some larbs have a > > one > > more "gals" clock while the others still only need "apb"/"smi" > > clocks. > > then the minItems is 2 and the maxItems is 3. > > > > Fixes: 27bb0e42855a ("dt-bindings: memory: mediatek: Convert SMI to > > DT schema") > > Signed-off-by: Yong Wu <yong.wu@mediatek.com> > > --- > > .../bindings/memory-controllers/mediatek,smi-larb.yaml | > > 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > 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/1563127 > > > larb@14016000: 'mediatek,larb-id' is a required property > arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml I will fix this in next version. This property is not needed in mt8167. > > larb@14017000: clock-names: ['apb', 'smi'] is too short > arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > burnet.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > fennel14.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku6.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml Some larbs only have two clocks(apb/smi) in mt8183. thus it is reasonable for me. I won't fix this in next version. Please tell me if I miss something. Thanks. > > larb@15001000: 'mediatek,larb-id' is a required property > arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml > > larb@16010000: clock-names: ['apb', 'smi'] is too short > arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > burnet.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > fennel14.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku6.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml > > larb@16010000: 'mediatek,larb-id' is a required property > arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml > > larb@17010000: clock-names: ['apb', 'smi'] is too short > arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > burnet.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > fennel14.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > sku6.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > sku1.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml > arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml >
On Mon, Dec 13, 2021 at 02:48:52PM +0800, Yong Wu wrote: > On Fri, 2021-12-03 at 17:34 -0600, Rob Herring wrote: > > On Fri, 03 Dec 2021 14:40:24 +0800, Yong Wu wrote: > > > If a platform's larb support gals, there will be some larbs have a > > > one > > > more "gals" clock while the others still only need "apb"/"smi" > > > clocks. > > > then the minItems is 2 and the maxItems is 3. > > > > > > Fixes: 27bb0e42855a ("dt-bindings: memory: mediatek: Convert SMI to > > > DT schema") > > > Signed-off-by: Yong Wu <yong.wu@mediatek.com> > > > --- > > > .../bindings/memory-controllers/mediatek,smi-larb.yaml | > > > 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > 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/1563127 > > > > > > larb@14016000: 'mediatek,larb-id' is a required property > > arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml > > I will fix this in next version. This property is not needed in mt8167. > > > > > larb@14017000: clock-names: ['apb', 'smi'] is too short > > arch/arm64/boot/dts/mediatek/mt8183-evb.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > > burnet.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi- > > fennel14.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > sku1.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel- > > sku6.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper- > > sku16.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > sku0.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow- > > sku1.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml > > arch/arm64/boot/dts/mediatek/mt8183-pumpkin.dt.yaml > > Some larbs only have two clocks(apb/smi) in mt8183. thus it is > reasonable for me. I won't fix this in next version. You also need to adjust 'clock-names' to allow for 2 items. Rob
diff --git a/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml index eaeff1ada7f8..a1402f3b8344 100644 --- a/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml +++ b/Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml @@ -81,7 +81,7 @@ allOf: properties: clock: items: - minItems: 3 + minItems: 2 maxItems: 3 clock-names: items:
If a platform's larb support gals, there will be some larbs have a one more "gals" clock while the others still only need "apb"/"smi" clocks. then the minItems is 2 and the maxItems is 3. Fixes: 27bb0e42855a ("dt-bindings: memory: mediatek: Convert SMI to DT schema") Signed-off-by: Yong Wu <yong.wu@mediatek.com> --- .../bindings/memory-controllers/mediatek,smi-larb.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)