Subject: [PATCH v2 5/6] arm: dts: imx7-colibri: Use pwm polarity control

From: [email-masked] (Lukasz Majewski)
Date: Fri, 7 Oct 2016 08:41:20 +0200

On Fri, 7 Oct 2016 11:38:52 +0530
Bhuvanchandra DV <[email-masked]> wrote:

> On 10/06/16 12:10, Lukasz Majewski wrote:
> 
> > On Sat, 1 Oct 2016 15:42:34 +0530
> > Bhuvanchandra DV <[email-masked]> wrote:
> >
> >> Configure PWM polarity control.
> >>
> >> Signed-off-by: Bhuvanchandra DV <[email-masked]>
> >> ---
> >>   arch/arm/boot/dts/imx7-colibri.dtsi | 2 +-
> >>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/arch/arm/boot/dts/imx7-colibri.dtsi
> >> b/arch/arm/boot/dts/imx7-colibri.dtsi index a9cc657..2af5e3e 100644
> >> --- a/arch/arm/boot/dts/imx7-colibri.dtsi
> >> +++ b/arch/arm/boot/dts/imx7-colibri.dtsi
> >> @@ -43,7 +43,7 @@
> >>   / {
> >>   	bl: backlight {
> >>   		compatible = "pwm-backlight";
> >> -		pwms = <&pwm1 0 5000000>;
> >> +		pwms = <&pwm1 0 5000000 0>;
> > My recommendation would be to add:
> > #include <dt-bindings/pwm/pwm.h>
> >
> > and then define pwms as:
> >
> > pwms = <&pwm1 0 5000000 PWM_POLARITY_NORMAL>;
> 
> Addendum:
> There is no "PWM_POLARITY_NORMAL" defined in "#include
> <dt-bindings/pwm/pwm.h>" I guess it is helpful when polarity
> inversion is enabled.

Yes, you are right :-), the normal polarity is assumed to be the
default.

There is only defined flag for inversion.

Sorry for the noise.

Best regards,
?ukasz Majewski

> 
> --
> Bhuvan
> 
> >
> > It would be more readable
> >
> > Best regards,
> > ?ukasz Majewski
> >
> >>   	};
> >>   
> >>   	reg_module_3v3: regulator-module-3v3 {
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161007/d9a2f031/attachment.sig>