Nami: Add keyboard backlight control

This implements keyboard backlight control for Nami, Vayne, Pantheon,
and Sona. On Sona, GPIOC4 is directly connected to the LED strings.
Thus, we use PWM to control the brightness. On the other variants,
the LED strings are connected to LM3509. Thus, we control the brightness
through I2C.

Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>

BUG=b:76182445,b:78141647
BRANCH=none
TEST=Verify keyboard backlight brightness changes on Nami.
Verify keyboard backlight turns on/off on lid close/open, sleep/suspend.
on Nami. Verify 'kblight' returns x set by 'kblight x' on Sona.

Change-Id: I400ea2bc7a58a3cc57eb959179d2139a99ac176c
Reviewed-on: https://chromium-review.googlesource.com/1022833
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@google.com>

5 files changed, 134 insertions, 13 deletions

diff --git a/board/nami/board.c b/board/nami/board.c
index 5878515149..836e3f978f 100644
--- a/board/nami/board.c
+++ b/board/nami/board.c
@@ -497,6 +497,7 @@ const struct pwm_t pwm_channels[] = {
 	[PWM_CH_LED1]   = { 3, PWM_CONFIG_DSLEEP, 100 },
 	[PWM_CH_LED2] = { 5, PWM_CONFIG_DSLEEP, 100 },
 	[PWM_CH_FAN] = {4, PWM_CONFIG_OPEN_DRAIN, 25000},
+	[PWM_CH_KBLIGHT] = { 2, 0, 100 },
 };
 BUILD_ASSERT(ARRAY_SIZE(pwm_channels) == PWM_CH_COUNT);
 
@@ -650,8 +651,6 @@ void lid_angle_peripheral_enable(int enable)
 static void board_chipset_resume(void)
 {
 	gpio_set_level(GPIO_ENABLE_BACKLIGHT_L, 0);
-	if (lid_is_open())
-		lm3509_poweron();
 }
 DECLARE_HOOK(HOOK_CHIPSET_RESUME, board_chipset_resume, HOOK_PRIO_DEFAULT);
 
@@ -659,20 +658,9 @@ DECLARE_HOOK(HOOK_CHIPSET_RESUME, board_chipset_resume, HOOK_PRIO_DEFAULT);
 static void board_chipset_suspend(void)
 {
 	gpio_set_level(GPIO_ENABLE_BACKLIGHT_L, 1);
-	lm3509_poweroff();
 }
 DECLARE_HOOK(HOOK_CHIPSET_SUSPEND, board_chipset_suspend, HOOK_PRIO_DEFAULT);
 
-/* Control keyboard backlight when Lid status change */
-static void lm3509_kblight_lid_change(void)
-{
-	if (lid_is_open())
-		lm3509_poweron();
-	else
-		lm3509_poweroff();
-}
-DECLARE_HOOK(HOOK_LID_CHANGE, lm3509_kblight_lid_change, HOOK_PRIO_DEFAULT);
-
 static void board_set_motion_sensor_count(void)
 {
 	/* There are two possible sensor configurations.
diff --git a/board/nami/board.h b/board/nami/board.h
index c1ccc4350f..e7ed9e3a44 100644
--- a/board/nami/board.h
+++ b/board/nami/board.h
@@ -241,6 +241,7 @@ enum pwm_channel {
 	PWM_CH_LED1,
 	PWM_CH_LED2,
 	PWM_CH_FAN,
+	PWM_CH_KBLIGHT,
 	/* Number of PWM channels */
 	PWM_CH_COUNT,
 };
diff --git a/board/nami/build.mk b/board/nami/build.mk
index f4bf21113d..3d83e52bfe 100644
--- a/board/nami/build.mk
+++ b/board/nami/build.mk
@@ -13,3 +13,4 @@ board-y=board.o
 board-$(CONFIG_BATTERY_SMART)+=battery.o
 board-$(CONFIG_LED_COMMON)+=led.o
 board-$(CONFIG_USB_POWER_DELIVERY)+=usb_pd_policy.o
+board-y+=keyboard_backlight.o
diff --git a/board/nami/gpio.inc b/board/nami/gpio.inc
index ce69db4da9..c12db6dceb 100644
--- a/board/nami/gpio.inc
+++ b/board/nami/gpio.inc
@@ -109,6 +109,7 @@ ALTERNATE(PIN_MASK(B, 0x40), 1, MODULE_PWM, 0)	/* GPIOB6 PWM1 Fan control */
 ALTERNATE(PIN_MASK(8, 0x01), 1, MODULE_PWM, 0)	/* GPIO80 PWM3 LED White */
 ALTERNATE(PIN_MASK(B, 0x80), 1, MODULE_PWM, 0)	/* GPIOB7 PWM5 LED Yellow */
 ALTERNATE(PIN_MASK(A, 0x40), 1, MODULE_PWM, 0)	/* GPIOA6 TA2 */
+ALTERNATE(PIN_MASK(C, 0x10), 1, MODULE_PWM, 0)	/* GPIOC4 PWM2 */
 
 /* Keyboard pins */
 #define GPIO_KB_INPUT (GPIO_INPUT | GPIO_PULL_UP)
diff --git a/board/nami/keyboard_backlight.c b/board/nami/keyboard_backlight.c
new file mode 100644
index 0000000000..29f74a8131
--- /dev/null
+++ b/board/nami/keyboard_backlight.c
@@ -0,0 +1,130 @@
+/* Copyright 2018 The Chromium OS Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ *
+ * Keyboard backlight control
+ */
+
+#include "console.h"
+#include "cros_board_info.h"
+#include "hooks.h"
+#include "lid_switch.h"
+#include "lm3509.h"
+#include "pwm.h"
+#include "util.h"
+
+static void (*kblight_set)(int percent);
+static int (*kblight_get)(void);
+static void (*kblight_power)(int enable);
+
+/*
+ * PWM routines
+ */
+static void kblight_pwm_set(int percent)
+{
+	pwm_set_duty(PWM_CH_KBLIGHT, percent);
+}
+
+static int kblight_pwm_get(void)
+{
+	return pwm_get_duty(PWM_CH_KBLIGHT);
+}
+
+static void kblight_pwm_power(int enable)
+{
+	pwm_enable(PWM_CH_KBLIGHT, enable);
+}
+
+/*
+ * I2C routines
+ */
+static void kblight_i2c_set(int percent)
+{
+	lm3509_set_brightness(percent);
+}
+
+static int kblight_i2c_get(void)
+{
+	int percent;
+	if (lm3509_get_brightness(&percent))
+		percent = 0;
+	return percent;
+}
+
+static void kblight_i2c_power(int enable)
+{
+	lm3509_power(enable);
+}
+
+static void kblight_init(void)
+{
+	uint32_t oem = PROJECT_NAMI;
+	uint32_t sku = 0;
+
+	cbi_get_oem_id(&oem);
+	cbi_get_sku_id(&sku);
+
+	switch (oem) {
+	default:
+	case PROJECT_NAMI:
+	case PROJECT_VAYNE:
+	case PROJECT_PANTHEON:
+		kblight_set = kblight_i2c_set;
+		kblight_get = kblight_i2c_get;
+		kblight_power = kblight_i2c_power;
+		break;
+	case PROJECT_SONA:
+		if (sku == 0x3AE2)
+			break;
+		kblight_set = kblight_pwm_set;
+		kblight_get = kblight_pwm_get;
+		kblight_power = kblight_pwm_power;
+		break;
+	}
+}
+DECLARE_HOOK(HOOK_INIT, kblight_init, HOOK_PRIO_DEFAULT);
+
+static void kblight_suspend(void)
+{
+	if (kblight_power)
+		kblight_power(0);
+}
+DECLARE_HOOK(HOOK_CHIPSET_SUSPEND, kblight_suspend, HOOK_PRIO_DEFAULT);
+
+static void kblight_resume(void)
+{
+	if (kblight_power)
+		kblight_power(lid_is_open());
+}
+DECLARE_HOOK(HOOK_CHIPSET_RESUME, kblight_resume, HOOK_PRIO_DEFAULT);
+
+static void kblight_lid_change(void)
+{
+	if (kblight_power)
+		kblight_power(lid_is_open());
+}
+DECLARE_HOOK(HOOK_LID_CHANGE, kblight_lid_change, HOOK_PRIO_DEFAULT);
+
+static int cc_kblight(int argc, char **argv)
+{
+	int i;
+	char *e;
+
+	if (argc < 2) {
+		if (!kblight_get)
+			return EC_ERROR_UNIMPLEMENTED;
+		ccprintf("%d\n", kblight_get());
+		return EC_SUCCESS;
+	}
+
+	if (!kblight_set)
+		return EC_ERROR_UNIMPLEMENTED;
+	i = strtoi(argv[1], &e, 0);
+	if (*e)
+		return EC_ERROR_PARAM1;
+	kblight_set(i);
+	return EC_SUCCESS;
+}
+DECLARE_CONSOLE_COMMAND(kblight, cc_kblight,
+			"kblight [percent]",
+			"Get/set keyboard backlight brightness");