linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [path] adding header info in raw.h
@ 2011-11-24 11:09 Rahul Bedarkar
  2011-11-24 16:14 ` Alexey Dobriyan
  0 siblings, 1 reply; 4+ messages in thread
From: Rahul Bedarkar @ 2011-11-24 11:09 UTC (permalink / raw)
  To: linux-kernel

Hi,

    Adding header info in raw.h

[master 1e3c43b] Added info header in raw.h 1 files changed, 6
insertions(+), 0 deletions(-)


diff --git a/include/linux/raw.h b/include/linux/raw.h
index 62d543e..7ca527a 100644
--- a/include/linux/raw.h
+++ b/include/linux/raw.h
@@ -1,3 +1,9 @@
+/*
+* include/linux/raw.h
+*
+* include file for raw block device drivers
+*/
+
 #ifndef __LINUX_RAW_H
 #define __LINUX_RAW_H


Regards,
Rahul Bedarkar

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [path] adding header info in raw.h
  2011-11-24 11:09 [path] adding header info in raw.h Rahul Bedarkar
@ 2011-11-24 16:14 ` Alexey Dobriyan
  2011-11-25  4:28   ` Rahul Bedarkar
  0 siblings, 1 reply; 4+ messages in thread
From: Alexey Dobriyan @ 2011-11-24 16:14 UTC (permalink / raw)
  To: Rahul Bedarkar; +Cc: linux-kernel

On Thu, Nov 24, 2011 at 1:09 PM, Rahul Bedarkar <rpal143@gmail.com> wrote:
> --- a/include/linux/raw.h
> +++ b/include/linux/raw.h
> @@ -1,3 +1,9 @@
> +/*
> +* include/linux/raw.h
> +*
> +* include file for raw block device drivers
> +*/

Quite a useless comment.
Filename inside file especially.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [path] adding header info in raw.h
  2011-11-24 16:14 ` Alexey Dobriyan
@ 2011-11-25  4:28   ` Rahul Bedarkar
  2011-11-25  4:33     ` Randy Dunlap
  0 siblings, 1 reply; 4+ messages in thread
From: Rahul Bedarkar @ 2011-11-25  4:28 UTC (permalink / raw)
  To: Alexey Dobriyan; +Cc: linux-kernel

On Thu, Nov 24, 2011 at 9:44 PM, Alexey Dobriyan <adobriyan@gmail.com> wrote:
> On Thu, Nov 24, 2011 at 1:09 PM, Rahul Bedarkar <rpal143@gmail.com> wrote:
>> --- a/include/linux/raw.h
>> +++ b/include/linux/raw.h
>> @@ -1,3 +1,9 @@
>> +/*
>> +* include/linux/raw.h
>> +*
>> +* include file for raw block device drivers
>> +*/
>
> Quite a useless comment.
> Filename inside file especially.
>
I have seen File name inside file rbtree.h, it is 19th line in my repo.

Regards,
Rahul Bedarkar

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [path] adding header info in raw.h
  2011-11-25  4:28   ` Rahul Bedarkar
@ 2011-11-25  4:33     ` Randy Dunlap
  0 siblings, 0 replies; 4+ messages in thread
From: Randy Dunlap @ 2011-11-25  4:33 UTC (permalink / raw)
  To: Rahul Bedarkar; +Cc: Alexey Dobriyan, linux-kernel

On 11/24/2011 08:28 PM, Rahul Bedarkar wrote:
> On Thu, Nov 24, 2011 at 9:44 PM, Alexey Dobriyan <adobriyan@gmail.com> wrote:
>> On Thu, Nov 24, 2011 at 1:09 PM, Rahul Bedarkar <rpal143@gmail.com> wrote:
>>> --- a/include/linux/raw.h
>>> +++ b/include/linux/raw.h
>>> @@ -1,3 +1,9 @@
>>> +/*
>>> +* include/linux/raw.h
>>> +*
>>> +* include file for raw block device drivers
>>> +*/
>>
>> Quite a useless comment.
>> Filename inside file especially.
>>
> I have seen File name inside file rbtree.h, it is 19th line in my repo.

It's still useless (and sometimes incorrect in files that have it).

-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-11-25  4:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-24 11:09 [path] adding header info in raw.h Rahul Bedarkar
2011-11-24 16:14 ` Alexey Dobriyan
2011-11-25  4:28   ` Rahul Bedarkar
2011-11-25  4:33     ` Randy Dunlap

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).