hi, On Tue, Aug 06, 2019 at 06:11:50PM -0400, thom...@fastmail.cn wrote: > > > On Tue, Aug 6, 2019, at 5:38 PM, Ben Hutchings wrote: > > Control: severity -1 wishlist > > > > On Tue, 2019-08-06 at 17:21 -0400, thom...@fastmail.cn wrote: > > > package: src:linux > > > I think less running code when possible and a smaller linux image is > > > always a good thing, therefore, I propose these as modules instead of > > > built-in. > > > > > > CONFIG_PINCTRL_AMD > > > CONFIG_PINCTRL_CHERRYVIEW > > > CONFIG_PINCTRL_INTEL > > > CONFIG_PINCTRL_CANNONLAKE > > > CONFIG_PINCTRL_CEDARFORK > > > CONFIG_PINCTRL_DENVERTON > > > CONFIG_PINCTRL_ICELAKE > > > CONFIG_PINCTRL_LEWISBURG > > > CONFIG_PINCTRL_SUNRISEPOINT > > > > Ideally, yes; however pinctrl drivers may be needed very early during > > boot so I don't know if these will work properly as modules. > Fair enough. I don't have the hardware to test/verify. I have one > laptop that uses canonlake one and its sd card reader works even if > the pinctrl is a module but this is a big change so lets not fix it > since its not broken. Feel free to close.
I have closed now this bugreport. Regards, Salvatore
