On 06/06/2012 01:07 AM, Hauke Mehrtens wrote: > --- a/drivers/net/wireless/brcm80211/brcmsmac/dma.c > +++ b/drivers/net/wireless/brcm80211/brcmsmac/dma.c > @@ -633,11 +634,14 @@ struct dma_pub *dma_attach(char *name, struct si_pub *sih, > */ > di->ddoffsetlow = 0; > di->dataoffsetlow = 0; > - /* add offset for pcie with DMA64 bus */ > - di->ddoffsetlow = 0; > - di->ddoffsethigh = SI_PCIE_DMA_H32; > - di->dataoffsetlow = di->ddoffsetlow; > - di->dataoffsethigh = di->ddoffsethigh; The two statements above can move after the if statement below. > + /* for pci bus, add offset */ > + if (sii->icbus->hosttype == BCMA_HOSTTYPE_PCI) { > + /* add offset for pcie with DMA64 bus */ > + di->ddoffsetlow = 0; > + di->ddoffsethigh = SI_PCIE_DMA_H32; > + } > + di->dataoffsetlow = di->ddoffsetlow; > + di->dataoffsethigh = di->ddoffsethigh; > + > /* WAR64450 : DMACtl.Addr ext fields are not supported in SDIOD core. */ > if ((core->id.id == BCMA_CORE_SDIO_DEV) > && ((rev > 0) && (rev <= 2))) -- 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