Re: [PATCH v4 2/3] iio: light: add support for veml6046x00 RGBIR color sensor
From: Andreas Klinger
Date: Sun May 25 2025 - 03:35:16 EST
Hi Andy,
thanks for the review. I have a question and a comment below.
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> schrieb am Mo, 19. Mai 14:04:
> > +/*
> > + * veml6046x00_gain_pd - translation from gain index (used in the driver) to
> > + * gain (sensor) and PD
> > + * @gain_sen: Gain used in the sensor as described in the datasheet of the
> > + * sensor
> > + * @pd: Photodiode size in the sensor
>
> This is made to look like kernel-doc, but it's not marked as a such, why?
I'll remove the '@'
...
> > + ret = regmap_clear_bits(data->regmap, VEML6046X00_REG_CONF0,
> > + VEML6046X00_CONF0_ON_0);
>
> Something wrong with the indentation. Please, fix all places like this...
>
> > + if (ret) {
> > + dev_err(dev, "Failed to set bit for power on %d\n", ret);
> > + return ret;
> > + }
> > +
> > + return regmap_clear_bits(data->regmap, VEML6046X00_REG_CONF1,
> > + VEML6046X00_CONF1_ON_1);
>
> ...or like this.
>
> > +}
I don't get the point what is wrong with the indentation. In the coding-style it
says the decendant line should be placed to the right.
Did i miss something?
Best regards,
Andreas
Attachment:
signature.asc
Description: PGP signature