Hi Farid, On Thu, 22 Apr 2010 23:30:34 +0200, Farid Hammane wrote: > Hi Jean and Wolfram, > > On Thursday 22 April 2010 09:59:37 you wrote: > > On Thu, Apr 22, 2010 at 09:25:24AM +0200, Jean Delvare wrote: > > > On Thu, 22 Apr 2010 01:57:38 +0200, Wolfram Sang wrote: > > > > Hi Farid, > > > > > > > > thanks for this approach. Have you checked that the binary is the same > > > > before/after your patch? If so, please mention in your patch > > > > description. > > Ok, I think that nothing can be broken with this kind of patch. I just built > i2c-algo-pca.c and everything was ok. > > For further patchs I will mention that it has been tested. Thanks for the > advice ! It's not just a matter of building it. When you only fix coding style issue, the binary object before and after your patch should be exactly the same. It's a good idea to verify this, especially for drivers you aren't using yourself. -- Jean Delvare -- 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