[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <44d7771e-5600-19c2-888a-dd226cbc4b50@nvidia.com>
Date: Tue, 23 Jun 2020 11:13:12 +0100
From: Jon Hunter <jonathanh@...dia.com>
To: Dinghao Liu <dinghao.liu@....edu.cn>, <kjlu@....edu>
CC: Laxman Dewangan <ldewangan@...dia.com>,
Vinod Koul <vkoul@...nel.org>,
Dan Williams <dan.j.williams@...el.com>,
Thierry Reding <thierry.reding@...il.com>,
<dmaengine@...r.kernel.org>, <linux-tegra@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] [v4] dmaengine: tegra210-adma: Fix runtime PM imbalance
on error
On 21/06/2020 06:47, Dinghao Liu wrote:
> pm_runtime_get_sync() increments the runtime PM usage counter even
> when it returns an error code. Thus a pairing decrement is needed on
> the error handling path to keep the counter balanced.
So you have not mentioned here why you are using _noidle and not _put.
Furthermore, in this patch [0] you are not using _noidle to fix the same
problem in another driver. We should fix this in a consistent manner
across all drivers, otherwise it leads to more confusion.
Finally, Rafael mentions we should just use _put [0] and so I think we
should follow his recommendation.
Jon
[0] https://lkml.org/lkml/2020/5/21/601
--
nvpublic
Powered by blists - more mailing lists