On Thursday 18 September 2014 07:47 PM, Shawn Guo wrote: > On Wed, Sep 17, 2014 at 01:46:30PM +0530, Bhuvanchandra DV wrote: >> Signed-off-by: Bhuvanchandra DV <bhuvanchandra...@toradex.com> >> --- >> arch/arm/boot/dts/vf610-colibri-eval-v3.dts | 6 ++++++ >> arch/arm/boot/dts/vf610-colibri.dtsi | 7 +++++++ >> 2 files changed, 13 insertions(+) >> >> diff --git a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts >> b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts >> index ba98039..f1988a0 100644 >> --- a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts >> +++ b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts >> @@ -52,3 +52,9 @@ >> &pwm1 { >> status = "okay"; >> }; >> + >> +&backlight { >> + brightness-levels = <0 4 8 16 32 64 128 255>; >> + default-brightness-level = <6>; >> + status = "okay"; >> +}; >> diff --git a/arch/arm/boot/dts/vf610-colibri.dtsi >> b/arch/arm/boot/dts/vf610-colibri.dtsi >> index 9bd1415..e725306 100644 >> --- a/arch/arm/boot/dts/vf610-colibri.dtsi >> +++ b/arch/arm/boot/dts/vf610-colibri.dtsi >> @@ -25,6 +25,13 @@ >> }; >> }; >> >> + backlight: bl { > I would suggest we name the node following the example in > Documentation/devicetree/bindings/pwm/pwm.txt, i.e. bl: backlight.
Will update the node name to bl: backlight >> + compitable = "pwm-backlight"; >> + pwms = <&pwm0 0 5000000 0>; >> + pwm-names = " backlight"; > Drop the leading space in the string. > > Shawn Ok >> + status = "disabled"; >> + }; >> + >> }; >> >> &esdhc1 { >> -- >> 1.7.9.5 >> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/