2007-08-13 11:14:36

by Joe Perches

[permalink] [raw]
Subject: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

Add file pattern to MAINTAINER entry

Signed-off-by: Joe Perches <[email protected]>

diff --git a/MAINTAINERS b/MAINTAINERS
index 9da5a44..3e51fc6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4406,6 +4406,7 @@ P: Nathan Laredo
M: [email protected]
W: http://www.stradis.com/
S: Maintained
+F: drivers/media/video/stradis.c

SUPERH (sh)
P: Paul Mundt


2007-08-14 04:04:40

by Nathan Laredo

[permalink] [raw]
Subject: Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

Well, technically speaking, there are multiple files to the stradis
driver, not just stradis.c.

The tar file, http://kernel.org/pub/linux/kernel/people/laredo/stradis.tar.gz
should make clear which files originated in the stradis driver.

- Nathan Laredo
[email protected]

On 8/12/07, [email protected] <[email protected]> wrote:
> Add file pattern to MAINTAINER entry
>
> Signed-off-by: Joe Perches <[email protected]>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 9da5a44..3e51fc6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4406,6 +4406,7 @@ P: Nathan Laredo
> M: [email protected]
> W: http://www.stradis.com/
> S: Maintained
> +F: drivers/media/video/stradis.c
>
> SUPERH (sh)
> P: Paul Mundt
>

2007-08-14 04:33:27

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

On Mon, 2007-08-13 at 21:04 -0700, Nathan Laredo wrote:
> Well, technically speaking, there are multiple files to the stradis
> driver, not just stradis.c.

These files seem to be shared between drivers.

the Kconfig file shows:

obj-$(CONFIG_VIDEO_STRADIS) += stradis.o

as the only STRADIS file.

Are you the maintainer for other entries?
Should I add these entries?

drivers/media/video/cs8420.h
drivers/media/video/ibmmpeg2.h
drivers/media/video/saa5249.c
drivers/media/video/saa7110.c
drivers/media/video/saa7111.c
drivers/media/video/saa7121.h
drivers/media/video/saa7146*.h
drivers/media/video/saa7185.h
drivers/media/video/saa7196.h
drivers/media/video/videodev.c



2007-08-14 22:13:42

by Nathan Laredo

[permalink] [raw]
Subject: Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

No, not all of them. Just the ones that show my name at the top of
the source file.

cs8240.h, ibmmpeg2.h, saa7121.h, saa7146*.h, stradis.c

stradis.c includes all of the above .h files, so they weren't listed separately.

Additionally, it should probably be noted that I still use a 2.2
kernel with this device and haven't tested the 3rd party ports to 2.6
yet but I've never actually gotten a bug report yet from someone
claiming it isn't working (but I'm pretty confident it will crash).

Thanks,
- Nathan Laredo
[email protected]

On 8/13/07, Joe Perches <[email protected]> wrote:
> On Mon, 2007-08-13 at 21:04 -0700, Nathan Laredo wrote:
> > Well, technically speaking, there are multiple files to the stradis
> > driver, not just stradis.c.
>
> These files seem to be shared between drivers.
>
> the Kconfig file shows:
>
> obj-$(CONFIG_VIDEO_STRADIS) += stradis.o
>
> as the only STRADIS file.
>
> Are you the maintainer for other entries?
> Should I add these entries?
>
> drivers/media/video/cs8420.h
> drivers/media/video/ibmmpeg2.h
> drivers/media/video/saa5249.c
> drivers/media/video/saa7110.c
> drivers/media/video/saa7111.c
> drivers/media/video/saa7121.h
> drivers/media/video/saa7146*.h
> drivers/media/video/saa7185.h
> drivers/media/video/saa7196.h
> drivers/media/video/videodev.c
>
>
>
>

2007-08-14 22:46:56

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

On Tue, 2007-08-14 at 14:51 -0700, Nathan Laredo wrote:
> Just the ones that show my name at the top of the source file.
> cs8240.h, ibmmpeg2.h, saa7121.h, saa7146*.h, stradis.c

STRADIS MPEG-2 DECODER DRIVER
P: Nathan Laredo
M: [email protected]
W: http://www.stradis.com/
S: Maintained
F: drivers/media/video/cs8240.h
F: drivers/media/video/ibmmpeg2.h
F: drivers/media/video/saa7121.h
F: drivers/media/video/saa7146*.h
F: drivers/media/video/stradis.c


2007-08-14 23:28:35

by Nathan Laredo

[permalink] [raw]
Subject: Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER

Looks good to me.

Signed-off-by: Nathan Laredo <[email protected]>

- Nathan


On 8/14/07, Joe Perches <[email protected]> wrote:
> On Tue, 2007-08-14 at 14:51 -0700, Nathan Laredo wrote:
> > Just the ones that show my name at the top of the source file.
> > cs8240.h, ibmmpeg2.h, saa7121.h, saa7146*.h, stradis.c
>
> STRADIS MPEG-2 DECODER DRIVER
> P: Nathan Laredo
> M: [email protected]
> W: http://www.stradis.com/
> S: Maintained
> F: drivers/media/video/cs8240.h
> F: drivers/media/video/ibmmpeg2.h
> F: drivers/media/video/saa7121.h
> F: drivers/media/video/saa7146*.h
> F: drivers/media/video/stradis.c
>
>
>