Re: [PATCH v1 0/2] pinctrl: qcom: Make pinctrl drivers default y

[Sumit Garg <sumit.garg@kernel.org>]

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/20251112164758.1560041-1-aswin.murugan@oss.qualcomm.com/
> 
> 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 <aswin.murugan@oss.qualcomm.com>
> 
> 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
>