Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-26 Thread Rafał Miłecki
On 24 December 2014 at 08:20, Kalle Valo wrote: > Oscar Forner Martinez writes: > >> Three lines with more than 80 characters per line have been split in several >> lines. >> >> Signed-off-by: Oscar Forner Martinez >> --- >> drivers/bcma/driver_chipcommon.c | 10 +++--- >> 1 file changed,

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-26 Thread Rafał Miłecki
On 24 December 2014 at 08:20, Kalle Valo kv...@codeaurora.org wrote: Oscar Forner Martinez oscar.forner.marti...@gmail.com writes: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinez oscar.forner.marti...@gmail.com ---

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-24 Thread Òscar Forner Martínez
Hi Kalle, I am doing it from London with the timezone of Spain, maybe is that the odd stuff? I can change it and do the patch again. I will check the style guide to do the changes properly this time. I have a couple of questions, as it is my first patch I am not sure how to do them. Do I have

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-24 Thread Arend van Spriel
On 12/24/14 08:20, Kalle Valo wrote: Oscar Forner Martinez writes: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinez --- drivers/bcma/driver_chipcommon.c | 10 +++--- 1 file changed, 7 insertions(+), 3

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-24 Thread Arend van Spriel
On 12/24/14 08:20, Kalle Valo wrote: Oscar Forner Martinezoscar.forner.marti...@gmail.com writes: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinezoscar.forner.marti...@gmail.com --- drivers/bcma/driver_chipcommon.c |

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-24 Thread Òscar Forner Martínez
Hi Kalle, I am doing it from London with the timezone of Spain, maybe is that the odd stuff? I can change it and do the patch again. I will check the style guide to do the changes properly this time. I have a couple of questions, as it is my first patch I am not sure how to do them. Do I have

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Kalle Valo
Oscar Forner Martinez writes: > Three lines with more than 80 characters per line have been split in several > lines. > > Signed-off-by: Oscar Forner Martinez > --- > drivers/bcma/driver_chipcommon.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) Just to handle the

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Joe Perches
On Wed, 2014-12-24 at 00:46 +0100, Rafał Miłecki wrote: > On 24 December 2014 at 00:55, Oscar Forner Martinez > wrote: > > Three lines with more than 80 characters per line have been split in > > several lines. > > > > Signed-off-by: Oscar Forner Martinez > > --- > >

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Rafał Miłecki
On 24 December 2014 at 00:55, Oscar Forner Martinez wrote: > Three lines with more than 80 characters per line have been split in several > lines. Since you will need V2 anyway, you may also drop this "Drivers:" prefix :) -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Rafał Miłecki
On 24 December 2014 at 00:55, Oscar Forner Martinez wrote: > Three lines with more than 80 characters per line have been split in several > lines. > > Signed-off-by: Oscar Forner Martinez > --- > drivers/bcma/driver_chipcommon.c | 10 +++--- > 1 file changed, 7 insertions(+), 3

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Rafał Miłecki
On 24 December 2014 at 00:55, Oscar Forner Martinez oscar.forner.marti...@gmail.com wrote: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinez oscar.forner.marti...@gmail.com --- drivers/bcma/driver_chipcommon.c | 10

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Rafał Miłecki
On 24 December 2014 at 00:55, Oscar Forner Martinez oscar.forner.marti...@gmail.com wrote: Three lines with more than 80 characters per line have been split in several lines. Since you will need V2 anyway, you may also drop this Drivers: prefix :) -- To unsubscribe from this list: send the

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Joe Perches
On Wed, 2014-12-24 at 00:46 +0100, Rafał Miłecki wrote: On 24 December 2014 at 00:55, Oscar Forner Martinez oscar.forner.marti...@gmail.com wrote: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinez

Re: [PATCH] Drivers: bcma: Fix three coding style issues, more than 80 characters per line.

2014-12-23 Thread Kalle Valo
Oscar Forner Martinez oscar.forner.marti...@gmail.com writes: Three lines with more than 80 characters per line have been split in several lines. Signed-off-by: Oscar Forner Martinez oscar.forner.marti...@gmail.com --- drivers/bcma/driver_chipcommon.c | 10 +++--- 1 file changed, 7