]> git.dujemihanovic.xyz Git - u-boot.git/commit
arm64: a37xx: pinctrl: Correct PWM pins definitions
authorMarek Behún <marek.behun@nic.cz>
Fri, 23 Jul 2021 17:57:11 +0000 (19:57 +0200)
committerStefan Roese <sr@denx.de>
Sat, 31 Jul 2021 07:59:58 +0000 (09:59 +0200)
commit5534fb4f4833eda4e1a2df1c89a75db72e5e2008
tree32f96dd1b2ac08a5fe9f24ed00bafaeaf3de63ab
parent3d7b93d51c26092010626708ba9adaf9a65a764f
arm64: a37xx: pinctrl: Correct PWM pins definitions

The PWM pins on North Bridge on Armada 37xx can be configured into PWM
or GPIO functions. When in PWM function, each pin can also be configured
to drive low on 0 and tri-state on 1 (LED mode).

The current definitions handle this by declaring two pin groups for each
pin:
- group "pwmN" with functions "pwm" and "gpio"
- group "ledN_od" ("od" for open drain) with functions "led" and "gpio"

This is semantically incorrect. The correct definition for each pin
should be one group with three functions: "pwm", "led" and "gpio".

Change the "pwmN" groups to support "led" function.

Remove "ledN_od" groups. This cannot break backwards compatibility with
older device trees: no device tree uses it since there is no PWM driver
for this SOC yet. Also "ledN_od" groups are not even documented.

Signed-off-by: Marek Behún <marek.behun@nic.cz>
Reviewed-by: Stefan Roese <sr@denx.de>
doc/device-tree-bindings/pinctrl/marvell,armada-37xx-pinctrl.txt
drivers/pinctrl/mvebu/pinctrl-armada-37xx.c