linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS
@ 2011-08-29 21:26 Jonathan Neuschäfer
  2011-08-29 22:00 ` Dave Jones
  2011-08-29 22:07 ` Dmitry Torokhov
  0 siblings, 2 replies; 5+ messages in thread
From: Jonathan Neuschäfer @ 2011-08-29 21:26 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: linux-input, Dmitry Torokhov, Stephen Hemminger, linux-kernel,
	Jonathan Neuschäfer

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 drivers/input/misc/apanel.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/apanel.c b/drivers/input/misc/apanel.c
index a8d2b8d..1dd1bd3 100644
--- a/drivers/input/misc/apanel.c
+++ b/drivers/input/misc/apanel.c
@@ -346,5 +346,4 @@ MODULE_DESCRIPTION(APANEL_NAME " driver");
 MODULE_LICENSE("GPL");
 MODULE_VERSION(APANEL_VERSION);
 
-MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifeBook*:pvr*:rvnFUJITSU:*");
 MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifebook*:pvr*:rvnFUJITSU:*");
-- 
1.7.5.4

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS
  2011-08-29 21:26 [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS Jonathan Neuschäfer
@ 2011-08-29 22:00 ` Dave Jones
  2011-08-29 22:07 ` Dmitry Torokhov
  1 sibling, 0 replies; 5+ messages in thread
From: Dave Jones @ 2011-08-29 22:00 UTC (permalink / raw)
  To: Jonathan Neuschäfer
  Cc: Jiri Kosina, linux-input, Dmitry Torokhov, Stephen Hemminger,
	linux-kernel

On Mon, Aug 29, 2011 at 11:26:29PM +0200, Jonathan Neuschäfer wrote:
 
 > -MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifeBook*:pvr*:rvnFUJITSU:*");
 >  MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifebook*:pvr*:rvnFUJITSU:*");
 
                                         ^
Is the module matching code case-insensitive ?

	Dave

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS
  2011-08-29 21:26 [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS Jonathan Neuschäfer
  2011-08-29 22:00 ` Dave Jones
@ 2011-08-29 22:07 ` Dmitry Torokhov
  2011-08-29 23:10   ` Stephen Hemminger
  1 sibling, 1 reply; 5+ messages in thread
From: Dmitry Torokhov @ 2011-08-29 22:07 UTC (permalink / raw)
  To: Jonathan Neuschäfer
  Cc: Jiri Kosina, linux-input, Stephen Hemminger, linux-kernel

On Monday, August 29, 2011 02:26:29 PM Jonathan Neuschäfer wrote:
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>  drivers/input/misc/apanel.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/input/misc/apanel.c b/drivers/input/misc/apanel.c
> index a8d2b8d..1dd1bd3 100644
> --- a/drivers/input/misc/apanel.c
> +++ b/drivers/input/misc/apanel.c
> @@ -346,5 +346,4 @@ MODULE_DESCRIPTION(APANEL_NAME " driver");
>  MODULE_LICENSE("GPL");
>  MODULE_VERSION(APANEL_VERSION);
> 
> -MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifeBook*:pvr*:rvnFUJITSU:*");
>  MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifebook*:pvr*:rvnFUJITSU:*");
                                        ^

Not necessarily redundant, if you notice the register is different.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS
  2011-08-29 22:07 ` Dmitry Torokhov
@ 2011-08-29 23:10   ` Stephen Hemminger
  2011-08-30 17:14     ` Jonathan Neuschäfer
  0 siblings, 1 reply; 5+ messages in thread
From: Stephen Hemminger @ 2011-08-29 23:10 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Jonathan Neuschäfer, Jiri Kosina, linux-input, linux-kernel

On Mon, 29 Aug 2011 15:07:14 -0700
Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:

> On Monday, August 29, 2011 02:26:29 PM Jonathan Neuschäfer wrote:
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> > ---
> >  drivers/input/misc/apanel.c |    1 -
> >  1 files changed, 0 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/input/misc/apanel.c b/drivers/input/misc/apanel.c
> > index a8d2b8d..1dd1bd3 100644
> > --- a/drivers/input/misc/apanel.c
> > +++ b/drivers/input/misc/apanel.c
> > @@ -346,5 +346,4 @@ MODULE_DESCRIPTION(APANEL_NAME " driver");
> >  MODULE_LICENSE("GPL");
> >  MODULE_VERSION(APANEL_VERSION);
> > 
> > -MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifeBook*:pvr*:rvnFUJITSU:*");
> >  MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifebook*:pvr*:rvnFUJITSU:*");
>                                         ^
> 
> Not necessarily redundant, if you notice the register is different.
> 

At one point I found machines with both values, that is why there
are two entries.
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS
  2011-08-29 23:10   ` Stephen Hemminger
@ 2011-08-30 17:14     ` Jonathan Neuschäfer
  0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Neuschäfer @ 2011-08-30 17:14 UTC (permalink / raw)
  To: Stephen Hemminger
  Cc: Dmitry Torokhov, Jonathan Neuschäfer, Jiri Kosina,
	linux-input, linux-kernel

On Mon, Aug 29, 2011 at 04:10:15PM -0700, Stephen Hemminger wrote:
> On Mon, 29 Aug 2011 15:07:14 -0700
> Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> 
> > On Monday, August 29, 2011 02:26:29 PM Jonathan Neuschäfer wrote:
> > > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> > > ---
> > >  drivers/input/misc/apanel.c |    1 -
> > >  1 files changed, 0 insertions(+), 1 deletions(-)
> > > 
> > > diff --git a/drivers/input/misc/apanel.c b/drivers/input/misc/apanel.c
> > > index a8d2b8d..1dd1bd3 100644
> > > --- a/drivers/input/misc/apanel.c
> > > +++ b/drivers/input/misc/apanel.c
> > > @@ -346,5 +346,4 @@ MODULE_DESCRIPTION(APANEL_NAME " driver");
> > >  MODULE_LICENSE("GPL");
> > >  MODULE_VERSION(APANEL_VERSION);
> > > 
> > > -MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifeBook*:pvr*:rvnFUJITSU:*");
> > >  MODULE_ALIAS("dmi:*:svnFUJITSU:pnLifebook*:pvr*:rvnFUJITSU:*");
> >                                         ^
> > 
> > Not necessarily redundant, if you notice the register is different.
> > 
> 
> At one point I found machines with both values, that is why there
> are two entries.

Okay, I didn't see the difference, sorry.

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

end of thread, other threads:[~2011-08-30 17:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-29 21:26 [PATCH trivial] input: apanel: remove a redundant MODULE_ALIAS Jonathan Neuschäfer
2011-08-29 22:00 ` Dave Jones
2011-08-29 22:07 ` Dmitry Torokhov
2011-08-29 23:10   ` Stephen Hemminger
2011-08-30 17:14     ` Jonathan Neuschäfer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).