2023-06-12 12:32:55

by Herve Codina

Subject: [PATCH v3 08/12] iio: inkern: Replace a FIXME comment by a TODO one

This FIXME comment is more a TODO one.
It is a note when someone will need for this currently unsupported case.

Change from FIXME to TODO.

Signed-off-by: Herve Codina <[email protected]>
---
drivers/iio/inkern.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/inkern.c b/drivers/iio/inkern.c
index ae1a41d3a559..4331e74191cf 100644
--- a/drivers/iio/inkern.c
+++ b/drivers/iio/inkern.c
@@ -879,7 +879,7 @@ static int iio_channel_read_max(struct iio_channel *chan,
*val = max_array(vals, length);
break;
default:
- /* FIXME: learn about max for other iio values */
+ /* TODO: learn about max for other iio values */
return -EINVAL;
}
return 0;
--
2.40.1

2023-06-12 14:28:52

by Andy Shevchenko

Subject: Re: [PATCH v3 08/12] iio: inkern: Replace a FIXME comment by a TODO one

On Mon, Jun 12, 2023 at 3:30 PM Herve Codina <[email protected]> wrote:
>
> This FIXME comment is more a TODO one.
> It is a note when someone will need for this currently unsupported case.

> Change from FIXME to TODO.

Suggested-by: Jonathan ... ?

Reviewed-by: Andy Shevchenko <[email protected]>

--
With Best Regards,
Andy Shevchenko