Re: [PATCH v4] bcma: fix two coding style issues, more than 80 characters per line.

From: Kalle Valo
Date: Wed Jan 07 2015 - 13:39:54 EST


RafaÅ MiÅecki <zajec5@xxxxxxxxx> writes:

> On 7 January 2015 at 18:55, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote:
>> Oscar Forner Martinez <oscar.forner.martinez@xxxxxxxxx> writes:
>>
>>> Two lines with more than 80 characters per line have been split in several lines.
>>>
>>> Signed-off-by: Oscar Forner Martinez <oscar.forner.martinez@xxxxxxxxx>
>>
>> Thanks, applied to wireless-drivers-next.git.
>
> Could you still replace it (V4) with an older version (V3)? Before I
> explained V3 was OK Oscar already sent this (worse) V4.

No. I'm not going to rebase a public git tree unless there's a huge
disaster.

But if I see V4 I'm going to assume that's the one I need to apply, not
some older version. If V4 is not good for some reason, make it
ABSOLUTELY clear that I should not apply it. I do not want to start
guessing what patch I should apply.

> This is not critical though, we can always send separated patch for
> one more line > 80 chars.

Please send a followup patch if you want to fix something.

--
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/