Re: [PATCH v3 06/14] dt-bindings: pinctrl: qcom: Add MSM8917 pinctrl
From: Krzysztof Kozlowski
Date: Fri Nov 08 2024 - 07:24:26 EST
On Thu, Nov 07, 2024 at 06:02:47PM +0100, Barnabás Czémán wrote:
> +$defs:
> + qcom-msm8917-tlmm-state:
> + type: object
> + description:
> + Pinctrl node's client devices use subnodes for desired pin configuration.
> + Client device subnodes use below standard properties.
> + $ref: qcom,tlmm-common.yaml#/$defs/qcom-tlmm-state
> + unevaluatedProperties: false
> +
> + properties:
> + pins:
> + description:
> + List of gpio pins affected by the properties specified in this
> + subnode.
> + items:
> + oneOf:
> + - pattern: "^gpio([0-9]|[1-9][0-9]|1[0-3][0-3])$"
That's not a good pattern, unless really gpio129 is not existing?
> + - enum: [ sdc1_clk, sdc1_cmd, sdc1_data, sdc1_rclk, sdc2_clk,
> + sdc2_cmd, sdc2_data, qdsd_clk, qdsd_cmd, qdsd_data0,
> + qdsd_data1, qdsd_data2, qdsd_data3 ]
> + minItems: 1
> + maxItems: 16
Best regards,
Krzysztof