linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: am335x-pocketbeagle: switch to pinconf-single
@ 2021-08-25 20:25 Drew Fustini
  2021-10-06  7:29 ` Tony Lindgren
  0 siblings, 1 reply; 2+ messages in thread
From: Drew Fustini @ 2021-08-25 20:25 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring, linux-omap,
	devicetree, linux-kernel
  Cc: Drew Fustini

Switch the compatible for the am33xx_pinmux pin controller node from
pinctrl-single to pinconf-single. The only change between these two
compatibles is that PCS_HAS_PINCONF will be true. This then allows
pinconf properties to be utilized.

The purpose of this change is to allow the PocketBeagle to use:

  pinctrl-single,bias-pullup
  pinctrl-single,bias-pulldown

This dts already defines these properites for gpio pins in the default
pinctrl state but it has no effect unless PCS_HAS_PINCONF is set.

The bias properties can then be modified on the corresponding gpio lines
through the gpiod uapi. The mapping between the pins and gpio lines is
defined by gpio-ranges under the gpio controller nodes in am33xx-l4.dtsi

Signed-off-by: Drew Fustini <drew@pdp7.com>
---
 arch/arm/boot/dts/am335x-pocketbeagle.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/am335x-pocketbeagle.dts b/arch/arm/boot/dts/am335x-pocketbeagle.dts
index 209cdd17dc1e..5e415d8ffdd8 100644
--- a/arch/arm/boot/dts/am335x-pocketbeagle.dts
+++ b/arch/arm/boot/dts/am335x-pocketbeagle.dts
@@ -205,6 +205,7 @@ &gpio3 {
 
 &am33xx_pinmux {
 
+	compatible = "pinconf-single";
 	pinctrl-names = "default";
 
 	pinctrl-0 =   < &P2_03_gpio &P1_34_gpio &P2_19_gpio &P2_24_gpio
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ARM: dts: am335x-pocketbeagle: switch to pinconf-single
  2021-08-25 20:25 [PATCH] ARM: dts: am335x-pocketbeagle: switch to pinconf-single Drew Fustini
@ 2021-10-06  7:29 ` Tony Lindgren
  0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2021-10-06  7:29 UTC (permalink / raw)
  To: Drew Fustini
  Cc: Benoît Cousson, Rob Herring, linux-omap, devicetree, linux-kernel

* Drew Fustini <drew@pdp7.com> [210826 01:20]:
> Switch the compatible for the am33xx_pinmux pin controller node from
> pinctrl-single to pinconf-single. The only change between these two
> compatibles is that PCS_HAS_PINCONF will be true. This then allows
> pinconf properties to be utilized.
> 
> The purpose of this change is to allow the PocketBeagle to use:
> 
>   pinctrl-single,bias-pullup
>   pinctrl-single,bias-pulldown
> 
> This dts already defines these properites for gpio pins in the default
> pinctrl state but it has no effect unless PCS_HAS_PINCONF is set.
> 
> The bias properties can then be modified on the corresponding gpio lines
> through the gpiod uapi. The mapping between the pins and gpio lines is
> defined by gpio-ranges under the gpio controller nodes in am33xx-l4.dtsi

Thanks applying into omap-for-v5.16/dt.

Tony

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-10-06  7:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-25 20:25 [PATCH] ARM: dts: am335x-pocketbeagle: switch to pinconf-single Drew Fustini
2021-10-06  7:29 ` 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).