On Mon, Mar 09, 2020 at 04:45:05PM +0000, Philippe Schenker wrote:
> On Mon, 2020-03-09 at 14:55 +0000, Paul Barker wrote:
> > The pwm-imx27 driver supports a third argument in a pwms reference to
> > provide flags. However #pwm-cells needs to be set to <3> in the pwm
> > node
> > so that these flags can be passed through to the driver. Once #pwm-
> > cells
> > is <3>, all pwms references need to use the 3-cell form.
> >
> > Signed-off-by: Paul Barker <pbarker@xxxxxxxxxxxx>
> >
> > Cc: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx>
> > Cc: Pantelis Antoniou <pantelis.antoniou@xxxxxxxxxxxx>
> > Cc: Rob Herring robh+dt@xxxxxxxxxx
> > Cc: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> > Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> > Cc: Shawn Guo <shawnguo@xxxxxxxxxx>
> > ---
> > arch/arm/boot/dts/imx6dl-aristainetos_4.dts | 2 +-
> > arch/arm/boot/dts/imx6dl-aristainetos_7.dts | 2 +-
> > arch/arm/boot/dts/imx6dl-mamoj.dts | 2 +-
> > arch/arm/boot/dts/imx6dl-yapp4-common.dtsi | 1 -
> > arch/arm/boot/dts/imx6q-ba16.dtsi | 2 +-
> > arch/arm/boot/dts/imx6q-display5.dtsi | 1 -
> > arch/arm/boot/dts/imx6q-kp.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6q-logicpd.dts | 2 +-
> > arch/arm/boot/dts/imx6q-mccmon6.dts | 1 -
> > arch/arm/boot/dts/imx6q-novena.dts | 2 +-
> > arch/arm/boot/dts/imx6q-pistachio.dts | 2 +-
> > arch/arm/boot/dts/imx6q-var-dt6customboard.dts | 2 +-
> > arch/arm/boot/dts/imx6qdl-apalis.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-apf6dev.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-aristainetos2.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-colibri.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-cubox-i.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-emcon.dtsi | 6 +++---
> > arch/arm/boot/dts/imx6qdl-gw52xx.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-gw53xx.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-gw54xx.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-gw560x.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-gw5903.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-gw5904.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-icore.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-kontron-samx6i.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi | 6 +++---
> > arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6qdl-phytec-mira.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-sabrelite.dtsi | 6 +++---
> > arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-savageboard.dtsi | 2 +-
> > arch/arm/boot/dts/imx6qdl-tx6.dtsi | 2 --
> > arch/arm/boot/dts/imx6qdl.dtsi | 8 ++++----
> > arch/arm/boot/dts/imx6sl-evk.dts | 2 +-
> > arch/arm/boot/dts/imx6sl.dtsi | 8 ++++----
> > arch/arm/boot/dts/imx6sll-evk.dts | 2 +-
> > arch/arm/boot/dts/imx6sll.dtsi | 8 ++++----
> > arch/arm/boot/dts/imx6sx-nitrogen6sx.dts | 2 +-
> > arch/arm/boot/dts/imx6sx-sdb.dtsi | 2 +-
> > arch/arm/boot/dts/imx6sx-softing-vining-2000.dts | 6 +++---
> > arch/arm/boot/dts/imx6sx.dtsi | 16 ++++++++-------
> > -
> > arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 2 +-
> > arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts | 2 +-
> > arch/arm/boot/dts/imx6ul-geam.dts | 2 +-
> > arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi | 2 +-
> > arch/arm/boot/dts/imx6ul-isiot.dtsi | 2 +-
> > arch/arm/boot/dts/imx6ul-kontron-n6310-s-43.dts | 2 +-
> > arch/arm/boot/dts/imx6ul-kontron-n6x1x-s.dtsi | 2 +-
> > arch/arm/boot/dts/imx6ul-pico.dtsi | 2 +-
> > arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 1 -
> > arch/arm/boot/dts/imx6ul.dtsi | 16 ++++++++-------
> > -
> > arch/arm/boot/dts/imx6ull-colibri.dtsi | 4 ----
> > 56 files changed, 85 insertions(+), 95 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/imx6dl-aristainetos_4.dts
> > b/arch/arm/boot/dts/imx6dl-aristainetos_4.dts
> > index 37f80ab8ccd0..f87ac95b70bc 100644
> > --- a/arch/arm/boot/dts/imx6dl-aristainetos_4.dts
> > +++ b/arch/arm/boot/dts/imx6dl-aristainetos_4.dts
> > @@ -14,7 +14,7 @@
> >
> > backlight {
> > compatible = "pwm-backlight";
> > - pwms = <&pwm1 0 5000000>;
> > + pwms = <&pwm1 0 5000000 0>;
>
> There is a header file existing that has PWM_POLARITY_INVERTED defined
> in include/dt-bindings/pwm/pwm.h. I feel we should add there a
> PWM_POLARITY_NORMAL instead of just using 0.
ack.
> But then I'm asking myself if we can't get rid of the file include/dt-
> bindings/pwm/pwm.h and instead use include/linux/pwm.h everywhere.
>
> Or shouldn't we use defines in devicetrees from there?
devicetrees are supposed to be stand-alone. So yes indeed, no linux
includes in device tree files.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
