Messages in this thread | | | Subject | Re: [PATCH] platform: dell: all DELL_WMI driver options depend on DELL_WMI | From | Randy Dunlap <> | Date | Fri, 8 Oct 2021 07:49:01 -0700 |
| |
Still seeing this build error in linux-next, so the patch is still needed...
On 9/27/21 1:53 PM, Randy Dunlap wrote: > A randconfig or a user can set/enable DELL_WMI_PRIVACY even when > DELL_WMI is not enabled. This leads to a build error in > dell-laptop.o: > > ld: drivers/platform/x86/dell/dell-laptop.o: in function `dell_init': > dell-laptop.c:(.init.text+0x719): undefined reference to `dell_privacy_has_mic_mute' > > Fix this by enclosing the DELL_WMI driver options inside a > if DELL_WMI / endif bracket. > > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Perry Yuan <Perry.Yuan@dell.com> > Cc: Dell.Client.Kernel@dell.com > Cc: platform-driver-x86@vger.kernel.org > Cc: Hans de Goede <hdegoede@redhat.com> > Cc: Mark Gross <mgross@linux.intel.com> > --- > Found in linux-next, applies to mainline. > > The last time that I sent a patch to Perry Yuan (about 2 weeks ago), > it bounced... > > drivers/platform/x86/dell/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > --- linux-next-20210927.orig/drivers/platform/x86/dell/Kconfig > +++ linux-next-20210927/drivers/platform/x86/dell/Kconfig > @@ -164,6 +164,8 @@ config DELL_WMI > To compile this driver as a module, choose M here: the module will > be called dell-wmi. > > +if DELL_WMI > + > config DELL_WMI_PRIVACY > bool "Dell WMI Hardware Privacy Support" > depends on LEDS_TRIGGER_AUDIO = y || DELL_WMI = LEDS_TRIGGER_AUDIO > @@ -198,6 +200,8 @@ config DELL_WMI_LED > This adds support for the Latitude 2100 and similar > notebooks that have an external LED. > > +endif # DELL_WMI > + > config DELL_WMI_SYSMAN > tristate "Dell WMI-based Systems management driver" > default m >
-- ~Randy
| |