> From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Sent: Friday, March 4, 2022 7:03 PM > To: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>; linux- > iio@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Cc: Sa, Nuno <Nuno.Sa@xxxxxxxxxx>; Jonathan Cameron > <jic23@xxxxxxxxxx>; Lars-Peter Clausen <lars@xxxxxxxxxx> > Subject: [PATCH v4 3/3] iio: temperature: ltc2983: Make use of device > properties > > [External] > > Convert the module to be property provider agnostic and allow > it to be used on non-OF platforms. > > The conversion slightly changes the logic behind property reading for > the configuration values. Original code allocates just as much memory > as needed. Then for each separate 32- or 64-bit value it reads it from > the property and converts to a raw one which will be fed to the sensor. > In the new code we allocate the amount of memory needed to > retrieve all > values at once from the property and then convert them as required. > > Signed-off-by: Andy Shevchenko > <andriy.shevchenko@xxxxxxxxxxxxxxx> > Tested-by: Nuno Sá <nuno.sa@xxxxxxxxxx> > --- > v4: added checks for error pointer (Nuno), added Tb tag (Nuno) > v3: no changes > Hi Andy, So the patch now just works after applying. Also, removed the if()...else() in ltc2983_thermocouple_new() and turned it into if (!IS_ERR_OR_NULL(ref)) { ... } and applied your fwnode fix and things work.... - Nuno Sá