Re: [PATCH v1 0/9] Use HWMON_CHANNEL_INFO macro to simplify code

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

 



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




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux