dm: pinctrl: introduce PINCONF_RECURSIVE option

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 <patrick.delaunay@st.com>
Reviewed-by: Simon Glass <sjg@chromium.org>

diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
index a0ac167d145a548bd738d2440e215889a3955bc5..deee92411c901920cf513c5bf30f733470814344 100644 (file)
--- a/drivers/pinctrl/Kconfig
+++ b/drivers/pinctrl/Kconfig
@@ -59,6 +59,22 @@ config PINCONF
          This option enables pin configuration through the generic pinctrl
          framework.
 
+config PINCONF_RECURSIVE
+       bool "Support recursive binding for pin configuration nodes"
+       depends on PINCTRL_FULL
+       default n if ARCH_STM32MP
+       default y
+       help
+         In the Linux pinctrl binding, the pin configuration nodes need not be
+         direct children of the pin controller device (may be grandchildren for
+         example). It is define is each individual pin controller device.
+         Say Y here if you want to keep this behavior with the pinconfig
+         u-class: all sub are recursivelly bounded.
+         If the option is disabled, this behavior is deactivated and only
+         the direct children of pin controller will be assumed as pin
+         configuration; you can save memory footprint when this feature is
+         no needed.
+
 config SPL_PINCTRL
        bool "Support pin controllers in SPL"
        depends on SPL && SPL_DM
@@ -104,6 +120,15 @@ config SPL_PINCONF
          This option is an SPL-variant of the PINCONF option.
          See the help of PINCONF for details.
 
+config SPL_PINCONF_RECURSIVE
+       bool "Support recursive binding for pin configuration nodes in SPL"
+       depends on SPL_PINCTRL_FULL
+       default n if ARCH_STM32MP
+       default y
+       help
+         This option is an SPL-variant of the PINCONF_RECURSIVE option.
+         See the help of PINCONF_RECURSIVE for details.
+
 if PINCTRL || SPL_PINCTRL
 
 config PINCTRL_AR933X

diff --git a/drivers/pinctrl/pinctrl-uclass.c b/drivers/pinctrl/pinctrl-uclass.c
index 5b1cd29d862dc421405b90e3a691dea8a78c525b..bf799a701c3840646a3d75e6c88faf2a9601c46c 100644 (file)
--- a/drivers/pinctrl/pinctrl-uclass.c
+++ b/drivers/pinctrl/pinctrl-uclass.c
@@ -151,7 +151,9 @@ static int pinconfig_post_bind(struct udevice *dev)
 
 UCLASS_DRIVER(pinconfig) = {
        .id = UCLASS_PINCONFIG,
+#if CONFIG_IS_ENABLED(PINCONFIG_RECURSIVE)
        .post_bind = pinconfig_post_bind,
+#endif
        .name = "pinconfig",
 };