Re: [PATCH] dt-bindings: Use 'enum' instead of 'oneOf' plus 'const' entries

From: Lee Jones
Date: Mon Sep 06 2021 - 05:35:23 EST


On Tue, 24 Aug 2021, Rob Herring wrote:

> 'enum' is equivalent to 'oneOf' with a list of 'const' entries, but 'enum'
> is more concise and yields better error messages.
>
> Cc: Vinod Koul <vkoul@xxxxxxxxxx>
> Cc: Maxime Ripard <mripard@xxxxxxxxxx>
> Cc: Vignesh R <vigneshr@xxxxxx>
> Cc: Marc Zyngier <maz@xxxxxxxxxx>
> Cc: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: Lee Jones <lee.jones@xxxxxxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
> Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> Cc: Kishon Vijay Abraham I <kishon@xxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Cc: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> Cc: dmaengine@xxxxxxxxxxxxxxx
> Cc: linux-i2c@xxxxxxxxxxxxxxx
> Cc: linux-media@xxxxxxxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx
> Cc: linux-pci@xxxxxxxxxxxxxxx
> Cc: linux-phy@xxxxxxxxxxxxxxxxxxx
> Cc: linux-serial@xxxxxxxxxxxxxxx
> Cc: alsa-devel@xxxxxxxxxxxxxxxx
> Cc: linux-spi@xxxxxxxxxxxxxxx
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> .../bindings/display/msm/dsi-phy-10nm.yaml | 6 +++---
> .../bindings/display/msm/dsi-phy-14nm.yaml | 6 +++---
> .../bindings/display/msm/dsi-phy-28nm.yaml | 8 ++++----
> .../bindings/dma/allwinner,sun6i-a31-dma.yaml | 12 ++++++------
> .../devicetree/bindings/firmware/arm,scpi.yaml | 6 +++---
> .../devicetree/bindings/i2c/ti,omap4-i2c.yaml | 10 +++++-----
> .../interrupt-controller/loongson,liointc.yaml | 8 ++++----
> .../devicetree/bindings/media/i2c/mipi-ccs.yaml | 8 ++++----
> .../devicetree/bindings/mfd/ti,lp87565-q1.yaml | 6 +++---

Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>

> .../devicetree/bindings/net/realtek-bluetooth.yaml | 8 ++++----
> .../bindings/net/ti,k3-am654-cpsw-nuss.yaml | 8 ++++----
> .../devicetree/bindings/net/ti,k3-am654-cpts.yaml | 6 +++---
> Documentation/devicetree/bindings/pci/loongson.yaml | 8 ++++----
> .../devicetree/bindings/phy/intel,lgm-emmc-phy.yaml | 6 +++---
> .../devicetree/bindings/serial/8250_omap.yaml | 9 +++++----
> .../devicetree/bindings/sound/qcom,sm8250.yaml | 6 +++---
> .../devicetree/bindings/sound/tlv320adcx140.yaml | 8 ++++----
> .../devicetree/bindings/spi/realtek,rtl-spi.yaml | 12 ++++++------
> .../devicetree/bindings/timer/arm,sp804.yaml | 6 +++---
> 19 files changed, 74 insertions(+), 73 deletions(-)

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog