lkml.org 
[lkml]   [2022]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 5/5] platform/chrome: cros_kbd_led_backlight: support EC PWM backend
On Fri, Mar 18, 2022 at 10:07:51AM -0700, Guenter Roeck wrote:
> On Mon, Mar 14, 2022 at 05:08:35PM +0800, Tzung-Bi Shih wrote:
> > @@ -142,6 +142,12 @@ config CROS_KBD_LED_BACKLIGHT_ACPI
> > help
> > ChromeOS keyboard backlight ACPI backend.
> >
> > +config CROS_KBD_LED_BACKLIGHT_EC_PWM
> > + tristate "ChromeOS keyboard backlight EC PWM backend"
>
> This is not a standa-alone module and should therefore be bool,
> not tristate.

Ack.

> > @@ -123,34 +130,122 @@ static const struct keyboard_led_drvdata keyboard_led_drvdata_acpi = {
> >
> > #endif /* IS_ENABLED(CONFIG_CROS_KBD_LED_BACKLIGHT_ACPI) */
> >
> > +#if IS_ENABLED(CONFIG_CROS_KBD_LED_BACKLIGHT_EC_PWM)
>
> #ifdef (with bool) should do.

Ack. Will apply the fix for CONFIG_CROS_KBD_LED_BACKLIGHT_ACPI too.

\
 
 \ /
  Last update: 2022-03-21 08:19    [W:0.324 / U:0.300 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site