Re: [PATCH v5 3/4] iio: chemical: add support for winsen MHZ19B CO2 sensor

[Gyeyoung Baek <gye976@gmail.com>]

> > +	if (ppm) {
> > +		if (!in_range(ppm, 1000, 4001)) {
> > +			dev_dbg(&indio_dev->dev,
> > +				"span point ppm should be between 1000 and 5000 inclusive.");
> > +			return -EINVAL;
> > +		}
>
> I proposed to define the _MIN and _MAX constants for the range and use them in
> the parameters. Any objection?

I'm sorry, my Gmail didn’t show your replies for some reason, so I missed your previous review. 
I only noticed it after checking iio lore directly.

It seems more readable to use _MIN and _MAX. I'll use those, thanks!

---

> > +
> > +	return len;
> > +}
> 
> Otherwise LGTM, thanks!

I'll send new patch based on your other review comments.
thanks.

--
Regards,
Gyeyoung