[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fca9ca93-16e4-44db-8fbb-90bc6af952e7@lunn.ch>
Date: Fri, 24 Jan 2025 16:08:22 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Huisong Li <lihuisong@...wei.com>
Cc: linux-hwmon@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org, netdev@...r.kernel.org,
linux-rtc@...r.kernel.org, oss-drivers@...igine.com,
matt@...ostay.sg, mchehab@...nel.org, irusskikh@...vell.com,
andrew+netdev@...n.ch, davem@...emloft.net, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com, louis.peens@...igine.com,
hkallweit1@...il.com, linux@...linux.org.uk, kabel@...nel.org,
alexandre.belloni@...tlin.com, krzk@...nel.org,
zhanjie9@...ilicon.com, zhenglifeng1@...wei.com,
liuyonglong@...wei.com
Subject: Re: [PATCH v1 0/9] Use HWMON_CHANNEL_INFO macro to simplify code
On Fri, Jan 24, 2025 at 10:26:26AM +0800, Huisong Li wrote:
> The HWMON_CHANNEL_INFO macro is provided by hwmon.h and used widely by many
> other drivers. This series use HWMON_CHANNEL_INFO macro to simplify code.
>
> Huisong Li (9):
> media: video-i2c: Use HWMON_CHANNEL_INFO macro to simplify code
> net: aquantia: Use HWMON_CHANNEL_INFO macro to simplify code
> net: nfp: Use HWMON_CHANNEL_INFO macro to simplify code
> net: phy: marvell: Use HWMON_CHANNEL_INFO macro to simplify code
> net: phy: marvell10g: Use HWMON_CHANNEL_INFO macro to simplify code
> rtc: ab-eoz9: Use HWMON_CHANNEL_INFO macro to simplify code
> rtc: ds3232: Use HWMON_CHANNEL_INFO macro to simplify code
> w1: w1_therm: w1: Use HWMON_CHANNEL_INFO macro to simplify code
> net: phy: aquantia: Use HWMON_CHANNEL_INFO macro to simplify code
Please split these patches per subsystem. Maintainers generally have
scripts to accept patches, and those scripts don't work when there are
patches for other Maintainers mixed in. So you need 4 patchsets,
media, net, rtc and w1.
I would also like to see the HWMON Maintainers opinion on these. A
patchset containing these have already been NACKed once. These patches
do however look like valid cleanups. But are they just so you can do
something bad in your vendor tree, which will never make it to
mainline?
Andrew
Powered by blists - more mailing lists