Re: [PATCH] ARM: OMAP: Fix warning in clock.c

[Tony Lindgren <tony@atomide.com>]

* Dirk Behme <dirk.behme@googlemail.com> [061005 16:28]:
> Ladislav Michl wrote:
> >On Wed, Oct 04, 2006 at 03:28:43PM +0200, Dirk Behme wrote:
> >
> >>Fix warning
> >>
> >>arch/arm/mach-omap1/clock.c: In function
> >>'omap1_clk_enable_generic':
> >>arch/arm/mach-omap1/clock.c:499: warning: 'return' with no
> >>value, in function returning non-void
> >>
> >>Signed-off-by: Dirk Behme <dirk.behme_at_gmail.com>
> >>
> >
> >>--- ./arch/arm/mach-omap1/clock.c_orig	2006-10-04 
> >>15:07:39.000000000 +0200
> >>+++ ./arch/arm/mach-omap1/clock.c	2006-10-04 15:09:41.000000000 +0200
> >>@@ -496,7 +496,7 @@ static int omap1_clk_enable_generic(stru
> >>		}
> >>	}
> >>
> >>-	return;
> >>+	return 0;
> >>}
> >
> >
> >Does it make sense to return any vaule at all? This functions return
> >always 0. Perhaps in case there is no enable reg it should return
> >something else. Btw, can this code run on omap2 cpu?
> 
> What's about something like in attachment?
> 

> --- ./arch/arm/mach-omap1/clock.c_orig	2006-10-05 15:10:17.000000000 +0200
> +++ ./arch/arm/mach-omap1/clock.c	2006-10-05 15:14:26.000000000 +0200
> @@ -432,8 +432,7 @@ static int omap1_clk_enable(struct clk *
>  			}
>  
>  			if (clk->flags & CLOCK_NO_IDLE_PARENT)
> -				if (!cpu_is_omap24xx())
> -					omap1_clk_deny_idle(clk->parent);
> +				omap1_clk_deny_idle(clk->parent);
>  		}
>  
>  		ret = clk->enable(clk);
> @@ -454,8 +453,7 @@ static void omap1_clk_disable(struct clk
>  		if (likely(clk->parent)) {
>  			omap1_clk_disable(clk->parent);
>  			if (clk->flags & CLOCK_NO_IDLE_PARENT)
> -				if (!cpu_is_omap24xx())
> -					omap1_clk_allow_idle(clk->parent);
> +				omap1_clk_allow_idle(clk->parent);
>  		}
>  	}
>  }
> @@ -471,7 +469,7 @@ static int omap1_clk_enable_generic(stru
>  	if (unlikely(clk->enable_reg == 0)) {
>  		printk(KERN_ERR "clock.c: Enable for %s without enable code\n",
>  		       clk->name);
> -		return 0;
> +		return -EINVAL;
>  	}
>  
>  	if (clk->flags & ENABLE_REG_32BIT) {
> @@ -496,7 +494,7 @@ static int omap1_clk_enable_generic(stru
>  		}
>  	}
>  
> -	return;
> +	return 0;
>  }
>  
>  static void omap1_clk_disable_generic(struct clk *clk)
> 

Pushed this one too.

Tony