| Message ID | 20221118162101.145267-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | dt-bindings: pinctrl: qcom,sdm845-pinctrl: add GPIO hogs | expand |
On Fri, Nov 18, 2022 at 05:21:01PM +0100, Krzysztof Kozlowski wrote: > Allow GPIO hogs in pin controller node. qcom/sdm845-cheza.dtsi already > uses it. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/pinctrl/qcom,sdm845-pinctrl.yaml | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml > index c9627777ceb3..57bac7f7a4fc 100644 > --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml > +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml > @@ -48,6 +48,11 @@ patternProperties: > $ref: "#/$defs/qcom-sdm845-tlmm-state" > additionalProperties: false > > + "-hog(-[0-9]+)?$": > + $ref: /schemas/gpio/gpio-hog.yaml You can drop this and just do: required: - gpio-hog As that's what selects the hog schema. Kind of an odd pattern I guess. We could make gpio.yaml define hog nodes instead I suppose. > + type: object > + unevaluatedProperties: false > +
On 20/11/2022 17:41, Rob Herring wrote: > On Fri, Nov 18, 2022 at 05:21:01PM +0100, Krzysztof Kozlowski wrote: >> Allow GPIO hogs in pin controller node. qcom/sdm845-cheza.dtsi already >> uses it. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> --- >> .../bindings/pinctrl/qcom,sdm845-pinctrl.yaml | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml >> index c9627777ceb3..57bac7f7a4fc 100644 >> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml >> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml >> @@ -48,6 +48,11 @@ patternProperties: >> $ref: "#/$defs/qcom-sdm845-tlmm-state" >> additionalProperties: false >> >> + "-hog(-[0-9]+)?$": >> + $ref: /schemas/gpio/gpio-hog.yaml > > You can drop this and just do: > > required: > - gpio-hog > > As that's what selects the hog schema. Kind of an odd pattern I guess. > We could make gpio.yaml define hog nodes instead I suppose. Thanks, that's much less code. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml index c9627777ceb3..57bac7f7a4fc 100644 --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml @@ -48,6 +48,11 @@ patternProperties: $ref: "#/$defs/qcom-sdm845-tlmm-state" additionalProperties: false + "-hog(-[0-9]+)?$": + $ref: /schemas/gpio/gpio-hog.yaml + type: object + unevaluatedProperties: false + $defs: qcom-sdm845-tlmm-state: type: object @@ -117,6 +122,7 @@ additionalProperties: false examples: - | + #include <dt-bindings/gpio/gpio.h> #include <dt-bindings/interrupt-controller/arm-gic.h> pinctrl@3400000 { @@ -130,6 +136,12 @@ examples: gpio-ranges = <&tlmm 0 0 151>; wakeup-parent = <&pdc_intc>; + ap-suspend-l-hog { + gpio-hog; + gpios = <126 GPIO_ACTIVE_LOW>; + output-low; + }; + cci0-default-state { pins = "gpio17", "gpio18"; function = "cci_i2c";
Allow GPIO hogs in pin controller node. qcom/sdm845-cheza.dtsi already uses it. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../bindings/pinctrl/qcom,sdm845-pinctrl.yaml | 12 ++++++++++++ 1 file changed, 12 insertions(+)