Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 28 Aug 2022 16:47:08 +0100
From:   Jonathan Cameron <jic23@kernel.org>
To:     Dmitry Rokosov <DDRokosov@sberdevices.ru>
Cc:     "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>,
        "daniel.lezcano@linaro.org" <daniel.lezcano@linaro.org>,
        "wsa@kernel.org" <wsa@kernel.org>,
        "andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
        "lars@metafoo.de" <lars@metafoo.de>,
        "Michael.Hennerich@analog.com" <Michael.Hennerich@analog.com>,
        "jbhayana@google.com" <jbhayana@google.com>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        kernel <kernel@sberdevices.ru>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 0/3] units: complement the set of Hz units
Message-ID: <20220828164708.017c400d@jic23-huawei>
In-Reply-To: <20220801143811.14817-1-ddrokosov@sberdevices.ru>
References: <20220801143811.14817-1-ddrokosov@sberdevices.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

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(-)
>