Re: [PATCH] drivers/input/keyboard: convert to dynamic input_dev allocation

[Dmitry Torokhov]

On Friday 28 October 2005 02:05, Dmitry Torokhov wrote:
> On Friday 28 October 2005 01:55, Jan-Benedict Glaw wrote:
> > On Thu, 2005-10-27 23:30:24 -0700, Greg KH <gregkh@xxxxxxx> wrote:
> > > diff --git a/drivers/input/keyboard/lkkbd.c b/drivers/input/keyboard/lkkbd.c
> > > index 098963c..7f06780 100644
> > > @@ -435,14 +434,14 @@ lkkbd_interrupt (struct serio *serio, un
> > >  
> > >  	switch (data) {
> > >  		case LK_ALL_KEYS_UP:
> > > -			input_regs (&lk->dev, regs);
> > > +			input_regs (lk->dev, regs);
> > >  			for (i = 0; i < ARRAY_SIZE (lkkbd_keycode); i++)
> > >  				if (lk->keycode[i] != KEY_RESERVED)
> > > -					input_report_key (&lk->dev, lk->keycode[i], 0);
> > > -			input_sync (&lk->dev);
> > > +					input_report_key (lk->dev, lk->keycode[i], 0);
> > > +			input_sync (lk->dev);
> > >  			break;
> > >  		case LK_METRONOME:
> > > -			DBG (KERN_INFO "Got LK_METRONOME and don't "
> > > +			DBG (KERN_INFO "Got %#d and don't "
> > >  					"know how to handle...\n");
> > >  			break;
> > >  		case LK_OUTPUT_ERROR:
> > 
> > The format change (%#d) should take an argument on stack, shouldn't
> > it? But there's nothing pushed? ...or is it just a typo?
> >
> 
> I think I messed it up... Was probably trying to do "default" case and then
> reverted back. Will fix.
> 

That's what I wanted to do...

-- 
Dmitry

Input: lkkbd - consolidate messages in lkkbd_interrup()

Signed-off-by: Dmitry Torokhov <dtor@xxxxxxx>
---

 drivers/input/keyboard/lkkbd.c |   28 +++++-----------------------
 1 files changed, 5 insertions(+), 23 deletions(-)

Index: work/drivers/input/keyboard/lkkbd.c
===================================================================
--- work.orig/drivers/input/keyboard/lkkbd.c
+++ work/drivers/input/keyboard/lkkbd.c
@@ -440,38 +440,20 @@ lkkbd_interrupt (struct serio *serio, un
 					input_report_key (lk->dev, lk->keycode[i], 0);
 			input_sync (lk->dev);
 			break;
+
 		case LK_METRONOME:
-			DBG (KERN_INFO "Got %#d and don't "
-					"know how to handle...\n");
-			break;
 		case LK_OUTPUT_ERROR:
-			DBG (KERN_INFO "Got LK_OUTPUT_ERROR and don't "
-					"know how to handle...\n");
-			break;
 		case LK_INPUT_ERROR:
-			DBG (KERN_INFO "Got LK_INPUT_ERROR and don't "
-					"know how to handle...\n");
-			break;
 		case LK_KBD_LOCKED:
-			DBG (KERN_INFO "Got LK_KBD_LOCKED and don't "
-					"know how to handle...\n");
-			break;
 		case LK_KBD_TEST_MODE_ACK:
-			DBG (KERN_INFO "Got LK_KBD_TEST_MODE_ACK and don't "
-					"know how to handle...\n");
-			break;
 		case LK_PREFIX_KEY_DOWN:
-			DBG (KERN_INFO "Got LK_PREFIX_KEY_DOWN and don't "
-					"know how to handle...\n");
-			break;
 		case LK_MODE_CHANGE_ACK:
-			DBG (KERN_INFO "Got LK_MODE_CHANGE_ACK and ignored "
-					"it properly...\n");
-			break;
 		case LK_RESPONSE_RESERVED:
-			DBG (KERN_INFO "Got LK_RESPONSE_RESERVED and don't "
-					"know how to handle...\n");
+			DBG (KERN_INFO
+			     "Got 0x%02x and don't know how to handle...\n",
+			     data);
 			break;
+
 		case 0x01:
 			DBG (KERN_INFO "Got 0x01, scheduling re-initialization\n");
 			lk->ignore_bytes = LK_NUM_IGNORE_BYTES;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/