From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1764300AbcINQEA (ORCPT ); Wed, 14 Sep 2016 12:04:00 -0400 Received: from mail.kmu-office.ch ([178.209.48.109]:55606 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761966AbcINQD5 (ORCPT ); Wed, 14 Sep 2016 12:03:57 -0400 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Wed, 14 Sep 2016 08:58:31 -0700 From: Stefan Agner To: Sanchayan Maity Cc: shawnguo@kernel.org, marcel.ziswiler@toradex.com, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1 2/3] ARM: dts: imx6q-apalis-ixora: Remove use of pwm-leds In-Reply-To: <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan@gmail.com> References: <9d2c17aa2e00e6ff4349a53de8247860d2382607.1473833908.git.maitysanchayan@gmail.com> <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan@gmail.com> Message-ID: <5eee36b0a60c41a05ca2a161b8ef3eb7@agner.ch> User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016-09-13 23:35, Sanchayan Maity wrote: > Remove use of pwm-leds and use the standard /sys/class/pwm > interface from PWM subsystem. Don't we have actual leds on the Ixora? -- Stefan > > Signed-off-by: Sanchayan Maity > --- > arch/arm/boot/dts/imx6q-apalis-ixora.dts | 22 ---------------------- > 1 file changed, 22 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > index d99979e..70a3da0 100644 > --- a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > +++ b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > @@ -146,28 +146,6 @@ > gpios = <&gpio2 2 GPIO_ACTIVE_HIGH>; > }; > }; > - > - pwmleds { > - compatible = "pwm-leds"; > - > - ledpwm1 { > - label = "PWM1"; > - pwms = <&pwm1 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm2 { > - label = "PWM2"; > - pwms = <&pwm2 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm3 { > - label = "PWM3"; > - pwms = <&pwm3 0 50000>; > - max-brightness = <255>; > - }; > - }; > }; > > &backlight { From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Agner Subject: Re: [PATCH v1 2/3] ARM: dts: imx6q-apalis-ixora: Remove use of pwm-leds Date: Wed, 14 Sep 2016 08:58:31 -0700 Message-ID: <5eee36b0a60c41a05ca2a161b8ef3eb7@agner.ch> References: <9d2c17aa2e00e6ff4349a53de8247860d2382607.1473833908.git.maitysanchayan@gmail.com> <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sanchayan Maity Cc: shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, marcel.ziswiler-2KBjVHiyJgBBDgjK7y7TUQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 2016-09-13 23:35, Sanchayan Maity wrote: > Remove use of pwm-leds and use the standard /sys/class/pwm > interface from PWM subsystem. Don't we have actual leds on the Ixora? -- Stefan > > Signed-off-by: Sanchayan Maity > --- > arch/arm/boot/dts/imx6q-apalis-ixora.dts | 22 ---------------------- > 1 file changed, 22 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > index d99979e..70a3da0 100644 > --- a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > +++ b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > @@ -146,28 +146,6 @@ > gpios = <&gpio2 2 GPIO_ACTIVE_HIGH>; > }; > }; > - > - pwmleds { > - compatible = "pwm-leds"; > - > - ledpwm1 { > - label = "PWM1"; > - pwms = <&pwm1 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm2 { > - label = "PWM2"; > - pwms = <&pwm2 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm3 { > - label = "PWM3"; > - pwms = <&pwm3 0 50000>; > - max-brightness = <255>; > - }; > - }; > }; > > &backlight { -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: stefan@agner.ch (Stefan Agner) Date: Wed, 14 Sep 2016 08:58:31 -0700 Subject: [PATCH v1 2/3] ARM: dts: imx6q-apalis-ixora: Remove use of pwm-leds In-Reply-To: <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan@gmail.com> References: <9d2c17aa2e00e6ff4349a53de8247860d2382607.1473833908.git.maitysanchayan@gmail.com> <6111feb57b5fbaa29eeb6ba089bb2824bbf2f8ff.1473833908.git.maitysanchayan@gmail.com> Message-ID: <5eee36b0a60c41a05ca2a161b8ef3eb7@agner.ch> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 2016-09-13 23:35, Sanchayan Maity wrote: > Remove use of pwm-leds and use the standard /sys/class/pwm > interface from PWM subsystem. Don't we have actual leds on the Ixora? -- Stefan > > Signed-off-by: Sanchayan Maity > --- > arch/arm/boot/dts/imx6q-apalis-ixora.dts | 22 ---------------------- > 1 file changed, 22 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > index d99979e..70a3da0 100644 > --- a/arch/arm/boot/dts/imx6q-apalis-ixora.dts > +++ b/arch/arm/boot/dts/imx6q-apalis-ixora.dts > @@ -146,28 +146,6 @@ > gpios = <&gpio2 2 GPIO_ACTIVE_HIGH>; > }; > }; > - > - pwmleds { > - compatible = "pwm-leds"; > - > - ledpwm1 { > - label = "PWM1"; > - pwms = <&pwm1 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm2 { > - label = "PWM2"; > - pwms = <&pwm2 0 50000>; > - max-brightness = <255>; > - }; > - > - ledpwm3 { > - label = "PWM3"; > - pwms = <&pwm3 0 50000>; > - max-brightness = <255>; > - }; > - }; > }; > > &backlight {