Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
My only suggestion for adding all these chips' orientation features, is to start the discussion independently from this driver. Are there other device series that provide such an orientation interrupt? Is it worth finding a representation in iio? Given the number of accelerometers these days have

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
My only suggestion for adding all these chips' orientation features, is to start the discussion independently from this driver. Are there other device series that provide such an orientation interrupt? Is it worth finding a representation in iio? Given the number of accelerometers these days have

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Harinath Nampally
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Jonathan Cameron
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: > This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 > and > fxls8471. Almost all these devices have more than one event. Current driver > design > hardcodes the event specific

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Jonathan Cameron
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: > This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 > and > fxls8471. Almost all these devices have more than one event. Current driver > design > hardcodes the event specific information, so only one event

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Jonathan Cameron
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: > This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 > and > fxls8471. Almost all these devices have more than one event. Current driver > design > hardcodes the event specific

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-09 Thread Jonathan Cameron
On Mon, 31 Jul 2017 07:17:38 -0400 Harinath Nampally wrote: > This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 > and > fxls8471. Almost all these devices have more than one event. Current driver > design > hardcodes the event specific information, so only one event

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-01 Thread Martin Kepplinger
On 2017-08-01 05:08, Harinath Nampally wrote: >> Thanks for doing that work. I have had it on my list for a long time >> and you seem to fix it. Although I'd happily review and possibly test >> it, unfortunately I can't do so before the week of August 21st. >> >> If this might go in quick, nothing

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-08-01 Thread Martin Kepplinger
On 2017-08-01 05:08, Harinath Nampally wrote: >> Thanks for doing that work. I have had it on my list for a long time >> and you seem to fix it. Although I'd happily review and possibly test >> it, unfortunately I can't do so before the week of August 21st. >> >> If this might go in quick, nothing

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Harinath Nampally
Thanks for doing that work. I have had it on my list for a long time and you seem to fix it. Although I'd happily review and possibly test it, unfortunately I can't do so before the week of August 21st. If this might go in quick, nothing will stop me from reviewing either, so, whatever.

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Harinath Nampally
Thanks for doing that work. I have had it on my list for a long time and you seem to fix it. Although I'd happily review and possibly test it, unfortunately I can't do so before the week of August 21st. If this might go in quick, nothing will stop me from reviewing either, so, whatever.

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Martin Kepplinger
Am 31. Juli 2017 13:17:38 MESZ schrieb Harinath Nampally : >This driver supports multiple devices like mma8653, mma8652, mma8452, >mma8453 and >fxls8471. Almost all these devices have more than one event. Current >driver design >hardcodes the event specific information,

Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Martin Kepplinger
Am 31. Juli 2017 13:17:38 MESZ schrieb Harinath Nampally : >This driver supports multiple devices like mma8653, mma8652, mma8452, >mma8453 and >fxls8471. Almost all these devices have more than one event. Current >driver design >hardcodes the event specific information, so only one event can be

[PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Harinath Nampally
This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be supported by this driver and current design doesn't have the

[PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-31 Thread Harinath Nampally
This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be supported by this driver and current design doesn't have the

[PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-30 Thread Harinath Nampally
This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be supported by this driver and current design doesn't have the

[PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely.

2017-07-30 Thread Harinath Nampally
This driver supports multiple devices like mma8653, mma8652, mma8452, mma8453 and fxls8471. Almost all these devices have more than one event. Current driver design hardcodes the event specific information, so only one event can be supported by this driver and current design doesn't have the