diff mbox series

[-next,v2] clk: pm_clock: provide stubs for pm_clk_runtime_suspend/_resume

Message ID 20201109032115.10610-1-rdunlap@infradead.org
State New
Headers show
Series [-next,v2] clk: pm_clock: provide stubs for pm_clk_runtime_suspend/_resume | expand

Commit Message

Randy Dunlap Nov. 9, 2020, 3:21 a.m. UTC
Add stubs for pm_clk_runtime_suspend() and pm_clk_runtime_resume()
to fix build errors when CONFIG_PM and CONFIG_PM_CLK are not enabled.

Fixes these build errors:

../drivers/clk/qcom/camcc-sc7180.c: In function ‘cam_cc_sc7180_probe’:
../drivers/clk/qcom/camcc-sc7180.c:1672:8: error: implicit declaration of function ‘pm_clk_runtime_resume’; did you mean ‘pm_runtime_resume’? [-Werror=implicit-function-declaration]
  ret = pm_clk_runtime_resume(&pdev->dev);
        ^~~~~~~~~~~~~~~~~~~~~
../drivers/clk/qcom/camcc-sc7180.c:1681:3: error: implicit declaration of function ‘pm_clk_runtime_suspend’; did you mean ‘pm_runtime_suspend’? [-Werror=implicit-function-declaration]
   pm_clk_runtime_suspend(&pdev->dev);
   ^~~~~~~~~~~~~~~~~~~~~~

Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Len Brown <len.brown@intel.com>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: linux-pm@vger.kernel.org
Cc: Michael Turquette <mturquette@baylibre.com>
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: linux-clk@vger.kernel.org
Cc: Taniya Das <tdas@codeaurora.org>
Cc: linux-next@vger.kernel.org
Cc: Nathan Chancellor <natechancellor@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
---
v2: move the function stubs to be inside the #else (for !CONFIG_PM)
    as suggested by Nathan to fix another build error

 include/linux/pm_clock.h |    8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Nathan Chancellor Nov. 10, 2020, 1:09 a.m. UTC | #1
On Sun, Nov 08, 2020 at 07:21:15PM -0800, Randy Dunlap wrote:
> Add stubs for pm_clk_runtime_suspend() and pm_clk_runtime_resume()
> to fix build errors when CONFIG_PM and CONFIG_PM_CLK are not enabled.
> 
> Fixes these build errors:
> 
> ../drivers/clk/qcom/camcc-sc7180.c: In function ‘cam_cc_sc7180_probe’:
> ../drivers/clk/qcom/camcc-sc7180.c:1672:8: error: implicit declaration of function ‘pm_clk_runtime_resume’; did you mean ‘pm_runtime_resume’? [-Werror=implicit-function-declaration]
>   ret = pm_clk_runtime_resume(&pdev->dev);
>         ^~~~~~~~~~~~~~~~~~~~~
> ../drivers/clk/qcom/camcc-sc7180.c:1681:3: error: implicit declaration of function ‘pm_clk_runtime_suspend’; did you mean ‘pm_runtime_suspend’? [-Werror=implicit-function-declaration]
>    pm_clk_runtime_suspend(&pdev->dev);
>    ^~~~~~~~~~~~~~~~~~~~~~
> 
> Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> Cc: Len Brown <len.brown@intel.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: linux-pm@vger.kernel.org
> Cc: Michael Turquette <mturquette@baylibre.com>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: linux-clk@vger.kernel.org
> Cc: Taniya Das <tdas@codeaurora.org>
> Cc: linux-next@vger.kernel.org
> Cc: Nathan Chancellor <natechancellor@gmail.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>

Build-tested-by: Nathan Chancellor <natechancellor@gmail.com>
Reviewed-by: Nathan Chancellor <natechancellor@gmail.com>

> ---
> v2: move the function stubs to be inside the #else (for !CONFIG_PM)
>     as suggested by Nathan to fix another build error
> 
>  include/linux/pm_clock.h |    8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> --- linux-next-20201106.orig/include/linux/pm_clock.h
> +++ linux-next-20201106/include/linux/pm_clock.h
> @@ -27,6 +27,14 @@ extern int pm_clk_runtime_resume(struct
>  	.runtime_resume = pm_clk_runtime_resume,
>  #else
>  #define USE_PM_CLK_RUNTIME_OPS
> +static inline int pm_clk_runtime_suspend(struct device *dev)
> +{
> +	return 0;
> +}
> +static inline int pm_clk_runtime_resume(struct device *dev)
> +{
> +	return 0;
> +}
>  #endif
>  
>  #ifdef CONFIG_PM_CLK
Rafael J. Wysocki Nov. 10, 2020, 5:43 p.m. UTC | #2
On Mon, Nov 9, 2020 at 4:21 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>

> Add stubs for pm_clk_runtime_suspend() and pm_clk_runtime_resume()

> to fix build errors when CONFIG_PM and CONFIG_PM_CLK are not enabled.

>

> Fixes these build errors:

>

> ../drivers/clk/qcom/camcc-sc7180.c: In function ‘cam_cc_sc7180_probe’:

> ../drivers/clk/qcom/camcc-sc7180.c:1672:8: error: implicit declaration of function ‘pm_clk_runtime_resume’; did you mean ‘pm_runtime_resume’? [-Werror=implicit-function-declaration]

>   ret = pm_clk_runtime_resume(&pdev->dev);

>         ^~~~~~~~~~~~~~~~~~~~~

> ../drivers/clk/qcom/camcc-sc7180.c:1681:3: error: implicit declaration of function ‘pm_clk_runtime_suspend’; did you mean ‘pm_runtime_suspend’? [-Werror=implicit-function-declaration]

>    pm_clk_runtime_suspend(&pdev->dev);

>    ^~~~~~~~~~~~~~~~~~~~~~

>

> Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")

> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>

> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>

> Cc: Len Brown <len.brown@intel.com>

> Cc: Pavel Machek <pavel@ucw.cz>

> Cc: linux-pm@vger.kernel.org

> Cc: Michael Turquette <mturquette@baylibre.com>

> Cc: Stephen Boyd <sboyd@kernel.org>

> Cc: linux-clk@vger.kernel.org

> Cc: Taniya Das <tdas@codeaurora.org>

> Cc: linux-next@vger.kernel.org

> Cc: Nathan Chancellor <natechancellor@gmail.com>

> Cc: Andrew Morton <akpm@linux-foundation.org>

> ---

> v2: move the function stubs to be inside the #else (for !CONFIG_PM)

>     as suggested by Nathan to fix another build error

>

>  include/linux/pm_clock.h |    8 ++++++++

>  1 file changed, 8 insertions(+)

>

> --- linux-next-20201106.orig/include/linux/pm_clock.h

> +++ linux-next-20201106/include/linux/pm_clock.h

> @@ -27,6 +27,14 @@ extern int pm_clk_runtime_resume(struct

>         .runtime_resume = pm_clk_runtime_resume,

>  #else

>  #define USE_PM_CLK_RUNTIME_OPS

> +static inline int pm_clk_runtime_suspend(struct device *dev)

> +{

> +       return 0;

> +}

> +static inline int pm_clk_runtime_resume(struct device *dev)

> +{

> +       return 0;

> +}

>  #endif

>

>  #ifdef CONFIG_PM_CLK


Applied to the PM tree as 5.10-rc material, thanks!
Stephen Rothwell Nov. 10, 2020, 9:30 p.m. UTC | #3
Hi Rafael,

On Tue, 10 Nov 2020 18:43:04 +0100 "Rafael J. Wysocki" <rafael@kernel.org> wrote:
>

> > Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")

>

> Applied to the PM tree as 5.10-rc material, thanks!


The problem is that the commit that this one fixes is in the clk tree
not the pm tree, so this patch needs to (also) be in the clk tree to
reduced bisect problems.

-- 
Cheers,
Stephen Rothwell
Rafael J. Wysocki Nov. 12, 2020, 3:01 p.m. UTC | #4
Hi Stephen,

On Tue, Nov 10, 2020 at 10:30 PM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>

> Hi Rafael,

>

> On Tue, 10 Nov 2020 18:43:04 +0100 "Rafael J. Wysocki" <rafael@kernel.org> wrote:

> >

> > > Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")

> >

> > Applied to the PM tree as 5.10-rc material, thanks!

>

> The problem is that the commit that this one fixes is in the clk tree

> not the pm tree, so this patch needs to (also) be in the clk tree to

> reduced bisect problems.


OK, dropping from pm then.
diff mbox series

Patch

--- linux-next-20201106.orig/include/linux/pm_clock.h
+++ linux-next-20201106/include/linux/pm_clock.h
@@ -27,6 +27,14 @@  extern int pm_clk_runtime_resume(struct
 	.runtime_resume = pm_clk_runtime_resume,
 #else
 #define USE_PM_CLK_RUNTIME_OPS
+static inline int pm_clk_runtime_suspend(struct device *dev)
+{
+	return 0;
+}
+static inline int pm_clk_runtime_resume(struct device *dev)
+{
+	return 0;
+}
 #endif
 
 #ifdef CONFIG_PM_CLK