On Thu, Feb 28, 2013 at 4:32 PM, Jean Delvare <[email protected]> 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 <[email protected]> >> > >> > 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 <[email protected]> >> > Acked-by: Peter Huewe <[email protected]> >> > Signed-off-by: Jean Delvare <[email protected]> >> > --- >> > 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 [email protected] > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
