From: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> To: Ahmad Fatoum <a.fatoum@pengutronix.de> Cc: devicetree@vger.kernel.org, Fabio Estevam <festevam@gmail.com>, Richard Zhu <hongxing.zhu@nxp.com>, linux-amarula@amarulasolutions.com, Sascha Hauer <s.hauer@pengutronix.de>, Joakim Zhang <qiangqing.zhang@nxp.com>, linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, NXP Linux Team <linux-imx@nxp.com>, Pengutronix Kernel Team <kernel@pengutronix.de>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>, Tim Harvey <tharvey@gateworks.com>, linux-arm-kernel@lists.infradead.org, Li Jun <jun.li@nxp.com> Subject: Re: [PATCH v3] arm64: dts: imx8mm-evk: add pwm1/backlight support Date: Wed, 13 Apr 2022 23:35:46 +0200 [thread overview] Message-ID: <20220413213546.GA11567@tom-ThinkPad-T14s-Gen-2i> (raw) In-Reply-To: <11f8bf43-d6f7-12fe-f5da-1f7a397c4317@pengutronix.de> On Wed, Apr 13, 2022 at 06:31:34PM +0200, Ahmad Fatoum wrote: > Hello Tomasso, > > On 13.04.22 18:08, Tommaso Merciai wrote: > > Add pwm1/backlight support nodes for imx8mm_evk board. > > Align with u-boot dts > > > > References: > > - https://patchwork.ozlabs.org/project/uboot/patch/20220326111911.13720-9-tommaso.merciai@amarulasolutions.com/ > > > > Signed-off-by: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> > > --- > > Changes since v1: > > - Fix commit body > > - Enable pwm, backlight > > > > Changes since v2: > > - Remove status okay from pwm1,backlight (enable as default) > > > > arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi | 19 +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > index 6d67df7692f1..c50d2cc3ca71 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > @@ -59,6 +59,14 @@ reg_usdhc2_vmmc: regulator-usdhc2 { > > enable-active-high; > > }; > > > > + backlight: backlight { > > + compatible = "pwm-backlight"; > > + pwms = <&pwm1 0 5000000>; > > + brightness-levels = <0 255>; > > + num-interpolated-steps = <255>; > > + default-brightness-level = <250>; > > + }; > > + > > ir-receiver { > > compatible = "gpio-ir-receiver"; > > gpios = <&gpio1 13 GPIO_ACTIVE_LOW>; > > @@ -395,6 +403,11 @@ &wdog1 { > > status = "okay"; > > }; > > > > +&pwm1 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_backlight>; > > imx8mm.dtsi has &pwm1 { status = "disabled" }; > Where is it enabled? Hi Ahmad, Thanks for review, update in v4. Tommaso > > > +}; > > + > > &iomuxc { > > pinctrl_fec1: fec1grp { > > fsl,pins = < > > @@ -549,4 +562,10 @@ pinctrl_wdog: wdoggrp { > > MX8MM_IOMUXC_GPIO1_IO02_WDOG1_WDOG_B 0x166 > > >; > > }; > > + > > + pinctrl_backlight: backlightgrp { > > + fsl,pins = < > > + MX8MM_IOMUXC_GPIO1_IO01_PWM1_OUT 0x06 > > + >; > > + }; > > }; > > > -- > 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 | -- Tommaso Merciai Embedded Linux Engineer tommaso.merciai@amarulasolutions.com __________________________________ Amarula Solutions SRL Via Le Canevare 30, 31100 Treviso, Veneto, IT T. +39 042 243 5310 info@amarulasolutions.com www.amarulasolutions.com
WARNING: multiple messages have this Message-ID (diff)
From: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> To: Ahmad Fatoum <a.fatoum@pengutronix.de> Cc: devicetree@vger.kernel.org, Fabio Estevam <festevam@gmail.com>, Richard Zhu <hongxing.zhu@nxp.com>, linux-amarula@amarulasolutions.com, Sascha Hauer <s.hauer@pengutronix.de>, Joakim Zhang <qiangqing.zhang@nxp.com>, linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, NXP Linux Team <linux-imx@nxp.com>, Pengutronix Kernel Team <kernel@pengutronix.de>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>, Tim Harvey <tharvey@gateworks.com>, linux-arm-kernel@lists.infradead.org, Li Jun <jun.li@nxp.com> Subject: Re: [PATCH v3] arm64: dts: imx8mm-evk: add pwm1/backlight support Date: Wed, 13 Apr 2022 23:35:46 +0200 [thread overview] Message-ID: <20220413213546.GA11567@tom-ThinkPad-T14s-Gen-2i> (raw) In-Reply-To: <11f8bf43-d6f7-12fe-f5da-1f7a397c4317@pengutronix.de> On Wed, Apr 13, 2022 at 06:31:34PM +0200, Ahmad Fatoum wrote: > Hello Tomasso, > > On 13.04.22 18:08, Tommaso Merciai wrote: > > Add pwm1/backlight support nodes for imx8mm_evk board. > > Align with u-boot dts > > > > References: > > - https://patchwork.ozlabs.org/project/uboot/patch/20220326111911.13720-9-tommaso.merciai@amarulasolutions.com/ > > > > Signed-off-by: Tommaso Merciai <tommaso.merciai@amarulasolutions.com> > > --- > > Changes since v1: > > - Fix commit body > > - Enable pwm, backlight > > > > Changes since v2: > > - Remove status okay from pwm1,backlight (enable as default) > > > > arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi | 19 +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > index 6d67df7692f1..c50d2cc3ca71 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi > > @@ -59,6 +59,14 @@ reg_usdhc2_vmmc: regulator-usdhc2 { > > enable-active-high; > > }; > > > > + backlight: backlight { > > + compatible = "pwm-backlight"; > > + pwms = <&pwm1 0 5000000>; > > + brightness-levels = <0 255>; > > + num-interpolated-steps = <255>; > > + default-brightness-level = <250>; > > + }; > > + > > ir-receiver { > > compatible = "gpio-ir-receiver"; > > gpios = <&gpio1 13 GPIO_ACTIVE_LOW>; > > @@ -395,6 +403,11 @@ &wdog1 { > > status = "okay"; > > }; > > > > +&pwm1 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_backlight>; > > imx8mm.dtsi has &pwm1 { status = "disabled" }; > Where is it enabled? Hi Ahmad, Thanks for review, update in v4. Tommaso > > > +}; > > + > > &iomuxc { > > pinctrl_fec1: fec1grp { > > fsl,pins = < > > @@ -549,4 +562,10 @@ pinctrl_wdog: wdoggrp { > > MX8MM_IOMUXC_GPIO1_IO02_WDOG1_WDOG_B 0x166 > > >; > > }; > > + > > + pinctrl_backlight: backlightgrp { > > + fsl,pins = < > > + MX8MM_IOMUXC_GPIO1_IO01_PWM1_OUT 0x06 > > + >; > > + }; > > }; > > > -- > 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 | -- Tommaso Merciai Embedded Linux Engineer tommaso.merciai@amarulasolutions.com __________________________________ Amarula Solutions SRL Via Le Canevare 30, 31100 Treviso, Veneto, IT T. +39 042 243 5310 info@amarulasolutions.com www.amarulasolutions.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-04-13 21:35 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-13 16:08 [PATCH v3] arm64: dts: imx8mm-evk: add pwm1/backlight support Tommaso Merciai 2022-04-13 16:08 ` Tommaso Merciai 2022-04-13 16:31 ` Ahmad Fatoum 2022-04-13 16:31 ` Ahmad Fatoum 2022-04-13 21:35 ` Tommaso Merciai [this message] 2022-04-13 21:35 ` Tommaso Merciai
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=20220413213546.GA11567@tom-ThinkPad-T14s-Gen-2i \ --to=tommaso.merciai@amarulasolutions.com \ --cc=a.fatoum@pengutronix.de \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=hongxing.zhu@nxp.com \ --cc=jun.li@nxp.com \ --cc=kernel@pengutronix.de \ --cc=krzk+dt@kernel.org \ --cc=linux-amarula@amarulasolutions.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-imx@nxp.com \ --cc=linux-kernel@vger.kernel.org \ --cc=qiangqing.zhang@nxp.com \ --cc=robh+dt@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=shawnguo@kernel.org \ --cc=tharvey@gateworks.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: linkBe 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.