There's no need for 'items' when there's only one item.
Fixes: 4a803990aeb1 ("dt-bindings: ARM: Mediatek: Add new document bindings of MT8192 clock")
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
.../arm/mediatek/mediatek,mt8192-clock.yaml | 45 +++++++++----------
1 file changed, 22 insertions(+), 23 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
index c8c67c033f8c..bb410b178f33 100644
--- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
+++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml
@@ -14,29 +14,28 @@ description:
properties:
compatible:
- items:
- - enum:
- - mediatek,mt8192-scp_adsp
- - mediatek,mt8192-imp_iic_wrap_c
- - mediatek,mt8192-imp_iic_wrap_e
- - mediatek,mt8192-imp_iic_wrap_s
- - mediatek,mt8192-imp_iic_wrap_ws
- - mediatek,mt8192-imp_iic_wrap_w
- - mediatek,mt8192-imp_iic_wrap_n
- - mediatek,mt8192-msdc_top
- - mediatek,mt8192-msdc
- - mediatek,mt8192-mfgcfg
- - mediatek,mt8192-imgsys
- - mediatek,mt8192-imgsys2
- - mediatek,mt8192-vdecsys_soc
- - mediatek,mt8192-vdecsys
- - mediatek,mt8192-vencsys
- - mediatek,mt8192-camsys
- - mediatek,mt8192-camsys_rawa
- - mediatek,mt8192-camsys_rawb
- - mediatek,mt8192-camsys_rawc
- - mediatek,mt8192-ipesys
- - mediatek,mt8192-mdpsys
+ enum:
+ - mediatek,mt8192-scp_adsp
+ - mediatek,mt8192-imp_iic_wrap_c
+ - mediatek,mt8192-imp_iic_wrap_e
+ - mediatek,mt8192-imp_iic_wrap_s
+ - mediatek,mt8192-imp_iic_wrap_ws
+ - mediatek,mt8192-imp_iic_wrap_w
+ - mediatek,mt8192-imp_iic_wrap_n
+ - mediatek,mt8192-msdc_top
+ - mediatek,mt8192-msdc
+ - mediatek,mt8192-mfgcfg
+ - mediatek,mt8192-imgsys
+ - mediatek,mt8192-imgsys2
+ - mediatek,mt8192-vdecsys_soc
+ - mediatek,mt8192-vdecsys
+ - mediatek,mt8192-vencsys
+ - mediatek,mt8192-camsys
+ - mediatek,mt8192-camsys_rawa
+ - mediatek,mt8192-camsys_rawb
+ - mediatek,mt8192-camsys_rawc
+ - mediatek,mt8192-ipesys
+ - mediatek,mt8192-mdpsys
reg:
maxItems: 1
--
2.35.1
On Tue, May 17, 2022 at 12:15:13PM +0200, AngeloGioacchino Del Regno wrote: > There's no need for 'items' when there's only one item. > > Fixes: 4a803990aeb1 ("dt-bindings: ARM: Mediatek: Add new document bindings of MT8192 clock") > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > .../arm/mediatek/mediatek,mt8192-clock.yaml | 45 +++++++++---------- > 1 file changed, 22 insertions(+), 23 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml > index c8c67c033f8c..bb410b178f33 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml > +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mt8192-clock.yaml > @@ -14,29 +14,28 @@ description: > > properties: > compatible: > - items: > - - enum: > - - mediatek,mt8192-scp_adsp > - - mediatek,mt8192-imp_iic_wrap_c > - - mediatek,mt8192-imp_iic_wrap_e > - - mediatek,mt8192-imp_iic_wrap_s > - - mediatek,mt8192-imp_iic_wrap_ws > - - mediatek,mt8192-imp_iic_wrap_w > - - mediatek,mt8192-imp_iic_wrap_n > - - mediatek,mt8192-msdc_top > - - mediatek,mt8192-msdc > - - mediatek,mt8192-mfgcfg > - - mediatek,mt8192-imgsys > - - mediatek,mt8192-imgsys2 > - - mediatek,mt8192-vdecsys_soc > - - mediatek,mt8192-vdecsys > - - mediatek,mt8192-vencsys > - - mediatek,mt8192-camsys > - - mediatek,mt8192-camsys_rawa > - - mediatek,mt8192-camsys_rawb > - - mediatek,mt8192-camsys_rawc > - - mediatek,mt8192-ipesys > - - mediatek,mt8192-mdpsys > + enum: > + - mediatek,mt8192-scp_adsp > + - mediatek,mt8192-imp_iic_wrap_c > + - mediatek,mt8192-imp_iic_wrap_e > + - mediatek,mt8192-imp_iic_wrap_s > + - mediatek,mt8192-imp_iic_wrap_ws > + - mediatek,mt8192-imp_iic_wrap_w > + - mediatek,mt8192-imp_iic_wrap_n > + - mediatek,mt8192-msdc_top > + - mediatek,mt8192-msdc > + - mediatek,mt8192-mfgcfg > + - mediatek,mt8192-imgsys > + - mediatek,mt8192-imgsys2 > + - mediatek,mt8192-vdecsys_soc > + - mediatek,mt8192-vdecsys > + - mediatek,mt8192-vencsys > + - mediatek,mt8192-camsys > + - mediatek,mt8192-camsys_rawa > + - mediatek,mt8192-camsys_rawb > + - mediatek,mt8192-camsys_rawc > + - mediatek,mt8192-ipesys > + - mediatek,mt8192-mdpsys This will conflict with the patch removing msdc: [1] https://lore.kernel.org/all/20220429123133.28869-1-matthias.bgg@kernel.org/ Although if this series is merged before that one, it should be fine. Thanks, Nícolas
On 17/05/2022 12:15, AngeloGioacchino Del Regno wrote: > There's no need for 'items' when there's only one item. > > Fixes: 4a803990aeb1 ("dt-bindings: ARM: Mediatek: Add new document bindings of MT8192 clock") The same - no bugs to fix here and this can be squashed. Best regards, Krzysztof
© 2016 - 2024 Red Hat, Inc.