On Mon, Dec 22, 2025 at 05:21:42PM +0530, Aswin Murugan wrote: > This is a follow-up patch series to address review feedback on the > QCS615 pinctrl driver addition: > https://lore.kernel.org/u-boot/[email protected]/ > > During review, it was suggested that since all Qualcomm pinctrl > drivers are already within a menu conditional on ARCH_SNAPDRAGON || > ARCH_IPQ40XX, they should be made default y based on the appropriate > architecture rather than requiring manual enablement in defconfig > files. > > This series implements that suggestion by: > 1. Adding 'default y if ARCH_SNAPDRAGON' to all Snapdragon pinctrl > drivers and 'default y if ARCH_IPQ40XX' to all IPQ pinctrl > drivers in the Kconfig
I see the usefullness of this feature but in turn it will limit the capability to reduce U-Boot size to an essential minimum. In scenarios where there are contraints on flash size or on-chip RAM for U-Boot SPL. I would rather favour something like following like: 'default y if PINCTRL_QCOM_GENERIC' where PINCTRL_QCOM_GENERIC can be enabled from generic Qcom defconfig. -Sumit > 2. Removing the now-redundant explicit pinctrl driver selections > from qcom_defconfig > > Signed-off-by: Aswin Murugan <[email protected]> > > Aswin Murugan (2): > pinctrl: qcom: Make pinctrl drivers default y based on architecture > configs: qcom_defconfig: Remove redundant pinctrl driver selections > > configs/qcom_defconfig | 18 ------------------ > drivers/pinctrl/qcom/Kconfig | 21 +++++++++++++++++++++ > 2 files changed, 21 insertions(+), 18 deletions(-) > > -- > 2.34.1 >
