Messages in this thread | | | From | Andy Shevchenko <> | Date | Thu, 24 Jan 2019 21:15:54 +0200 | Subject | Re: [for next][PATCH v2 2/2] platform/x86: Fix unmet dependency warning for SAMSUNG_Q10 |
| |
On Thu, Jan 24, 2019 at 7:57 PM Sinan Kaya <okaya@kernel.org> wrote: > > Add BACKLIGHT_LCD_SUPPORT for SAMSUNG_Q10 to fix the > warning: unmet direct dependencies detected for BACKLIGHT_CLASS_DEVICE. > > SAMSUNG_Q10 selects BACKLIGHT_CLASS_DEVICE but BACKLIGHT_CLASS_DEVICE > depends on BACKLIGHT_LCD_SUPPORT. > > Copy BACKLIGHT_LCD_SUPPORT dependency into SAMSUNG_Q10 to fix: > > WARNING: unmet direct dependencies detected for BACKLIGHT_CLASS_DEVICE > Depends on [n]: HAS_IOMEM [=y] && BACKLIGHT_LCD_SUPPORT [=n] > Selected by [y]: > - SAMSUNG_Q10 [=y] && X86 [=y] && X86_PLATFORM_DEVICES [=y] && ACPI [=y]
Thanks for the fix. One comment below though.
> config SAMSUNG_Q10 > tristate "Samsung Q10 Extras" > - depends on ACPI
> + depends on ACPI && BACKLIGHT_LCD_SUPPORT
I would like rather see it on a separate line in both patches.
Rafael, you may take it if it feels comfortable.
> select BACKLIGHT_CLASS_DEVICE > ---help--- > This driver provides support for backlight control on Samsung Q10 > -- > 2.19.0 >
-- With Best Regards, Andy Shevchenko
| |