Messages in this thread | | | Date | Sat, 8 Nov 2014 16:16:44 +0200 | Subject | Re: [PATCH] iio: imu: Add support for Kionix KMX61 sensor | From | Daniel Baluta <> |
| |
On Sat, Nov 8, 2014 at 2:37 PM, Jonathan Cameron <jic23@kernel.org> wrote: > On 06/11/14 12:29, Daniel Baluta wrote: >> Minimal implementation for KMX61 6-axis accelerometer/magnetometer. It exports >> raw accel/magn readings together with scale and sampling frequency. >> >> Datasheet will be available at: >> http://www.kionix.com/6-axis-accelerometer-magnetometer/kmx61 >> >> Signed-off-by: Daniel Baluta <daniel.baluta@intel.com> > Very nice. Only one comment inline. > > There are some possible race conditions (I think) based on functions > doing read, modify, write. Ideally they may want protecting with > a mutex.
You are right. I will analyze all possible race conditions and send v2.
Thanks.
| |