Re: [PATCH 1/1 linux-next] iio: common: remove unnecessary sizeof(u8)
On 22/11/14 01:26, Hartmut Knaack wrote: > Fabian Frederick schrieb am 16.11.2014 13:33: >> sizeof(u8) is always 1. >> >> Signed-off-by: Fabian Frederick > Acked-by: Hartmut Knaack Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan >> --- >> drivers/iio/common/st_sensors/st_sensors_spi.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/iio/common/st_sensors/st_sensors_spi.c >> b/drivers/iio/common/st_sensors/st_sensors_spi.c >> index 78a6a1a..5b37737 100644 >> --- a/drivers/iio/common/st_sensors/st_sensors_spi.c >> +++ b/drivers/iio/common/st_sensors/st_sensors_spi.c >> @@ -54,7 +54,7 @@ static int st_sensors_spi_read(struct >> st_sensor_transfer_buffer *tb, >> if (err) >> goto acc_spi_read_error; >> >> -memcpy(data, tb->rx_buf, len*sizeof(u8)); >> +memcpy(data, tb->rx_buf, len); >> mutex_unlock(&tb->buf_lock); >> return len; >> >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1 linux-next] iio: common: remove unnecessary sizeof(u8)
Fabian Frederick schrieb am 16.11.2014 13:33: > sizeof(u8) is always 1. > > Signed-off-by: Fabian Frederick Acked-by: Hartmut Knaack > --- > drivers/iio/common/st_sensors/st_sensors_spi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/common/st_sensors/st_sensors_spi.c > b/drivers/iio/common/st_sensors/st_sensors_spi.c > index 78a6a1a..5b37737 100644 > --- a/drivers/iio/common/st_sensors/st_sensors_spi.c > +++ b/drivers/iio/common/st_sensors/st_sensors_spi.c > @@ -54,7 +54,7 @@ static int st_sensors_spi_read(struct > st_sensor_transfer_buffer *tb, > if (err) > goto acc_spi_read_error; > > - memcpy(data, tb->rx_buf, len*sizeof(u8)); > + memcpy(data, tb->rx_buf, len); > mutex_unlock(&tb->buf_lock); > return len; > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/1 linux-next] iio: common: remove unnecessary sizeof(u8)
sizeof(u8) is always 1. Signed-off-by: Fabian Frederick --- drivers/iio/common/st_sensors/st_sensors_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/common/st_sensors/st_sensors_spi.c b/drivers/iio/common/st_sensors/st_sensors_spi.c index 78a6a1a..5b37737 100644 --- a/drivers/iio/common/st_sensors/st_sensors_spi.c +++ b/drivers/iio/common/st_sensors/st_sensors_spi.c @@ -54,7 +54,7 @@ static int st_sensors_spi_read(struct st_sensor_transfer_buffer *tb, if (err) goto acc_spi_read_error; - memcpy(data, tb->rx_buf, len*sizeof(u8)); + memcpy(data, tb->rx_buf, len); mutex_unlock(&tb->buf_lock); return len; -- 1.9.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/