On Fri, 01 Apr 2022 09:12:47 -0500, 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@kernel.org Cc: Thierry Reding thierry.reding@gmail.com Cc: Jonathan Hunter jonathanh@nvidia.com Cc: Mauro Carvalho Chehab mchehab@kernel.org Cc: Charles Keepax ckeepax@opensource.cirrus.com Cc: Linus Walleij linus.walleij@linaro.org Cc: Sebastian Reichel sre@kernel.org Cc: Tony Lindgren tony@atomide.com Cc: Yunfei Dong yunfei.dong@mediatek.com Cc: - patches@opensource.cirrus.com Cc: linux-media@vger.kernel.org Cc: alsa-devel@alsa-project.org Cc: linux-gpio@vger.kernel.org Cc: linux-pm@vger.kernel.org Signed-off-by: Rob Herring robh@kernel.org
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/20220401030847epcms1p8cf7a8e1d8cd7d325dacf30f78da3...
.../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(-)
Applied, thanks!