Tomas Winkler wrote: > Add Intel Wireless MultiCom 3200 SDIO BT driver > IWMC3200 is 4Wireless Com CHIP (GPS/BT/WiFi/WiMAX). > wmc3200bt is derived from btsdio driver Can you list the hardware differences between this chip and the standard? > + unsigned char tx_buf[2048]; > + unsigned char rx_buf[2048]; I don't think these buffers are cache line aligned so can't be used for DMA. > + /* in IWMC_BT we need to read the transfer length > + * before interrupt clearing */ > + val1 = sdio_readb(data->func, REG_H2D_COUNT_L, &err); > + IBT_DBG(data->func, "%s to read len1," > + " reg_addr=%x ret=%d val=%x\n", > + err ? "Failed" : "Succeeded", > + REG_H2D_COUNT_L, err, err ? 0xff : val1); > + if (err == 0) { > + val2 = sdio_readb(data->func, REG_H2D_COUNT_H, &err); > + IBT_DBG(data->func, "%s to read len2," > + " reg_addr=%x ret=%d val=%x\n", > + err ? "Failed" : "Succeeded", > + REG_H2D_COUNT_H, err, err ? 0xff : val2); Wow. Intel really messed up the hardware here. An extra two SDIO commands per packet? I guess no one was thinking of performance... David -- David Vrabel, Senior Software Engineer, Drivers CSR, Churchill House, Cambridge Business Park, Tel: +44 (0)1223 692562 Cowley Road, Cambridge, CB4 0WZ http://www.csr.com/ Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html