Re: [PATCH 1/4] dt-bindings: memory: mediatek: Correct the minItems of clk for larbs
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 > > > --- > > > .../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 ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH 1/4] dt-bindings: memory: mediatek: Correct the minItems of clk for larbs
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 > > --- > > .../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@1601: 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@1601: 'mediatek,larb-id' is a required property > arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml > > larb@1701: 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-
Re: [PATCH 1/4] dt-bindings: memory: mediatek: Correct the minItems of clk for larbs
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 > --- > .../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@1601: 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@1601: 'mediatek,larb-id' is a required property arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dt.yaml larb@1701: 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/
[PATCH 1/4] dt-bindings: memory: mediatek: Correct the minItems of clk for larbs
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 --- .../bindings/memory-controllers/mediatek,smi-larb.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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: -- 2.18.0 ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu