On Wed, Apr 1, 2015 at 2:01 PM, Cristina Opriceana <cristina.opriceana@xxxxxxxxx> wrote: > Power down device when an error occurs in order to avoid wasting > power. Also move powerdown function up to be seen by the new call. > > Signed-off-by: Cristina Opriceana <cristina.opriceana@xxxxxxxxx> > --- > drivers/iio/light/ltr501.c | 18 ++++++++++-------- > 1 file changed, 10 insertions(+), 8 deletions(-) > > diff --git a/drivers/iio/light/ltr501.c b/drivers/iio/light/ltr501.c > index 62b7072..29ca4b8 100644 > --- a/drivers/iio/light/ltr501.c > +++ b/drivers/iio/light/ltr501.c > @@ -333,6 +333,13 @@ static int ltr501_init(struct ltr501_data *data) > data->ps_contr); > } > > +static int ltr501_powerdown(struct ltr501_data *data) > +{ > + return ltr501_write_contr(data->client, > + data->als_contr & ~LTR501_CONTR_ACTIVE, > + data->ps_contr & ~LTR501_CONTR_ACTIVE); > +} > + While at it please also fix the alignment for ltr501_write_contr parameters. Don't forget to mention this in the commit message. When running checkpatch.pl please use --strict parameter. thanks, Daniel. -- 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