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.
