| Message ID | 20220208041606.144039-1-bjorn.andersson@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | a28106a2734f602d852a9269526f5880df352b51 |
| Headers | show |
| Series | [1/2] arm64: dts: qcom: c630: Add backlight controller | expand |
On 2/7/22 10:16 PM, Bjorn Andersson wrote: > The Lenovo Yoga C630 uses the PWM controller in the TI SN65DSI86 bridge > chip to provide a signal for the backlight control and has TLMM GPIO 11 > attached to some regulator that drives the backlight. > > Unfortunately the regulator attached to this gpio is also powering the > camera, so turning off backlight result in the detachment of the camera > as well. > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > index 58845a14805f..55fb7302245b 100644 > --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > @@ -59,6 +59,7 @@ mode { > panel { > compatible = "boe,nv133fhm-n61"; > no-hpd; > + backlight = <&backlight>; > > ports { > port { > @@ -98,6 +99,12 @@ sn65dsi86_refclk: sn65dsi86-refclk { > > clock-frequency = <19200000>; > }; > + > + backlight: backlight { > + compatible = "pwm-backlight"; > + pwms = <&sn65dsi86 1000000>; > + enable-gpios = <&tlmm 11 GPIO_ACTIVE_HIGH>; > + }; > }; > > &adsp_pas { > @@ -419,6 +426,7 @@ sn65dsi86: bridge@2c { > clock-names = "refclk"; > > no-hpd; > + #pwm-cells = <1>; > > ports { > #address-cells = <1>; Tested-by: Steev Klimaszewski <steev@kali.org>
Hello: This series was applied to qcom/linux.git (for-next) by Bjorn Andersson <bjorn.andersson@linaro.org>: On Mon, 7 Feb 2022 22:16:05 -0600 you wrote: > The Lenovo Yoga C630 uses the PWM controller in the TI SN65DSI86 bridge > chip to provide a signal for the backlight control and has TLMM GPIO 11 > attached to some regulator that drives the backlight. > > Unfortunately the regulator attached to this gpio is also powering the > camera, so turning off backlight result in the detachment of the camera > as well. > > [...] Here is the summary with links: - [1/2] arm64: dts: qcom: c630: Add backlight controller https://git.kernel.org/qcom/c/a28106a2734f - [2/2] arm64: dts: qcom: c630: Move panel to aux-bus https://git.kernel.org/qcom/c/ff899133fdae You are awesome, thank you!
diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts index 58845a14805f..55fb7302245b 100644 --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts @@ -59,6 +59,7 @@ mode { panel { compatible = "boe,nv133fhm-n61"; no-hpd; + backlight = <&backlight>; ports { port { @@ -98,6 +99,12 @@ sn65dsi86_refclk: sn65dsi86-refclk { clock-frequency = <19200000>; }; + + backlight: backlight { + compatible = "pwm-backlight"; + pwms = <&sn65dsi86 1000000>; + enable-gpios = <&tlmm 11 GPIO_ACTIVE_HIGH>; + }; }; &adsp_pas { @@ -419,6 +426,7 @@ sn65dsi86: bridge@2c { clock-names = "refclk"; no-hpd; + #pwm-cells = <1>; ports { #address-cells = <1>;
The Lenovo Yoga C630 uses the PWM controller in the TI SN65DSI86 bridge chip to provide a signal for the backlight control and has TLMM GPIO 11 attached to some regulator that drives the backlight. Unfortunately the regulator attached to this gpio is also powering the camera, so turning off backlight result in the detachment of the camera as well. Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 8 ++++++++ 1 file changed, 8 insertions(+)