On Thu, Feb 28, 2013 at 4:32 PM, Jean Delvare <kh...@linux-fr.org> wrote: > On Thu, 28 Feb 2013 23:12:51 +0100, Peter Hüwe wrote: >> thanks for resending. > > You're welcome. > >> Am Donnerstag, 28. Februar 2013, 11:06:11 schrieb Jean Delvare: >> > From: Shubhrajyoti Datta <omaplinuxker...@gmail.com> >> > >> > Convert the struct i2c_msg initialization to C99 format. This makes >> > maintaining and editing the code simpler. Also helps once other fields >> > like transferred are added in future. >> > >> > Thanks to Julia Lawall for automating the conversion. >> > >> > Signed-off-by: Shubhrajyoti D <shubhrajy...@ti.com> >> > Acked-by: Peter Huewe <peter.hu...@infineon.com> >> > Signed-off-by: Jean Delvare <kh...@linux-fr.org> >> > --- >> > v2: removed zero initialization of flags. >> > >> > Patch already sent by Shubhrajyoti Datta on 2012-10-10. >> > >> > drivers/char/tpm/tpm_i2c_infineon.c | 19 ++++++++++++++++--- >> > 1 file changed, 16 insertions(+), 3 deletions(-) >> >> As it was not yet applied I folded it into my patch >> "Add support for new Infineon I2C TPM (SLB 9645 TT 1.2 I2C)" > > I'd suggest not to. Mixing actual changes with cleanups is generally > discouraged.
Yes, please don't do this. Also because it makes it really confusing for me to figure out whether something has been applied or not and if I have the right people on the sign off. Peter, can you resend without it? I'll have a new staging tree with everything up in the next day or two. Thanks, Kent > -- > Jean Delvare > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/