Re: [PATCH v3] memory/samsung: reduce protected code area in IRQ
On Tue, May 12, 2020 at 05:31:49AM -0700, Bernard Zhao wrote: > This change will speed-up a bit this IRQ processing and there > is no need to protect return value or printing. > > Signed-off-by: Bernard Zhao > --- > Changes since v1: > *change release lock before the if statement. > *revert dmc->df->lock mutex lock to protect function > exynos5_dmc_perf_events_check > > Changes since v2: > *Improve subject and commit message > > Link for V1: > *https://lore.kernel.org/patchwork/patch/123/ > --- > drivers/memory/samsung/exynos5422-dmc.c | 6 ++ Thanks, applied. Best regards, Krzysztof
Re: [PATCH v3] memory/samsung: reduce protected code area in IRQ
On 5/12/20 1:31 PM, Bernard Zhao wrote: This change will speed-up a bit this IRQ processing and there is no need to protect return value or printing. Signed-off-by: Bernard Zhao LGTM Acked-by: Lukasz Luba --- Changes since v1: *change release lock before the if statement. *revert dmc->df->lock mutex lock to protect function exynos5_dmc_perf_events_check Changes since v2: *Improve subject and commit message Link for V1: *https://lore.kernel.org/patchwork/patch/123/ --- drivers/memory/samsung/exynos5422-dmc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/memory/samsung/exynos5422-dmc.c b/drivers/memory/samsung/exynos5422-dmc.c index 22a43d662833..25196d6268e2 100644 --- a/drivers/memory/samsung/exynos5422-dmc.c +++ b/drivers/memory/samsung/exynos5422-dmc.c @@ -1346,15 +1346,13 @@ static irqreturn_t dmc_irq_thread(int irq, void *priv) struct exynos5_dmc *dmc = priv; mutex_lock(>df->lock); - exynos5_dmc_perf_events_check(dmc); - res = update_devfreq(dmc->df); + mutex_unlock(>df->lock); + if (res) dev_warn(dmc->dev, "devfreq failed with %d\n", res); - mutex_unlock(>df->lock); - return IRQ_HANDLED; }
[PATCH v3] memory/samsung: reduce protected code area in IRQ
This change will speed-up a bit this IRQ processing and there is no need to protect return value or printing. Signed-off-by: Bernard Zhao --- Changes since v1: *change release lock before the if statement. *revert dmc->df->lock mutex lock to protect function exynos5_dmc_perf_events_check Changes since v2: *Improve subject and commit message Link for V1: *https://lore.kernel.org/patchwork/patch/123/ --- drivers/memory/samsung/exynos5422-dmc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/memory/samsung/exynos5422-dmc.c b/drivers/memory/samsung/exynos5422-dmc.c index 22a43d662833..25196d6268e2 100644 --- a/drivers/memory/samsung/exynos5422-dmc.c +++ b/drivers/memory/samsung/exynos5422-dmc.c @@ -1346,15 +1346,13 @@ static irqreturn_t dmc_irq_thread(int irq, void *priv) struct exynos5_dmc *dmc = priv; mutex_lock(>df->lock); - exynos5_dmc_perf_events_check(dmc); - res = update_devfreq(dmc->df); + mutex_unlock(>df->lock); + if (res) dev_warn(dmc->dev, "devfreq failed with %d\n", res); - mutex_unlock(>df->lock); - return IRQ_HANDLED; } -- 2.26.2
