| Message ID | 20230621230958.3815818-1-robh@kernel.org |
|---|---|
| State | Accepted |
| Commit | a454850a815e62fa5d7c1eded0e8d56742613b94 |
| Headers | show |
| Series | dt-bindings: phy: brcm,brcmstb-usb-phy: Fix error in "compatible" conditional schema | expand |
On 22.06.2023 01:09, Rob Herring wrote: > The conditional if/then schema has an error as the "enum" values have > "const" in them. Drop the "const". > > Signed-off-by: Rob Herring <robh@kernel.org> Oops, it's my mistake. Fixes: 46b616c1574d ("dt-bindings: phy: brcm, brcmstb-usb-phy: add BCM4908 binding") Acked-by: Rafał Miłecki <rafal@milecki.pl> Thanks! > --- > .../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml b/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml > index 43a4b880534c..580fbe37b37f 100644 > --- a/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml > +++ b/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml > @@ -115,8 +115,8 @@ allOf: > compatible: > contains: > enum: > - - const: brcm,bcm4908-usb-phy > - - const: brcm,brcmstb-usb-phy > + - brcm,bcm4908-usb-phy > + - brcm,brcmstb-usb-phy > then: > properties: > reg:
On 21-06-23, 17:09, Rob Herring wrote: > The conditional if/then schema has an error as the "enum" values have > "const" in them. Drop the "const". Applied, thanks
diff --git a/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml b/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml index 43a4b880534c..580fbe37b37f 100644 --- a/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml +++ b/Documentation/devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml @@ -115,8 +115,8 @@ allOf: compatible: contains: enum: - - const: brcm,bcm4908-usb-phy - - const: brcm,brcmstb-usb-phy + - brcm,bcm4908-usb-phy + - brcm,brcmstb-usb-phy then: properties: reg:
The conditional if/then schema has an error as the "enum" values have "const" in them. Drop the "const". Signed-off-by: Rob Herring <robh@kernel.org> --- .../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)