[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <803c76e7-30fc-4a0d-ab30-ea779747300b@collabora.com>
Date: Mon, 12 Jan 2026 17:15:33 +0100
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Andi Shyti <andi@...da.it>, Wolfram Sang <wsa+renesas@...g-engineering.com>,
Qianfeng Rong <rongqianfeng@...o.com>, linux-i2c@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org, linux-rockchip@...ts.infradead.org
Cc: Khalil Blaiech <kblaiech@...dia.com>, Asmaa Mnebhi <asmaa@...dia.com>,
Andi Shyti <andi.shyti@...nel.org>, Qii Wang <qii.wang@...iatek.com>,
Matthias Brugger <matthias.bgg@...il.com>, Linus Walleij
<linusw@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
Patrice Chotard <patrice.chotard@...s.st.com>,
Ard Biesheuvel <ardb@...nel.org>
Subject: Re: [PATCH v1 0/7] i2c: busses: Introduce and use HZ_PER_GHZ
Il 12/01/26 14:46, Andy Shevchenko ha scritto:
> A few drivers for I²C host controllers use HZ_PER_GHZ in numeric form.
> This is less readable and prone to subtle mistakes. Instead, introduce
> a well defined constant and use it in the drivers. No functional change
> implied except one case where 64-bit division replaced with 32-bit one.
>
> Andy Shevchenko (7):
> units: Add HZ_PER_GHZ
> i2c: mlxbf: Use HZ_PER_GHZ constant instead of custom one
> i2c: mt65xx: Use HZ_PER_GHZ constant instead of plain number
> i2c: nomadik: Use HZ_PER_GHZ constant instead of plain number
> i2c: rk3x: Use HZ_PER_GHZ constant instead of plain number
> i2c: st: Use HZ_PER_GHZ constant instead of plain number
> i2c: synquacer: Use HZ_PER_GHZ constant instead of plain number
>
> drivers/i2c/busses/i2c-mlxbf.c | 5 ++---
> drivers/i2c/busses/i2c-mt65xx.c | 11 ++++++-----
> drivers/i2c/busses/i2c-nomadik.c | 7 ++++---
> drivers/i2c/busses/i2c-rk3x.c | 12 ++++++------
> drivers/i2c/busses/i2c-st.c | 3 ++-
> drivers/i2c/busses/i2c-synquacer.c | 3 ++-
> include/linux/units.h | 3 +++
> 7 files changed, 25 insertions(+), 19 deletions(-)
>
Whole series is
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Powered by blists - more mailing lists