On Fri, Dec 26, 2025 at 02:07:18PM +0530, Sumit Garg wrote: > 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.
If something is merely "default y" it can still be easily turned off. If we want to make the "if ..." part more precise, OK. But I don't want to go back to (nearly?) every snapdragon and ipq40xx defconfig being modified, as it was in an earlier iteration. -- Tom
signature.asc
Description: PGP signature
