All of lore.kernel.org
 help / color / mirror / Atom feed
From: Oleksij Rempel <o.rempel@pengutronix.de>
To: Fabio Estevam <festevam@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
	<devicetree@vger.kernel.org>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	DRI mailing list <dri-devel@lists.freedesktop.org>,
	Robin van der Gracht <robin@protonic.nl>,
	David Airlie <airlied@linux.ie>, Sam Ravnborg <sam@ravnborg.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Daniel Vetter <daniel@ffwll.ch>, David Jander <david@protonic.nl>,
	Shawn Guo <shawnguo@kernel.org>,
	"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 6/7] ARM: dts: imx6dl-prtvt7: fix PWM cell count for the backlight node.
Date: Fri, 22 Jan 2021 10:42:07 +0100	[thread overview]
Message-ID: <20210122094207.w2upvl5k5sad7u7p@pengutronix.de> (raw)
In-Reply-To: <CAOMZO5C6RM2vEMFJB-+Nen1Et8wn39JJVM1UcOcar0aMebKykQ@mail.gmail.com>

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 |

WARNING: multiple messages have this Message-ID (diff)
From: Oleksij Rempel <o.rempel@pengutronix.de>
To: Fabio Estevam <festevam@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
	<devicetree@vger.kernel.org>, Daniel Vetter <daniel@ffwll.ch>,
	David Airlie <airlied@linux.ie>,
	Robin van der Gracht <robin@protonic.nl>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	DRI mailing list <dri-devel@lists.freedesktop.org>,
	Rob Herring <robh+dt@kernel.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	David Jander <david@protonic.nl>, Shawn Guo <shawnguo@kernel.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 6/7] ARM: dts: imx6dl-prtvt7: fix PWM cell count for the backlight node.
Date: Fri, 22 Jan 2021 10:42:07 +0100	[thread overview]
Message-ID: <20210122094207.w2upvl5k5sad7u7p@pengutronix.de> (raw)
In-Reply-To: <CAOMZO5C6RM2vEMFJB-+Nen1Et8wn39JJVM1UcOcar0aMebKykQ@mail.gmail.com>

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 |

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Oleksij Rempel <o.rempel@pengutronix.de>
To: Fabio Estevam <festevam@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
	<devicetree@vger.kernel.org>, David Airlie <airlied@linux.ie>,
	Robin van der Gracht <robin@protonic.nl>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	DRI mailing list <dri-devel@lists.freedesktop.org>,
	Rob Herring <robh+dt@kernel.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	David Jander <david@protonic.nl>, Shawn Guo <shawnguo@kernel.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 6/7] ARM: dts: imx6dl-prtvt7: fix PWM cell count for the backlight node.
Date: Fri, 22 Jan 2021 10:42:07 +0100	[thread overview]
Message-ID: <20210122094207.w2upvl5k5sad7u7p@pengutronix.de> (raw)
In-Reply-To: <CAOMZO5C6RM2vEMFJB-+Nen1Et8wn39JJVM1UcOcar0aMebKykQ@mail.gmail.com>

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 |
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2021-01-22  9:48 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-21  6:11 [PATCH v2 0/7] devicetree fixes for Protonic and Plymovent boards Oleksij Rempel
2021-01-21  6:11 ` Oleksij Rempel
2021-01-21  6:11 ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 1/7] dt-bindings: display: simple: add Innolux G070Y2-T02 panel Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-02-09  2:36   ` Rob Herring
2021-02-09  2:36     ` Rob Herring
2021-02-09  2:36     ` Rob Herring
2021-01-21  6:11 ` [PATCH v2 2/7] drm: panel-simple: Add support for the " Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 3/7] ARM: dts: imx6dl-prtvt7: Add display and panel nodes Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 4/7] ARM: dts: imx6dl-prtvt7: add TSC2046 touchscreen node Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 5/7] ARM: dts: imx6dl-prtvt7: Remove backlight enable gpio Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 6/7] ARM: dts: imx6dl-prtvt7: fix PWM cell count for the backlight node Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21 14:17   ` Fabio Estevam
2021-01-21 14:17     ` Fabio Estevam
2021-01-21 14:17     ` Fabio Estevam
2021-01-22  9:42     ` Oleksij Rempel [this message]
2021-01-22  9:42       ` Oleksij Rempel
2021-01-22  9:42       ` Oleksij Rempel
2021-01-21  6:11 ` [PATCH v2 7/7] ARM: dts: imx6dl-plym2m: remove touchscreen-size-* properties Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel
2021-01-21  6:11   ` Oleksij Rempel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210122094207.w2upvl5k5sad7u7p@pengutronix.de \
    --to=o.rempel@pengutronix.de \
    --cc=airlied@linux.ie \
    --cc=daniel@ffwll.ch \
    --cc=david@protonic.nl \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=robin@protonic.nl \
    --cc=s.hauer@pengutronix.de \
    --cc=sam@ravnborg.org \
    --cc=shawnguo@kernel.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.