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
