Re: [PATCHv2 0/6] media: convert to c99 format

From: Hans Verkuil
Date: Tue Sep 18 2012 - 06:00:30 EST


On Tue 18 September 2012 11:50:37 Shubhrajyoti D wrote:
> The series tries to convert the i2c_msg to c99 struct.
> This may avoid issues like below if someone tries to add an
> element to the structure.
> http://www.mail-archive.com/linux-i2c@xxxxxxxxxxxxxxx/msg08972.html
>
> Special thanks to Julia Lawall for helping it automate.
> By the below script.
> http://www.mail-archive.com/cocci@xxxxxxx/msg02753.html

That looks much better.

For the whole series:

Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>

Regards,

Hans

>
>
>
> Shubhrajyoti D (6):
> media: Convert struct i2c_msg initialization to C99 format
> media: Convert struct i2c_msg initialization to C99 format
> media: Convert struct i2c_msg initialization to C99 format
> media: Convert struct i2c_msg initialization to C99 format
> media: Convert struct i2c_msg initialization to C99 format
> media: Convert struct i2c_msg initialization to C99 format
>
> drivers/media/i2c/ks0127.c | 14 +++++++-
> drivers/media/i2c/msp3400-driver.c | 42 +++++++++++++++++++++---
> drivers/media/i2c/tvaudio.c | 14 +++++++-
> drivers/media/radio/radio-tea5764.c | 14 ++++++--
> drivers/media/radio/saa7706h.c | 16 ++++++++-
> drivers/media/radio/si470x/radio-si470x-i2c.c | 24 ++++++++++---
> 6 files changed, 103 insertions(+), 21 deletions(-)
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/