Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-14 Thread hermann pitton
Am Dienstag, den 14.08.2007, 18:02 +0400 schrieb Manu Abraham:
> On 8/14/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
> >
> > >
> > > > F: drivers/media/*
> > >
> > >
> > > This is NOT OK !
> >
> > This IS ok. You just need to read the definition of the 'F' tag:
> >
> > F: Files and directories with wildcard patterns.
> >A trailing slash includes all files and subdirectory files.
> > F:  drivers/net/all files in and below drivers/net
> > F:  drivers/net/*   all files in drivers/net, but not below
> > F:  */net/* all files in "any top level directory"/net
> >One pattern per line.  Multiple F: lines acceptable.
> >
> > This tag just includes Kconfig and Makefile.
> 
> 
> Just as much as you state that, the reverse is as well true for
> Kconfig and Makefile.
> 

Please explain what you still want and why.

After two almost wasted years, you should be able to do so.


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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-14 Thread Manu Abraham
On 8/14/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
>
> >
> > > F: drivers/media/*
> >
> >
> > This is NOT OK !
>
> This IS ok. You just need to read the definition of the 'F' tag:
>
> F: Files and directories with wildcard patterns.
>A trailing slash includes all files and subdirectory files.
> F:  drivers/net/all files in and below drivers/net
> F:  drivers/net/*   all files in drivers/net, but not below
> F:  */net/* all files in "any top level directory"/net
>One pattern per line.  Multiple F: lines acceptable.
>
> This tag just includes Kconfig and Makefile.


Just as much as you state that, the reverse is as well true for
Kconfig and Makefile.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-14 Thread Mauro Carvalho Chehab

> 
> > F: drivers/media/*
> 
> 
> This is NOT OK !

This IS ok. You just need to read the definition of the 'F' tag:

F: Files and directories with wildcard patterns.
   A trailing slash includes all files and subdirectory files.
F:  drivers/net/all files in and below drivers/net
F:  drivers/net/*   all files in drivers/net, but not below
F:  */net/* all files in "any top level directory"/net
   One pattern per line.  Multiple F: lines acceptable.

This tag just includes Kconfig and Makefile.

-- 
Cheers,
Mauro

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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-14 Thread Manu Abraham
On 8/14/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
>
> Em Seg, 2007-08-13 às 19:44 -0700, Joe Perches escreveu:
> > On Mon, 2007-08-13 at 22:08 -0400, Michael Krufky wrote:
> > > drivers/media/video  -- v4l
> > > drivers/media/radio  -- v4l
> > >
> > > drivers/media/dvb   -- dvb
> >
> > VIDEO FOR LINUX
> > P:Mauro Carvalho Chehab
> > M:[EMAIL PROTECTED]
> > P:LinuxTV.org Project
> > M:[EMAIL PROTECTED]
> > L:[EMAIL PROTECTED]
> > W:http://linuxtv.org
> > T:git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
> > S:Maintained
> > F:Documentation/video4linux/
> > F:drivers/media/video/
> > F:drivers/media/radio/
> > F:include/linux/video_decoder.h
> > F include/linux/videodev.h
> You missed ':'
> > F:include/linux/videodev2.h
> > F:include/media/
>
> This is not ok. Some maintained files won't be hit. You should add also
> something like:


> F: drivers/media/*


This is NOT OK !
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-14 Thread Mauro Carvalho Chehab

Em Seg, 2007-08-13 às 19:44 -0700, Joe Perches escreveu:
> On Mon, 2007-08-13 at 22:08 -0400, Michael Krufky wrote:
> > drivers/media/video  -- v4l
> > drivers/media/radio  -- v4l
> > 
> > drivers/media/dvb   -- dvb
> 
> VIDEO FOR LINUX
> P:Mauro Carvalho Chehab
> M:[EMAIL PROTECTED]
> P:LinuxTV.org Project
> M:[EMAIL PROTECTED]
> L:[EMAIL PROTECTED]
> W:http://linuxtv.org
> T:git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
> S:Maintained
> F:Documentation/video4linux/
> F:drivers/media/video/
> F:drivers/media/radio/
> F:include/linux/video_decoder.h
> F include/linux/videodev.h
You missed ':'
> F:include/linux/videodev2.h
> F:include/media/

This is not ok. Some maintained files won't be hit. You should add also
something like:
F: drivers/media/*
F: drivers/media/common/ir-functions.c
F: drivers/media/common/ir-keymaps.c
F: drivers/media/common/Kconfig
F: drivers/media/common/Makefile

-- 
Cheers,
Mauro

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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Joe Perches
On Mon, 2007-08-13 at 22:08 -0400, Michael Krufky wrote:
> drivers/media/video  -- v4l
> drivers/media/radio  -- v4l
> 
> drivers/media/dvb   -- dvb

VIDEO FOR LINUX
P:  Mauro Carvalho Chehab
M:  [EMAIL PROTECTED]
P:  LinuxTV.org Project
M:  [EMAIL PROTECTED]
L:  [EMAIL PROTECTED]
W:  http://linuxtv.org
T:  git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
S:  Maintained
F:  Documentation/video4linux/
F:  drivers/media/video/
F:  drivers/media/radio/
F:  include/linux/video_decoder.h
F   include/linux/videodev.h
F:  include/linux/videodev2.h
F:  include/media/

DVB SUBSYSTEM AND DRIVERS
P:  LinuxTV.org Project
M:  [EMAIL PROTECTED]
L:  [EMAIL PROTECTED] (subscribers-only)
W:  http://linuxtv.org/
T:  git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
S:  Maintained
F:  Documentation/dvb/
F:  drivers/media/dvb/
F:  drivers/media/common/saa7146*.c
F:  include/linux/dvb/


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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Manu Abraham
On 8/14/07, Joe Perches <[EMAIL PROTECTED]> wrote:
> On Mon, 2007-08-13 at 23:22 +0400, Manu Abraham wrote:
> > > F:  drivers/media/
> > This means everything under media is maintained under V4L. Incorrect
>
> What would you like?
>

media/video
and
media/radio

*only* belong under V4L

Manu
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Michael Krufky
Joe Perches wrote:
> On Mon, 2007-08-13 at 23:22 +0400, Manu Abraham wrote:
>   
>>> F:  drivers/media/
>>>   
>> This means everything under media is maintained under V4L. Incorrect
>> 
>
> What would you like?
>
>   
Just dont list "drivers/media".

Instead:

drivers/media/video  -- v4l
drivers/media/radio  -- v4l


drivers/media/dvb   -- dvb

Cheers,

Michael Krufky
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Joe Perches
On Mon, 2007-08-13 at 23:22 +0400, Manu Abraham wrote:
> > F:  drivers/media/
> This means everything under media is maintained under V4L. Incorrect

What would you like?

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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Joe Perches
On Mon, 2007-08-13 at 20:32 +0200, Hans Verkuil wrote:
> --- MAINTAINERS.org 2007-08-13 20:23:47.0 +0200
> +++ MAINTAINERS 2007-08-13 20:28:17.0 +0200
> +IVTV VIDEO4LINUX DRIVER
> +P: Hans Verkuil
> +M: [EMAIL PROTECTED]
> +L: [EMAIL PROTECTED]
> +L: [EMAIL PROTECTED]
> +L: [EMAIL PROTECTED]
> +W: http://www.ivtvdriver.org
> +S: Maintained
> +F: Documentation/video4linux/README.ivtv
> +F: drivers/media/video/ivtv/

Added locally.

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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Manu Abraham
On 8/13/07, Joe Perches <[EMAIL PROTECTED]> wrote:
> On Mon, 2007-08-13 at 15:15 -0300, Mauro Carvalho Chehab wrote:
> > There are more include files to be added, for the older V4L APIs:
> > F: include/linux/video_decoder.h
> > F: include/linux/videodev.h
>
> VIDEO FOR LINUX
> P:  Mauro Carvalho Chehab
> M:  [EMAIL PROTECTED]
> M:  [EMAIL PROTECTED]
> L:  [EMAIL PROTECTED]
> W:  http://linuxtv.org
> T:  git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
> S:  Maintained
> F:  Documentation/video4linux/


> F:  drivers/media/

This means everything under media is maintained under V4L. Incorrect
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Mauro Carvalho Chehab

Em Seg, 2007-08-13 às 11:33 -0700, Joe Perches escreveu:
> On Mon, 2007-08-13 at 15:15 -0300, Mauro Carvalho Chehab wrote:
> > There are more include files to be added, for the older V4L APIs:
> > F: include/linux/video_decoder.h
> > F: include/linux/videodev.h
> 
> VIDEO FOR LINUX
> P:Mauro Carvalho Chehab
> M:[EMAIL PROTECTED]
> M:[EMAIL PROTECTED]
> L:[EMAIL PROTECTED]
> W:http://linuxtv.org
> T:git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
> S:Maintained
> F:Documentation/video4linux/
> F:drivers/media/
> F:include/linux/video_decoder.h
> F include/linux/videodev.h
> F:include/linux/videodev2.h
> F:include/media/

Acked-by: Mauro Carvalho Chehab <[EMAIL PROTECTED]>
> 
> 
-- 
Cheers,
Mauro

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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Joe Perches
On Mon, 2007-08-13 at 15:15 -0300, Mauro Carvalho Chehab wrote:
> There are more include files to be added, for the older V4L APIs:
> F: include/linux/video_decoder.h
> F: include/linux/videodev.h

VIDEO FOR LINUX
P:  Mauro Carvalho Chehab
M:  [EMAIL PROTECTED]
M:  [EMAIL PROTECTED]
L:  [EMAIL PROTECTED]
W:  http://linuxtv.org
T:  git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
S:  Maintained
F:  Documentation/video4linux/
F:  drivers/media/
F:  include/linux/video_decoder.h
F   include/linux/videodev.h
F:  include/linux/videodev2.h
F:  include/media/


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


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Hans Verkuil
On Monday 13 August 2007 08:39:28 [EMAIL PROTECTED] wrote:
> Add file pattern to MAINTAINER entry
>
> Signed-off-by: Joe Perches <[EMAIL PROTECTED]>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3e962b1..25bc39a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5046,6 +5046,10 @@ L: [EMAIL PROTECTED]
>  W:   http://linuxtv.org
>  T:   git kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
>  S:   Maintained
> +F:   Documentation/video4linux/
> +F:   drivers/media/
> +F:   include/linux/videodev2.h
> +F:   include/media/
>
>  VT1211 HARDWARE MONITOR DRIVER
>  P:   Juerg Haefliger
>
> ___
> v4l-dvb-maintainer mailing list
> [EMAIL PROTECTED]
> http://www.linuxtv.org/cgi-bin/mailman/listinfo/v4l-dvb-maintainer

While we're at it: I noticed that there was no entry for the ivtv driver 
(part of the kernel since 2.6.22). So here it is:

Signed-off-by: Hans Verkuil <[EMAIL PROTECTED]>

Regards,

Hans

--- MAINTAINERS.org 2007-08-13 20:23:47.0 +0200
+++ MAINTAINERS 2007-08-13 20:28:17.0 +0200
@@ -2036,6 +2036,17 @@
 W: http://www.melware.de
 S: Maintained

+IVTV VIDEO4LINUX DRIVER
+P: Hans Verkuil
+M: [EMAIL PROTECTED]
+L: [EMAIL PROTECTED]
+L: [EMAIL PROTECTED]
+L: [EMAIL PROTECTED]
+W: http://www.ivtvdriver.org
+S: Maintained
+F: Documentation/video4linux/README.ivtv
+F: drivers/media/video/ivtv/
+
 JOURNALLING FLASH FILE SYSTEM V2 (JFFS2)
 P: David Woodhouse
 M: [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [v4l-dvb-maintainer] [PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

2007-08-13 Thread Mauro Carvalho Chehab

Em Dom, 2007-08-12 às 23:39 -0700, [EMAIL PROTECTED] escreveu:
> Add file pattern to MAINTAINER entry

> @@ -5046,6 +5046,10 @@ L: [EMAIL PROTECTED]
> +F:   Documentation/video4linux/
> +F:   drivers/media/
> +F:   include/linux/videodev2.h
> +F:   include/media/

There are more include files to be added, for the older V4L APIs:
F: include/linux/video_decoder.h
F: include/linux/videodev.h

Cheers,
Mauro

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