linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/1] clk: Fix incorrect return type in clk.c
@ 2013-03-13  9:47 Sachin Kamat
  2013-03-13 17:09 ` Pankaj Jangra
  0 siblings, 1 reply; 3+ messages in thread
From: Sachin Kamat @ 2013-03-13  9:47 UTC (permalink / raw)
  To: linux-arm-kernel

Return type of function clk_propagate_rate_change is a pointer.
But 0 was being returned. Change it to NULL.
Silences the following warning:
drivers/clk/clk.c:977:24: warning: Using plain integer as NULL pointer

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/clk/clk.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index ed87b24..467b31f 100644
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -974,7 +974,7 @@ static struct clk *clk_propagate_rate_change(struct clk *clk, unsigned long even
 	int ret = NOTIFY_DONE;
 
 	if (clk->rate == clk->new_rate)
-		return 0;
+		return NULL;
 
 	if (clk->notifier_count) {
 		ret = __clk_notify(clk, event, clk->rate, clk->new_rate);
-- 
1.7.4.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 1/1] clk: Fix incorrect return type in clk.c
  2013-03-13  9:47 [PATCH 1/1] clk: Fix incorrect return type in clk.c Sachin Kamat
@ 2013-03-13 17:09 ` Pankaj Jangra
  2013-03-22 19:19   ` Mike Turquette
  0 siblings, 1 reply; 3+ messages in thread
From: Pankaj Jangra @ 2013-03-13 17:09 UTC (permalink / raw)
  To: linux-arm-kernel

Hi.

On Wed, Mar 13, 2013 at 3:17 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> Return type of function clk_propagate_rate_change is a pointer.
> But 0 was being returned. Change it to NULL.
> Silences the following warning:
> drivers/clk/clk.c:977:24: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  drivers/clk/clk.c |    2 +-

Looks good to me.
Reviewed-by: Pankaj Jangra <jangra.pankaj9@gmail.com>

Regards,
Pankaj Jangra

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/1] clk: Fix incorrect return type in clk.c
  2013-03-13 17:09 ` Pankaj Jangra
@ 2013-03-22 19:19   ` Mike Turquette
  0 siblings, 0 replies; 3+ messages in thread
From: Mike Turquette @ 2013-03-22 19:19 UTC (permalink / raw)
  To: linux-arm-kernel

Quoting Pankaj Jangra (2013-03-13 10:09:18)
> Hi.
> 
> On Wed, Mar 13, 2013 at 3:17 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> > Return type of function clk_propagate_rate_change is a pointer.
> > But 0 was being returned. Change it to NULL.
> > Silences the following warning:
> > drivers/clk/clk.c:977:24: warning: Using plain integer as NULL pointer
> >
> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> > ---
> >  drivers/clk/clk.c |    2 +-
> 
> Looks good to me.
> Reviewed-by: Pankaj Jangra <jangra.pankaj9@gmail.com>
> 

Thanks for the fix.  Taken into clk-next.

Regards,
Mike

> Regards,
> Pankaj Jangra

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-03-22 19:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-13  9:47 [PATCH 1/1] clk: Fix incorrect return type in clk.c Sachin Kamat
2013-03-13 17:09 ` Pankaj Jangra
2013-03-22 19:19   ` Mike Turquette

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).