Oh... Sorry. I got your email confused with someone else. > > diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c > > index 2a5b36f..0805050 100644 > > --- a/drivers/staging/wilc1000/linux_wlan.c > > +++ b/drivers/staging/wilc1000/linux_wlan.c > > @@ -1780,6 +1780,7 @@ int wilc_netdev_init(struct wilc **wilc) > > /*The 1st function called after module inserted*/ > > static int __init init_wilc_driver(void) > > { > > + struct wilc *wilc; > > #ifdef WILC_SPI > > struct wilc *wilc; > > #endif The problem is that now if WILC_SPI is defined it's declared twice so this patch is definitely wrong. I haven't looked at it further than that, sorry. CC the atmel people with these patches. Use ./scripts/get_maintainer.pl to find the right maintainers. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel