| Message ID | 20220801143811.14817-1-ddrokosov@sberdevices.ru (mailing list archive) |
|---|---|
| Headers | show |
| Series | units: complement the set of Hz units | expand |
On Mon, 1 Aug 2022 14:37:23 +0000 Dmitry Rokosov <DDRokosov@sberdevices.ru> wrote: > During msa311 accel IIO driver development > > https://lore.kernel.org/linux-iio/20220616104211.9257-1-ddrokosov@sberdevices.ru/ > > Andy requested to use proper units in the hz->ms calculation. Current > units.h header doesn't have milli, micro and nano HZ coefficients, so > some drivers (in the IIO subsystem) implement their own copies for that. > > The current patchset resolves such a problem and intoduces general > MILLIHZ_PER_HZ, MICROHZ_PER_HZ and NANOHZ_PER_HZ definitions in the units.h, > and fixes all drivers which duplicate these units. Ok, I'm just about convinced this is worth doing given the precedence of similar cases. Applied to the togreg branch of iio.git and pushed out as testing. Thanks, Jonathan > > Changes: > * v2->v3: > - changed UHZ_PER_HZ to MICROHZ_PER_HZ and NHZ_PER_HZ to > NANOHZ_PER_HZ to save name consistency for all new HZ units > > * v1->v2: > - changed MHZ_PER_HZ to a different name as Andy suggested > (suppose MILLIHZ_PER_HZ is good enough) > > Dmitry Rokosov (3): > units: complement the set of Hz units > iio: accel: adxl345: use HZ macro from units.h > iio: common: scmi_sensors: use HZ macro from units.h > > drivers/iio/accel/adxl345_core.c | 7 ++++--- > drivers/iio/common/scmi_sensors/scmi_iio.c | 8 ++++---- > include/linux/units.h | 3 +++ > 3 files changed, 11 insertions(+), 7 deletions(-) >
During msa311 accel IIO driver development https://lore.kernel.org/linux-iio/20220616104211.9257-1-ddrokosov@sberdevices.ru/ Andy requested to use proper units in the hz->ms calculation. Current units.h header doesn't have milli, micro and nano HZ coefficients, so some drivers (in the IIO subsystem) implement their own copies for that. The current patchset resolves such a problem and intoduces general MILLIHZ_PER_HZ, MICROHZ_PER_HZ and NANOHZ_PER_HZ definitions in the units.h, and fixes all drivers which duplicate these units. Changes: * v2->v3: - changed UHZ_PER_HZ to MICROHZ_PER_HZ and NHZ_PER_HZ to NANOHZ_PER_HZ to save name consistency for all new HZ units * v1->v2: - changed MHZ_PER_HZ to a different name as Andy suggested (suppose MILLIHZ_PER_HZ is good enough) Dmitry Rokosov (3): units: complement the set of Hz units iio: accel: adxl345: use HZ macro from units.h iio: common: scmi_sensors: use HZ macro from units.h drivers/iio/accel/adxl345_core.c | 7 ++++--- drivers/iio/common/scmi_sensors/scmi_iio.c | 8 ++++---- include/linux/units.h | 3 +++ 3 files changed, 11 insertions(+), 7 deletions(-)