Re: [PATCH v3 2/4] dt-bindings: phy: qcom,usb-hs-phy: Add compatible

From: Krzysztof Kozlowski
Date: Sun Jun 04 2023 - 06:30:52 EST


On 04/06/2023 08:30, Rudraksha Gupta wrote:
> Adds qcom,usb-hs-phy-msm8960 compatible
>
> Signed-off-by: Rudraksha Gupta <guptarud@xxxxxxxxx>
> ---
> Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.yaml | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.yaml
> index aa97478dd016..63b6914993fe 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.yaml
> @@ -13,7 +13,9 @@ if:
> properties:
> compatible:
> contains:
> - const: qcom,usb-hs-phy-apq8064
> + items:
> + - const: qcom,usb-hs-phy-apq8064
> + - const: qcom,usb-hs-phy-msm8960

I have no clue what you want to achieve here. You break the bindings.

It does not look like you tested the bindings, at least after quick
look. Please run `make dt_binding_check` (see
Documentation/devicetree/bindings/writing-schema.rst for instructions).
Maybe you need to update your dtschema and yamllint.

Best regards,
Krzysztof