* [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings @ 2015-05-31 15:17 Gaston Gonzalez 2015-05-31 22:17 ` Greg KH 0 siblings, 1 reply; 9+ messages in thread From: Gaston Gonzalez @ 2015-05-31 15:17 UTC (permalink / raw) To: gregkh, cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker Cc: devel, linux-kernel, gascoar Fix the following sparse warnings: drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> --- drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c index d2e8b12..0477ba1 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be auth = (struct ieee80211_authentication *) skb_put(skb, sizeof(struct ieee80211_authentication)); - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); + if (challengelen) + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); auth->header.duration_id = 0x013a; //FIXME -- 2.1.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-05-31 15:17 [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings Gaston Gonzalez @ 2015-05-31 22:17 ` Greg KH 2015-05-31 23:07 ` Gaston Gonzalez 0 siblings, 1 reply; 9+ messages in thread From: Greg KH @ 2015-05-31 22:17 UTC (permalink / raw) To: Gaston Gonzalez Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > Fix the following sparse warnings: > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > --- > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Still doesn't apply, but this might be a git bug, see below: > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > index d2e8b12..0477ba1 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > auth = (struct ieee80211_authentication *) > skb_put(skb, sizeof(struct ieee80211_authentication)); > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > + if (challengelen) > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > auth->header.duration_id = 0x013a; //FIXME > > -- > 2.1.4 This patch is not created properly. Did you edit it by hand? If not, we need to report a bug as the diff is obviously wrong, look at the @@@ line. Can you update your version of git to see if it's fixed there? Also, patch tells you that this is broken: checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-05-31 22:17 ` Greg KH @ 2015-05-31 23:07 ` Gaston Gonzalez 2015-05-31 23:24 ` Greg KH 0 siblings, 1 reply; 9+ messages in thread From: Gaston Gonzalez @ 2015-05-31 23:07 UTC (permalink / raw) To: Greg KH Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > Fix the following sparse warnings: > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > --- > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > Still doesn't apply, but this might be a git bug, see below: > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > index d2e8b12..0477ba1 100644 > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > auth = (struct ieee80211_authentication *) > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > + if (challengelen) > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > auth->header.duration_id = 0x013a; //FIXME > > > > -- > > 2.1.4 > > This patch is not created properly. Did you edit it by hand? If not, > we need to report a bug as the diff is obviously wrong, look at the @@@ > line. > > Can you update your version of git to see if it's fixed there? > > Also, patch tells you that this is broken: > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); Greg, The patch was prepared using git format-patch. The git version I'm using: gaston22:\-$ git --version git version 2.1.4 I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. Could that drive the issue? However, I could properly apply the patch using 'git am' from an email and get the git log right. I'll try with a newer version of git as you suggest. Any other check I could do before the submission? I really don't like making noise :-( Thanks, Gaston > > > > > > > > _______________________________________________ > > devel mailing list > > devel@linuxdriverproject.org > > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-05-31 23:07 ` Gaston Gonzalez @ 2015-05-31 23:24 ` Greg KH 2015-06-01 1:41 ` Gaston Gonzalez 0 siblings, 1 reply; 9+ messages in thread From: Greg KH @ 2015-05-31 23:24 UTC (permalink / raw) To: Gaston Gonzalez Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > Fix the following sparse warnings: > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > --- > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > index d2e8b12..0477ba1 100644 > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > auth = (struct ieee80211_authentication *) > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > + if (challengelen) > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > -- > > > 2.1.4 > > > > This patch is not created properly. Did you edit it by hand? If not, > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > line. > > > > Can you update your version of git to see if it's fixed there? > > > > Also, patch tells you that this is broken: > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > Greg, > > The patch was prepared using git format-patch. The git version I'm using: > > gaston22:\-$ git --version > git version 2.1.4 > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > Could that drive the issue? However, I could properly apply the patch using 'git > am' from an email and get the git log right. Yes, but is there really a commit there? That's what messed me up last time, git applied it, but with no actual diff (which is probably a bug too, just a different one...) And if you edited it by hand, that might explain the issue, what did the "original" patch look like? > I'll try with a newer version of git as you suggest. > Any other check I could do before the submission? I really don't like making > noise :-( Let's see the original version first before blaming git. Odds are you got something wrong when editing the patch by hand. thanks, greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-05-31 23:24 ` Greg KH @ 2015-06-01 1:41 ` Gaston Gonzalez 2015-06-01 1:53 ` Greg KH 0 siblings, 1 reply; 9+ messages in thread From: Gaston Gonzalez @ 2015-06-01 1:41 UTC (permalink / raw) To: Greg KH Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel [-- Attachment #1: Type: text/plain, Size: 4189 bytes --] On Mon, Jun 01, 2015 at 08:24:17AM +0900, Greg KH wrote: > On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > > Fix the following sparse warnings: > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > > --- > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > index d2e8b12..0477ba1 100644 > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > > auth = (struct ieee80211_authentication *) > > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > > + if (challengelen) > > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > > > -- > > > > 2.1.4 > > > > > > This patch is not created properly. Did you edit it by hand? If not, > > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > > line. > > > > > > Can you update your version of git to see if it's fixed there? > > > > > > Also, patch tells you that this is broken: > > > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > Greg, > > > > The patch was prepared using git format-patch. The git version I'm using: > > > > gaston22:\-$ git --version > > git version 2.1.4 > > > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > > Could that drive the issue? However, I could properly apply the patch using 'git > > am' from an email and get the git log right. > > Yes, but is there really a commit there? That's what messed me up last > time, git applied it, but with no actual diff (which is probably a bug > too, just a different one...) > Yes, there was a commit. In fact the patch prepare: git format-patch -1 > And if you edited it by hand, that might explain the issue, what did the > "original" patch look like? Attached the original patch I submitted: 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch Attached is also the patch without the hand-edited deletions: ieee80211_softmac_fixes_v3.patch My steps once having this were: - run perl checkpatch -f - I got total: 4 errors, 0 warnings, 28 lines checked. The 4 warnings were extra white spaces. - Delete the spaces - git send-email --annotate Patch > > > I'll try with a newer version of git as you suggest. > > Any other check I could do before the submission? I really don't like making > > noise :-( > > Let's see the original version first before blaming git. Odds are you > got something wrong when editing the patch by hand. > Ok. > thanks, > > greg k-h thanks, Gaston [-- Attachment #2: 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch --] [-- Type: text/x-diff, Size: 1845 bytes --] >From b689a6d0de2a2d483e012698996b7bda9a33d6e2 Mon Sep 17 00:00:00 2001 From: Gaston Gonzalez <gascoar@gmail.com> Date: Sun, 31 May 2015 01:07:54 -0300 Subject: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings Fix the following sparse warnings: drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> --- drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c index d2e8b12..0477ba1 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be auth = (struct ieee80211_authentication *) skb_put(skb, sizeof(struct ieee80211_authentication)); - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); + if (challengelen) + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); auth->header.duration_id = 0x013a; //FIXME -- 2.1.4 [-- Attachment #3: ieee80211_softmac_fixes_v3.patch --] [-- Type: text/x-diff, Size: 1139 bytes --] >From b689a6d0de2a2d483e012698996b7bda9a33d6e2 Mon Sep 17 00:00:00 2001 From: Gaston Gonzalez <gascoar@gmail.com> Date: Sun, 31 May 2015 01:07:54 -0300 Subject: [PATCH] Regen ieee80211_softmac.c endianness fix patch --- drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c index d2e8b12..0477ba1 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c @@ -660,8 +660,9 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be auth = (struct ieee80211_authentication *) skb_put(skb, sizeof(struct ieee80211_authentication)); - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); + if (challengelen) + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); auth->header.duration_id = 0x013a; //FIXME -- 2.1.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-06-01 1:41 ` Gaston Gonzalez @ 2015-06-01 1:53 ` Greg KH 2015-06-01 2:24 ` Gaston Gonzalez 0 siblings, 1 reply; 9+ messages in thread From: Greg KH @ 2015-06-01 1:53 UTC (permalink / raw) To: Gaston Gonzalez Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Sun, May 31, 2015 at 10:41:52PM -0300, Gaston Gonzalez wrote: > On Mon, Jun 01, 2015 at 08:24:17AM +0900, Greg KH wrote: > > On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > > > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > > > Fix the following sparse warnings: > > > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > > > --- > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > index d2e8b12..0477ba1 100644 > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > > > auth = (struct ieee80211_authentication *) > > > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > > > + if (challengelen) > > > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > > > > > -- > > > > > 2.1.4 > > > > > > > > This patch is not created properly. Did you edit it by hand? If not, > > > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > > > line. > > > > > > > > Can you update your version of git to see if it's fixed there? > > > > > > > > Also, patch tells you that this is broken: > > > > > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > Greg, > > > > > > The patch was prepared using git format-patch. The git version I'm using: > > > > > > gaston22:\-$ git --version > > > git version 2.1.4 > > > > > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > > > Could that drive the issue? However, I could properly apply the patch using 'git > > > am' from an email and get the git log right. > > > > Yes, but is there really a commit there? That's what messed me up last > > time, git applied it, but with no actual diff (which is probably a bug > > too, just a different one...) > > > > Yes, there was a commit. In fact the patch prepare: git format-patch -1 > > > And if you edited it by hand, that might explain the issue, what did the > > "original" patch look like? > > Attached the original patch I submitted: > 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch Which is broken. > Attached is also the patch without the hand-edited deletions: > > ieee80211_softmac_fixes_v3.patch That patch is correct. Why did you edit the patch metadata in a way that was incorrect? > > My steps once having this were: > > - run perl checkpatch -f > - I got total: 4 errors, 0 warnings, 28 lines checked. The 4 warnings were extra white > spaces. > - Delete the spaces > - git send-email --annotate Patch You did a few other things here, you hand-edited the patch, which broke it. Yes, 'git am' does apply it, but not really, as there's a bug in git that you see I notified the git developers about. So I still can't apply this patch, please resend it in a way that is not broken, like your attachment here :) thanks, greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-06-01 1:53 ` Greg KH @ 2015-06-01 2:24 ` Gaston Gonzalez 2015-06-01 4:03 ` Greg KH 0 siblings, 1 reply; 9+ messages in thread From: Gaston Gonzalez @ 2015-06-01 2:24 UTC (permalink / raw) To: Greg KH Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Mon, Jun 01, 2015 at 10:53:20AM +0900, Greg KH wrote: > On Sun, May 31, 2015 at 10:41:52PM -0300, Gaston Gonzalez wrote: > > On Mon, Jun 01, 2015 at 08:24:17AM +0900, Greg KH wrote: > > > On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > > > > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > > > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > > > > Fix the following sparse warnings: > > > > > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > > > > --- > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > index d2e8b12..0477ba1 100644 > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > > > > auth = (struct ieee80211_authentication *) > > > > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > > > > + if (challengelen) > > > > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > > > > > > > -- > > > > > > 2.1.4 > > > > > > > > > > This patch is not created properly. Did you edit it by hand? If not, > > > > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > > > > line. > > > > > > > > > > Can you update your version of git to see if it's fixed there? > > > > > > > > > > Also, patch tells you that this is broken: > > > > > > > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > Greg, > > > > > > > > The patch was prepared using git format-patch. The git version I'm using: > > > > > > > > gaston22:\-$ git --version > > > > git version 2.1.4 > > > > > > > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > > > > Could that drive the issue? However, I could properly apply the patch using 'git > > > > am' from an email and get the git log right. > > > > > > Yes, but is there really a commit there? That's what messed me up last > > > time, git applied it, but with no actual diff (which is probably a bug > > > too, just a different one...) > > > > > > > Yes, there was a commit. In fact the patch prepare: git format-patch -1 > > > > > And if you edited it by hand, that might explain the issue, what did the > > > "original" patch look like? > > > > Attached the original patch I submitted: > > 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch > > Which is broken. > > > Attached is also the patch without the hand-edited deletions: > > > > ieee80211_softmac_fixes_v3.patch > > That patch is correct. > > Why did you edit the patch metadata in a way that was incorrect? The only edit I did was from within git send-email --annotate. Nothing else. I get now that that editor is not for everything... as I get the rest of the apply process right, I thought it was ok that way. My bad :( > > > > > My steps once having this were: > > > > - run perl checkpatch -f > > - I got total: 4 errors, 0 warnings, 28 lines checked. The 4 warnings were extra white > > spaces. > > - Delete the spaces > > - git send-email --annotate Patch > > You did a few other things here, you hand-edited the patch, which broke > it. > > Yes, 'git am' does apply it, but not really, as there's a bug in git > that you see I notified the git developers about. So I still can't > apply this patch, please resend it in a way that is not broken, like > your attachment here :) > Ok, will do. Only one question left, I always get checkpatch.pl white spaces errors after git format-patch, some of which weren't there before. So, what is the correct way of fixing them? > thanks, > > greg k-h Thanks for all the help, Gaston ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-06-01 2:24 ` Gaston Gonzalez @ 2015-06-01 4:03 ` Greg KH 2015-06-01 12:11 ` Gaston Gonzalez 0 siblings, 1 reply; 9+ messages in thread From: Greg KH @ 2015-06-01 4:03 UTC (permalink / raw) To: Gaston Gonzalez Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Sun, May 31, 2015 at 11:24:42PM -0300, Gaston Gonzalez wrote: > On Mon, Jun 01, 2015 at 10:53:20AM +0900, Greg KH wrote: > > On Sun, May 31, 2015 at 10:41:52PM -0300, Gaston Gonzalez wrote: > > > On Mon, Jun 01, 2015 at 08:24:17AM +0900, Greg KH wrote: > > > > On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > > > > > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > > > > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > > > > > Fix the following sparse warnings: > > > > > > > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > > > > > --- > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > index d2e8b12..0477ba1 100644 > > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > > > > > auth = (struct ieee80211_authentication *) > > > > > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > > > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > > > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > > > > > + if (challengelen) > > > > > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > > > > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > > > > > > > > > -- > > > > > > > 2.1.4 > > > > > > > > > > > > This patch is not created properly. Did you edit it by hand? If not, > > > > > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > > > > > line. > > > > > > > > > > > > Can you update your version of git to see if it's fixed there? > > > > > > > > > > > > Also, patch tells you that this is broken: > > > > > > > > > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > Greg, > > > > > > > > > > The patch was prepared using git format-patch. The git version I'm using: > > > > > > > > > > gaston22:\-$ git --version > > > > > git version 2.1.4 > > > > > > > > > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > > > > > Could that drive the issue? However, I could properly apply the patch using 'git > > > > > am' from an email and get the git log right. > > > > > > > > Yes, but is there really a commit there? That's what messed me up last > > > > time, git applied it, but with no actual diff (which is probably a bug > > > > too, just a different one...) > > > > > > > > > > Yes, there was a commit. In fact the patch prepare: git format-patch -1 > > > > > > > And if you edited it by hand, that might explain the issue, what did the > > > > "original" patch look like? > > > > > > Attached the original patch I submitted: > > > 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch > > > > Which is broken. > > > > > Attached is also the patch without the hand-edited deletions: > > > > > > ieee80211_softmac_fixes_v3.patch > > > > That patch is correct. > > > > Why did you edit the patch metadata in a way that was incorrect? > > The only edit I did was from within git send-email --annotate. Nothing else. I > get now that that editor is not for everything... as I get the rest of the apply process > right, I thought it was ok that way. My bad :( That's worrysome, why would --annotate change the lines that relate to the patch linenumbers? Did you do a search/replace or something accidentally? > > > My steps once having this were: > > > > > > - run perl checkpatch -f > > > - I got total: 4 errors, 0 warnings, 28 lines checked. The 4 warnings were extra white > > > spaces. > > > - Delete the spaces > > > - git send-email --annotate Patch > > > > You did a few other things here, you hand-edited the patch, which broke > > it. > > > > Yes, 'git am' does apply it, but not really, as there's a bug in git > > that you see I notified the git developers about. So I still can't > > apply this patch, please resend it in a way that is not broken, like > > your attachment here :) > > > Ok, will do. Only one question left, I always get checkpatch.pl white spaces > errors after git format-patch, some of which weren't there before. So, what is the correct way of > fixing them? Fix your editor to not add them :) thanks, greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings 2015-06-01 4:03 ` Greg KH @ 2015-06-01 12:11 ` Gaston Gonzalez 0 siblings, 0 replies; 9+ messages in thread From: Gaston Gonzalez @ 2015-06-01 12:11 UTC (permalink / raw) To: Greg KH Cc: cristina.opriceana, hamohammed.sa, gdonald, mahfouz.saif.elyazal, paul.gortmaker, devel, linux-kernel On Mon, Jun 01, 2015 at 01:03:16PM +0900, Greg KH wrote: > On Sun, May 31, 2015 at 11:24:42PM -0300, Gaston Gonzalez wrote: > > On Mon, Jun 01, 2015 at 10:53:20AM +0900, Greg KH wrote: > > > On Sun, May 31, 2015 at 10:41:52PM -0300, Gaston Gonzalez wrote: > > > > On Mon, Jun 01, 2015 at 08:24:17AM +0900, Greg KH wrote: > > > > > On Sun, May 31, 2015 at 08:07:21PM -0300, Gaston Gonzalez wrote: > > > > > > On Mon, Jun 01, 2015 at 07:17:39AM +0900, Greg KH wrote: > > > > > > > On Sun, May 31, 2015 at 12:17:48PM -0300, Gaston Gonzalez wrote: > > > > > > > > Fix the following sparse warnings: > > > > > > > > > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int > > > > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Gaston Gonzalez <gascoar@gmail.com> > > > > > > > > --- > > > > > > > > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 5 +++-- > > > > > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > > > > > > > Still doesn't apply, but this might be a git bug, see below: > > > > > > > > > > > > > > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > > index d2e8b12..0477ba1 100644 > > > > > > > > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > > @@ -660,2 +660,2 @@ inline struct sk_buff *ieee80211_authentication_req(struct ieee80211_network *be > > > > > > > > auth = (struct ieee80211_authentication *) > > > > > > > > skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > > > > > > > - auth->header.frame_ctl = IEEE80211_STYPE_AUTH; > > > > > > > > - if (challengelen) auth->header.frame_ctl |= IEEE80211_FCTL_WEP; > > > > > > > > + auth->header.frame_ctl = cpu_to_le16(IEEE80211_STYPE_AUTH); > > > > > > > > + if (challengelen) > > > > > > > > + auth->header.frame_ctl |= cpu_to_le16(IEEE80211_FCTL_WEP); > > > > > > > > > > > > > > > > auth->header.duration_id = 0x013a; //FIXME > > > > > > > > > > > > > > > > -- > > > > > > > > 2.1.4 > > > > > > > > > > > > > > This patch is not created properly. Did you edit it by hand? If not, > > > > > > > we need to report a bug as the diff is obviously wrong, look at the @@@ > > > > > > > line. > > > > > > > > > > > > > > Can you update your version of git to see if it's fixed there? > > > > > > > > > > > > > > Also, patch tells you that this is broken: > > > > > > > > > > > > > > checking file drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > > > > > > > patch: **** malformed patch at line 132: skb_put(skb, sizeof(struct ieee80211_authentication)); > > > > > > > > > > > > Greg, > > > > > > > > > > > > The patch was prepared using git format-patch. The git version I'm using: > > > > > > > > > > > > gaston22:\-$ git --version > > > > > > git version 2.1.4 > > > > > > > > > > > > I did deleted 4 whitespaces by hand after running checkpatch.pl on the patch. > > > > > > Could that drive the issue? However, I could properly apply the patch using 'git > > > > > > am' from an email and get the git log right. > > > > > > > > > > Yes, but is there really a commit there? That's what messed me up last > > > > > time, git applied it, but with no actual diff (which is probably a bug > > > > > too, just a different one...) > > > > > > > > > > > > > Yes, there was a commit. In fact the patch prepare: git format-patch -1 > > > > > > > > > And if you edited it by hand, that might explain the issue, what did the > > > > > "original" patch look like? > > > > > > > > Attached the original patch I submitted: > > > > 0001-Regen-ieee80211_softmac.c-endianness-fix-patch.patch > > > > > > Which is broken. > > > > > > > Attached is also the patch without the hand-edited deletions: > > > > > > > > ieee80211_softmac_fixes_v3.patch > > > > > > That patch is correct. > > > > > > Why did you edit the patch metadata in a way that was incorrect? > > > > The only edit I did was from within git send-email --annotate. Nothing else. I > > get now that that editor is not for everything... as I get the rest of the apply process > > right, I thought it was ok that way. My bad :( > > That's worrysome, why would --annotate change the lines that relate to > the patch linenumbers? Did you do a search/replace or something > accidentally? To be honest I don't know, using --annotate I edited: the body 'Fix the following sparse warnings', the signoff and the subject. I thought the subject could be the problem. Nothing below the --- line. Besides that, only the whitespaces deletion using emacs before the git send-email step. I really don't think I've accidentally replaced anything. In some hours I'll be able to build the patch again without the deletions and from scratch just in case. We'll see after the resend, hope it end up ok this time... Thanks again for all the help and sorry for the noise, Gaston > > > > > My steps once having this were: > > > > > > > > - run perl checkpatch -f > > > > - I got total: 4 errors, 0 warnings, 28 lines checked. The 4 warnings were extra white > > > > spaces. > > > > - Delete the spaces > > > > - git send-email --annotate Patch > > > > > > You did a few other things here, you hand-edited the patch, which broke > > > it. > > > > > > Yes, 'git am' does apply it, but not really, as there's a bug in git > > > that you see I notified the git developers about. So I still can't > > > apply this patch, please resend it in a way that is not broken, like > > > your attachment here :) > > > > > Ok, will do. Only one question left, I always get checkpatch.pl white spaces > > errors after git format-patch, some of which weren't there before. So, what is the correct way of > > fixing them? > > Fix your editor to not add them :) > > thanks, > > greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-06-01 12:09 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-05-31 15:17 [PATCH RESEND] staging: rtl8192u: ieee80211: Fix sparse endianness warnings Gaston Gonzalez 2015-05-31 22:17 ` Greg KH 2015-05-31 23:07 ` Gaston Gonzalez 2015-05-31 23:24 ` Greg KH 2015-06-01 1:41 ` Gaston Gonzalez 2015-06-01 1:53 ` Greg KH 2015-06-01 2:24 ` Gaston Gonzalez 2015-06-01 4:03 ` Greg KH 2015-06-01 12:11 ` Gaston Gonzalez
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).