Re: [PATCH] coresight: cti: Correct the parameter for pm_runtime_put
From: Mathieu Poirier <mathieu.poirier@linaro.org>
Date: 2021-09-02 17:19:12
Also in:
lkml
Hi Tao, Apologies for the late reply - this patch fell through the cracks. On Thu, Aug 19, 2021 at 05:29:37PM +0800, Tao Zhang wrote:
quoted hunk ↗ jump to hunk
The input parameter of the function pm_runtime_put should be the same in the function cti_enable_hw and cti_disable_hw. The correct parameter to use here should be dev->parent. Signed-off-by: Tao Zhang <redacted> --- drivers/hwtracing/coresight/coresight-cti-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/hwtracing/coresight/coresight-cti-core.c b/drivers/hwtracing/coresight/coresight-cti-core.c index e2a3620..8988b2e 100644 --- a/drivers/hwtracing/coresight/coresight-cti-core.c +++ b/drivers/hwtracing/coresight/coresight-cti-core.c@@ -175,7 +175,7 @@ static int cti_disable_hw(struct cti_drvdata *drvdata) coresight_disclaim_device_unlocked(csdev); CS_LOCK(drvdata->base); spin_unlock(&drvdata->spinlock); - pm_runtime_put(dev); + pm_runtime_put(dev->parent);
You are correct - I have added this patch to my next tree. Thanks, Mathieu
return 0; /* not disabled this call */ -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel