[PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

[PATCH] [534/2many] MAINTAINERS - VIDEO FOR LINUX

[joe]

Add file pattern to MAINTAINER entry

Signed-off-by: Joe Perches <joe@perches.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index 3e962b1..25bc39a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5046,6 +5046,10 @@ L:	video4linux-list@redhat.com
 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

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

[Mauro Carvalho Chehab]

Em Dom, 2007-08-12 às 23:39 -0700, joe@perches.com escreveu:
> Add file pattern to MAINTAINER entry

> @@ -5046,6 +5046,10 @@ L:	video4linux-list@redhat.com
> +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

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

[Hans Verkuil]

On Monday 13 August 2007 08:39:28 joe@perches.com wrote:
> Add file pattern to MAINTAINER entry
>
> Signed-off-by: Joe Perches <joe@perches.com>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3e962b1..25bc39a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5046,6 +5046,10 @@ L:	video4linux-list@redhat.com
>  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
> v4l-dvb-maintainer@linuxtv.org
> 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 <hverkuil@xs4all.nl>

Regards,

	Hans

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

+IVTV VIDEO4LINUX DRIVER
+P:     Hans Verkuil
+M:     hverkuil@xs4all.nl
+L:     ivtv-devel@ivtvdriver.org
+L:     ivtv-users@ivtvdriver.org
+L:     video4linux-list@redhat.com
+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:     dwmw2@infradead.org

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

[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:	mchehab@infradead.org
M:	v4l-dvb-maintainer@linuxtv.org
L:	video4linux-list@redhat.com
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/

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

[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:	mchehab@infradead.org
> M:	v4l-dvb-maintainer@linuxtv.org
> L:	video4linux-list@redhat.com
> 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 <mchehab@infradead.org>
> 
> 
-- 
Cheers,
Mauro

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

[Manu Abraham]

On 8/13/07, Joe Perches <joe@perches.com> 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:      mchehab@infradead.org
> M:      v4l-dvb-maintainer@linuxtv.org
> L:      video4linux-list@redhat.com
> 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

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

[Joe Perches]

On Mon, 2007-08-13 at 20:32 +0200, Hans Verkuil wrote:
> --- MAINTAINERS.org     2007-08-13 20:23:47.000000000 +0200
> +++ MAINTAINERS 2007-08-13 20:28:17.000000000 +0200
> +IVTV VIDEO4LINUX DRIVER
> +P:     Hans Verkuil
> +M:     hverkuil@xs4all.nl
> +L:     ivtv-devel@ivtvdriver.org
> +L:     ivtv-users@ivtvdriver.org
> +L:     video4linux-list@redhat.com
> +W:     http://www.ivtvdriver.org
> +S:     Maintained
> +F:     Documentation/video4linux/README.ivtv
> +F:     drivers/media/video/ivtv/

Added locally.

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

[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?

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

[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

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

[Manu Abraham]

On 8/14/07, Joe Perches <joe@perches.com> 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

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

[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:	mchehab@infradead.org
P:	LinuxTV.org Project
M:	v4l-dvb-maintainer@linuxtv.org
L:	video4linux-list@redhat.com
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:	v4l-dvb-maintainer@linuxtv.org
L:	linux-dvb@linuxtv.org (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/

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

[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:	mchehab@infradead.org
> P:	LinuxTV.org Project
> M:	v4l-dvb-maintainer@linuxtv.org
> L:	video4linux-list@redhat.com
> 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

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

[Manu Abraham]

On 8/14/07, Mauro Carvalho Chehab <mchehab@infradead.org> 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:    mchehab@infradead.org
> > P:    LinuxTV.org Project
> > M:    v4l-dvb-maintainer@linuxtv.org
> > L:    video4linux-list@redhat.com
> > 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 !

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

[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

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

[Manu Abraham]

On 8/14/07, Mauro Carvalho Chehab <mchehab@infradead.org> 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.

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

[hermann pitton]

Am Dienstag, den 14.08.2007, 18:02 +0400 schrieb Manu Abraham:
> On 8/14/07, Mauro Carvalho Chehab <mchehab@infradead.org> 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.