On Tue, Oct 19, 2010 at 10:30:54AM +0300, Felipe Balbi wrote: > Get rid of the following warning: > > arch/arm/mach-omap1/devices.c: In function 'omap_init_wdt': > arch/arm/mach-omap1/devices.c:298: warning: 'return' with > no value, in function returning non-void > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> > --- > arch/arm/mach-omap1/devices.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap1/devices.c b/arch/arm/mach-omap1/devices.c > index 63d68cf..a037aba 100644 > --- a/arch/arm/mach-omap1/devices.c > +++ b/arch/arm/mach-omap1/devices.c > @@ -295,7 +295,7 @@ static struct platform_device omap_wdt_device = { > static int __init omap_init_wdt(void) > { > if (!cpu_is_omap16xx()) > - return; > + return -ENODEV; > > platform_device_register(&omap_wdt_device); > return 0; It's unrelated to this patch, but using return platform_device_register(&omap_wdt_device); would be a bit cleaner. Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html