lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 9 Nov 2015 11:03:23 +0900 From: glen lee <glen.lee@...el.com> To: Dan Carpenter <dan.carpenter@...cle.com>, Punit Vara <punitvara@...il.com> CC: <gregkh@...uxfoundation.org>, <devel@...verdev.osuosl.org>, <chris.park@...el.com>, <austin.shin@...el.com>, <linux-wireless@...r.kernel.org>, <johnny.kim@...el.com>, <linux-kernel@...r.kernel.org>, <tony.cho@...el.com>, <leo.kim@...el.com> Subject: Re: [PATCH V2] Staging: wilc1000: Fix build break due to undeclared *wilc and implicit declaration of init_irq I just check the email, I will get back here after I check the patches related with this issue. regards, glen lee. On 2015년 11월 09일 06:59, Dan Carpenter wrote: > Top posting beucase I am a bad person. Punit, you should have CC'd Glen > since this is his code. > > The fix is still not right. It will cause an unused variable warning on > some configs. > > To be honest, this code makes no sense. Why do we even have the wilc > variable in this function when we never use it? Just declare move it to > wilc_netdev_init(). Also there are too many ifdefs in this code. > > regards, > dan carpenter > > On Mon, Nov 09, 2015 at 03:01:50AM +0530, Punit Vara wrote: >> This patch is to the linux_wlan.c file that fixes up break found during >> make drivers/staging/wilc1000/linux_wlan.o >> >> Patch add following things to file : >> -init_irq declaration >> -At preprocessor (!defined WILC_SDIO) to defination of init_irq >> -At preprocessor (!defined WILC_SDIO) to defination isr_uh_routine >> -removes unnecessary lines to declare *wilc >> >> Patch fixes 702c0e50f and 2c1d05d10 tags. >> >> Signed-off-by: Punit Vara <punitvara@...il.com> >> --- >> -Fixes tag added suggested by Dan carpenter. >> -Remove declaration of autovariable with same type and same name suggested by Joe Perches >> >> drivers/staging/wilc1000/linux_wlan.c | 10 ++++++---- >> 1 file changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c >> index 086f1db..5bd14ed 100644 >> --- a/drivers/staging/wilc1000/linux_wlan.c >> +++ b/drivers/staging/wilc1000/linux_wlan.c >> @@ -208,7 +208,7 @@ static int dev_state_ev_handler(struct notifier_block *this, unsigned long event >> return NOTIFY_DONE; >> } >> >> -#if (defined WILC_SPI) || (defined WILC_SDIO_IRQ_GPIO) >> +#if (defined WILC_SPI) || (defined WILC_SDIO_IRQ_GPIO) || (!defined WILC_SDIO) >> static irqreturn_t isr_uh_routine(int irq, void *user_data) >> { >> perInterface_wlan_t *nic; >> @@ -246,7 +246,7 @@ irqreturn_t isr_bh_routine(int irq, void *userdata) >> return IRQ_HANDLED; >> } >> >> -#if (defined WILC_SPI) || (defined WILC_SDIO_IRQ_GPIO) >> +#if (defined WILC_SPI) || (defined WILC_SDIO_IRQ_GPIO) || (!defined WILC_SDIO) >> static int init_irq(struct net_device *dev) >> { >> int ret = 0; >> @@ -937,6 +937,10 @@ static void wlan_deinitialize_threads(struct net_device *dev) >> } >> } >> >> +#if (!defined WILC_SDIO) || (defined WILC_SDIO_IRQ_GPIO) >> +static int init_irq(struct net_device *dev); >> +#endif >> + >> int wilc1000_wlan_init(struct net_device *dev, perInterface_wlan_t *p_nic) >> { >> wilc_wlan_inp_t nwi; >> @@ -1578,9 +1582,7 @@ int wilc_netdev_init(struct wilc **wilc) >> >> static int __init init_wilc_driver(void) >> { >> -#ifdef WILC_SPI >> struct wilc *wilc; >> -#endif >> >> #if defined(WILC_DEBUGFS) >> if (wilc_debugfs_init() < 0) { >> -- >> 2.6.2 >> >> _______________________________________________ >> devel mailing list >> devel@...uxdriverproject.org >> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists