Re: [PATCH v4 0/4] iio: temperature: mlx90632: Add extended calibration calculations

From: Andy Shevchenko
Date: Sat Aug 08 2020 - 16:07:27 EST


On Sat, Aug 8, 2020 at 3:10 PM Crt Mori <cmo@xxxxxxxxxxx> wrote:
>
> Since the second patch is dependent on the first and was still not
> merged, I have decided to send them together. First patch just makes
> second one more readable as it splits out the repeated calculation and
> that enables the second patch to tweak the variable to the new
> condition.

So, we are closer, but here is one remark, it's not good to send a new
version if you are not going to address _all_ comments (it's fine to
do, but you need to answer first why you are not going to satisfy
them). For example, explicit castings here and there, voodoo
arithmetics is left uncommented.


--
With Best Regards,
Andy Shevchenko