Hi guys, Jean made a recent review of i2c-hid, and before this driver goes into Linus' tree, here are some cleanups. Patch 2 has not been detected by Jean, but it appeared while playing with buffers allocation. So this is the only change not asked by the previous review. I still need to work on the mutex to protect the potential race on .open and .close, but meanwhile, here is a first batch of patches. Cheers, Benjamin Benjamin Tissoires (14): HID: i2c-hid: change I2C name HID: i2c-hid: fix memory corruption due to missing hid declarations HID: i2c-hid: enhance Kconfig HID: i2c-hid: fix checkpatch.pl warning HID: i2c-hid: fix i2c_hid_dbg macro HID: i2c-hid: remove unused static declarations HID: i2c-hid: fix return paths HID: i2c-hid: fix error messages HID: i2c-hid: i2c_hid_get_report may fail HID: i2c-hid: reorder allocation/free of buffers HID: i2c-hid: remove unneeded test in i2c_hid_remove HID: i2c-hid: remove extra .irq field in struct i2c_hid HID: i2c-hid: also call i2c_hid_free_buffers in i2c_hid_remove HID: i2c-hid: fix i2c_hid_get_raw_report count mismatches drivers/hid/i2c-hid/Kconfig | 7 +- drivers/hid/i2c-hid/i2c-hid.c | 184 ++++++++++++++++++++---------------------- 2 files changed, 89 insertions(+), 102 deletions(-) -- 1.8.0.1 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html