# \[PATCH 1/2\] dt-bindings: leds: backlight: qcom-wled: Add PMI8994 compatible - Subsystems: ARM/QUALCOMM MAILING LIST, LED SUBSYSTEM, OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, THE REST - Maintainers: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Torvalds - List: linux-fbdev - 13 messages, 5 authors - Page: https://ratatoskr.run/linux-fbdev/2021/02/6280851/t ## \[PATCH 1/2\] dt-bindings: leds: backlight: qcom-wled: Add PMI8994 compatible - From: Konrad Dybcio \ - Date: 2021-02-28 12:43:07 - Message: https://ratatoskr.run/linux-fbdev/2021/02/6280851.md ```text Document the newly added PMI8994 compatible. Signed-off-by: Konrad Dybcio --- Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml index 47938e372987..d839e75d9788 100644 --- a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml +++ b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml @@ -19,6 +19,7 @@ properties: compatible: enum: - qcom,pm8941-wled + - qcom,pmi8994-wled - qcom,pmi8998-wled - qcom,pm660l-wled - qcom,pm8150l-wled -- 2.30.1 ``` ## \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Konrad Dybcio \ - Date: 2021-02-28 12:43:08 - Message: https://ratatoskr.run/linux-fbdev/2021/02/6280852.md ```text Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. Signed-off-by: Konrad Dybcio --- drivers/video/backlight/qcom-wled.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c index 3bc7800eb0a9..497b9035a908 100644 --- a/drivers/video/backlight/qcom-wled.c +++ b/drivers/video/backlight/qcom-wled.c @@ -1704,6 +1704,7 @@ static int wled_remove(struct platform_device *pdev) static const struct of_device_id wled_match_table[] = { { .compatible = "qcom,pm8941-wled", .data = (void *)3 }, + { .compatible = "qcom,pmi8994-wled", .data = (void *)4 }, { .compatible = "qcom,pmi8998-wled", .data = (void *)4 }, { .compatible = "qcom,pm660l-wled", .data = (void *)4 }, { .compatible = "qcom,pm8150l-wled", .data = (void *)5 }, -- 2.30.1 ``` ## Re: \[PATCH 1/2\] dt-bindings: leds: backlight: qcom-wled: Add PMI8994 compatible - From: Rob Herring \ - Date: 2021-03-08 17:23:59 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6292521.md ```text On Sun, 28 Feb 2021 13:41:04 +0100, Konrad Dybcio wrote: > Document the newly added PMI8994 compatible. > > Signed-off-by: Konrad Dybcio > --- > Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml | 1 + > 1 file changed, 1 insertion(+) > Acked-by: Rob Herring ``` ## Re: \[PATCH 1/2\] dt-bindings: leds: backlight: qcom-wled: Add PMI8994 compatible - From: Lee Jones \ - Date: 2021-03-10 09:22:26 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6294736.md ```text On Sun, 28 Feb 2021, Konrad Dybcio wrote: > Document the newly added PMI8994 compatible. > > Signed-off-by: Konrad Dybcio Applied, thanks. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ``` ## Re: \[PATCH 1/2\] dt-bindings: leds: backlight: qcom-wled: Add PMI8994 compatible - From: Daniel Thompson \ - Date: 2021-03-22 16:18:48 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6310914.md ```text On Sun, Feb 28, 2021 at 01:41:04PM +0100, Konrad Dybcio wrote: > Document the newly added PMI8994 compatible. > > Signed-off-by: Konrad Dybcio (For Lee) Acked-by: Daniel Thompson [ref] Daniel. > --- > Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml > index 47938e372987..d839e75d9788 100644 > --- a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml > +++ b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml > @@ -19,6 +19,7 @@ properties: > compatible: > enum: > - qcom,pm8941-wled > + - qcom,pmi8994-wled > - qcom,pmi8998-wled > - qcom,pm660l-wled > - qcom,pm8150l-wled > -- > 2.30.1 ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Daniel Thompson \ - Date: 2021-03-22 16:19:21 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6310915.md ```text On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > Signed-off-by: Konrad Dybcio Reviewed-by: Daniel Thompson Daniel. > --- > drivers/video/backlight/qcom-wled.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > index 3bc7800eb0a9..497b9035a908 100644 > --- a/drivers/video/backlight/qcom-wled.c > +++ b/drivers/video/backlight/qcom-wled.c > @@ -1704,6 +1704,7 @@ static int wled_remove(struct platform_device *pdev) > > static const struct of_device_id wled_match_table[] = { > { .compatible = "qcom,pm8941-wled", .data = (void *)3 }, > + { .compatible = "qcom,pmi8994-wled", .data = (void *)4 }, > { .compatible = "qcom,pmi8998-wled", .data = (void *)4 }, > { .compatible = "qcom,pm660l-wled", .data = (void *)4 }, > { .compatible = "qcom,pm8150l-wled", .data = (void *)5 }, > -- > 2.30.1 ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Lee Jones \ - Date: 2021-03-23 08:40:31 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6311854.md ```text On Mon, 22 Mar 2021, Daniel Thompson wrote: > On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > > > Signed-off-by: Konrad Dybcio > > Reviewed-by: Daniel Thompson Why are you Reviewing/Acking a patch that was applied on the 10th? > > --- > > drivers/video/backlight/qcom-wled.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > > index 3bc7800eb0a9..497b9035a908 100644 > > --- a/drivers/video/backlight/qcom-wled.c > > +++ b/drivers/video/backlight/qcom-wled.c > > @@ -1704,6 +1704,7 @@ static int wled_remove(struct platform_device *pdev) > > > > static const struct of_device_id wled_match_table[] = { > > { .compatible = "qcom,pm8941-wled", .data = (void *)3 }, > > + { .compatible = "qcom,pmi8994-wled", .data = (void *)4 }, > > { .compatible = "qcom,pmi8998-wled", .data = (void *)4 }, > > { .compatible = "qcom,pm660l-wled", .data = (void *)4 }, > > { .compatible = "qcom,pm8150l-wled", .data = (void *)5 }, -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Daniel Thompson \ - Date: 2021-03-23 09:36:39 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6311933.md ```text On Tue, Mar 23, 2021 at 08:39:35AM +0000, Lee Jones wrote: > On Mon, 22 Mar 2021, Daniel Thompson wrote: > > > On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > > > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > > > > > Signed-off-by: Konrad Dybcio > > > > Reviewed-by: Daniel Thompson > > Why are you Reviewing/Acking a patch that was applied on the 10th? Simply an oversight. Looks like I forgot to remove it from my backlog when it was applied. Daniel. > > > > --- > > > drivers/video/backlight/qcom-wled.c | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > > > index 3bc7800eb0a9..497b9035a908 100644 > > > --- a/drivers/video/backlight/qcom-wled.c > > > +++ b/drivers/video/backlight/qcom-wled.c > > > @@ -1704,6 +1704,7 @@ static int wled_remove(struct platform_device *pdev) > > > > > > static const struct of_device_id wled_match_table[] = { > > > { .compatible = "qcom,pm8941-wled", .data = (void *)3 }, > > > + { .compatible = "qcom,pmi8994-wled", .data = (void *)4 }, > > > { .compatible = "qcom,pmi8998-wled", .data = (void *)4 }, > > > { .compatible = "qcom,pm660l-wled", .data = (void *)4 }, > > > { .compatible = "qcom,pm8150l-wled", .data = (void *)5 }, > > -- > Lee Jones [李琼斯] > Senior Technical Lead - Developer Services > Linaro.org │ Open source software for Arm SoCs > Follow Linaro: Facebook | Twitter | Blog ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Lee Jones \ - Date: 2021-03-23 09:43:36 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6311947.md ```text On Tue, 23 Mar 2021, Daniel Thompson wrote: > On Tue, Mar 23, 2021 at 08:39:35AM +0000, Lee Jones wrote: > > On Mon, 22 Mar 2021, Daniel Thompson wrote: > > > > > On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > > > > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > > > > > > > Signed-off-by: Konrad Dybcio > > > > > > Reviewed-by: Daniel Thompson > > > > Why are you Reviewing/Acking a patch that was applied on the 10th? > > Simply an oversight. Looks like I forgot to remove it from my backlog > when it was applied. I wondered if I'd made a mistake. No problem. Thanks for the clarification. > > > > --- > > > > drivers/video/backlight/qcom-wled.c | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > > > > index 3bc7800eb0a9..497b9035a908 100644 > > > > --- a/drivers/video/backlight/qcom-wled.c > > > > +++ b/drivers/video/backlight/qcom-wled.c > > > > @@ -1704,6 +1704,7 @@ static int wled_remove(struct platform_device *pdev) > > > > > > > > static const struct of_device_id wled_match_table[] = { > > > > { .compatible = "qcom,pm8941-wled", .data = (void *)3 }, > > > > + { .compatible = "qcom,pmi8994-wled", .data = (void *)4 }, > > > > { .compatible = "qcom,pmi8998-wled", .data = (void *)4 }, > > > > { .compatible = "qcom,pm660l-wled", .data = (void *)4 }, > > > > { .compatible = "qcom,pm8150l-wled", .data = (void *)5 }, > > -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Geert Uytterhoeven \ - Date: 2021-03-24 08:55:42 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6313450.md ```text Hi Lee, On Tue, Mar 23, 2021 at 9:40 AM Lee Jones [ref] wrote: > On Mon, 22 Mar 2021, Daniel Thompson wrote: > > On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > > > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > > > > > Signed-off-by: Konrad Dybcio > > > > Reviewed-by: Daniel Thompson > > Why are you Reviewing/Acking a patch that was applied on the 10th? Only 12 days later?!? It's not uncommon to receive acks for patches after they have been applied upstream. But it is if the patch was applied 10 years and 9 months ago! https://lore.kernel.org/linux-m68k/F5513AE92A5A1047AC2F91AEBB9202680288CBBA3983@E2K7-MS2.ds.strath.ac.uk/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Lee Jones \ - Date: 2021-03-24 09:15:12 - Message: https://ratatoskr.run/linux-fbdev/2021/03/6313471.md ```text On Wed, 24 Mar 2021, Geert Uytterhoeven wrote: > Hi Lee, > > On Tue, Mar 23, 2021 at 9:40 AM Lee Jones [ref] wrote: > > On Mon, 22 Mar 2021, Daniel Thompson wrote: > > > On Sun, Feb 28, 2021 at 01:41:05PM +0100, Konrad Dybcio wrote: > > > > Add a compatible for PMI8994 WLED. It uses the V4 of WLED IP. > > > > > > > > Signed-off-by: Konrad Dybcio > > > > > > Reviewed-by: Daniel Thompson > > > > Why are you Reviewing/Acking a patch that was applied on the 10th? > > Only 12 days later?!? > > It's not uncommon to receive acks for patches after they have been > applied upstream. But it is if the patch was applied 10 years and 9 > months ago! > https://lore.kernel.org/linux-m68k/F5513AE92A5A1047AC2F91AEBB9202680288CBBA3983@E2K7-MS2.ds.strath.ac.uk/ That truly is next level! :) -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Konrad Dybcio \ - Date: 2021-05-27 16:10:11 - Message: https://ratatoskr.run/linux-fbdev/2021/05/1394276.md ```text Hi, > Why are you Reviewing/Acking a patch that was applied on the 10th? Forgive me if it turns out I'm blind, but I can't see the patch being in either -next, backlight/for-next, or 5.13-rc3. Perhaps it was omitted after all? Konrad ``` ## Re: \[PATCH 2/2\] video: backlight: qcom-wled: Add PMI8994 compatible - From: Lee Jones \ - Date: 2021-05-27 16:26:14 - Message: https://ratatoskr.run/linux-fbdev/2021/05/1394311.md ```text On Thu, 27 May 2021, Konrad Dybcio wrote: > Hi, > > > Why are you Reviewing/Acking a patch that was applied on the 10th? > > Forgive me if it turns out I'm blind, but I can't see the patch > being in either -next, backlight/for-next, or 5.13-rc3. Perhaps it > was omitted after all? Interesting. It's not you, it's me! No idea what happened exactly TBH, but it's applied again. Should be in -next tomorrow. Apologies for the delay. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ```