Hi,

since backlight will only come in 3.9, I can add some remarks myself, so I 
don't forget them next time.

On Saturday 17 November 2012 00:20:32 Marc Dietrich wrote:
> This enables backlight using power sequences.
> 
> Signed-off-by: Marc Dietrich <[email protected]>
> ---
>  arch/arm/boot/dts/tegra20-paz00.dts |   72
> +++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts
> b/arch/arm/boot/dts/tegra20-paz00.dts index 8a14e1d..7349aa3 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -15,6 +15,10 @@
>                       rgb {
>                               status = "okay";
>                               nvidia,ddc-i2c-bus = <&lvds_ddc>;
> +
> +                             display {
> +                                     nvidia,backlight = <&backlight>;
> +                             };
>                       };
>               };
>       };
> @@ -461,6 +465,74 @@
>               };
>       };
> 
> +     backlight: backlight {
> +             compatible = "pwm-backlight";
> +             brightness-levels = <0 16 32 48 64 80 96 112 128 144 160 176 
> 192 208 
224
> 240 255>; +           default-brightness-level = <12>;
> +
> +             /* resources used by the power sequences */
> +             pwms = <&pwm 0 5000000>;
> +             pwm-names = "backlight";
> +
> +             power-sequences {
> +                     power-on {
> +                             step0 {
> +                                     type = "pwm";
> +                                     id = "backlight";
> +                                     enable;
> +                             };
> +                             step1 {
> +                                     type = "gpio";
> +                                     gpio = <&gpio 4 0>;   /* gpio PA4, 
> en_vdd_pnl  */
> +                                     value = <1>;
> +                             };
> +                             step2 {
> +                                     type = "gpio";
> +                                     gpio = <&gpio 176 0>; /* gpio PW0, 
> bl_vdd */

this one needs to be removed as it is not used. There is bl_pwm (gpio U3) 
however which I cannot sort in here.

> +                                     value = <1>;
> +                             };
> +                             step3 {
> +                                     type = "gpio";
> +                                     gpio = <&gpio 102 0>; /* gpio PM6, 
> lvds_shdn */
> +                                     value = <1>;
> +                             };
> +                             step4 {
> +                                     type = "gpio";
> +                                     gpio = <&gpio 164 0>; /* gpio PU4, 
> bl_enb */
> +                                     value = <1>;
> +                             };
> +                     };
> +
> +                     power-off {
> +                             step4 {
> +                                     type = "gpio";
> +                                     gpio = <&gpio 164 0>; /* gpio PU4, 
> bl_enb */
> +                                     value = <0>;

and this is totally bogus.

/me notes: better do not send patches after midnight!

Marc

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to