public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
@ 2007-08-13  6:25 joe
  2007-08-16  9:16 ` Miguel Ojeda
  0 siblings, 1 reply; 7+ messages in thread
From: joe @ 2007-08-13  6:25 UTC (permalink / raw)
  To: torvalds, maxextreme, linux-kernel, akpm

Add file pattern to MAINTAINER entry

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

diff --git a/MAINTAINERS b/MAINTAINERS
index fdedf32..0f5a92f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1169,6 +1169,8 @@ M:	maxextreme@gmail.com
 L:	linux-kernel@vger.kernel.org
 W:	http://auxdisplay.googlepages.com/
 S:	Maintained
+F:	drivers/auxdisplay/cfag12864bfb.c
+F:	include/linux/cfag12864b.h
 
 CFG80211 and NL80211
 P:	Johannes Berg

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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-13  6:25 [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER joe
@ 2007-08-16  9:16 ` Miguel Ojeda
  2007-08-16 13:55   ` Joe Perches
  0 siblings, 1 reply; 7+ messages in thread
From: Miguel Ojeda @ 2007-08-16  9:16 UTC (permalink / raw)
  To: joe@perches.com; +Cc: torvalds, linux-kernel, akpm

On 8/13/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 fdedf32..0f5a92f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1169,6 +1169,8 @@ M:        maxextreme@gmail.com
>  L:     linux-kernel@vger.kernel.org
>  W:     http://auxdisplay.googlepages.com/
>  S:     Maintained
> +F:     drivers/auxdisplay/cfag12864bfb.c
> +F:     include/linux/cfag12864b.h
>
>  CFG80211 and NL80211
>  P:     Johannes Berg
>

I think you could add also:

 +F:     Documentation/auxdisplay/cfag12864bfb

as you did at the ks0108 driver entry.

Thank you for the patch.

-- 
Miguel Ojeda
http://maxextreme.googlepages.com/index.htm

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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-16  9:16 ` Miguel Ojeda
@ 2007-08-16 13:55   ` Joe Perches
  2007-08-17  4:27     ` Miguel Ojeda
  2007-08-17  4:37     ` Miguel Ojeda
  0 siblings, 2 replies; 7+ messages in thread
From: Joe Perches @ 2007-08-16 13:55 UTC (permalink / raw)
  To: Miguel Ojeda; +Cc: torvalds, linux-kernel, akpm

On Thu, 2007-08-16 at 11:16 +0200, Miguel Ojeda wrote:
> I think you could add also:
>  +F:     Documentation/auxdisplay/cfag12864bfb

This is what I have now:

CFAG12864BFB LCD FRAMEBUFFER DRIVER
P:	Miguel Ojeda Sandonis
M:	maxextreme@gmail.com
L:	linux-kernel@vger.kernel.org
W:	http://auxdisplay.googlepages.com/
S:	Maintained
F:	Documentation/auxdisplay/cfag12864bfb
F:	drivers/auxdisplay/cfag12864bfb.c
F:	include/linux/cfag12864b.h

Joe


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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-16 13:55   ` Joe Perches
@ 2007-08-17  4:27     ` Miguel Ojeda
  2007-08-17  4:37     ` Miguel Ojeda
  1 sibling, 0 replies; 7+ messages in thread
From: Miguel Ojeda @ 2007-08-17  4:27 UTC (permalink / raw)
  To: Joe Perches; +Cc: torvalds, linux-kernel, akpm

On 8/16/07, Joe Perches <joe@perches.com> wrote:
> On Thu, 2007-08-16 at 11:16 +0200, Miguel Ojeda wrote:
> > I think you could add also:
> >  +F:     Documentation/auxdisplay/cfag12864bfb
>
> This is what I have now:
>
> CFAG12864BFB LCD FRAMEBUFFER DRIVER
> P:      Miguel Ojeda Sandonis
> M:      maxextreme@gmail.com
> L:      linux-kernel@vger.kernel.org
> W:      http://auxdisplay.googlepages.com/
> S:      Maintained
> F:      Documentation/auxdisplay/cfag12864bfb
> F:      drivers/auxdisplay/cfag12864bfb.c
> F:      include/linux/cfag12864b.h
>
> Joe
>
>

Fine, thank you for the patch!

Signed-off-by: Miguel Ojeda <maxextreme@gmail.com>

-- 
Miguel Ojeda
http://maxextreme.googlepages.com/index.htm

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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-16 13:55   ` Joe Perches
  2007-08-17  4:27     ` Miguel Ojeda
@ 2007-08-17  4:37     ` Miguel Ojeda
  2007-08-17  4:55       ` Joe Perches
  1 sibling, 1 reply; 7+ messages in thread
From: Miguel Ojeda @ 2007-08-17  4:37 UTC (permalink / raw)
  To: Joe Perches; +Cc: torvalds, linux-kernel, akpm

On 8/16/07, Joe Perches <joe@perches.com> wrote:
> On Thu, 2007-08-16 at 11:16 +0200, Miguel Ojeda wrote:
> > I think you could add also:
> >  +F:     Documentation/auxdisplay/cfag12864bfb
>
> This is what I have now:
>
> CFAG12864BFB LCD FRAMEBUFFER DRIVER
> P:      Miguel Ojeda Sandonis
> M:      maxextreme@gmail.com
> L:      linux-kernel@vger.kernel.org
> W:      http://auxdisplay.googlepages.com/
> S:      Maintained
> F:      Documentation/auxdisplay/cfag12864bfb
> F:      drivers/auxdisplay/cfag12864bfb.c
> F:      include/linux/cfag12864b.h
>
> Joe
>
>

Oh, I got confused by the different emails:

   Documentation/auxdisplay/cfag12864bfb

does not exist. I meant

   Documentation/auxdisplay/cfag12864b

Also, in this patch:

    [PATCH] [119/2many] MAINTAINERS - CFAG12864B LCD DRIVER

You could add the same line:

   Documentation/auxdisplay/cfag12864b

As it is related to the same stuff: cfag12864b is the driver that
makes the LCD work along with ks0108; and cfag12864bfb is a
framebuffer to work with the LCD from userspace. I named it
"cfag12864b" because that is the name of the LCD, so people will not
get confused with the "fb".

As Documentation/auxdisplay/cfag12864b describes both at the same
time, it is OK to add them in both entries.

Sorry for the mistake.

-- 
Miguel Ojeda
http://maxextreme.googlepages.com/index.htm

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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-17  4:37     ` Miguel Ojeda
@ 2007-08-17  4:55       ` Joe Perches
  2007-08-17  5:21         ` Miguel Ojeda
  0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2007-08-17  4:55 UTC (permalink / raw)
  To: Miguel Ojeda; +Cc: torvalds, linux-kernel, akpm

On Fri, 2007-08-17 at 06:37 +0200, Miguel Ojeda wrote:
> On 8/16/07, Joe Perches <joe@perches.com> wrote:
> > On Thu, 2007-08-16 at 11:16 +0200, Miguel Ojeda wrote:
> > > I think you could add also:
> > >  +F:     Documentation/auxdisplay/cfag12864bfb
> does not exist. I meant
>    Documentation/auxdisplay/cfag12864b
> As Documentation/auxdisplay/cfag12864b describes both at the same
> time, it is OK to add them in both entries.
> Sorry for the mistake.

No worries.  We all make a bunch of 'em.

This is what I have now:

CFAG12864B LCD DRIVER
P:	Miguel Ojeda Sandonis
M:	maxextreme@gmail.com
L:	linux-kernel@vger.kernel.org
W:	http://auxdisplay.googlepages.com/
S:	Maintained
F:	Documentation/auxdisplay/cfag12864b
F:	drivers/auxdisplay/cfag12864b.c
F:	include/linux/cfag12864b.h

CFAG12864BFB LCD FRAMEBUFFER DRIVER
P:	Miguel Ojeda Sandonis
M:	maxextreme@gmail.com
L:	linux-kernel@vger.kernel.org
W:	http://auxdisplay.googlepages.com/
S:	Maintained
F:	Documentation/auxdisplay/cfag12864b
F:	drivers/auxdisplay/cfag12864bfb.c
F:	include/linux/cfag12864b.h

cheers, Joe


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

* Re: [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER
  2007-08-17  4:55       ` Joe Perches
@ 2007-08-17  5:21         ` Miguel Ojeda
  0 siblings, 0 replies; 7+ messages in thread
From: Miguel Ojeda @ 2007-08-17  5:21 UTC (permalink / raw)
  To: Joe Perches; +Cc: torvalds, linux-kernel, akpm

On 8/17/07, Joe Perches <joe@perches.com> wrote:
> On Fri, 2007-08-17 at 06:37 +0200, Miguel Ojeda wrote:
> > On 8/16/07, Joe Perches <joe@perches.com> wrote:
> > > On Thu, 2007-08-16 at 11:16 +0200, Miguel Ojeda wrote:
> > > > I think you could add also:
> > > >  +F:     Documentation/auxdisplay/cfag12864bfb
> > does not exist. I meant
> >    Documentation/auxdisplay/cfag12864b
> > As Documentation/auxdisplay/cfag12864b describes both at the same
> > time, it is OK to add them in both entries.
> > Sorry for the mistake.
>
> No worries.  We all make a bunch of 'em.
>
> This is what I have now:
>
> CFAG12864B LCD DRIVER
> P:      Miguel Ojeda Sandonis
> M:      maxextreme@gmail.com
> L:      linux-kernel@vger.kernel.org
> W:      http://auxdisplay.googlepages.com/
> S:      Maintained
> F:      Documentation/auxdisplay/cfag12864b
> F:      drivers/auxdisplay/cfag12864b.c
> F:      include/linux/cfag12864b.h
>
> CFAG12864BFB LCD FRAMEBUFFER DRIVER
> P:      Miguel Ojeda Sandonis
> M:      maxextreme@gmail.com
> L:      linux-kernel@vger.kernel.org
> W:      http://auxdisplay.googlepages.com/
> S:      Maintained
> F:      Documentation/auxdisplay/cfag12864b
> F:      drivers/auxdisplay/cfag12864bfb.c
> F:      include/linux/cfag12864b.h
>
> cheers, Joe
>
>

Fine!

Btw, you did me to look at the Documentation to check the correct name
of the "cfag12864b" file, and now, looking at the code, I have
discovered a one-line bug... So the mistake will solve two things at
the same thing! I'm creating the patch now.

Signed-off-by: Miguel Ojeda <maxextreme@gmail.com>

-- 
Miguel Ojeda
http://maxextreme.googlepages.com/index.htm

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

end of thread, other threads:[~2007-08-17  5:21 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-13  6:25 [PATCH] [120/2many] MAINTAINERS - CFAG12864BFB LCD FRAMEBUFFER DRIVER joe
2007-08-16  9:16 ` Miguel Ojeda
2007-08-16 13:55   ` Joe Perches
2007-08-17  4:27     ` Miguel Ojeda
2007-08-17  4:37     ` Miguel Ojeda
2007-08-17  4:55       ` Joe Perches
2007-08-17  5:21         ` Miguel Ojeda

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