Re: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-30 Thread David Miller
From: Fabio Porcedda Date: Wed, 25 Sep 2013 11:21:26 +0200 > Signed-off-by: Fabio Porcedda Applied to net-next, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/ma

Re: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-30 Thread David Miller
From: Bjørn Mork Date: Wed, 25 Sep 2013 15:31:39 +0200 > Sorry, I really don't see the point of this. Yes, the lines are longer > than 80 columns, but breaking them don't improve the readability at > all. On the contrary, IMHO. > > So NAK from me for this part. Sorry, breaking things up like

RE: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-26 Thread David Laight
> Anything that breaks a previously unbroken argument list will reduce the > readability in my opinion. The lines can of course not be unlimited, > but there is no need to set the limit as low as 80 columns. Feedback > I've got from developers using e.g. 80 column braille devices is that > longer

Re: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-26 Thread Bjørn Mork
Fabio Porcedda writes: > Hi Bjørn, > thanks for reviewing. > > On Wed, Sep 25, 2013 at 3:31 PM, Bjørn Mork wrote: >> Sorry, I really don't see the point of this. Yes, the lines are longer >> than 80 columns, but breaking them don't improve the readability at >> all. On the contrary, IMHO. >> >

Re: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-26 Thread Fabio Porcedda
Hi Bjørn, thanks for reviewing. On Wed, Sep 25, 2013 at 3:31 PM, Bjørn Mork wrote: > Sorry, I really don't see the point of this. Yes, the lines are longer > than 80 columns, but breaking them don't improve the readability at > all. On the contrary, IMHO. > > So NAK from me for this part. Whic

Re: [PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-25 Thread Bjørn Mork
Sorry, I really don't see the point of this. Yes, the lines are longer than 80 columns, but breaking them don't improve the readability at all. On the contrary, IMHO. So NAK from me for this part. Bjørn Fabio Porcedda writes: > Signed-off-by: Fabio Porcedda > --- > drivers/net/usb/qmi_wwa

[PATCH 2/2] net: qmi_wwan: fix checkpatch warnings

2013-09-25 Thread Fabio Porcedda
Signed-off-by: Fabio Porcedda --- drivers/net/usb/qmi_wwan.c | 56 +- 1 file changed, 36 insertions(+), 20 deletions(-) diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c index 5f6b6fa..0e59f9e 100644 --- a/drivers/net/usb/qmi_wwan.c