[PATCH 02/32] staging: brcm80211: new Kconfig item to enable IO reordering fix
Greg KH
greg at kroah.com
Tue May 17 19:19:29 UTC 2011
On Sun, May 15, 2011 at 02:53:08PM +0200, Arend van Spriel wrote:
> On 05/14/2011 05:46 PM, Greg KH wrote:
> >On Fri, May 13, 2011 at 11:59:22AM +0200, Arend van Spriel wrote:
> >>From: Roland Vossen<rvossen at broadcom.com>
> >>
> >>bcm4716 (which includes 4717& 4718), plus 4706 on PCIe can reorder
> >>transactions. As a fix, a read after write is performed on certain places
> >>in the code. Older chips and the newer 5357 family don't require this fix.
> >>
> >>Cc: devel at linuxdriverproject.org
> >>Cc: linux-wireless at vger.kernel.org
> >>Reviewed-by: Henry Ptasinski<henryp at broadcom.com>
> >>Reviewed-by: Brett Rudley<brudley at broadcom.com>
> >>Signed-off-by: Arend van Spriel<arend at broadcom.com>
> >>---
> >> drivers/staging/brcm80211/Kconfig | 8 ++++++++
> >> .../staging/brcm80211/brcmsmac/phy/wlc_phy_cmn.c | 4 ++--
> >> drivers/staging/brcm80211/include/bcmutils.h | 11 +++++++++++
> >> 3 files changed, 21 insertions(+), 2 deletions(-)
> >>
> >>diff --git a/drivers/staging/brcm80211/Kconfig b/drivers/staging/brcm80211/Kconfig
> >>index f4cf9b2..a11ffc7 100644
> >>--- a/drivers/staging/brcm80211/Kconfig
> >>+++ b/drivers/staging/brcm80211/Kconfig
> >>@@ -36,3 +36,11 @@ config BRCMDBG
> >> depends on BRCMSMAC || BRCMFMAC
> >> ---help---
> >> Selecting this enables additional code for debug purposes.
> >>+
> >>+config BRCM_FIX_IO_ORDER
> >>+ bool "Broadcom I/O order fix"
> >>+ default n
> >>+ depends on BRCMSMAC
> >>+ ---help---
> >>+ Select this for for platforms running on bcm4706, 4716, 4717 or 4718.
> >>+ It resolves an I/O issue with the PCIe core used in these chips.
> >No, don't make a new kconfig option for something that everyone has to
> >enable as they don't know what they have on their system (think
> >universal kernels built by distros.)
> >
> >Do this check dynamically, depending on the chip the driver is running
> >on, don't rely on a user to know what to do here.
> >
> >So I'm not going to accept this patch, sorry.
> Hi Greg,
>
> No problem. Thanks for the feedback. There are two other patches in
> this series which depend on this one. Could you also drop the
> following:
>
> [PATCH 03/32] staging: brcm80211: replaced #ifdef __mips__ sections
> by W_REG_FLUSH
> [PATCH 11/32] staging: brcm80211: Kconfig fix for 'Broadcom I/O order fix'
Ok, now dropped.
thanks,
greg k-h
More information about the devel
mailing list