On Fri, Apr 2, 2021 at 11:14 AM Lars-Peter Clausen <lars@xxxxxxxxxx> wrote: > > On 4/1/21 11:36 AM, Andy Shevchenko wrote: > > [...] > >> + case IIO_CHAN_INFO_SCALE: > >> + /* Conversion from 10s of uC to mC > >> + * as IIO reports temperature in mC > >> + */ > >> + *val = TMP117_RESOLUTION_10UC / 10000; > >> + *val2 = (TMP117_RESOLUTION_10UC % 10000) * 100; > >> + return IIO_VAL_INT_PLUS_MICRO; > > You use 10000 many times, can you give it an appropriate name (via #define)? > #define TENTHOUSAND 10000 ;) Actually, besides April fool's day, this can have a value if 1000 is defined as mC _PER_uC or so. Oh, wait. We have it already. #define MILLIDEGREE_PER_DEGREE 1000 -- With Best Regards, Andy Shevchenko