[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <50AB67DB.3030309@mvista.com>
Date: Tue, 20 Nov 2012 15:22:03 +0400
From: Sergei Shtylyov <sshtylyov@...sta.com>
To: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
CC: Jason Cooper <jason@...edaemon.net>,
Lior Amsalem <alior@...vell.com>, Andrew Lunn <andrew@...n.ch>,
netdev@...r.kernel.org,
Gregory Clement <gregory.clement@...e-electrons.com>,
"David S. Miller" <davem@...emloft.net>,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 3/3] net: mvneta: adjust multiline comments to net/ style
Hello.
On 19-11-2012 16:56, Thomas Petazzoni wrote:
>>> -/*
>>> - * The two bytes Marvell header. Either contains a special value used
>>> +/* The two bytes Marvell header. Either contains a special value used
>>
>> Why the heck you're doing this? It's the preferred style, see
>> Documentation/CodingStyle, chapter 8.
> No. Please read Documentation/CodingStyle, chapter 8 entirely:
> ===============================================================
> The preferred style for long (multi-line) comments is:
>
> /*
> * This is the preferred style for multi-line
> * comments in the Linux kernel source code.
> * Please use it consistently.
> *
> * Description: A column of asterisks on the left side,
> * with beginning and ending almost-blank lines.
> */
>
> For files in net/ and drivers/net/ the preferred style for long
> (multi-line) comments is a little different.
>
> /* The preferred comment style for files in net/ and drivers/net
> * looks like this.
> *
> * It is nearly the same as the generally preferred comment style,
> * but there is no initial almost-blank line.
> */
> ===============================================================
> Seen the second part?
Not until now probably. Sorry for the noise then. :-/
WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists