Message ID | 1506275376-5037-1-git-send-email-harinath922@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Sun, 24 Sep 2017 13:49:36 -0400 Harinath Nampally <harinath922@gmail.com> wrote: > Improves code readability, no impact on functionality. > > Signed-off-by: Harinath Nampally <harinath922@gmail.com> I'm unconvinced by this one. At some point we have to map from the devices own description to that of IIO. The naming currently does this when the structures are created - all this rename does is push this remapping of naming down into the functions. If others thing this helps then I don't really care either way.. Jonathan > --- > drivers/iio/accel/mma8452.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/iio/accel/mma8452.c b/drivers/iio/accel/mma8452.c > index 6194169..3472e7e 100644 > --- a/drivers/iio/accel/mma8452.c > +++ b/drivers/iio/accel/mma8452.c > @@ -135,7 +135,7 @@ struct mma8452_event_regs { > u8 ev_count; > }; > > -static const struct mma8452_event_regs ev_regs_accel_falling = { > +static const struct mma8452_event_regs ff_mt_ev_regs = { > .ev_cfg = MMA8452_FF_MT_CFG, > .ev_cfg_ele = MMA8452_FF_MT_CFG_ELE, > .ev_cfg_chan_shift = MMA8452_FF_MT_CHAN_SHIFT, > @@ -145,7 +145,7 @@ static const struct mma8452_event_regs ev_regs_accel_falling = { > .ev_count = MMA8452_FF_MT_COUNT > }; > > -static const struct mma8452_event_regs ev_regs_accel_rising = { > +static const struct mma8452_event_regs trans_ev_regs = { > .ev_cfg = MMA8452_TRANSIENT_CFG, > .ev_cfg_ele = MMA8452_TRANSIENT_CFG_ELE, > .ev_cfg_chan_shift = MMA8452_TRANSIENT_CHAN_SHIFT, > @@ -777,12 +777,12 @@ static int mma8452_get_event_regs(struct mma8452_data *data, > & MMA8452_INT_TRANS) && > (data->chip_info->enabled_events > & MMA8452_INT_TRANS)) > - *ev_reg = &ev_regs_accel_rising; > + *ev_reg = &trans_ev_regs; > else > - *ev_reg = &ev_regs_accel_falling; > + *ev_reg = &ff_mt_ev_regs; > return 0; > case IIO_EV_DIR_FALLING: > - *ev_reg = &ev_regs_accel_falling; > + *ev_reg = &ff_mt_ev_regs; > return 0; > default: > return -EINVAL; -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/iio/accel/mma8452.c b/drivers/iio/accel/mma8452.c index 6194169..3472e7e 100644 --- a/drivers/iio/accel/mma8452.c +++ b/drivers/iio/accel/mma8452.c @@ -135,7 +135,7 @@ struct mma8452_event_regs { u8 ev_count; }; -static const struct mma8452_event_regs ev_regs_accel_falling = { +static const struct mma8452_event_regs ff_mt_ev_regs = { .ev_cfg = MMA8452_FF_MT_CFG, .ev_cfg_ele = MMA8452_FF_MT_CFG_ELE, .ev_cfg_chan_shift = MMA8452_FF_MT_CHAN_SHIFT, @@ -145,7 +145,7 @@ static const struct mma8452_event_regs ev_regs_accel_falling = { .ev_count = MMA8452_FF_MT_COUNT }; -static const struct mma8452_event_regs ev_regs_accel_rising = { +static const struct mma8452_event_regs trans_ev_regs = { .ev_cfg = MMA8452_TRANSIENT_CFG, .ev_cfg_ele = MMA8452_TRANSIENT_CFG_ELE, .ev_cfg_chan_shift = MMA8452_TRANSIENT_CHAN_SHIFT, @@ -777,12 +777,12 @@ static int mma8452_get_event_regs(struct mma8452_data *data, & MMA8452_INT_TRANS) && (data->chip_info->enabled_events & MMA8452_INT_TRANS)) - *ev_reg = &ev_regs_accel_rising; + *ev_reg = &trans_ev_regs; else - *ev_reg = &ev_regs_accel_falling; + *ev_reg = &ff_mt_ev_regs; return 0; case IIO_EV_DIR_FALLING: - *ev_reg = &ev_regs_accel_falling; + *ev_reg = &ff_mt_ev_regs; return 0; default: return -EINVAL;
Improves code readability, no impact on functionality. Signed-off-by: Harinath Nampally <harinath922@gmail.com> --- drivers/iio/accel/mma8452.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)