[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0af59f13-7b88-3ede-ef90-00457d105c3c@huawei.com>
Date: Wed, 5 Feb 2025 10:05:43 +0800
From: "lihuisong (C)" <lihuisong@...wei.com>
To: Andrew Lunn <andrew@...n.ch>
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
Hi Andrew,
在 2025/1/24 23:08, Andrew Lunn 写道:
> 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.
Many thanks for your reply and good suggestion.
I will split them.
>
> 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