* [PATCH] clk: berlin: Pass correct type to hw provider registration
@ 2016-11-16 19:08 Stephen Boyd
2016-11-17 23:04 ` Stephen Boyd
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Boyd @ 2016-11-16 19:08 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-kernel, linux-clk, Dan Carpenter, Jisheng Zhang,
Alexandre Belloni, Sebastian Hesselbarth, Stephen Boyd
Dan Carpenter reports that we're passing a pointer to a pointer
here when we should just be passing a pointer. Pass the right
pointer so that the of_clk_hw_onecell_get() sees the appropriate
data pointer on its end.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Jisheng Zhang <jszhang@marvell.com>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Cc: Stephen Boyd <stephen.boyd@linaro.org>
Fixes: f6475e298297 ("clk: berlin: Migrate to clk_hw based registration and OF APIs")
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
drivers/clk/berlin/bg2.c | 2 +-
drivers/clk/berlin/bg2q.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/berlin/bg2.c b/drivers/clk/berlin/bg2.c
index edf3b96b3b73..1d99292e2039 100644
--- a/drivers/clk/berlin/bg2.c
+++ b/drivers/clk/berlin/bg2.c
@@ -685,7 +685,7 @@ static void __init berlin2_clock_setup(struct device_node *np)
}
/* register clk-provider */
- of_clk_add_hw_provider(np, of_clk_hw_onecell_get, &clk_data);
+ of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_data);
return;
diff --git a/drivers/clk/berlin/bg2q.c b/drivers/clk/berlin/bg2q.c
index 0718e831475f..3b784b593afd 100644
--- a/drivers/clk/berlin/bg2q.c
+++ b/drivers/clk/berlin/bg2q.c
@@ -382,7 +382,7 @@ static void __init berlin2q_clock_setup(struct device_node *np)
}
/* register clk-provider */
- of_clk_add_hw_provider(np, of_clk_hw_onecell_get, &clk_data);
+ of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_data);
return;
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] clk: berlin: Pass correct type to hw provider registration
2016-11-16 19:08 [PATCH] clk: berlin: Pass correct type to hw provider registration Stephen Boyd
@ 2016-11-17 23:04 ` Stephen Boyd
0 siblings, 0 replies; 2+ messages in thread
From: Stephen Boyd @ 2016-11-17 23:04 UTC (permalink / raw)
To: Michael Turquette
Cc: linux-kernel, linux-clk, Dan Carpenter, Jisheng Zhang,
Alexandre Belloni, Sebastian Hesselbarth, Stephen Boyd
On 11/16, Stephen Boyd wrote:
> Dan Carpenter reports that we're passing a pointer to a pointer
> here when we should just be passing a pointer. Pass the right
> pointer so that the of_clk_hw_onecell_get() sees the appropriate
> data pointer on its end.
>
> Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
> Cc: Jisheng Zhang <jszhang@marvell.com>
> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Cc: Stephen Boyd <stephen.boyd@linaro.org>
> Fixes: f6475e298297 ("clk: berlin: Migrate to clk_hw based registration and OF APIs")
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> ---
Applied to clk-fixes
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-11-17 23:04 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-16 19:08 [PATCH] clk: berlin: Pass correct type to hw provider registration Stephen Boyd
2016-11-17 23:04 ` Stephen Boyd
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).