[PATCH 05/30] iio: chemical: sgp30: Add description for sgp_read_cmd()'s 'duration_us'
Jonathan Cameron
jic23 at kernel.org
Sat Jul 18 10:33:45 EDT 2020
On Thu, 16 Jul 2020 14:59:03 +0100
Lee Jones <lee.jones at linaro.org> wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/iio/chemical/sgp30.c:236: warning: Function parameter or member 'duration_us' not described in 'sgp_read_cmd'
>
> Cc: Andreas Brauchli <a.brauchli at elementarea.net>
> Cc: Pascal Sachs <pascal.sachs at sensirion.com>
> Signed-off-by: Lee Jones <lee.jones at linaro.org>
Perhaps we can do better on the wording.
> ---
> drivers/iio/chemical/sgp30.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/iio/chemical/sgp30.c b/drivers/iio/chemical/sgp30.c
> index 403e8803471a9..82e407d48a0aa 100644
> --- a/drivers/iio/chemical/sgp30.c
> +++ b/drivers/iio/chemical/sgp30.c
> @@ -227,6 +227,7 @@ static int sgp_verify_buffer(const struct sgp_data *data,
> * @cmd: SGP Command to issue
> * @buf: Raw data buffer to use
> * @word_count: Num words to read, excluding CRC bytes
> + * @duration_us: Time to sleep in microseconds
Time taken to sensor to take a reading and data to be ready.
Something like that maybe? I've not checked the datasheet but from
code that looks to be what this describes..
Jonathan
> *
> * Return: 0 on success, negative error otherwise.
> */
More information about the linux-arm-kernel
mailing list