| commit | aa9922579fee20ac4f2bb959a79bbfbff2c04585 | |
| author | Jonathan Cameron <jic23@kernel.org> | |
| Sun, 24 Mar 2013 16:58:00 +0000 (24 16:58 +0000) | ||
| committer | Jonathan Cameron <jic23@kernel.org> | |
| Tue, 2 Apr 2013 17:36:24 +0000 (2 18:36 +0100) | ||
| tree | ccbd00def5fc4076f4c46a95181b4ad78f9c95c6 | treesnapshot (tar.gz zip) |
| parent | dea120e665712a9be41a6c846e73c6af19e00af7 | commitdiff |
staging:iio:magnetometer:ak8975 drop I2C_M_NOSTART flag in read_data
This flag makes no sense whatsoever where it is.
Documentation/i2c/i2c-protocol states:
If you set the I2C_M_NOSTART variable for the first partial message,
we do not generate Addr, but we do generate the startbit S. This will
probably confuse all other clients on your bus, so don't try this.
This is exactly what is going on here. Likelihood given that the
driver never checked for this protocol mangling being available is that
it wasn't present on the test boards and hence this flag was simply
ignored. No indication of why it would be necessary has been found in
the datasheets.
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
This flag makes no sense whatsoever where it is.
Documentation/i2c/i2c-protocol states:
If you set the I2C_M_NOSTART variable for the first partial message,
we do not generate Addr, but we do generate the startbit S. This will
probably confuse all other clients on your bus, so don't try this.
This is exactly what is going on here. Likelihood given that the
driver never checked for this protocol mangling being available is that
it wasn't present on the test boards and hence this flag was simply
ignored. No indication of why it would be necessary has been found in
the datasheets.
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
| drivers/staging/iio/magnetometer/ak8975.c | diffblobblamehistory |