diff mbox series

[2/9] dt-bindings: arm: qcom: fix Longcheer L8150 compatibles

Message ID 20220520123252.365762-2-krzysztof.kozlowski@linaro.org
State Accepted
Commit 25d203d0751ca191301bc578ba5d59fa401f1fbf
Headers show
Series [1/9] dt-bindings: arm: qcom: fix Alcatel OneTouch Idol 3 compatibles | expand

Commit Message

Krzysztof Kozlowski May 20, 2022, 12:32 p.m. UTC 
The MSM8916 Longcheer L8150 uses a fallback in compatible:

  msm8916-longcheer-l8150.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
    ['longcheer,l8150', 'qcom,msm8916-v1-qrd/9-v1', 'qcom,msm8916'] is too long

Fixes: b72160fa886d ("dt-bindings: qcom: Document bindings for new MSM8916 devices")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Stephan Gerhold May 20, 2022, 6:17 p.m. UTC | #1 
On Fri, May 20, 2022 at 02:32:45PM +0200, Krzysztof Kozlowski wrote:
> The MSM8916 Longcheer L8150 uses a fallback in compatible:
> 
>   msm8916-longcheer-l8150.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
>     ['longcheer,l8150', 'qcom,msm8916-v1-qrd/9-v1', 'qcom,msm8916'] is too long
> 
> Fixes: b72160fa886d ("dt-bindings: qcom: Document bindings for new MSM8916 devices")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Stephan Gerhold <stephan@gerhold.net>

> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index b7dd61df7ec0..e15012035093 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -160,11 +160,15 @@ properties:
>        - items:
>            - enum:
>                - alcatel,idol347
> -              - longcheer,l8150
>                - samsung,a3u-eur
>                - samsung,a5u-eur
>            - const: qcom,msm8916
>  
> +      - items:
> +          - const: longcheer,l8150
> +          - const: qcom,msm8916-v1-qrd/9-v1
> +          - const: qcom,msm8916
> +
>        - items:
>            - enum:
>                - sony,karin_windy
> -- 
> 2.32.0
>
Rob Herring (Arm) May 26, 2022, 2:14 a.m. UTC | #2 
On Fri, 20 May 2022 14:32:45 +0200, Krzysztof Kozlowski wrote:
> The MSM8916 Longcheer L8150 uses a fallback in compatible:
> 
>   msm8916-longcheer-l8150.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
>     ['longcheer,l8150', 'qcom,msm8916-v1-qrd/9-v1', 'qcom,msm8916'] is too long
> 
> Fixes: b72160fa886d ("dt-bindings: qcom: Document bindings for new MSM8916 devices")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index b7dd61df7ec0..e15012035093 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -160,11 +160,15 @@  properties:
       - items:
           - enum:
               - alcatel,idol347
-              - longcheer,l8150
               - samsung,a3u-eur
               - samsung,a5u-eur
           - const: qcom,msm8916
 
+      - items:
+          - const: longcheer,l8150
+          - const: qcom,msm8916-v1-qrd/9-v1
+          - const: qcom,msm8916
+
       - items:
           - enum:
               - sony,karin_windy