Re: [PATCH][RFC]: Clean up resource allocation in i8042 driver

[Dave Jones <davej@redhat.com>]

On Fri, Jan 21, 2005 at 10:14:46AM -0500, Prarit Bhargava wrote:

 > Signed-off-by: Prarit Bhargava <prarit@sgi.com>
 > 
 > ===== i8042.c 1.71 vs edited =====
 > --- 1.71/drivers/input/serio/i8042.c    2005-01-03 08:11:49 -05:00
 > +++ edited/i8042.c      2005-01-21 10:02:20 -05:00
 > @@ -696,7 +696,10 @@
 >                unsigned char param;
 > 
 >                if (i8042_command(&param, I8042_CMD_CTL_TEST)) {
 > -                       printk(KERN_ERR "i8042.c: i8042 controller self 
 > test timeout.\n");

wordwrapped patch.
		
 > +                       if (i8042_read_status() != 0xFF)
 > +                               printk(KERN_ERR "i8042.c: i8042 controller 
 > self test timeout.\n");
 > +                       else
 > +                               printk(KERN_ERR "i8042.c: no i8042 
 > controller found.\n");
 >                        return -1;
 >                }
 > 
 >                }

I doubt these }'s should line up, broken indentation ?
 
 > @@ -1011,21 +1014,34 @@
 >        i8042_timer.function = i8042_timer_func;
 > 
 >        if (i8042_platform_init())
 > +       {
 > +               del_timer_sync(&i8042_timer);
 >                return -EBUSY;
 > +       }

Spaces instead of tabs. Oops.
(Also the { should be on the same line as the if)

Nitpicking..

		Dave