2014-04-15 4:06 GMT+08:00 Mark Brown <broonie@xxxxxxxxxx>: > On Mon, Apr 14, 2014 at 02:30:01PM +0800, Barry Song wrote: >> From: Qipan Li <Qipan.Li@xxxxxxx> >> >> sometimes t->tx can be equal with t->rx. for example, spidev will make >> tx and rx point to spidev->buffer at the same time. currently, for this >> case, we map the buffer BIDIRECTION to fix the cache consistency. > > I've applied this but such usage is out of spec - do we have any drivers > doing this in mainline? i felt strange too at the first look from internal gerrit. qipan told me there is one: drivers/spi/spidev.c static int spidev_message(struct spidev_data *spidev, struct spi_ioc_transfer *u_xfers, unsigned n_xfers) { buf = spidev->buffer; .. k_tmp->rx_buf = buf; ... k_tmp->tx_buf = buf; ... spi_message_add_tail(k_tmp, &msg); spidev_sync(spidev, &msg); } -barry -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html