Re: [PATCH] ethtool: fixed trailing statements in ethtool

2013-07-16 Thread David Miller
From: Dragos Foianu Date: Sat, 13 Jul 2013 14:43:00 +0100 > Applied fixes suggested by checkpatch.pl. > > Signed-off-by: Dragos Foianu Applied, but only so that life is easier for the next person who makes changes to this file and wants to validate things with checkpatch. -- To unsubscribe fro

Re: [PATCH] ethtool: fixed trailing statements in ethtool

2013-07-15 Thread Ben Hutchings
On Sat, 2013-07-13 at 14:43 +0100, Dragos Foianu wrote: > Applied fixes suggested by checkpatch.pl. > > Signed-off-by: Dragos Foianu I think that it's fine to put the conditional statement on the same line for repetitive conversion code like this. I don't think anyone's likely to mis-read it, w

[PATCH] ethtool: fixed trailing statements in ethtool

2013-07-13 Thread Dragos Foianu
Applied fixes suggested by checkpatch.pl. Signed-off-by: Dragos Foianu --- net/core/ethtool.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/net/core/ethtool.c b/net/core/ethtool.c index ab5fa63..78e9d92 100644 --- a/net/core/ethtool.c +++