Bug#934090: built-in PINCTRL drivers should be modules
- To: thomasw@fastmail.cn, 934090@bugs.debian.org, 934090-done@bugs.debian.org
- Cc: Ben Hutchings <ben@decadent.org.uk>
- Subject: Bug#934090: built-in PINCTRL drivers should be modules
- From: Salvatore Bonaccorso <carnil@debian.org>
- Date: Fri, 23 Apr 2021 13:35:22 +0200
- Message-id: <YIKw+m5q/L8ISFQs@eldamar.lan>
- Reply-to: Salvatore Bonaccorso <carnil@debian.org>, 934090@bugs.debian.org
- In-reply-to: <9465ad55-6918-4d24-9598-97fa1a8a22c8@www.fastmail.com>
- References: <4fa7cafa-69b7-4c12-9d6f-27e490cadd3d@www.fastmail.com> <56e1bcf20b75fc97bde043a9168b26868356bb30.camel@decadent.org.uk> <4fa7cafa-69b7-4c12-9d6f-27e490cadd3d@www.fastmail.com> <9465ad55-6918-4d24-9598-97fa1a8a22c8@www.fastmail.com> <4fa7cafa-69b7-4c12-9d6f-27e490cadd3d@www.fastmail.com>
hi,
On Tue, Aug 06, 2019 at 06:11:50PM -0400, thomasw@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, 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.
I have closed now this bugreport.
Regards,
Salvatore
Reply to: