Re: [net-next PATCH] net: convert users of bitmap_foo() to linkmode_foo()

[Sean Anderson <sean.anderson@seco.com>]

On 10/25/21 3:20 PM, Jakub Kicinski wrote:
> On Sun, 24 Oct 2021 20:50:45 +0200 Andrew Lunn wrote:
>> On Fri, Oct 22, 2021 at 06:41:04PM -0400, Sean Anderson wrote:
>> > This converts instances of
>> > 	bitmap_foo(args..., __ETHTOOL_LINK_MODE_MASK_NBITS)
>> > to
>> > 	linkmode_foo(args...)
>>
>> It does touch a lot of files, but it does help keep the API uniform.
>>
>> > I manually fixed up some lines to prevent them from being excessively
>> > long. Otherwise, this change was generated with the following semantic
>> > patch:
>>
>> How many did you fix?
>
> Strange, I thought coccinelle does pretty well on checkpatch compliance.

It does, but the problem is there is no obvious place to break

	long_function_name(another_long_function_name(and_some_variable)))

without introducing a variable.

>> > Because this touches so many files in the net tree, you may want to
>> > generate a new diff using the semantic patch above when you apply this.
>>
>> If it still applies cleanly, i would just apply it.
>
> It seems to apply but does not build (missing include in mlx4?)

Hmm. I tried to determine if the correct headers were included, but it
looks like there was an error there. In any case, it seems like David
fixed it up when he applied it.

--Sean