Re: [PATCH] memory: samsung: exynos5422-dmc: Additional locking for 'curr_rate'

From: Lukasz Luba
Date: Mon Aug 17 2020 - 08:28:50 EST


Hi Krzysztof,

On 8/17/20 12:54 PM, Krzysztof Kozlowski wrote:
On Tue, Aug 11, 2020 at 11:17:27AM +0100, Lukasz Luba wrote:
The 'curr_rate' is protected by local 'dmc->lock' in various places, but
not in a function exynos5_dmc_get_status(). The lock protects frequency
(and voltage) change process and the corresponding value stored in
'curr_rate'. Add the locking mechanism to protect the 'curr_rate' reading
also in the exynos5_dmc_get_status().

Suggested-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Signed-off-by: Lukasz Luba <lukasz.luba@xxxxxxx>
---
drivers/memory/samsung/exynos5422-dmc.c | 3 +++

Thanks, applied.


Thank you for applying it.

Regards,
Lukasz