lkml.org 
[lkml]   [2019]   [Mar]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH V6 4/5] ARM: dts: imx7ulp: Add pwm0 support
Date


Best Regards!
Anson Huang

> -----Original Message-----
> From: Uwe Kleine-König [mailto:u.kleine-koenig@pengutronix.de]
> Sent: 2019年3月19日 14:55
> To: Anson Huang <anson.huang@nxp.com>
> Cc: thierry.reding@gmail.com; robh+dt@kernel.org; mark.rutland@arm.com;
> shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de;
> festevam@gmail.com; linux@armlinux.org.uk; stefan@agner.ch;
> otavio@ossystems.com.br; Leonard Crestez <leonard.crestez@nxp.com>;
> schnitzeltony@gmail.com; jan.tuerk@emtrion.com; Robin Gong
> <yibin.gong@nxp.com>; linux-pwm@vger.kernel.org;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> kernel@vger.kernel.org; dl-linux-imx <linux-imx@nxp.com>
> Subject: Re: [PATCH V6 4/5] ARM: dts: imx7ulp: Add pwm0 support
>
> On Tue, Mar 19, 2019 at 06:50:26AM +0000, Anson Huang wrote:
> > Add i.MX7ULP EVK board PWM0 support.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> > ---
> > Changes since V5:
> > - change #pwm-cells to 3.
> > ---
> > arch/arm/boot/dts/imx7ulp-evk.dts | 12 ++++++++++++
> > arch/arm/boot/dts/imx7ulp.dtsi | 10 ++++++++++
>
> IMHO the change to the evk.dts belongs into the last patch.

Ah, yes, the PWM pin settings in board dts should belongs into the backlight patch,
I will resend V6, thanks.

Anson.

>
> Best regards
> Uwe
>
> --
> Pengutronix e.K. | Uwe Kleine-König |
> Industrial Linux Solutions |
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.p
> engutronix.de%2F&amp;data=02%7C01%7Canson.huang%40nxp.com%7Ce3
> 645a6d4d6643a1f80f08d6ac37c725%7C686ea1d3bc2b4c6fa92cd99c5c301635
> %7C0%7C0%7C636885752897208572&amp;sdata=N67sms9lX6asXpYaq%2Ftf
> TNpNz%2BeDIKNx89%2F57CYQAwA%3D&amp;reserved=0 |
\
 
 \ /
  Last update: 2019-03-19 08:03    [W:0.032 / U:0.192 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site