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

2013-09-30 Thread David Miller
From: Bjørn Mork bj...@mork.no 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

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

2013-09-30 Thread David Miller
From: Fabio Porcedda fabio.porce...@gmail.com Date: Wed, 25 Sep 2013 11:21:26 +0200 Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com 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

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 bj...@mork.no 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

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

2013-09-26 Thread Bjørn Mork
Fabio Porcedda fabio.porce...@gmail.com writes: Hi Bjørn, thanks for reviewing. On Wed, Sep 25, 2013 at 3:31 PM, Bjørn Mork bj...@mork.no 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.

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

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

2013-09-25 Thread Fabio Porcedda
Signed-off-by: Fabio Porcedda fabio.porce...@gmail.com --- 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 ---

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 fabio.porce...@gmail.com writes: Signed-off-by: Fabio Porcedda