u-boot.git/drivers/pinctrl/pinctrl-uclass.c, branch v2020.01

pinctrol: dm: remove the function pinctrl_decode_pin_config

2019-11-14T13:09:34+00:00

Remove the pinctrl_decode_pin_config() API, because this
function is unused and not compatible with livetree
(it uses fdtdec_get_bool instead of ofnode API).

Signed-off-by: Patrick Delaunay 
Reviewed-by: Simon Glass

dm: pinctrl: fix for introduce PINCONF_RECURSIVE option

2019-10-21T13:40:54+00:00

Correct the name of the define used CONFIG_IS_ENABLED which is
not aligned with Kconfig name: CONFIG_$(SPL_)PINCONF_RECURSIVE.

The recursive calls is conditional only for UCLASS_PINCONFIG
"pinconfig" driver.
It is always needed to call pinctrl_post_bind for UCLASS_PINCTRL
"pinctrl", the test CONFIG_IS_ENABLED(PINCONF_RECURSIVE) need to
be removed for this driver.

This correct a regression introduced because the same patch is
applied twice times in u-boot-dm branch:
- commit e878b53a79d1 ("dm: pinctrl: introduce PINCONF_RECURSIVE
  option")
- commit c20851b3d850 ("dm: pinctrl: introduce PINCONF_RECURSIVE
  option")

Signed-off-by: Patrick Delaunay

dm: pinctrl: introduce PINCONF_RECURSIVE option

2019-10-15T14:40:03+00:00

In the Linux pinctrl binding, the pin configuration nodes don't need to
be direct children of the pin controller device (may be grandchildren for
example). This behavior is managed with the pinconfig u-class which
recursively bind all the sub-node of the pin controller.

But for some binding (when pin configuration is only children of pin
controller) that is not necessary. U-Boot can save memory and reduce
the number of pinconf instance when this feature is deactivated
(for arch stm32mp for example for SPL).

This patch allows to control this feature with a new option
CONFIG_PINCONF_RECURSIVE when it is possible for each individual
pin controller device.

Signed-off-by: Patrick Delaunay 
Reviewed-by: Simon Glass 
Fixed CONFIG_IF_ENABLED() condition, added __maybe_unused:
Signed-off-by: Simon Glass

dm: pinctrl: Skip not associated gpio phandle and rise a warning message

2019-10-15T14:40:02+00:00

Skip not associated gpio phandle let register the other gpios on a group.
We need anyway to send out a warning to the user to fix their uboot-board.dtsi.
Thhe handle id can be found inside the decompiled dtb

dtc -I dtb -O dts -o devicetree.dts spl/u-boot-spl.dtb

Signed-off-by: Michael Trimarchi 
Reviewed-by: Simon Glass

dm: pinctrl: introduce PINCONF_RECURSIVE option

2019-10-15T14:40:02+00:00

In the Linux pinctrl binding, the pin configuration nodes don't need to
be direct children of the pin controller device (may be grandchildren for
example). This behavior is managed with the pinconfig u-class which
recursively bind all the sub-node of the pin controller.

But for some binding (when pin configuration is only children of pin
controller) that is not necessary. U-Boot can save memory and reduce
the number of pinconf instance when this feature is deactivated
(for arch stm32mp for example for SPL).

This patch allows to control this feature with a new option
CONFIG_PINCONF_RECURSIVE when it is possible for each individual
pin controller device.

Signed-off-by: Patrick Delaunay 
Reviewed-by: Simon Glass

pinctrl: gpio: Add callback for configuring pin as GPIO

2019-05-04T17:26:49+00:00

Add callback to configure, and de-configure, pin as a GPIO on the
pin controller side. This matches similar functionality in Linux
and aims to replace the ad-hoc implementations present in U-Boot.

Signed-off-by: Marek Vasut 
Cc: Alex Kiernan 
Cc: Christoph Muellner 
Cc: Eugeniu Rosca 
Cc: Patrice Chotard 
Cc: Patrick DELAUNAY 
Cc: Philipp Tomsich 
Cc: Simon Glass

pinctrl: exit pinconfig_post_bind if there are no subnodes

2019-04-30T22:00:05+00:00

This fixes RK3288 SPL hanging or hitting this assert:
drivers/core/ofnode.c:183: ofnode_first_subnode: Assertion `ofnode_valid(node)' failed.

Signed-off-by: Urja Rannikko 
Reviewed-by: Simon Glass 
Reviewed-by: Philipp Tomsich

dm: pinctrl: Skip gpio-controller node in pinconfig_post_bind()

2019-04-12T02:10:05+00:00

Some binding define child node gpio-controller without compatible property.
This patch avoid to bind the pinconfig uclass to these node.

For example, the binding for st,stm32-pinctrl
(./device-tree-bindings/pinctrl/st,stm32-pinctrl.txt) defines the GPIO
controller/bank node as sub-node of pincontrol (st,stm32f429-pinctrl)
but without compatible (as it is not mandatory).

Signed-off-by: Patrick Delaunay 
Signed-off-by: Patrice Chotard 
Reviewed-by: Simon Glass

dm: pinctrl: Avoid race condition on probe for UCLASS_PINCTRL

2019-04-12T02:10:05+00:00

In case of system with several pin-controller device, probe the first
UCLASS_PINCTRL by seq number (defined by alias) to avoid race condition
with I2C PINCONTROL driver for GPIO expander (GPIO expander need I2C bus,
I2C driver need PINCONFIG).

Signed-off-by: Patrick DELAUNAY 
Signed-off-by: Patrice Chotard 
Reviewed-by: Simon Glass

dm: pinctrl: Remove obsolete function pinctrl_decode_pin_config_dm().

2019-04-12T02:10:05+00:00

This reverts commit 5ff776889212c080e3d1a33634ac904405ed6845.

As noted in the comment, the function pinctrl_decode_pin_config_dm()
only served as a temporary solution.

Since the function has no users anymore, we can remove it again.

Signed-off-by: Christoph Muellner 
Reviewed-by: Simon Glass