mbox series

[v2,0/7] devicetree fixes for Protonic and Plymovent boards

Message ID 20210121061141.23062-1-o.rempel@pengutronix.de
Headers show
Series devicetree fixes for Protonic and Plymovent boards | expand

Message

Oleksij Rempel Jan. 21, 2021, 6:11 a.m. UTC 
changes v2:
- imx6dl-prtvt7: remove touchscreen-inverted-*

Oleksij Rempel (7):
  dt-bindings: display: simple: add Innolux G070Y2-T02 panel
  drm: panel-simple: Add support for the Innolux G070Y2-T02 panel
  ARM: dts: imx6dl-prtvt7: Add display and panel nodes
  ARM: dts: imx6dl-prtvt7: add TSC2046 touchscreen node
  ARM: dts: imx6dl-prtvt7: Remove backlight enable gpio
  ARM: dts: imx6dl-prtvt7: fix PWM cell count for the backlight node.
  ARM: dts: imx6dl-plym2m: remove touchscreen-size-* properties

 .../bindings/display/panel/panel-simple.yaml  |  2 +
 arch/arm/boot/dts/imx6dl-plym2m.dts           |  2 -
 arch/arm/boot/dts/imx6dl-prtvt7.dts           | 74 ++++++++++++++++---
 drivers/gpu/drm/panel/panel-simple.c          | 16 ++++
 4 files changed, 81 insertions(+), 13 deletions(-)

Comments

Fabio Estevam Jan. 21, 2021, 2:17 p.m. UTC | #1 
Hi Oleksij,

On Thu, Jan 21, 2021 at 3:12 AM Oleksij Rempel <o.rempel@pengutronix.de> wrote:
>
> At some point PWM cell count was changed, but it didn't triggered any

It changed in this commit:

commit fa28d8212ede9c533ae87a737571a9d3b3eebb29
Author: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Date:   Fri Jul 10 07:19:37 2020 +0200

    ARM: dts: imx: default to #pwm-cells = <3> in the SoC dtsi files

    The imx-pwm driver supports 3 cells and this is the more flexible setting.
    So use it by default and overwrite it back to two for the files that
    reference the PWMs with just 2 cells to minimize changes.

    This allows to drop explicit setting to 3 cells for the boards that already
    depend on this. The boards that are now using 2 cells explicitly can be
    converted to 3 individually.

    Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
    Signed-off-by: Shawn Guo <shawnguo@kernel.org>
Oleksij Rempel Jan. 22, 2021, 9:42 a.m. UTC | #2 
On Thu, Jan 21, 2021 at 11:17:42AM -0300, Fabio Estevam wrote:
> Hi Oleksij,

> 

> On Thu, Jan 21, 2021 at 3:12 AM Oleksij Rempel <o.rempel@pengutronix.de> wrote:

> >

> > At some point PWM cell count was changed, but it didn't triggered any

> 

> It changed in this commit:

> 

> commit fa28d8212ede9c533ae87a737571a9d3b3eebb29

> Author: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

> Date:   Fri Jul 10 07:19:37 2020 +0200

> 

>     ARM: dts: imx: default to #pwm-cells = <3> in the SoC dtsi files

> 

>     The imx-pwm driver supports 3 cells and this is the more flexible setting.

>     So use it by default and overwrite it back to two for the files that

>     reference the PWMs with just 2 cells to minimize changes.

> 

>     This allows to drop explicit setting to 3 cells for the boards that already

>     depend on this. The boards that are now using 2 cells explicitly can be

>     converted to 3 individually.

> 

>     Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

>     Signed-off-by: Shawn Guo <shawnguo@kernel.org>


OK, nice. Thx!

Should I resend this series without this patch?

Regards,
Oleksij
-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |