From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: "thierry.reding@gmail.com" <thierry.reding@gmail.com>,
"linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
"linux-renesas-soc@vger.kernel.org"
<linux-renesas-soc@vger.kernel.org>
Subject: RE: [PATCH v2 2/4] pwm: rcar: Use "atomic" API on rcar_pwm_resume()
Date: Tue, 8 Jan 2019 08:46:30 +0000 [thread overview]
Message-ID: <TY2PR01MB481233D0989EEA601A584377D88A0@TY2PR01MB4812.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <20190108074749.d23mpcr3wscr6s5j@pengutronix.de>
Hello Uwe,
> From: Uwe Kleine-Konig, Sent: Tuesday, January 8, 2019 4:48 PM
> > diff --git a/drivers/pwm/pwm-rcar.c b/drivers/pwm/pwm-rcar.c
> > index ba70e83..4987c12 100644
> > --- a/drivers/pwm/pwm-rcar.c
> > +++ b/drivers/pwm/pwm-rcar.c
> > @@ -316,18 +316,16 @@ static int rcar_pwm_suspend(struct device *dev)
> > static int rcar_pwm_resume(struct device *dev)
> > {
> > struct pwm_device *pwm = rcar_pwm_dev_to_pwm_dev(dev);
> > + struct pwm_state state;
> >
> > if (!test_bit(PWMF_REQUESTED, &pwm->flags))
> > return 0;
> >
> > pm_runtime_get_sync(dev);
> >
> > - rcar_pwm_config(pwm->chip, pwm, pwm->state.duty_cycle,
> > - pwm->state.period);
> > - if (pwm_is_enabled(pwm))
> > - rcar_pwm_enable(pwm->chip, pwm);
> > + pwm_get_state(pwm, &state);
> >
> > - return 0;
> > + return rcar_pwm_apply(pwm->chip, pwm, &state);
> > }
>
> Orthogonal to this patch I wonder what the intended behaviour for a pwm
> is on suspend. Should it stop oscilating unconditionally? Or should it
> only stop if the consumer stops it as part of its own suspend callback?
I think the second one is better. I checked drivers/video/backlight/pwm_bl.c
and it is possible to call pwm_apply_state() by the driver after rcar_pwm_suspend()
was called. So, I'll fix this as other patch.
> As the patch only reworks and improves the code without a change in
> behaviour, it is fine for me.
>
> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thanks!
Best regards,
Yoshihiro Shimoda
> Best regards
> Uwe
>
> --
> Pengutronix e.K. | Uwe Kleine-König |
> Industrial Linux Solutions | http://www.pengutronix.de/ |
next prev parent reply other threads:[~2019-01-08 8:46 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-08 3:28 [PATCH v2 0/4] pwm: rcar: Add support "atomic" API and improve calculation Yoshihiro Shimoda
2019-01-08 3:28 ` [PATCH v2 1/4] pwm: rcar: Add support "atomic" API Yoshihiro Shimoda
2019-01-08 7:43 ` Uwe Kleine-König
2019-01-08 8:11 ` Yoshihiro Shimoda
2019-01-08 3:28 ` [PATCH v2 2/4] pwm: rcar: Use "atomic" API on rcar_pwm_resume() Yoshihiro Shimoda
2019-01-08 7:47 ` Uwe Kleine-König
2019-01-08 8:46 ` Yoshihiro Shimoda [this message]
2019-01-08 9:12 ` Uwe Kleine-König
2019-01-08 8:56 ` Geert Uytterhoeven
2019-01-08 9:19 ` Uwe Kleine-König
2019-01-08 9:35 ` Geert Uytterhoeven
2019-01-10 9:51 ` Uwe Kleine-König
2019-01-08 3:28 ` [PATCH v2 3/4] pwm: rcar: remove legacy APIs Yoshihiro Shimoda
2019-01-08 7:49 ` Uwe Kleine-König
2019-01-08 3:28 ` [PATCH v2 4/4] pwm: rcar: improve calculation of divider Yoshihiro Shimoda
2019-01-08 7:49 ` Uwe Kleine-König
2019-01-08 8:21 ` Geert Uytterhoeven
2019-01-08 9:30 ` Yoshihiro Shimoda
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=TY2PR01MB481233D0989EEA601A584377D88A0@TY2PR01MB4812.jpnprd01.prod.outlook.com \
--to=yoshihiro.shimoda.uh@renesas.com \
--cc=linux-pwm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=thierry.reding@gmail.com \
--cc=u.kleine-koenig@pengutronix.de \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).