Re: [PATCH] net: intel: Cleanup e1000 - add space between }}

From: Jeff Kirsher
Date: Mon Aug 26 2019 - 04:03:37 EST

On Fri, 2019-08-23 at 19:14 +0000, Forrest Fleming wrote:
> suggested by checkpatch
> Signed-off-by: Forrest Fleming <ffleming@xxxxxxxxx>
> ---
> .../net/ethernet/intel/e1000/e1000_param.c | 28 +++++++++----------
> 1 file changed, 14 insertions(+), 14 deletions(-)

While I do not see an issue with this change, I wonder how important it is
to make such a change. Especially since most of the hardware supported by
this driver is not available for testing. In addition, this is one
suggested change by that I personally do not agree with.

This is not a hard NAK, but you have to explain how this change makes the
code more readable before I consider it.

Attachment: signature.asc
Description: This is a digitally signed message part