Re: [PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-05 Thread Jacek Anaszewski
Hi Gene, Thank you for the patch. Please find my comments in the code below. On 6/4/20 8:26 AM, Gene Chen wrote: From: Gene Chen Add MT6360 LED driver include 2-channel Flash LED with torch/strobe mode, and 3-channel RGB LED support Register/Flash/Breath Mode Signed-off-by: Gene Chen

Re: [PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-04 Thread Dan Murphy
Gene On 6/4/20 1:26 AM, Gene Chen wrote: From: Gene Chen Add MT6360 LED driver include 2-channel Flash LED with torch/strobe mode, and 3-channel RGB LED support Register/Flash/Breath Mode Signed-off-by: Gene Chen base-commit: 098c4adf249c198519a4abebe482b1e6b8c50e47 ---

Re: [PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-04 Thread Pavel Machek
Hi! > diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig > index c664d84..c47be91 100644 > --- a/drivers/leds/Kconfig > +++ b/drivers/leds/Kconfig > @@ -229,6 +229,17 @@ config LEDS_MT6323 > This option enables support for on-chip LED drivers found on > Mediatek MT6323 PMIC.

Re: [PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-04 Thread Pavel Machek
Hi! > + > +#define MT6360_LED_DESC(_id) { > \ > + .cdev = { \ > + .name = "mt6360_isink" #_id,\ > + .max_brightness =

Re: [PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-04 Thread Pavel Machek
Hi! > Add MT6360 LED driver include 2-channel Flash LED with torch/strobe mode, > and 3-channel RGB LED support Register/Flash/Breath Mode > > Signed-off-by: Gene Chen > base-commit: 098c4adf249c198519a4abebe482b1e6b8c50e47 Does this need device tree binding, too? > +++

[PATCH] leds: mt6360: Add LED driver for MT6360

2020-06-04 Thread Gene Chen
From: Gene Chen Add MT6360 LED driver include 2-channel Flash LED with torch/strobe mode, and 3-channel RGB LED support Register/Flash/Breath Mode Signed-off-by: Gene Chen base-commit: 098c4adf249c198519a4abebe482b1e6b8c50e47 --- drivers/leds/Kconfig | 11 + drivers/leds/Makefile