Bug#934090: built-in PINCTRL drivers should be modules
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, thomasw@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.
Reply to: