public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
@ 2007-08-13  6:36 joe
  2007-08-14  4:04 ` Nathan Laredo
  0 siblings, 1 reply; 6+ messages in thread
From: joe @ 2007-08-13  6:36 UTC (permalink / raw)
  To: torvalds, linux-kernel, laredo, akpm

Add file pattern to MAINTAINER entry

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

diff --git a/MAINTAINERS b/MAINTAINERS
index 9da5a44..3e51fc6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4406,6 +4406,7 @@ P:	Nathan Laredo
 M:	laredo@gnu.org
 W:	http://www.stradis.com/
 S:	Maintained
+F:	drivers/media/video/stradis.c
 
 SUPERH (sh)
 P:	Paul Mundt

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

* Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
  2007-08-13  6:36 [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER joe
@ 2007-08-14  4:04 ` Nathan Laredo
  2007-08-14  4:32   ` Joe Perches
  0 siblings, 1 reply; 6+ messages in thread
From: Nathan Laredo @ 2007-08-14  4:04 UTC (permalink / raw)
  To: joe@perches.com; +Cc: torvalds, linux-kernel, akpm

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
laredo@gnu.org

On 8/12/07, joe@perches.com <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 9da5a44..3e51fc6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4406,6 +4406,7 @@ P:        Nathan Laredo
>  M:     laredo@gnu.org
>  W:     http://www.stradis.com/
>  S:     Maintained
> +F:     drivers/media/video/stradis.c
>
>  SUPERH (sh)
>  P:     Paul Mundt
>

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

* Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
  2007-08-14  4:04 ` Nathan Laredo
@ 2007-08-14  4:32   ` Joe Perches
  2007-08-14 21:51     ` Nathan Laredo
  0 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2007-08-14  4:32 UTC (permalink / raw)
  To: Nathan Laredo; +Cc: torvalds, linux-kernel, akpm

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




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

* Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
  2007-08-14  4:32   ` Joe Perches
@ 2007-08-14 21:51     ` Nathan Laredo
  2007-08-14 22:45       ` Joe Perches
  0 siblings, 1 reply; 6+ messages in thread
From: Nathan Laredo @ 2007-08-14 21:51 UTC (permalink / raw)
  To: Joe Perches; +Cc: torvalds, linux-kernel, akpm

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
laredo@gnu.org

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

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

* Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
  2007-08-14 21:51     ` Nathan Laredo
@ 2007-08-14 22:45       ` Joe Perches
  2007-08-14 23:27         ` Nathan Laredo
  0 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2007-08-14 22:45 UTC (permalink / raw)
  To: Nathan Laredo; +Cc: torvalds, linux-kernel, akpm

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:	laredo@gnu.org
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



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

* Re: [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER
  2007-08-14 22:45       ` Joe Perches
@ 2007-08-14 23:27         ` Nathan Laredo
  0 siblings, 0 replies; 6+ messages in thread
From: Nathan Laredo @ 2007-08-14 23:27 UTC (permalink / raw)
  To: Joe Perches; +Cc: torvalds, linux-kernel, akpm

Looks good to me.

Signed-off-by: Nathan Laredo <laredo@gnu.org>

- Nathan


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

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

end of thread, other threads:[~2007-08-14 23:28 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-13  6:36 [PATCH] [463/2many] MAINTAINERS - STRADIS MPEG-2 DECODER DRIVER joe
2007-08-14  4:04 ` Nathan Laredo
2007-08-14  4:32   ` Joe Perches
2007-08-14 21:51     ` Nathan Laredo
2007-08-14 22:45       ` Joe Perches
2007-08-14 23:27         ` Nathan Laredo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox