[PATCH v1] clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

[PATCH v1] clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

[Sowjanya Komatineni]

This patch includes fix to remove pm_runtime_irq_safe which was added
mistakenly during dfll suspend and resume implementation as a part of
Tegra210 SC7 suspend and resume patch series.

This patch removes pm_runtime_irq_safe and also has minor description
update for argument of dfll_suspend in comments.

Some of the patch series are applied so sending this fix as incremental
fix to latest linux-next.

Sowjanya Komatineni (1):
  clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

 drivers/clk/tegra/clk-dfll.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
2.7.4

[PATCH v1] clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

[Sowjanya Komatineni]

pm_runtime_irq_safe is not needed as interrupts are allowed during
suspend and resume. This was added mistakenly during dfll suspend
and resume support patch.

This patch includes fix to remove pm_runtime_irq_safe.

Also updated description of dfll_suspend argument.

Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
---
 drivers/clk/tegra/clk-dfll.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c
index c051d92c2bbf..cfbaa90c7adb 100644
--- a/drivers/clk/tegra/clk-dfll.c
+++ b/drivers/clk/tegra/clk-dfll.c
@@ -1487,7 +1487,6 @@ static int dfll_init(struct tegra_dfll *td)
 	td->last_unrounded_rate = 0;
 
 	pm_runtime_enable(td->dev);
-	pm_runtime_irq_safe(td->dev);
 	pm_runtime_get_sync(td->dev);
 
 	dfll_set_mode(td, DFLL_DISABLED);
@@ -1516,7 +1515,7 @@ static int dfll_init(struct tegra_dfll *td)
 
 /**
  * tegra_dfll_suspend - check DFLL is disabled
- * @dev: DFLL device *
+ * @dev: DFLL instance
  *
  * DFLL clock should be disabled by the CPUFreq driver. So, make
  * sure it is disabled and disable all clocks needed by the DFLL.
-- 
2.7.4

Re: [PATCH v1] clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

[Stephen Boyd]

Quoting Sowjanya Komatineni (2019-11-12 08:17:05)
> This patch includes fix to remove pm_runtime_irq_safe which was added
> mistakenly during dfll suspend and resume implementation as a part of
> Tegra210 SC7 suspend and resume patch series.
> 
> This patch removes pm_runtime_irq_safe and also has minor description
> update for argument of dfll_suspend in comments.
> 
> Some of the patch series are applied so sending this fix as incremental
> fix to latest linux-next.

Please don't send cover letters for single patches. The value add is
minimal vs. just putting extra text next to the diffstat in the single
patch.

> 
> Sowjanya Komatineni (1):
>   clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe
> 
>  drivers/clk/tegra/clk-dfll.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Re: [PATCH v1] clk: tegra: clk-dfll: Fix to remove pm_runtime_irq_safe

[Dmitry Osipenko]

12.11.2019 19:17, Sowjanya Komatineni пишет:
> pm_runtime_irq_safe is not needed as interrupts are allowed during
> suspend and resume. This was added mistakenly during dfll suspend
> and resume support patch.
> 
> This patch includes fix to remove pm_runtime_irq_safe.
> 
> Also updated description of dfll_suspend argument.
> 
> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
> ---
>  drivers/clk/tegra/clk-dfll.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c
> index c051d92c2bbf..cfbaa90c7adb 100644
> --- a/drivers/clk/tegra/clk-dfll.c
> +++ b/drivers/clk/tegra/clk-dfll.c
> @@ -1487,7 +1487,6 @@ static int dfll_init(struct tegra_dfll *td)
>  	td->last_unrounded_rate = 0;
>  
>  	pm_runtime_enable(td->dev);
> -	pm_runtime_irq_safe(td->dev);
>  	pm_runtime_get_sync(td->dev);
>  
>  	dfll_set_mode(td, DFLL_DISABLED);
> @@ -1516,7 +1515,7 @@ static int dfll_init(struct tegra_dfll *td)
>  
>  /**
>   * tegra_dfll_suspend - check DFLL is disabled
> - * @dev: DFLL device *
> + * @dev: DFLL instance
>   *
>   * DFLL clock should be disabled by the CPUFreq driver. So, make
>   * sure it is disabled and disable all clocks needed by the DFLL.
> 

Hello Sowjanya,

It should be better to have the commit's title saying something more
generic like "Clean up code" or to have two separate patches because
this patch does two things that are unrelated to each other in
accordance to the title and this is a bit confusing at first glance.
Although, that's not a big deal for a trivial patch like this and it
should be fine as is (IMO). Please take into account in the future
patches that both commit's title and description should be accurate.

Reviewed-by: Dmitry Osipenko <digetx@gmail.com>