Re: [PATCH] dt-bindings: Fix 'enum' lists with duplicate entries
From: Krzysztof Kozlowski
Date: Sat Apr 02 2022 - 07:40:06 EST
On 01/04/2022 16:12, Rob Herring wrote:
> There's no reason to list the same value twice in an 'enum'. Fix all the
> occurrences in the tree. A meta-schema change will catch future ones.
>
> Cc: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
> Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> Cc: Jonathan Hunter <jonathanh@xxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Cc: Sebastian Reichel <sre@xxxxxxxxxx>
> Cc: Tony Lindgren <tony@xxxxxxxxxxx>
> Cc: Yunfei Dong <yunfei.dong@xxxxxxxxxxxx>
> Cc: - <patches@xxxxxxxxxxxxxxxxxxxxx>
> Cc: linux-media@xxxxxxxxxxxxxxx
> Cc: alsa-devel@xxxxxxxxxxxxxxxx
> Cc: linux-gpio@xxxxxxxxxxxxxxx
> Cc: linux-pm@xxxxxxxxxxxxxxx
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> There's also one other occurrence in snps,dwmac.yaml I didn't fix as
> there's a patch[1] for it which prompted this patch.
>
> Rob
>
> [1] https://lore.kernel.org/r/20220401030847epcms1p8cf7a8e1d8cd7d325dacf30f78da36328@epcms1p8
>
> .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 1 -
> Documentation/devicetree/bindings/bus/ti-sysc.yaml | 1 -
> .../bindings/media/mediatek,vcodec-encoder.yaml | 1 -
> .../devicetree/bindings/pinctrl/cirrus,madera.yaml | 11 +++++------
> .../devicetree/bindings/power/supply/bq2415x.yaml | 1 -
> 5 files changed, 5 insertions(+), 10 deletions(-)
>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Best regards,
Krzysztof