On Thu, 2010-07-22 at 10:45 +0200, ext Kalle Valo wrote:
> Hi John,
>
> On 07/21/2010 06:31 PM, John W. Linville wrote:
> > @@ -225,7 +225,7 @@ static void wl1251_boot_set_ecpu_ctrl(struct wl1251 *wl, u32 flag)
> > int wl1251_boot_run_firmware(struct wl1251 *wl)
> > {
> > int loop, ret;
> > - u32 chip_id, interrupt;
> > + u32 chip_id, acx_intr;
>
> I don't get why you did this change. I don't object renaming the
> variable, but I can't figure out the reasoning behind it.
Sparse complains that interrupt is shadowing the definition in hw_irq.h:
> drivers/net/wireless/wl12xx/wl1251_boot.c:228:22: warning: symbol
> 'interrupt' shadows an earlier one
> /home/linville/git/wireless-next-2.6/arch/x86/include/asm/hw_irq.h:132:13: originally declared here
So, it's just a cosmetic change, I guess.
--
Cheers,
Luca.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
