On 11/29/2012 08:25 PM, Luis R. Rodriguez wrote: > On Wed, Nov 28, 2012 at 3:07 PM, Hauke Mehrtens <hauke@xxxxxxxxxx> wrote: >> On kernel version <= 2.6.28, I saw the following compile error without >> this patch: >> >> /net/wireless/util.c: In function ‘cfg80211_get_p2p_attr’: >> /net/wireless/util.c:1120:4: error: implicit declaration of function >> ‘get_unaligned_le16’ [-Werror=implicit-function-declaration] >> >> Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx> >> --- >> patches/network/06-header-changes.patch | 10 ++++++++++ >> patches/network/30-bridge-port.patch | 2 +- >> 2 files changed, 11 insertions(+), 1 deletion(-) >> >> diff --git a/patches/network/06-header-changes.patch b/patches/network/06-header-changes.patch >> index cd587cd..5da05c0 100644 >> --- a/patches/network/06-header-changes.patch >> +++ b/patches/network/06-header-changes.patch >> @@ -125,3 +125,13 @@ cases. >> >> #define USB_VERSION "1.0" >> >> +--- a/net/wireless/util.c >> ++++ b/net/wireless/util.c >> +@@ -10,6 +10,7 @@ >> + #include <net/cfg80211.h> >> + #include <net/ip.h> >> + #include <net/dsfield.h> >> ++#include <asm/unaligned.h> > > Interesting, I wonder if we should just get a compat header to add > this therefore not requiring any patching. Probably some of the header files included also include asm/unaligned.h on kernel versions > 2.6.28. You could add all the includes added in this patch to compat-X.h to get rid of this entire patch, but this will increase the risk of name clashes. >> + #include "core.h" >> + #include "rdev-ops.h" >> + >> diff --git a/patches/network/30-bridge-port.patch b/patches/network/30-bridge-port.patch >> index b4106b7..a426274 100644 >> --- a/patches/network/30-bridge-port.patch >> +++ b/patches/network/30-bridge-port.patch >> @@ -23,7 +23,7 @@ compat-drivers. >> >> --- a/net/wireless/util.c >> +++ b/net/wireless/util.c >> -@@ -812,7 +812,7 @@ int cfg80211_change_iface(struct cfg8021 >> +@@ -813,7 +813,7 @@ int cfg80211_change_iface(struct cfg8021 >> return -EOPNOTSUPP; >> > > This hunk does not seem to be relevant so I'll discard it. The offset of this change in the patch changed because of the additional header being added in the previous patch. I am always refreshing the patches after doing some changes to them to make all apply cleanly. Hauke -- To unsubscribe from this list: send the line "unsubscribe backports" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
