Re: [PATCH v3 0/3] units: complement the set of Hz units

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 1 Aug 2022 14:37:23 +0000
Dmitry Rokosov <DDRokosov@xxxxxxxxxxxxxx> wrote:

> During msa311 accel IIO driver development
> 
> https://lore.kernel.org/linux-iio/20220616104211.9257-1-ddrokosov@xxxxxxxxxxxxxx/
> 
> 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(-)
> 




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux