> 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 <julia.law...@lip6.fr>  for automating the conversion

Looks good to me,
Reviewed-by: Peter Huewe <peter.hu...@infineon.com>

BUT maybe we should probably wait for the outcome of the 
"introduce macros for i2c_msg initialization" discussion also by Julia:
https://lkml.org/lkml/2012/10/7/92
which would wrap this initialization into a macro:

struct i2c_msg msg1 = I2C_MSG_WRITE(tpm_dev.client->addr, 1, &addr);
struct i2c_msg msg2 I2C_MSG_READ(tpm_dev.client->addr, len, buffer);
...
struct i2c_msg msg1 = I2C_MSG_WRITE(tpm_dev.client->addr, len + 1, tpm_dev.buf);

When the discussion is settled I'd probably go directly to I2C_MSG_ macros in 
order to avoid a double conversion.

Thanks,
Peter

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to