Re: dell_smbios in 4.15 and keyboard backlight

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2018-02-23 at 19:19 +0000, Mario.Limonciello@xxxxxxxx wrote:
> > -----Original Message-----
> > From: Marco d'Itri [mailto:md@xxxxxxxx]
> > Sent: Friday, February 23, 2018 1:06 PM
> > To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx>
> > Cc: pali.rohar@xxxxxxxxx
> > Subject: Re: dell_smbios in 4.15 and keyboard backlight
> > 
> > On Feb 23, Mario Limonciello <superm1@xxxxxxxxx> wrote:
> > 
> > > Can you please email me on mario.limonciello@xxxxxxxx for this
> > > topic?
> > > 
> > > I would be interested to know do you have dell-smbios-smm or
> > > dell-smbios-wmi not loading or compiling?
> > 
> > Oops... The new modules have not been enabled yet by the Debian
> > kernel
> > team, so this explains why I could not find them.
> > Thank you for your help, I will get back to you if I will still have
> > troubles with the new kernel.
> > 
> 
> If you don't mind I'm going to CC this to the mailing list for
> discussion.  I think
> you identified a valid config problem.
> 
> Andy, Darren,
> 
> Marco (CC'ed) privately emailed Pali and I to discuss an issue that
> dell-laptop
> wasn't working properly for him and dell-smbios couldn't find a
> backend.
> 
> I thought at first it was an issue of the race condition recently
> discussed but
> it's actually a case that the distro kernel he's using compiled:
> 
> DELL_SMBIOS
> DELL_LAPTOP
> 
> But didn't select DELL_SMBIOS_WMI or DELL_SMBIOS_SMM.

Distros have to enable whatever they want to.

Can it be the Dell model, that survives w/o one of above or even both?

> I remember hypothesizing about this with one of the earlier versions
> of the
> patch series but the decision was to just hide config DELL_SMBIOS as
> an invisible
> tristate.


> Because this is a real problem happening I think we need something
> like this:
> 
> diff --git a/drivers/platform/x86/Kconfig
> b/drivers/platform/x86/Kconfig
> index 9a8f964..f2f548b 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -107,6 +107,7 @@ config ASUS_LAPTOP
>  
>  config DELL_SMBIOS
>         tristate
> +       depends on DELL_SMBIOS_WMI || DELL_SMBIOS_SMM

This will simple not work.

>  
>  config DELL_SMBIOS_WMI
>         tristate "Dell SMBIOS calling interface (WMI implementation)"
> 
> Thoughts?

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux