On Sat, 2014-10-18 at 13:11 +0100, Jonathan Cameron wrote:
> On 10/10/14 17:30, Srinivas Pandruvada wrote:
> > This chip is capable to identify motion across x, y and z axes. So
> > send different events.
> >
> > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
> This is an odd mix of a true fix and change of functionality. It probably wants
> to be broken up into clearer sections. We 'could' take this through
> the fixes branch... I'm just not entirely sure if we want to.
>
> I suppose it probably makes sense as it is an abi change and the driver
> is just working it's way towards a mainline release.
>
> Can't take it just yet via that route due to backlogged pull request.
>
I realized that and I have already sent updated version on 10/10/14,
broken this into two parts: One fix and one enhancement.
Thanks,
Srinivas
> J
>
> > ---
> > drivers/iio/accel/bmc150-accel.c | 25 +++++++++++++++++++++----
> > 1 file changed, 21 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/iio/accel/bmc150-accel.c b/drivers/iio/accel/bmc150-accel.c
> > index fdb101c..352d959 100644
> > --- a/drivers/iio/accel/bmc150-accel.c
> > +++ b/drivers/iio/accel/bmc150-accel.c
> > @@ -44,6 +44,9 @@
> >
> > #define BMC150_ACCEL_REG_INT_STATUS_2 0x0B
> > #define BMC150_ACCEL_ANY_MOTION_MASK 0x07
> > +#define BMC150_ACCEL_ANY_MOTION_BIT_X BIT(0)
> > +#define BMC150_ACCEL_ANY_MOTION_BIT_Y BIT(1)
> > +#define BMC150_ACCEL_ANY_MOTION_BIT_Z BIT(2)
> > #define BMC150_ACCEL_ANY_MOTION_BIT_SIGN BIT(3)
> >
> > #define BMC150_ACCEL_REG_PMU_LPW 0x11
> > @@ -850,7 +853,7 @@ static const struct attribute_group bmc150_accel_attrs_group = {
> >
> > static const struct iio_event_spec bmc150_accel_event = {
> > .type = IIO_EV_TYPE_ROC,
> > - .dir = IIO_EV_DIR_RISING | IIO_EV_DIR_FALLING,
> > + .dir = IIO_EV_DIR_EITHER,
> T
> > .mask_separate = BIT(IIO_EV_INFO_VALUE) |
> > BIT(IIO_EV_INFO_ENABLE) |
> > BIT(IIO_EV_INFO_PERIOD)
> > @@ -1097,12 +1100,26 @@ static irqreturn_t bmc150_accel_event_handler(int irq, void *private)
> > else
> > dir = IIO_EV_DIR_RISING;
> >
> > - if (ret & BMC150_ACCEL_ANY_MOTION_MASK)
> > + if (ret & BMC150_ACCEL_ANY_MOTION_BIT_X)
> > iio_push_event(indio_dev, IIO_MOD_EVENT_CODE(IIO_ACCEL,
> > 0,
> > - IIO_MOD_X_OR_Y_OR_Z,
> > + IIO_MOD_X,
> > IIO_EV_TYPE_ROC,
> > - IIO_EV_DIR_EITHER),
> > + dir),
> > + data->timestamp);
> > + if (ret & BMC150_ACCEL_ANY_MOTION_BIT_Y)
> > + iio_push_event(indio_dev, IIO_MOD_EVENT_CODE(IIO_ACCEL,
> > + 0,
> > + IIO_MOD_Y,
> > + IIO_EV_TYPE_ROC,
> > + dir),
> > + data->timestamp);
> > + if (ret & BMC150_ACCEL_ANY_MOTION_BIT_Z)
> > + iio_push_event(indio_dev, IIO_MOD_EVENT_CODE(IIO_ACCEL,
> > + 0,
> > + IIO_MOD_Z,
> > + IIO_EV_TYPE_ROC,
> > + dir),
> > data->timestamp);
> > ack_intr_status:
> > if (!data->dready_trigger_on)
> >
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
