* Re: [PATCH 3.1-stable] staging: brcm80211: fix for rate index in receive status [not found] ` <20111110151746.GB29104@suse.de> @ 2011-11-10 18:37 ` Arend van Spriel 2011-11-10 18:44 ` Greg KH 0 siblings, 1 reply; 3+ messages in thread From: Arend van Spriel @ 2011-11-10 18:37 UTC (permalink / raw) To: Greg KH Cc: linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org On 11/10/2011 04:17 PM, Greg KH wrote: > On Thu, Nov 10, 2011 at 10:48:34AM +0100, Arend van Spriel wrote: >> This fix resolves an issue on 5GHz as posted on LKML: >> >> BUG: All network processes hang (brcmsmac/wpa_supplicant) >> ID: <20111018111422.GA1979@schottelius.org> > > What are these tags for? I've not seen them in kernel patches before. > The first line was the email Subject as posted on LKML. The second is the message id. >> Cc: stable@kernel.org > > Wrong address, please use stable@vger.kernel.org now. > Noted. >> Reported-by: Nico Schottelius <nico-linux-20111017@schottelius.org> >> Signed-off-by: Arend van Spriel <arend@broadcom.com> >> --- >> Hi Greg, >> >> This change is in the mainline brcmsmac driver, but I do not have a commit >> message to refer to as we transitioned from staging to mainline in the last >> merge window. Can you process this patch without that info? > > No, I need the git commit id of the patch that fixed this in Linus's > tree, if the files are different due to the move, that's fine, but I > still need this to reference. > Here it is: commit 5a84d6ad4c9306afb467a600a4c6d7b50da49440 Author: Arend van Spriel <arend@broadcom.com> Date: Tue Aug 23 14:14:00 2011 +0200 staging: brcm80211: fill in proper rx rate in mac80211 rx status The brcmsmac driver supports different rates on 5GHz but this is not taken into account when providing the rate index in the receive status information passed to mac80211. Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: Henry Ptasinski <henryp@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> Gr. AvS ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3.1-stable] staging: brcm80211: fix for rate index in receive status 2011-11-10 18:37 ` [PATCH 3.1-stable] staging: brcm80211: fix for rate index in receive status Arend van Spriel @ 2011-11-10 18:44 ` Greg KH 2011-11-10 18:50 ` Arend van Spriel 0 siblings, 1 reply; 3+ messages in thread From: Greg KH @ 2011-11-10 18:44 UTC (permalink / raw) To: Arend van Spriel Cc: linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org On Thu, Nov 10, 2011 at 07:37:10PM +0100, Arend van Spriel wrote: > On 11/10/2011 04:17 PM, Greg KH wrote: > > On Thu, Nov 10, 2011 at 10:48:34AM +0100, Arend van Spriel wrote: > >> This fix resolves an issue on 5GHz as posted on LKML: > >> > >> BUG: All network processes hang (brcmsmac/wpa_supplicant) > >> ID: <20111018111422.GA1979@schottelius.org> > > > > What are these tags for? I've not seen them in kernel patches before. > > > > The first line was the email Subject as posted on LKML. The second is > the message id. Neither of those are something we normally put in changelog comments, right? > >> Cc: stable@kernel.org > > > > Wrong address, please use stable@vger.kernel.org now. > > > > Noted. > > >> Reported-by: Nico Schottelius <nico-linux-20111017@schottelius.org> > >> Signed-off-by: Arend van Spriel <arend@broadcom.com> > >> --- > >> Hi Greg, > >> > >> This change is in the mainline brcmsmac driver, but I do not have a commit > >> message to refer to as we transitioned from staging to mainline in the last > >> merge window. Can you process this patch without that info? > > > > No, I need the git commit id of the patch that fixed this in Linus's > > tree, if the files are different due to the move, that's fine, but I > > still need this to reference. > > > > Here it is: > > commit 5a84d6ad4c9306afb467a600a4c6d7b50da49440 > Author: Arend van Spriel <arend@broadcom.com> > Date: Tue Aug 23 14:14:00 2011 +0200 > > staging: brcm80211: fill in proper rx rate in mac80211 rx status > > The brcmsmac driver supports different rates on 5GHz but this is not > taken into account when providing the rate index in the receive > status information passed to mac80211. > > Reviewed-by: Roland Vossen <rvossen@broadcom.com> > Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> > Reviewed-by: Henry Ptasinski <henryp@broadcom.com> > Signed-off-by: Arend van Spriel <arend@broadcom.com> > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> Thanks, that's what I need. I'll queue this up for the next 3.1 kernel after this one gets out later this week. greg k-h ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 3.1-stable] staging: brcm80211: fix for rate index in receive status 2011-11-10 18:44 ` Greg KH @ 2011-11-10 18:50 ` Arend van Spriel 0 siblings, 0 replies; 3+ messages in thread From: Arend van Spriel @ 2011-11-10 18:50 UTC (permalink / raw) To: Greg KH Cc: linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org On 11/10/2011 07:44 PM, Greg KH wrote: > On Thu, Nov 10, 2011 at 07:37:10PM +0100, Arend van Spriel wrote: >> On 11/10/2011 04:17 PM, Greg KH wrote: >> >> Here it is: >> >> commit 5a84d6ad4c9306afb467a600a4c6d7b50da49440 >> Author: Arend van Spriel <arend@broadcom.com> >> Date: Tue Aug 23 14:14:00 2011 +0200 >> >> staging: brcm80211: fill in proper rx rate in mac80211 rx status >> >> The brcmsmac driver supports different rates on 5GHz but this is not >> taken into account when providing the rate index in the receive >> status information passed to mac80211. >> >> Reviewed-by: Roland Vossen <rvossen@broadcom.com> >> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> >> Reviewed-by: Henry Ptasinski <henryp@broadcom.com> >> Signed-off-by: Arend van Spriel <arend@broadcom.com> >> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> > > Thanks, that's what I need. I'll queue this up for the next 3.1 kernel > after this one gets out later this week. > > greg k-h > Fine by me. Gr. AvS ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-11-10 18:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1320918514-5916-1-git-send-email-arend@broadcom.com>
[not found] ` <20111110151746.GB29104@suse.de>
2011-11-10 18:37 ` [PATCH 3.1-stable] staging: brcm80211: fix for rate index in receive status Arend van Spriel
2011-11-10 18:44 ` Greg KH
2011-11-10 18:50 ` Arend van Spriel
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).