2023-07-12 09:13:53

by Naresh Solanki

Subject: [PATCH v2 1/2] dt-bindings: pinctrl: cypress,cy8c95x0: Add reset pin

From: Patrick Rudolph <[email protected]>

This patch adds support for an optional reset pin.

The reset pin is used to bring the chip into a known state and has an
internal pull-down, allowing it to be left floating if not needed.

Signed-off-by: Patrick Rudolph <[email protected]>
Signed-off-by: Naresh Solanki <[email protected]>
---
Changes in V2:
- Update subject
- Update reset-gpios description.
---
.../devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
index 222d57541b65..2fa22160336f 100644
--- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
@@ -51,6 +51,10 @@ properties:
description:
Optional power supply.

+ reset-gpios:
+ description: Reference to the GPIO connected to the XRES pin
+ maxItems: 1
+
patternProperties:
'-pins$':
type: object

base-commit: 3bc551a3007a751a53bfba5b37fa16157f4fb861
--
2.41.0

2023-07-12 13:25:41

by Krzysztof Kozlowski

Subject: Re: [PATCH v2 1/2] dt-bindings: pinctrl: cypress,cy8c95x0: Add reset pin

On 12/07/2023 10:52, Naresh Solanki wrote:
> From: Patrick Rudolph <[email protected]>
>
> This patch adds support for an optional reset pin.
>
> The reset pin is used to bring the chip into a known state and has an
> internal pull-down, allowing it to be left floating if not needed.
>
> Signed-off-by: Patrick Rudolph <[email protected]>
> Signed-off-by: Naresh Solanki <[email protected]>
> ---
> Changes in V2:
> - Update subject
> - Update reset-gpios description.
> ---
> .../devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> index 222d57541b65..2fa22160336f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/cypress,cy8c95x0.yaml
> @@ -51,6 +51,10 @@ properties:
> description:
> Optional power supply.
>
> + reset-gpios:
> + description: Reference to the GPIO connected to the XRES pin

I asked you to remove redundant part...

Best regards,
Krzysztof