Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933745AbaDIXP6 (ORCPT ); Wed, 9 Apr 2014 19:15:58 -0400 Received: from kirsty.vergenet.net ([202.4.237.240]:37892 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932820AbaDIXPy (ORCPT ); Wed, 9 Apr 2014 19:15:54 -0400 Date: Thu, 10 Apr 2014 08:15:49 +0900 From: Simon Horman To: Alexandre Belloni Cc: Thierry Reding , Tony Lindgren , Philipp Zabel , Paul Parsons , Magnus Damm , linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, linux-sh@vger.kernel.org Subject: Re: [PATCH 2/2] pwm: use PWM_LOOKUP to set the period and polarity Message-ID: <20140409231549.GD2187@verge.net.au> References: <1397066649-3767-1-git-send-email-alexandre.belloni@free-electrons.com> <1397066649-3767-3-git-send-email-alexandre.belloni@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1397066649-3767-3-git-send-email-alexandre.belloni@free-electrons.com> Organisation: Horms Solutions Ltd. User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Apr 09, 2014 at 08:04:09PM +0200, Alexandre Belloni wrote: > Now that the PWM core is able to set the period and polarity based on > the lookup table, add those to PWM_LOOKUP to ease their usage. I would prefer if this change was made in a non-atomic manner. 1. Add new infrastructure 2. Update users individually 3. Remove old infrastructure > > Signed-off-by: Alexandre Belloni > --- > Documentation/pwm.txt | 3 ++- > arch/arm/mach-omap2/board-omap3beagle.c | 3 ++- > arch/arm/mach-pxa/hx4700.c | 3 ++- > arch/arm/mach-shmobile/board-armadillo800eva.c | 3 ++- > include/linux/pwm.h | 4 +++- > 5 files changed, 11 insertions(+), 5 deletions(-) > > diff --git a/Documentation/pwm.txt b/Documentation/pwm.txt > index 93cb97974986..f38f99cda64f 100644 > --- a/Documentation/pwm.txt > +++ b/Documentation/pwm.txt > @@ -19,7 +19,8 @@ should instead register a static mapping that can be used to match PWM > consumers to providers, as given in the following example: > > static struct pwm_lookup board_pwm_lookup[] = { > - PWM_LOOKUP("tegra-pwm", 0, "pwm-backlight", NULL), > + PWM_LOOKUP("tegra-pwm", 0, "pwm-backlight", NULL, > + 50000, PWM_POLARITY_NORMAL), > }; > > static void __init board_init(void) > diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c > index d6ed819ff15c..54c135a5b4f7 100644 > --- a/arch/arm/mach-omap2/board-omap3beagle.c > +++ b/arch/arm/mach-omap2/board-omap3beagle.c > @@ -61,7 +61,8 @@ > > static struct pwm_lookup pwm_lookup[] = { > /* LEDB -> PMU_STAT */ > - PWM_LOOKUP("twl-pwmled", 1, "leds_pwm", "beagleboard::pmu_stat"), > + PWM_LOOKUP("twl-pwmled", 1, "leds_pwm", "beagleboard::pmu_stat", > + 7812500, PWM_POLARITY_NORMAL), > }; > > static struct led_pwm pwm_leds[] = { > diff --git a/arch/arm/mach-pxa/hx4700.c b/arch/arm/mach-pxa/hx4700.c > index a7c30eb0c8db..c66ad4edc5e3 100644 > --- a/arch/arm/mach-pxa/hx4700.c > +++ b/arch/arm/mach-pxa/hx4700.c > @@ -574,7 +574,8 @@ static struct platform_device backlight = { > }; > > static struct pwm_lookup hx4700_pwm_lookup[] = { > - PWM_LOOKUP("pxa27x-pwm.1", 0, "pwm-backlight", NULL), > + PWM_LOOKUP("pxa27x-pwm.1", 0, "pwm-backlight", NULL, > + 30923, PWM_POLARITY_NORMAL), > }; > > /* > diff --git a/arch/arm/mach-shmobile/board-armadillo800eva.c b/arch/arm/mach-shmobile/board-armadillo800eva.c > index 2858f380beae..e2c4c5010f19 100644 > --- a/arch/arm/mach-shmobile/board-armadillo800eva.c > +++ b/arch/arm/mach-shmobile/board-armadillo800eva.c > @@ -416,7 +416,8 @@ static struct platform_device pwm_device = { > }; > > static struct pwm_lookup pwm_lookup[] = { > - PWM_LOOKUP("renesas-tpu-pwm", 2, "pwm-backlight.0", NULL), > + PWM_LOOKUP("renesas-tpu-pwm", 2, "pwm-backlight.0", NULL, > + 33333, PWM_POLARITY_NORMAL), > }; > > /* LCDC and backlight */ > diff --git a/include/linux/pwm.h b/include/linux/pwm.h > index 2f45e2fe5b93..e90628cac8fa 100644 > --- a/include/linux/pwm.h > +++ b/include/linux/pwm.h > @@ -278,12 +278,14 @@ struct pwm_lookup { > enum pwm_polarity polarity; > }; > > -#define PWM_LOOKUP(_provider, _index, _dev_id, _con_id) \ > +#define PWM_LOOKUP(_provider, _index, _dev_id, _con_id, _period, _polarity) \ > { \ > .provider = _provider, \ > .index = _index, \ > .dev_id = _dev_id, \ > .con_id = _con_id, \ > + .period = _period, \ > + .polarity = _polarity \ > } > > #if IS_ENABLED(CONFIG_PWM) > -- > 1.8.3.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/