Messages in this thread | | | Date | Fri, 17 Jun 2022 15:28:59 +0300 | Subject | Re: [PATCH 11/11] ASoC: ti: davinci-mcasp: use pm_runtime_resume_and_get() | From | Péter Ujfalusi <> |
| |
Hi Pierre,
On 17/06/2022 01:04, Pierre-Louis Bossart wrote: > The use of pm_runtime_get_sync() is buggy with no use of put_noidle on > error. > > Use pm_runtime_resume_and_get() instead.
Thanks for the fix,
Acked-by: Peter Ujfalusi <peter.ujfalusi@gmail.com>
> > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com> > Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com> > Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com> > --- > sound/soc/ti/davinci-mcasp.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/sound/soc/ti/davinci-mcasp.c b/sound/soc/ti/davinci-mcasp.c > index e2aab4729f3ab..0201000b619f6 100644 > --- a/sound/soc/ti/davinci-mcasp.c > +++ b/sound/soc/ti/davinci-mcasp.c > @@ -2111,8 +2111,7 @@ static int davinci_mcasp_gpio_request(struct gpio_chip *chip, unsigned offset) > } > > /* Do not change the PIN yet */ > - > - return pm_runtime_get_sync(mcasp->dev); > + return pm_runtime_resume_and_get(mcasp->dev); > } > > static void davinci_mcasp_gpio_free(struct gpio_chip *chip, unsigned offset)
-- Péter
| |