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

On 10/25/21 3:20 PM, Jakub Kicinski wrote:

On Sun, 24 Oct 2021 20:50:45 +0200 Andrew Lunn wrote:

quoted

On Fri, Oct 22, 2021 at 06:41:04PM -0400, Sean Anderson wrote:

quoted

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.

quoted

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.

quoted

quoted

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