[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1521319566.2835.0@smtp.crapouillou.net>
Date: Sat, 17 Mar 2018 21:46:04 +0100
From: Paul Cercueil <paul@...pouillou.net>
To: Thierry Reding <thierry.reding@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>
Cc: linux-pwm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/4] pwm: pwm-jz4740: Implement set_polarity
Hi,
Could this patchset get a bit of love?
I have other changes waiting for this patchset to get in, so it'd be
great
to see it in 4.17-rc1.
Thanks,
-Paul
Le sam. 6 janv. 2018 à 17:58, Paul Cercueil <paul@...pouillou.net> a
écrit :
> This permits clients of this driver to specify the polarity to use for
> their PWM channel.
>
> Signed-off-by: Paul Cercueil <paul@...pouillou.net>
> ---
> drivers/pwm/pwm-jz4740.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/drivers/pwm/pwm-jz4740.c b/drivers/pwm/pwm-jz4740.c
> index 2e41ba213f39..6539c001fe32 100644
> --- a/drivers/pwm/pwm-jz4740.c
> +++ b/drivers/pwm/pwm-jz4740.c
> @@ -130,10 +130,29 @@ static int jz4740_pwm_config(struct pwm_chip
> *chip, struct pwm_device *pwm,
> return 0;
> }
>
> +static int jz4740_pwm_set_polarity(struct pwm_chip *chip,
> + struct pwm_device *pwm, enum pwm_polarity polarity)
> +{
> + uint32_t ctrl = jz4740_timer_get_ctrl(pwm->pwm);
> +
> + switch (polarity) {
> + case PWM_POLARITY_NORMAL:
> + ctrl &= ~JZ_TIMER_CTRL_PWM_ACTIVE_LOW;
> + break;
> + case PWM_POLARITY_INVERSED:
> + ctrl |= JZ_TIMER_CTRL_PWM_ACTIVE_LOW;
> + break;
> + }
> +
> + jz4740_timer_set_ctrl(pwm->hwpwm, ctrl);
> + return 0;
> +}
> +
> static const struct pwm_ops jz4740_pwm_ops = {
> .request = jz4740_pwm_request,
> .free = jz4740_pwm_free,
> .config = jz4740_pwm_config,
> + .set_polarity = jz4740_pwm_set_polarity,
> .enable = jz4740_pwm_enable,
> .disable = jz4740_pwm_disable,
> .owner = THIS_MODULE,
> --
> 2.11.0
>
Powered by blists - more mailing lists