On 08/01/2014 16:22, Wolfram Sang wrote: >> + new_compat->name = kstrdup("compatible", GFP_KERNEL); >> + new_compat->length = sizeof("marvell,mv78230-a0-i2c"); >> + new_compat->value = kstrdup("marvell,mv78230-a0-i2c", >> + GFP_KERNEL); >> + > > Very minor again: Strange whitespacing after "=" > arg I missed it :( -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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