On Wed, 20 Sep 2017 20:59:56 +0200 Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Makes sense as do the other drivers. Feel free to add Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> to all of them (though they hardly took a lot of reviewing given how simple the patches were :) > --- > drivers/i2c/i2c-dev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/i2c/i2c-dev.c b/drivers/i2c/i2c-dev.c > index 6f638bbc922db4..bbc7aadb4c899d 100644 > --- a/drivers/i2c/i2c-dev.c > +++ b/drivers/i2c/i2c-dev.c > @@ -280,6 +280,8 @@ static noinline int i2cdev_ioctl_rdwr(struct i2c_client *client, > res = PTR_ERR(rdwr_pa[i].buf); > break; > } > + /* memdup_user allocates with GFP_KERNEL, so DMA is ok */ > + rdwr_pa[i].flags |= I2C_M_DMA_SAFE; > > /* > * If the message length is received from the slave (similar
