On 06/06/2012 01:10 PM, Arend van Spriel wrote:
> 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.
I will change that.
>
>> + /* 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
