2023-08-03 11:12:46

by Dhruva Gole

[permalink] [raw]
Subject: [PATCH] dt-bindings: pinctrl: pinctrl-single: add am62x compatible

Add the am62x compatible property to add support for the new
wakeup enable and status bits positions

Cc: Nishanth Menon <[email protected]>
Cc: Vignesh Raghavendra <[email protected]>
CC: Tony Lindgren <[email protected]>
Signed-off-by: Dhruva Gole <[email protected]>
---

Base: tag: next-20230731 + below "depends on" patch
Depends on: https://lore.kernel.org/linux-omap/[email protected]/T/

Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
index b6b6bcd7074b..4c98035a1217 100644
--- a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
@@ -23,6 +23,7 @@ properties:
- pinconf-single
- items:
- enum:
+ - ti,am6-padconf
- ti,am437-padconf
- ti,dra7-padconf
- ti,omap2420-padconf
--
2.34.1



2023-08-07 15:07:07

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pinctrl: pinctrl-single: add am62x compatible

* Dhruva Gole <[email protected]> [230803 09:24]:
> Add the am62x compatible property to add support for the new
> wakeup enable and status bits positions
>
> Cc: Nishanth Menon <[email protected]>
> Cc: Vignesh Raghavendra <[email protected]>
> CC: Tony Lindgren <[email protected]>
> Signed-off-by: Dhruva Gole <[email protected]>

Reviewed-by: Tony Lindgren <[email protected]>

2023-08-07 16:10:46

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pinctrl: pinctrl-single: add am62x compatible

On 03/08/2023 11:23, Dhruva Gole wrote:
> Add the am62x compatible property to add support for the new
> wakeup enable and status bits positions
>
> Cc: Nishanth Menon <[email protected]>
> Cc: Vignesh Raghavendra <[email protected]>
> CC: Tony Lindgren <[email protected]>
> Signed-off-by: Dhruva Gole <[email protected]>
> ---
>
> Base: tag: next-20230731 + below "depends on" patch
> Depends on: https://lore.kernel.org/linux-omap/[email protected]/T/
>
> Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> index b6b6bcd7074b..4c98035a1217 100644
> --- a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> @@ -23,6 +23,7 @@ properties:
> - pinconf-single
> - items:
> - enum:
> + - ti,am6-padconf

This does not look like am62x. Except this, am62x looks like family, not
SoC.

Best regards,
Krzysztof


2023-08-07 16:20:25

by Dhruva Gole

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pinctrl: pinctrl-single: add am62x compatible

Krzysztof,

On Aug 07, 2023 at 16:53:28 +0200, Krzysztof Kozlowski wrote:
> On 03/08/2023 11:23, Dhruva Gole wrote:
> > Add the am62x compatible property to add support for the new
> > wakeup enable and status bits positions
> >
> > Cc: Nishanth Menon <[email protected]>
> > Cc: Vignesh Raghavendra <[email protected]>
> > CC: Tony Lindgren <[email protected]>
> > Signed-off-by: Dhruva Gole <[email protected]>
> > ---
> >
> > Base: tag: next-20230731 + below "depends on" patch
> > Depends on: https://lore.kernel.org/linux-omap/[email protected]/T/
> >
> > Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> > index b6b6bcd7074b..4c98035a1217 100644
> > --- a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> > +++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
> > @@ -23,6 +23,7 @@ properties:
> > - pinconf-single
> > - items:
> > - enum:
> > + - ti,am6-padconf
>
> This does not look like am62x. Except this, am62x looks like family, not
> SoC.

We are at V3 in this patch,
Tony, please can you review the latest rev?
https://lore.kernel.org/all/[email protected]/

Otherwise, I will re-spin a new series including the DT-Binding patch
and the driver changes in a single patch series, where I will pick Tony's R-by.

>
> Best regards,
> Krzysztof
>

--
Best regards,
Dhruva Gole <[email protected]>