Search Linux Wireless

Re: [PATCH] wlcore: drop INET dependency

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



(updated to my new email address).

On Sun, 2012-06-24 at 19:17 -0700, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> 
> Mainline build reports:
> 
> warning: (WL12XX) selects WLCORE which has unmet direct dependencies (NETDEVICES && WLAN && WL_TI && GENERIC_HARDIRQS && MAC80211 && INET)
> 
> The INET dependency was added in commit
> 3c6af5b54fe74b6e56efadc22927e4055d00e9fc:
>     wl1271_main.c:(.text+0x271052): undefined reference to `unregister_inetaddr_
> notifier'
>     wl1271_main.c:(.text+0x2714d7): undefined reference to `register_inetaddr_no
> tifier'
>     
>     Driver is doing some filtering based on IP addresses...
> 
> but this driver no longer has that code and it builds fine even when
> CONFIG_INET is not enabled, so drop that dependency and eliminate the
> kconfig warning message.
> 
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> Cc: Luciano Coelho <luciano.coelho@xxxxxxxxx>
> Cc: John W. Linville <linville@xxxxxxxxxxxxx>
> ---

Acked-by: Luciano Coelho <coelho@xxxxxx>

Thanks, Randy!

This one goes quite far back, the dependency in the code was removed in
commit eb887d (wl1271: Use the ARP configuration function from
mac80211), 2 years ago.

I think this should be applied to 3.5 and maybe even to stable (is this
kind of warning important enough to deserve going to stable releases?)

John, can you take this directly into wireless.git?

--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux