* [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible
@ 2023-08-04 5:07 Dhruva Gole
2023-08-04 21:14 ` Nishanth Menon
2023-08-08 10:21 ` Tony Lindgren
0 siblings, 2 replies; 5+ messages in thread
From: Dhruva Gole @ 2023-08-04 5:07 UTC (permalink / raw)
To: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Tony Lindgren
Cc: linux-gpio, devicetree, linux-kernel, linux-arm-kernel,
linux-omap, Dhruva Gole, Nishanth Menon, Vignesh Raghavendra,
Conor Dooley
Add the am625 compatible property to add support for the new
wakeup enable and status bits positions
Cc: Nishanth Menon <nm@ti.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
CC: Tony Lindgren <tony@atomide.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Dhruva Gole <d-gole@ti.com>
---
Base: tag: next-20230731 + below "depends on" patch
Depends on: https://lore.kernel.org/linux-omap/20230731061908.GG5194@atomide.com/T/
Changelog:
* v2 -> v3:
keep order sorted, Suggested-by: Nishanth Menon <nm@ti.com>
link to v2: https://lore.kernel.org/all/20230803150955.611717-1-d-gole@ti.com/
* v1 -> v2 changes:
rename to use am625 instead of am6
link to v1:
https://lore.kernel.org/all/20230803092311.604610-1-d-gole@ti.com/
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..b0a10ab22b74 100644
--- a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.yaml
@@ -24,6 +24,7 @@ properties:
- items:
- enum:
- ti,am437-padconf
+ - ti,am625-padconf
- ti,dra7-padconf
- ti,omap2420-padconf
- ti,omap2430-padconf
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible
2023-08-04 5:07 [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible Dhruva Gole
@ 2023-08-04 21:14 ` Nishanth Menon
2023-08-05 5:14 ` Dhruva Gole
2023-08-08 10:21 ` Tony Lindgren
1 sibling, 1 reply; 5+ messages in thread
From: Nishanth Menon @ 2023-08-04 21:14 UTC (permalink / raw)
To: Dhruva Gole
Cc: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Tony Lindgren, linux-gpio, devicetree, linux-kernel,
linux-arm-kernel, linux-omap, Vignesh Raghavendra, Conor Dooley
On 10:37-20230804, Dhruva Gole wrote:
> Add the am625 compatible property to add support for the new
> wakeup enable and status bits positions
>
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> CC: Tony Lindgren <tony@atomide.com>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> Signed-off-by: Dhruva Gole <d-gole@ti.com>
> ---
How about the driver changes to actually support this compatible?
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible
2023-08-04 21:14 ` Nishanth Menon
@ 2023-08-05 5:14 ` Dhruva Gole
2023-08-05 17:17 ` Nishanth Menon
0 siblings, 1 reply; 5+ messages in thread
From: Dhruva Gole @ 2023-08-05 5:14 UTC (permalink / raw)
To: Nishanth Menon
Cc: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Tony Lindgren, linux-gpio, devicetree, linux-kernel,
linux-arm-kernel, linux-omap, Vignesh Raghavendra, Conor Dooley
On Aug 04, 2023 at 16:14:32 -0500, Nishanth Menon wrote:
> On 10:37-20230804, Dhruva Gole wrote:
> > Add the am625 compatible property to add support for the new
> > wakeup enable and status bits positions
> >
> > Cc: Nishanth Menon <nm@ti.com>
> > Cc: Vignesh Raghavendra <vigneshr@ti.com>
> > CC: Tony Lindgren <tony@atomide.com>
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > Signed-off-by: Dhruva Gole <d-gole@ti.com>
> > ---
>
> How about the driver changes to actually support this compatible?
Wanted to post this patch before I posted the driver changes,
have posted them here now:
https://lore.kernel.org/all/20230805045554.786092-1-d-gole@ti.com
>
> --
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
--
Best regards,
Dhruva Gole <d-gole@ti.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible
2023-08-05 5:14 ` Dhruva Gole
@ 2023-08-05 17:17 ` Nishanth Menon
0 siblings, 0 replies; 5+ messages in thread
From: Nishanth Menon @ 2023-08-05 17:17 UTC (permalink / raw)
To: Dhruva Gole
Cc: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Tony Lindgren, linux-gpio, devicetree, linux-kernel,
linux-arm-kernel, linux-omap, Vignesh Raghavendra, Conor Dooley
On 10:44-20230805, Dhruva Gole wrote:
> On Aug 04, 2023 at 16:14:32 -0500, Nishanth Menon wrote:
> > On 10:37-20230804, Dhruva Gole wrote:
> > > Add the am625 compatible property to add support for the new
> > > wakeup enable and status bits positions
> > >
> > > Cc: Nishanth Menon <nm@ti.com>
> > > Cc: Vignesh Raghavendra <vigneshr@ti.com>
> > > CC: Tony Lindgren <tony@atomide.com>
> > > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > > Signed-off-by: Dhruva Gole <d-gole@ti.com>
> > > ---
> >
> > How about the driver changes to actually support this compatible?
>
> Wanted to post this patch before I posted the driver changes,
> have posted them here now:
>
> https://lore.kernel.org/all/20230805045554.786092-1-d-gole@ti.com
>
Please dont patches piecemeal - it makes no sense to review without
context.
Please send future series organized as follows:
* binding
* driver updates
* if any dts updates are relevant - mark them as DONOTMERGE to let the
driver maintainer know that the dts patches go via SoC tree - most
maintainers know this, but it is better to be safe.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible
2023-08-04 5:07 [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible Dhruva Gole
2023-08-04 21:14 ` Nishanth Menon
@ 2023-08-08 10:21 ` Tony Lindgren
1 sibling, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2023-08-08 10:21 UTC (permalink / raw)
To: Dhruva Gole
Cc: Linus Walleij, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
linux-gpio, devicetree, linux-kernel, linux-arm-kernel,
linux-omap, Nishanth Menon, Vignesh Raghavendra, Conor Dooley
* Dhruva Gole <d-gole@ti.com> [230804 05:10]:
> Add the am625 compatible property to add support for the new
> wakeup enable and status bits positions
Looks good to me:
Reviewed-by: Tony Lindgren <tony@atomide.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-08-08 17:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-04 5:07 [PATCH V3] dt-bindings: pinctrl: pinctrl-single: add am625 compatible Dhruva Gole
2023-08-04 21:14 ` Nishanth Menon
2023-08-05 5:14 ` Dhruva Gole
2023-08-05 17:17 ` Nishanth Menon
2023-08-08 10:21 ` Tony Lindgren
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).