* [PATCH v3] dt-bindings: pinctrl: cy8x95x0: Minor fix & update
@ 2024-01-30 12:56 Naresh Solanki
2024-01-31 8:32 ` Krzysztof Kozlowski
0 siblings, 1 reply; 3+ messages in thread
From: Naresh Solanki @ 2024-01-30 12:56 UTC (permalink / raw)
To: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Patrick Rudolph
Cc: mazziesaccount, Naresh Solanki, linux-gpio, devicetree,
linux-kernel
Update maxItems to 60 for gpio-reserved-ranges to allow multiple gpio
reserved ranges.
Add input-enable property to allow configuring a pin as input.
Update example & fix alignment.
Signed-off-by: Naresh Solanki <naresh.solanki@9elements.com>
---
.../bindings/pinctrl/cypress,cy8c95x0.yaml | 38 ++++++++++++++-----
1 file changed, 28 insertions(+), 10 deletions(-)
diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
index 7f30ec2f1e54..90dda5d3cc55 100644
--- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
@@ -45,7 +45,8 @@ properties:
maxItems: 1
gpio-reserved-ranges:
- maxItems: 1
+ minItems: 1
+ maxItems: 60
vdd-supply:
description:
@@ -85,6 +86,8 @@ patternProperties:
bias-disable: true
+ input-enable: true
+
output-high: true
output-low: true
@@ -125,14 +128,29 @@ examples:
#size-cells = <0>;
pinctrl@20 {
- compatible = "cypress,cy8c9520";
- reg = <0x20>;
- gpio-controller;
- #gpio-cells = <2>;
- #interrupt-cells = <2>;
- interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
- interrupt-controller;
- vdd-supply = <&p3v3>;
- gpio-reserved-ranges = <5 1>;
+ compatible = "cypress,cy8c9520";
+ reg = <0x20>;
+ gpio-controller;
+ #gpio-cells = <2>;
+ #interrupt-cells = <2>;
+ interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-controller;
+ vdd-supply = <&p3v3>;
+ gpio-reserved-ranges = <1 2>, <6 1>, <10 1>, <15 1>;
+
+ pinctrl-0 = <&U62160_pins>, <&U62160_ipins>;
+ pinctrl-names = "default";
+ U62160_pins: cfg-pins {
+ pins = "gp03", "gp16", "gp20", "gp50", "gp51";
+ function = "gpio";
+ input-enable;
+ bias-pull-up;
+ };
+ U62160_ipins: icfg-pins {
+ pins = "gp04", "gp17", "gp21", "gp52", "gp53";
+ function = "gpio";
+ input-enable;
+ bias-pull-up;
+ };
};
};
base-commit: 861c0981648f5b64c86fd028ee622096eb7af05a
--
2.42.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v3] dt-bindings: pinctrl: cy8x95x0: Minor fix & update
2024-01-30 12:56 [PATCH v3] dt-bindings: pinctrl: cy8x95x0: Minor fix & update Naresh Solanki
@ 2024-01-31 8:32 ` Krzysztof Kozlowski
2024-02-02 10:52 ` Naresh Solanki
0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-31 8:32 UTC (permalink / raw)
To: Naresh Solanki, Linus Walleij, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Patrick Rudolph
Cc: mazziesaccount, linux-gpio, devicetree, linux-kernel
On 30/01/2024 13:56, Naresh Solanki wrote:
> Update maxItems to 60 for gpio-reserved-ranges to allow multiple gpio
Subject: everything can be a fix and everything is update. Write
something useful.
> reserved ranges.
> Add input-enable property to allow configuring a pin as input.
> Update example & fix alignment.
>
> Signed-off-by: Naresh Solanki <naresh.solanki@9elements.com>
> ---
> .../bindings/pinctrl/cypress,cy8c95x0.yaml | 38 ++++++++++++++-----
> 1 file changed, 28 insertions(+), 10 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> index 7f30ec2f1e54..90dda5d3cc55 100644
> --- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> @@ -45,7 +45,8 @@ properties:
> maxItems: 1
>
> gpio-reserved-ranges:
> - maxItems: 1
> + minItems: 1
> + maxItems: 60
>
> vdd-supply:
> description:
> @@ -85,6 +86,8 @@ patternProperties:
>
> bias-disable: true
>
> + input-enable: true
> +
> output-high: true
>
> output-low: true
> @@ -125,14 +128,29 @@ examples:
> #size-cells = <0>;
>
> pinctrl@20 {
> - compatible = "cypress,cy8c9520";
> - reg = <0x20>;
> - gpio-controller;
> - #gpio-cells = <2>;
> - #interrupt-cells = <2>;
> - interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> - interrupt-controller;
> - vdd-supply = <&p3v3>;
> - gpio-reserved-ranges = <5 1>;
> + compatible = "cypress,cy8c9520";
I don't understand why you change from correct indentation to mixed one
(2 and 4 spaces). It does not make sense.
> + reg = <0x20>;
> + gpio-controller;
> + #gpio-cells = <2>;
> + #interrupt-cells = <2>;
> + interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-controller;
> + vdd-supply = <&p3v3>;
> + gpio-reserved-ranges = <1 2>, <6 1>, <10 1>, <15 1>;
> +
> + pinctrl-0 = <&U62160_pins>, <&U62160_ipins>;
> + pinctrl-names = "default";
Missing blank line.
> + U62160_pins: cfg-pins {
> + pins = "gp03", "gp16", "gp20", "gp50", "gp51";
> + function = "gpio";
> + input-enable;
> + bias-pull-up;
> + };
Missing blank line.
> + U62160_ipins: icfg-pins {
> + pins = "gp04", "gp17", "gp21", "gp52", "gp53";
> + function = "gpio";
> + input-enable;
> + bias-pull-up;
> + };
> };
> };
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v3] dt-bindings: pinctrl: cy8x95x0: Minor fix & update
2024-01-31 8:32 ` Krzysztof Kozlowski
@ 2024-02-02 10:52 ` Naresh Solanki
0 siblings, 0 replies; 3+ messages in thread
From: Naresh Solanki @ 2024-02-02 10:52 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Patrick Rudolph, mazziesaccount, linux-gpio, devicetree,
linux-kernel
Hi Krzysztof,
On Wed, 31 Jan 2024 at 14:02, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 30/01/2024 13:56, Naresh Solanki wrote:
> > Update maxItems to 60 for gpio-reserved-ranges to allow multiple gpio
>
> Subject: everything can be a fix and everything is update. Write
> something useful.
Sure. Will update as:
dt-bindings: pinctrl: cy8x95x0: Update gpio-reserved-ranges
>
> > reserved ranges.
> > Add input-enable property to allow configuring a pin as input.
> > Update example & fix alignment.
> >
> > Signed-off-by: Naresh Solanki <naresh.solanki@9elements.com>
> > ---
> > .../bindings/pinctrl/cypress,cy8c95x0.yaml | 38 ++++++++++++++-----
> > 1 file changed, 28 insertions(+), 10 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > index 7f30ec2f1e54..90dda5d3cc55 100644
> > --- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > +++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> > @@ -45,7 +45,8 @@ properties:
> > maxItems: 1
> >
> > gpio-reserved-ranges:
> > - maxItems: 1
> > + minItems: 1
> > + maxItems: 60
> >
> > vdd-supply:
> > description:
> > @@ -85,6 +86,8 @@ patternProperties:
> >
> > bias-disable: true
> >
> > + input-enable: true
> > +
> > output-high: true
> >
> > output-low: true
> > @@ -125,14 +128,29 @@ examples:
> > #size-cells = <0>;
> >
> > pinctrl@20 {
> > - compatible = "cypress,cy8c9520";
> > - reg = <0x20>;
> > - gpio-controller;
> > - #gpio-cells = <2>;
> > - #interrupt-cells = <2>;
> > - interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> > - interrupt-controller;
> > - vdd-supply = <&p3v3>;
> > - gpio-reserved-ranges = <5 1>;
> > + compatible = "cypress,cy8c9520";
>
> I don't understand why you change from correct indentation to mixed one
> (2 and 4 spaces). It does not make sense.
Yes indentation is missed. I'll correct to 2 space in next patch revision
>
>
> > + reg = <0x20>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
> > + #interrupt-cells = <2>;
> > + interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> > + interrupt-controller;
> > + vdd-supply = <&p3v3>;
> > + gpio-reserved-ranges = <1 2>, <6 1>, <10 1>, <15 1>;
> > +
> > + pinctrl-0 = <&U62160_pins>, <&U62160_ipins>;
> > + pinctrl-names = "default";
>
> Missing blank line.
Will fix in next revision.
>
> > + U62160_pins: cfg-pins {
> > + pins = "gp03", "gp16", "gp20", "gp50", "gp51";
> > + function = "gpio";
> > + input-enable;
> > + bias-pull-up;
> > + };
>
> Missing blank line.
Will fix in next revision.
Regards,
Naresh
>
>
> > + U62160_ipins: icfg-pins {
> > + pins = "gp04", "gp17", "gp21", "gp52", "gp53";
> > + function = "gpio";
> > + input-enable;
> > + bias-pull-up;
> > + };
> > };
> > };
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-02-02 10:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-30 12:56 [PATCH v3] dt-bindings: pinctrl: cy8x95x0: Minor fix & update Naresh Solanki
2024-01-31 8:32 ` Krzysztof Kozlowski
2024-02-02 10:52 ` Naresh Solanki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).