Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On Wed, Jul 15, 2015 at 4:02 PM, Bjorn Andersson bjorn.anders...@sonymobile.com wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com Tested-by: Rob Clark robdcl...@gmail.com (also on a z3, with work-in-progress panel driver for the AUO panel + drm/msm) BR, -R --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) diff --git a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt index a85a964d61f5..424f8444a6cd 100644 --- a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt +++ b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt @@ -5,10 +5,7 @@ Required properties: - reg: slave address Optional properties: -- label: The label for this led - See Documentation/devicetree/bindings/leds/common.txt -- linux,default-trigger: Default trigger assigned to the LED - See Documentation/devicetree/bindings/leds/common.txt +- label: The name of the backlight device - qcom,cs-out: bool; enable current sink output - qcom,cabc: bool; enable content adaptive backlight control - qcom,ext-gen: bool; use externally generated modulator signal to dim diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig index 9ad35f72ab4c..b8d4b965ca2a 100644 --- a/drivers/leds/Kconfig +++ b/drivers/leds/Kconfig @@ -578,14 +578,6 @@ config LEDS_VERSATILE This option enabled support for the LEDs on the ARM Versatile and RealView boards. Say Y to enabled these. -config LEDS_PM8941_WLED - tristate LED support for the Qualcomm PM8941 WLED block - depends on LEDS_CLASS - select REGMAP - help - This option enables support for the 'White' LED block - on Qualcomm PM8941 PMICs. - comment LED Triggers source drivers/leds/trigger/Kconfig diff --git a/drivers/leds/Makefile b/drivers/leds/Makefile index 8d6a24a2f513..abe96d960ebe 100644 --- a/drivers/leds/Makefile +++ b/drivers/leds/Makefile @@ -63,7 +63,6 @@ obj-$(CONFIG_LEDS_BLINKM) += leds-blinkm.o obj-$(CONFIG_LEDS_SYSCON) += leds-syscon.o obj-$(CONFIG_LEDS_VERSATILE) += leds-versatile.o obj-$(CONFIG_LEDS_MENF21BMC) += leds-menf21bmc.o -obj-$(CONFIG_LEDS_PM8941_WLED) += leds-pm8941-wled.o obj-$(CONFIG_LEDS_KTD2692) += leds-ktd2692.o # LED SPI Drivers diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig index 0505b796d743..6c67c5430933 100644 --- a/drivers/video/backlight/Kconfig +++ b/drivers/video/backlight/Kconfig @@ -299,6 +299,14 @@ config BACKLIGHT_TOSA If you have an Sharp SL-6000 Zaurus say Y to enable a driver for its backlight +config BACKLIGHT_PM8941_WLED + tristate Qualcomm PM8941 WLED Driver + depends on LEDS_CLASS + select REGMAP + help + If you have the Qualcomm PM8941, say Y to enable a driver for the + WLED block. + config BACKLIGHT_SAHARA tristate Tabletkiosk Sahara Touch-iT Backlight Driver depends on X86 diff --git a/drivers/video/backlight/Makefile b/drivers/video/backlight/Makefile index d67073f9d421..16ec534cff30 100644 --- a/drivers/video/backlight/Makefile +++ b/drivers/video/backlight/Makefile @@ -48,6 +48,7 @@ obj-$(CONFIG_BACKLIGHT_OMAP1) += omap1_bl.o obj-$(CONFIG_BACKLIGHT_OT200) += ot200_bl.o obj-$(CONFIG_BACKLIGHT_PANDORA)+= pandora_bl.o obj-$(CONFIG_BACKLIGHT_PCF50633) += pcf50633-backlight.o +obj-$(CONFIG_BACKLIGHT_PM8941_WLED)+= pm8941-wled.o obj-$(CONFIG_BACKLIGHT_PWM)+= pwm_bl.o obj-$(CONFIG_BACKLIGHT_SAHARA) += kb3886_bl.o obj-$(CONFIG_BACKLIGHT_SKY81452) += sky81452-backlight.o diff --git a/drivers/leds/leds-pm8941-wled.c b/drivers/video/backlight/pm8941-wled.c similarity index 90% rename from drivers/leds/leds-pm8941-wled.c rename to drivers/video/backlight/pm8941-wled.c index bf64a593fbf1..4f5ae95331a1 100644 --- a/drivers/leds/leds-pm8941-wled.c +++ b/drivers/video/backlight/pm8941-wled.c @@ -11,7 +11,7 @@ */ #include linux/kernel.h -#include linux/leds.h +#include linux/backlight.h #include linux/module.h #include linux/of.h #include linux/of_device.h @@ -76,30 +76,29 @@
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On Tue, 21 Jul 2015, Jingoo Han wrote: On 2015. 7. 21., at AM 6:11, Jacek Anaszewski jacek.anaszew...@gmail.com wrote: On 20.07.2015 16:15, Jingoo Han wrote: On Thursday, July 16, 2015 5:01 PM, Jacek Anaszewski wrote: On 07/15/2015 10:02 PM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig | 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) Acked-by: Jacek Anaszewski j.anaszew...@samsung.com If you are not an author of this driver or a maintainer affected by this patch, Just 'Reviewed-by' looks good. I am a LED subsystem maintainer. OK, I see. I didn't notice that you were added as the maintainer 2 months ago. Please read: Sections 12 and 13 of Documentation/SubmittingPatches. Granted, Acked-bys are used by Maintainers when reviewing changes in subsystems they are responsible for. However, that's not the sole use of them. Anyone can provide an Ack for code they have reviewed, but are not an expert in. Reviewed-bys have much stronger meaning and should only be used if the supplier knows the code and the subsystem well. Please see SubmittingPatches for a full meaning of the Reviewed-by tag. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On Thursday, July 16, 2015 5:01 PM, Jacek Anaszewski wrote: On 07/15/2015 10:02 PM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) Acked-by: Jacek Anaszewski j.anaszew...@samsung.com If you are not an author of this driver or a maintainer affected by this patch, Just 'Reviewed-by' looks good. Best regards, Jingoo Han -- Best Regards, Jacek Anaszewski -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On Mon 20 Jul 07:22 PDT 2015, Jingoo Han wrote: On Thursday, July 16, 2015 5:02 AM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) diff --git a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt index a85a964d61f5..424f8444a6cd 100644 --- a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt +++ b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt Please move this txt file to 'backlight' directory. Of course [..] diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig index 0505b796d743..6c67c5430933 100644 --- a/drivers/video/backlight/Kconfig +++ b/drivers/video/backlight/Kconfig @@ -299,6 +299,14 @@ config BACKLIGHT_TOSA If you have an Sharp SL-6000 Zaurus say Y to enable a driver for its backlight +config BACKLIGHT_PM8941_WLED + tristate Qualcomm PM8941 WLED Driver + depends on LEDS_CLASS LEDS_CLASS? Please check your patch carefully. Sorry about that. One more thing, Did you test this patch with the board? If not, I will not accept this patch, unless other people send 'Tested-by'. I've tested the driver on the Sony Xperia Z3 and it visually behaves as expected. I'll send you an updated patch shortly. Thank you, Bjorn -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On 20.07.2015 16:15, Jingoo Han wrote: On Thursday, July 16, 2015 5:01 PM, Jacek Anaszewski wrote: On 07/15/2015 10:02 PM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) Acked-by: Jacek Anaszewski j.anaszew...@samsung.com If you are not an author of this driver or a maintainer affected by this patch, Just 'Reviewed-by' looks good. I am a LED subsystem maintainer. -- Best Regards, Jacek Anaszewski -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
On 2015. 7. 21., at AM 6:11, Jacek Anaszewski jacek.anaszew...@gmail.com wrote: On 20.07.2015 16:15, Jingoo Han wrote: On Thursday, July 16, 2015 5:01 PM, Jacek Anaszewski wrote: On 07/15/2015 10:02 PM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig | 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) Acked-by: Jacek Anaszewski j.anaszew...@samsung.com If you are not an author of this driver or a maintainer affected by this patch, Just 'Reviewed-by' looks good. I am a LED subsystem maintainer. OK, I see. I didn't notice that you were added as the maintainer 2 months ago. Best regards, Jingoo Han-- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
Hi Bjorn, On 07/15/2015 10:02 PM, Bjorn Andersson wrote: The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) Acked-by: Jacek Anaszewski j.anaszew...@samsung.com -- Best Regards, Jacek Anaszewski -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] backlight: pm8941-wled: Move PM8941 WLED driver to backlight
The Qualcomm PM8941 WLED block is used for backlight and should therefor be in the backlight framework and not in the LED framework. This moves the driver and adapts to the backlight api instead. Signed-off-by: Bjorn Andersson bjorn.anders...@sonymobile.com --- .../devicetree/bindings/leds/leds-pm8941-wled.txt | 5 +- drivers/leds/Kconfig | 8 --- drivers/leds/Makefile | 1 - drivers/video/backlight/Kconfig| 8 +++ drivers/video/backlight/Makefile | 1 + .../backlight/pm8941-wled.c} | 59 ++ 6 files changed, 36 insertions(+), 46 deletions(-) rename drivers/{leds/leds-pm8941-wled.c = video/backlight/pm8941-wled.c} (90%) diff --git a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt index a85a964d61f5..424f8444a6cd 100644 --- a/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt +++ b/Documentation/devicetree/bindings/leds/leds-pm8941-wled.txt @@ -5,10 +5,7 @@ Required properties: - reg: slave address Optional properties: -- label: The label for this led - See Documentation/devicetree/bindings/leds/common.txt -- linux,default-trigger: Default trigger assigned to the LED - See Documentation/devicetree/bindings/leds/common.txt +- label: The name of the backlight device - qcom,cs-out: bool; enable current sink output - qcom,cabc: bool; enable content adaptive backlight control - qcom,ext-gen: bool; use externally generated modulator signal to dim diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig index 9ad35f72ab4c..b8d4b965ca2a 100644 --- a/drivers/leds/Kconfig +++ b/drivers/leds/Kconfig @@ -578,14 +578,6 @@ config LEDS_VERSATILE This option enabled support for the LEDs on the ARM Versatile and RealView boards. Say Y to enabled these. -config LEDS_PM8941_WLED - tristate LED support for the Qualcomm PM8941 WLED block - depends on LEDS_CLASS - select REGMAP - help - This option enables support for the 'White' LED block - on Qualcomm PM8941 PMICs. - comment LED Triggers source drivers/leds/trigger/Kconfig diff --git a/drivers/leds/Makefile b/drivers/leds/Makefile index 8d6a24a2f513..abe96d960ebe 100644 --- a/drivers/leds/Makefile +++ b/drivers/leds/Makefile @@ -63,7 +63,6 @@ obj-$(CONFIG_LEDS_BLINKM) += leds-blinkm.o obj-$(CONFIG_LEDS_SYSCON) += leds-syscon.o obj-$(CONFIG_LEDS_VERSATILE) += leds-versatile.o obj-$(CONFIG_LEDS_MENF21BMC) += leds-menf21bmc.o -obj-$(CONFIG_LEDS_PM8941_WLED) += leds-pm8941-wled.o obj-$(CONFIG_LEDS_KTD2692) += leds-ktd2692.o # LED SPI Drivers diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig index 0505b796d743..6c67c5430933 100644 --- a/drivers/video/backlight/Kconfig +++ b/drivers/video/backlight/Kconfig @@ -299,6 +299,14 @@ config BACKLIGHT_TOSA If you have an Sharp SL-6000 Zaurus say Y to enable a driver for its backlight +config BACKLIGHT_PM8941_WLED + tristate Qualcomm PM8941 WLED Driver + depends on LEDS_CLASS + select REGMAP + help + If you have the Qualcomm PM8941, say Y to enable a driver for the + WLED block. + config BACKLIGHT_SAHARA tristate Tabletkiosk Sahara Touch-iT Backlight Driver depends on X86 diff --git a/drivers/video/backlight/Makefile b/drivers/video/backlight/Makefile index d67073f9d421..16ec534cff30 100644 --- a/drivers/video/backlight/Makefile +++ b/drivers/video/backlight/Makefile @@ -48,6 +48,7 @@ obj-$(CONFIG_BACKLIGHT_OMAP1) += omap1_bl.o obj-$(CONFIG_BACKLIGHT_OT200) += ot200_bl.o obj-$(CONFIG_BACKLIGHT_PANDORA)+= pandora_bl.o obj-$(CONFIG_BACKLIGHT_PCF50633) += pcf50633-backlight.o +obj-$(CONFIG_BACKLIGHT_PM8941_WLED)+= pm8941-wled.o obj-$(CONFIG_BACKLIGHT_PWM)+= pwm_bl.o obj-$(CONFIG_BACKLIGHT_SAHARA) += kb3886_bl.o obj-$(CONFIG_BACKLIGHT_SKY81452) += sky81452-backlight.o diff --git a/drivers/leds/leds-pm8941-wled.c b/drivers/video/backlight/pm8941-wled.c similarity index 90% rename from drivers/leds/leds-pm8941-wled.c rename to drivers/video/backlight/pm8941-wled.c index bf64a593fbf1..4f5ae95331a1 100644 --- a/drivers/leds/leds-pm8941-wled.c +++ b/drivers/video/backlight/pm8941-wled.c @@ -11,7 +11,7 @@ */ #include linux/kernel.h -#include linux/leds.h +#include linux/backlight.h #include linux/module.h #include linux/of.h #include linux/of_device.h @@ -76,30 +76,29 @@ struct pm8941_wled_config { }; struct pm8941_wled { + const char *name; struct regmap *regmap; u16 addr; - struct led_classdev cdev; - struct pm8941_wled_config cfg; }; -static int pm8941_wled_set(struct led_classdev *cdev, - enum led_brightness value)