LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight
@ 2021-10-29 11:44 Satya Priya
  2021-10-29 11:44 ` [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support Satya Priya
                   ` (3 more replies)
  0 siblings, 4 replies; 13+ messages in thread
From: Satya Priya @ 2021-10-29 11:44 UTC (permalink / raw)
  To: Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	Satya Priya

This series depends on [1], which adds driver for Qualcomm LPG.

[1] https://patchwork.kernel.org/project/linux-arm-msm/list/?series=560587&state=%2A&archive=both

Satya Priya (1):
  arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2

satya priya (3):
  dt-bindings: leds: Add pm8350c pmic support
  leds: Add pm8350c support to Qualcomm LPG driver
  arm64: dts: qcom: pm8350c: Add pwm support

 Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml |  1 +
 arch/arm64/boot/dts/qcom/pm8350c.dtsi                     |  7 +++++++
 arch/arm64/boot/dts/qcom/sc7280-idp2.dts                  |  4 ++++
 drivers/leds/rgb/leds-qcom-lpg.c                          | 13 +++++++++++++
 4 files changed, 25 insertions(+)

-- 
2.7.4


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support
  2021-10-29 11:44 [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight Satya Priya
@ 2021-10-29 11:44 ` Satya Priya
  2021-10-29 17:23   ` Trilok Soni
  2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 13+ messages in thread
From: Satya Priya @ 2021-10-29 11:44 UTC (permalink / raw)
  To: Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya

From: satya priya <skakit@codeaurora.org>

Add pm8350c pmic pwm support.

Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Acked-by: Rob Herring <robh@kernel.org>
---
Changes in V2:
 - No changes.

Changes in V3:
 - No changes.

 Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
index 336bd8e..409a4c7 100644
--- a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
+++ b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
@@ -19,6 +19,7 @@ properties:
     enum:
       - qcom,pm8150b-lpg
       - qcom,pm8150l-lpg
+      - qcom,pm8350c-pwm
       - qcom,pm8916-pwm
       - qcom,pm8941-lpg
       - qcom,pm8994-lpg
-- 
2.7.4


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver
  2021-10-29 11:44 [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight Satya Priya
  2021-10-29 11:44 ` [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support Satya Priya
@ 2021-10-29 11:44 ` Satya Priya
  2021-10-29 17:23   ` Trilok Soni
                     ` (2 more replies)
  2021-10-29 11:44 ` [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support Satya Priya
  2021-10-29 11:44 ` [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2 Satya Priya
  3 siblings, 3 replies; 13+ messages in thread
From: Satya Priya @ 2021-10-29 11:44 UTC (permalink / raw)
  To: Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya

From: satya priya <skakit@codeaurora.org>

Add pm8350c compatible and lpg_data to the driver.

Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
---
Changes in V2:
 - Added const for lpg_channel_data[] struct.

Changes in V3:
 - Correct the num_channels and add respective base addresses.

 drivers/leds/rgb/leds-qcom-lpg.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/drivers/leds/rgb/leds-qcom-lpg.c b/drivers/leds/rgb/leds-qcom-lpg.c
index 45ef4ec..ad99a9b 100644
--- a/drivers/leds/rgb/leds-qcom-lpg.c
+++ b/drivers/leds/rgb/leds-qcom-lpg.c
@@ -1275,9 +1275,22 @@ static const struct lpg_data pm8150l_lpg_data = {
 	},
 };
 
+static const struct lpg_data pm8350c_pwm_data = {
+	.pwm_9bit_mask = BIT(2),
+
+	.num_channels = 4,
+	.channels = (const struct lpg_channel_data[]) {
+		{ .base = 0xe800 },
+		{ .base = 0xe900 },
+		{ .base = 0xea00 },
+		{ .base = 0xeb00 },
+	},
+};
+
 static const struct of_device_id lpg_of_table[] = {
 	{ .compatible = "qcom,pm8150b-lpg", .data = &pm8150b_lpg_data },
 	{ .compatible = "qcom,pm8150l-lpg", .data = &pm8150l_lpg_data },
+	{ .compatible = "qcom,pm8350c-pwm", .data = &pm8350c_pwm_data },
 	{ .compatible = "qcom,pm8916-pwm", .data = &pm8916_pwm_data },
 	{ .compatible = "qcom,pm8941-lpg", .data = &pm8941_lpg_data },
 	{ .compatible = "qcom,pm8994-lpg", .data = &pm8994_lpg_data },
-- 
2.7.4


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support
  2021-10-29 11:44 [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight Satya Priya
  2021-10-29 11:44 ` [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support Satya Priya
  2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
@ 2021-10-29 11:44 ` Satya Priya
  2021-10-29 19:02   ` Stephen Boyd
  2021-10-29 11:44 ` [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2 Satya Priya
  3 siblings, 1 reply; 13+ messages in thread
From: Satya Priya @ 2021-10-29 11:44 UTC (permalink / raw)
  To: Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya

From: satya priya <skakit@codeaurora.org>

Add pwm support for PM8350C pmic.

Signed-off-by: satya priya <skakit@codeaurora.org>
---
Changes in V2:
 - Dropped suffix '4' from pwm phandle and removed "status=ok".

Changes in V3:
 - Add reg property.

 arch/arm64/boot/dts/qcom/pm8350c.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/pm8350c.dtsi b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
index e1b75ae..1ba6318 100644
--- a/arch/arm64/boot/dts/qcom/pm8350c.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
@@ -29,6 +29,13 @@
 			interrupt-controller;
 			#interrupt-cells = <2>;
 		};
+
+		pm8350c_pwm: pwm@e800 {
+			compatible = "qcom,pm8350c-pwm";
+			reg = <0xe800>;
+			#pwm-cells = <2>;
+			status = "disabled";
+		};
 	};
 };
 
-- 
2.7.4


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2
  2021-10-29 11:44 [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight Satya Priya
                   ` (2 preceding siblings ...)
  2021-10-29 11:44 ` [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support Satya Priya
@ 2021-10-29 11:44 ` Satya Priya
  2021-10-29 19:02   ` Stephen Boyd
  3 siblings, 1 reply; 13+ messages in thread
From: Satya Priya @ 2021-10-29 11:44 UTC (permalink / raw)
  To: Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	Satya Priya

Enable pm8350c pmic pwm support for backlight on sc7280-idp2.

Signed-off-by: Satya Priya <quic_c_skakit@quicinc.com>
---
Changes in V3:
 - New patch added in V3, to enable pwm support on sc7280-idp2 board.

 arch/arm64/boot/dts/qcom/sc7280-idp2.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp2.dts b/arch/arm64/boot/dts/qcom/sc7280-idp2.dts
index 3ae9969..07ca295 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp2.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp2.dts
@@ -29,3 +29,7 @@
 &nvme_3v3_regulator {
 	gpio = <&tlmm 51 GPIO_ACTIVE_HIGH>;
 };
+
+&pm8350c_pwm {
+	status = "okay";
+};
-- 
2.7.4


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support
  2021-10-29 11:44 ` [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support Satya Priya
@ 2021-10-29 17:23   ` Trilok Soni
  2021-11-26  8:57     ` Satya Priya Kakitapalli (Temp)
  0 siblings, 1 reply; 13+ messages in thread
From: Trilok Soni @ 2021-10-29 17:23 UTC (permalink / raw)
  To: Satya Priya, Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya

On 10/29/2021 4:44 AM, Satya Priya wrote:
> From: satya priya <skakit@codeaurora.org>
> 
> Add pm8350c pmic pwm support.
> 
> Signed-off-by: satya priya <skakit@codeaurora.org>

If you are sending email form quic_c_skakit@quicinc.com then can you 
please use the same email for SOB as well?


> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
> Reviewed-by: Stephen Boyd <swboyd@chromium.org>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
> Changes in V2:
>   - No changes.
> 
> Changes in V3:
>   - No changes.
> 
>   Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> index 336bd8e..409a4c7 100644
> --- a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> @@ -19,6 +19,7 @@ properties:
>       enum:
>         - qcom,pm8150b-lpg
>         - qcom,pm8150l-lpg
> +      - qcom,pm8350c-pwm
>         - qcom,pm8916-pwm
>         - qcom,pm8941-lpg
>         - qcom,pm8994-lpg
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver
  2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
@ 2021-10-29 17:23   ` Trilok Soni
  2021-10-29 19:01   ` Stephen Boyd
  2022-02-17 22:26   ` Bjorn Andersson
  2 siblings, 0 replies; 13+ messages in thread
From: Trilok Soni @ 2021-10-29 17:23 UTC (permalink / raw)
  To: Satya Priya, Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya

On 10/29/2021 4:44 AM, Satya Priya wrote:
> From: satya priya <skakit@codeaurora.org>
> 
> Add pm8350c compatible and lpg_data to the driver.
> 
> Signed-off-by: satya priya <skakit@codeaurora.org>

Similar comment like earlier patch.

---Trilok Soni


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver
  2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
  2021-10-29 17:23   ` Trilok Soni
@ 2021-10-29 19:01   ` Stephen Boyd
  2022-02-17 22:26   ` Bjorn Andersson
  2 siblings, 0 replies; 13+ messages in thread
From: Stephen Boyd @ 2021-10-29 19:01 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Pavel Machek, Rob Herring, Satya Priya
  Cc: mka, linux-leds, devicetree, linux-kernel, linux-arm-msm, satya priya

Quoting Satya Priya (2021-10-29 04:44:51)
> From: satya priya <skakit@codeaurora.org>
>
> Add pm8350c compatible and lpg_data to the driver.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
> ---

Reviewed-by: Stephen Boyd <swboyd@chromium.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support
  2021-10-29 11:44 ` [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support Satya Priya
@ 2021-10-29 19:02   ` Stephen Boyd
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Boyd @ 2021-10-29 19:02 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Pavel Machek, Rob Herring, Satya Priya
  Cc: mka, linux-leds, devicetree, linux-kernel, linux-arm-msm, satya priya

Quoting Satya Priya (2021-10-29 04:44:52)
> From: satya priya <skakit@codeaurora.org>
>
> Add pwm support for PM8350C pmic.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
> ---

Reviewed-by: Stephen Boyd <swboyd@chromium.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2
  2021-10-29 11:44 ` [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2 Satya Priya
@ 2021-10-29 19:02   ` Stephen Boyd
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Boyd @ 2021-10-29 19:02 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Pavel Machek, Rob Herring, Satya Priya
  Cc: mka, linux-leds, devicetree, linux-kernel, linux-arm-msm

Quoting Satya Priya (2021-10-29 04:44:53)
> Enable pm8350c pmic pwm support for backlight on sc7280-idp2.
>
> Signed-off-by: Satya Priya <quic_c_skakit@quicinc.com>
> ---

Reviewed-by: Stephen Boyd <swboyd@chromium.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support
  2021-10-29 17:23   ` Trilok Soni
@ 2021-11-26  8:57     ` Satya Priya Kakitapalli (Temp)
  0 siblings, 0 replies; 13+ messages in thread
From: Satya Priya Kakitapalli (Temp) @ 2021-11-26  8:57 UTC (permalink / raw)
  To: Trilok Soni, Pavel Machek, Rob Herring, Andy Gross, Bjorn Andersson
  Cc: mka, swboyd, linux-leds, devicetree, linux-kernel, linux-arm-msm,
	satya priya


On 10/29/2021 10:53 PM, Trilok Soni wrote:
> On 10/29/2021 4:44 AM, Satya Priya wrote:
>> From: satya priya <skakit@codeaurora.org>
>>
>> Add pm8350c pmic pwm support.
>>
>> Signed-off-by: satya priya <skakit@codeaurora.org>
>
> If you are sending email form quic_c_skakit@quicinc.com then can you 
> please use the same email for SOB as well?
>

Okay, I'll resend the V3 by changing SOB.


>
>> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
>> Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>> Acked-by: Rob Herring <robh@kernel.org>
>> ---
>> Changes in V2:
>>   - No changes.
>>
>> Changes in V3:
>>   - No changes.
>>
>>   Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git 
>> a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml 
>> b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
>> index 336bd8e..409a4c7 100644
>> --- a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
>> +++ b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
>> @@ -19,6 +19,7 @@ properties:
>>       enum:
>>         - qcom,pm8150b-lpg
>>         - qcom,pm8150l-lpg
>> +      - qcom,pm8350c-pwm
>>         - qcom,pm8916-pwm
>>         - qcom,pm8941-lpg
>>         - qcom,pm8994-lpg
>>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver
  2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
  2021-10-29 17:23   ` Trilok Soni
  2021-10-29 19:01   ` Stephen Boyd
@ 2022-02-17 22:26   ` Bjorn Andersson
  2022-02-22  4:45     ` Satya Priya Kakitapalli (Temp)
  2 siblings, 1 reply; 13+ messages in thread
From: Bjorn Andersson @ 2022-02-17 22:26 UTC (permalink / raw)
  To: Satya Priya
  Cc: Pavel Machek, Rob Herring, Andy Gross, mka, swboyd, linux-leds,
	devicetree, linux-kernel, linux-arm-msm, satya priya

On Fri 29 Oct 04:44 PDT 2021, Satya Priya wrote:

> From: satya priya <skakit@codeaurora.org>
> 
> Add pm8350c compatible and lpg_data to the driver.
> 
> Signed-off-by: satya priya <skakit@codeaurora.org>

Sorry for the OCD, but I would really like for you to capitalize 's' and
'p' in your name.

> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
> ---
> Changes in V2:
>  - Added const for lpg_channel_data[] struct.
> 
> Changes in V3:
>  - Correct the num_channels and add respective base addresses.
> 
>  drivers/leds/rgb/leds-qcom-lpg.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/leds/rgb/leds-qcom-lpg.c b/drivers/leds/rgb/leds-qcom-lpg.c
> index 45ef4ec..ad99a9b 100644
> --- a/drivers/leds/rgb/leds-qcom-lpg.c
> +++ b/drivers/leds/rgb/leds-qcom-lpg.c
> @@ -1275,9 +1275,22 @@ static const struct lpg_data pm8150l_lpg_data = {
>  	},
>  };
>  
> +static const struct lpg_data pm8350c_pwm_data = {
> +	.pwm_9bit_mask = BIT(2),

As you might have seen in v12 I dropped pwm_9bit_mask.

> +

Afaict there's a tri-channel current sink in pm8350c as well, so you
should have:

        .triled_base = 0xef00,

And then associated .triled_mask in the relevant channels.

Regards,
Bjorn

> +	.num_channels = 4,
> +	.channels = (const struct lpg_channel_data[]) {
> +		{ .base = 0xe800 },
> +		{ .base = 0xe900 },
> +		{ .base = 0xea00 },
> +		{ .base = 0xeb00 },
> +	},
> +};
> +
>  static const struct of_device_id lpg_of_table[] = {
>  	{ .compatible = "qcom,pm8150b-lpg", .data = &pm8150b_lpg_data },
>  	{ .compatible = "qcom,pm8150l-lpg", .data = &pm8150l_lpg_data },
> +	{ .compatible = "qcom,pm8350c-pwm", .data = &pm8350c_pwm_data },
>  	{ .compatible = "qcom,pm8916-pwm", .data = &pm8916_pwm_data },
>  	{ .compatible = "qcom,pm8941-lpg", .data = &pm8941_lpg_data },
>  	{ .compatible = "qcom,pm8994-lpg", .data = &pm8994_lpg_data },
> -- 
> 2.7.4
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver
  2022-02-17 22:26   ` Bjorn Andersson
@ 2022-02-22  4:45     ` Satya Priya Kakitapalli (Temp)
  0 siblings, 0 replies; 13+ messages in thread
From: Satya Priya Kakitapalli (Temp) @ 2022-02-22  4:45 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Pavel Machek, Rob Herring, Andy Gross, mka, swboyd, linux-leds,
	devicetree, linux-kernel, linux-arm-msm, satya priya

On 2/18/2022 3:56 AM, Bjorn Andersson wrote:

> On Fri 29 Oct 04:44 PDT 2021, Satya Priya wrote:
>
>> From: satya priya <skakit@codeaurora.org>
>>
>> Add pm8350c compatible and lpg_data to the driver.
>>
>> Signed-off-by: satya priya <skakit@codeaurora.org>
> Sorry for the OCD, but I would really like for you to capitalize 's' and
> 'p' in your name.


I had corrected this in the v3-resend version.

https://patchwork.kernel.org/project/linux-arm-msm/patch/1637917920-22041-3-git-send-email-quic_c_skakit@quicinc.com/


>
>> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
>> ---
>> Changes in V2:
>>   - Added const for lpg_channel_data[] struct.
>>
>> Changes in V3:
>>   - Correct the num_channels and add respective base addresses.
>>
>>   drivers/leds/rgb/leds-qcom-lpg.c | 13 +++++++++++++
>>   1 file changed, 13 insertions(+)
>>
>> diff --git a/drivers/leds/rgb/leds-qcom-lpg.c b/drivers/leds/rgb/leds-qcom-lpg.c
>> index 45ef4ec..ad99a9b 100644
>> --- a/drivers/leds/rgb/leds-qcom-lpg.c
>> +++ b/drivers/leds/rgb/leds-qcom-lpg.c
>> @@ -1275,9 +1275,22 @@ static const struct lpg_data pm8150l_lpg_data = {
>>   	},
>>   };
>>   
>> +static const struct lpg_data pm8350c_pwm_data = {
>> +	.pwm_9bit_mask = BIT(2),
> As you might have seen in v12 I dropped pwm_9bit_mask.


Yeah, I'll drop this.


>> +
> Afaict there's a tri-channel current sink in pm8350c as well, so you
> should have:
>
>          .triled_base = 0xef00,
>
> And then associated .triled_mask in the relevant channels.


Okay, I'll add them.


> Regards,
> Bjorn
>
>> +	.num_channels = 4,
>> +	.channels = (const struct lpg_channel_data[]) {
>> +		{ .base = 0xe800 },
>> +		{ .base = 0xe900 },
>> +		{ .base = 0xea00 },
>> +		{ .base = 0xeb00 },
>> +	},
>> +};
>> +
>>   static const struct of_device_id lpg_of_table[] = {
>>   	{ .compatible = "qcom,pm8150b-lpg", .data = &pm8150b_lpg_data },
>>   	{ .compatible = "qcom,pm8150l-lpg", .data = &pm8150l_lpg_data },
>> +	{ .compatible = "qcom,pm8350c-pwm", .data = &pm8350c_pwm_data },
>>   	{ .compatible = "qcom,pm8916-pwm", .data = &pm8916_pwm_data },
>>   	{ .compatible = "qcom,pm8941-lpg", .data = &pm8941_lpg_data },
>>   	{ .compatible = "qcom,pm8994-lpg", .data = &pm8994_lpg_data },
>> -- 
>> 2.7.4
>>

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2022-02-22  5:30 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-29 11:44 [PATCH V3 0/4] Add PM8350C PMIC PWM support for backlight Satya Priya
2021-10-29 11:44 ` [PATCH V3 1/4] dt-bindings: leds: Add pm8350c pmic support Satya Priya
2021-10-29 17:23   ` Trilok Soni
2021-11-26  8:57     ` Satya Priya Kakitapalli (Temp)
2021-10-29 11:44 ` [PATCH V3 2/4] leds: Add pm8350c support to Qualcomm LPG driver Satya Priya
2021-10-29 17:23   ` Trilok Soni
2021-10-29 19:01   ` Stephen Boyd
2022-02-17 22:26   ` Bjorn Andersson
2022-02-22  4:45     ` Satya Priya Kakitapalli (Temp)
2021-10-29 11:44 ` [PATCH V3 3/4] arm64: dts: qcom: pm8350c: Add pwm support Satya Priya
2021-10-29 19:02   ` Stephen Boyd
2021-10-29 11:44 ` [PATCH V3 4/4] arm64: dts: qcom: Enable pm8350c pwm for sc7280-idp2 Satya Priya
2021-10-29 19:02   ` Stephen Boyd

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).