2010-07-18 14:39:27

by Wan ZongShun

Subject: [PATCH] keypad/nuc900: change keypad driver name from 'nuc900-keypad' to 'nuc900-kpi'

Hi Dmitry,

I ever changed the keypad platform device name from 'nuc900-keypad' to
'nuc900-kpi', but I forget to modify the related keypad driver name,so
the keypad driver does not work,I have to send the patch to fix this issue.

Signed-off-by: Wan ZongShun <[email protected]>
---
drivers/input/keyboard/w90p910_keypad.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/keyboard/w90p910_keypad.c b/drivers/input/keyboard/w90p910_keypad.c
index 4ef764c..ee2bf6b 100644
--- a/drivers/input/keyboard/w90p910_keypad.c
+++ b/drivers/input/keyboard/w90p910_keypad.c
@@ -258,7 +258,7 @@ static struct platform_driver w90p910_keypad_driver = {
.probe = w90p910_keypad_probe,
.remove = __devexit_p(w90p910_keypad_remove),
.driver = {
- .name = "nuc900-keypad",
+ .name = "nuc900-kpi",
.owner = THIS_MODULE,
},
};
--
1.6.3.3

2010-07-22 05:34:21

by Wan ZongShun

Subject: Re: [PATCH] keypad/nuc900: change keypad driver name from 'nuc900-keypad' to 'nuc900-kpi'

Dmitry,

Can it be applied?

Thanks!

> Hi Dmitry,
>
> I ever changed the keypad platform device name from 'nuc900-keypad' to
> 'nuc900-kpi', but I forget to modify the related keypad driver name,so
> the keypad driver does not work,I have to send the patch to fix this issue.
>
> Signed-off-by: Wan ZongShun <[email protected]>
> ---
>  drivers/input/keyboard/w90p910_keypad.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/input/keyboard/w90p910_keypad.c b/drivers/input/keyboard/w90p910_keypad.c
> index 4ef764c..ee2bf6b 100644
> --- a/drivers/input/keyboard/w90p910_keypad.c
> +++ b/drivers/input/keyboard/w90p910_keypad.c
> @@ -258,7 +258,7 @@ static struct platform_driver w90p910_keypad_driver = {
>        .probe          = w90p910_keypad_probe,
>        .remove         = __devexit_p(w90p910_keypad_remove),
>        .driver         = {
> -               .name   = "nuc900-keypad",
> +               .name   = "nuc900-kpi",
>                .owner  = THIS_MODULE,
>        },
>  };
> --
> 1.6.3.3
>



--
*linux-arm-kernel mailing list
mail addr:[email protected]
you can subscribe by:
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

* linux-arm-NUC900 mailing list
mail addr:[email protected]
main web: https://groups.google.com/group/NUC900
you can subscribe it by sending me mail:
[email protected]

2010-07-22 15:43:36

by Dmitry Torokhov

Subject: Re: [PATCH] keypad/nuc900: change keypad driver name from 'nuc900-keypad' to 'nuc900-kpi'

Hi Wan,

On Thu, Jul 22, 2010 at 01:34:17PM +0800, Wan ZongShun wrote:
> Dmitry,
>
> Can it be applied?
>

It is applied to my 'for-linus' branch.

Thanks.

> Thanks!
>
> > Hi Dmitry,
> >
> > I ever changed the keypad platform device name from 'nuc900-keypad' to
> > 'nuc900-kpi', but I forget to modify the related keypad driver name,so
> > the keypad driver does not work,I have to send the patch to fix this issue.
> >
> > Signed-off-by: Wan ZongShun <[email protected]>
> > ---
> > ?drivers/input/keyboard/w90p910_keypad.c | ? ?2 +-
> > ?1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/input/keyboard/w90p910_keypad.c b/drivers/input/keyboard/w90p910_keypad.c
> > index 4ef764c..ee2bf6b 100644
> > --- a/drivers/input/keyboard/w90p910_keypad.c
> > +++ b/drivers/input/keyboard/w90p910_keypad.c
> > @@ -258,7 +258,7 @@ static struct platform_driver w90p910_keypad_driver = {
> > ? ? ? ?.probe ? ? ? ? ?= w90p910_keypad_probe,
> > ? ? ? ?.remove ? ? ? ? = __devexit_p(w90p910_keypad_remove),
> > ? ? ? ?.driver ? ? ? ? = {
> > - ? ? ? ? ? ? ? .name ? = "nuc900-keypad",
> > + ? ? ? ? ? ? ? .name ? = "nuc900-kpi",
> > ? ? ? ? ? ? ? ?.owner ?= THIS_MODULE,
> > ? ? ? ?},
> > ?};
> > --
> > 1.6.3.3
> >
>
>
--
Dmitry