Messages in this thread | | | Subject | Re: [PATCH] net: intel: Cleanup e1000 - add space between }} | From | Jeff Kirsher <> | Date | Mon, 26 Aug 2019 01:03:33 -0700 |
| |
On Fri, 2019-08-23 at 19:14 +0000, Forrest Fleming wrote: > suggested by checkpatch > > Signed-off-by: Forrest Fleming <ffleming@gmail.com> > --- > .../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 checkpatch.pl 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. [unhandled content-type:application/pgp-signature] | |