return -EIO;
 
        /* it seems that 0xFE indicates that a button is still hold
-          down, while 0xff indicates that no button is hold
-          down. 0xfe sequences are sometimes interrupted by 0xFF */
+          down, while 0xff indicates that no button is hold down. */
 
        if (b == 0xff)
                return 0;
        unsigned char subaddr, keydetect, key;
 
        struct i2c_msg msg[] = { { .addr = i2c_dev->addr, .flags = 0, .buf = &subaddr, .len = 1},
-
-                               { .addr = i2c_dev->addr, .flags = I2C_M_RD, .buf = &keydetect, .len = 1} };
+                                { .addr = i2c_dev->addr, .flags = I2C_M_RD, .buf = &keydetect, .len = 1} };
 
        subaddr = 0x10;
        if (2 != i2c_transfer(i2c_dev->adapter, msg, 2))