[PATCH] ARM: davinci: add missing commas on last members of structure and arrays

Nori, Sekhar nsekhar at ti.com
Thu Dec 15 13:37:33 EST 2011


On Thu, Dec 15, 2011 at 03:30:05, Tony Lindgren wrote:
> * Nori, Sekhar <nsekhar at ti.com> [111214 04:42]:
> > Hi Akshay,
> > 
> > On Thu, Dec 08, 2011 at 14:35:51, Shankarmurthy, Akshay wrote:
> > > This patch adds missing commas on last members of structure and arrays.
> > > This makes less harder to add additional initializer at the end of the
> > > existing initializers and avoids the conflict of more line being
> > > modified. This was pointed out by Russell King in his pet peeves mail at
> > > http://www.spinics.net/lists/arm-kernel/msg147268.html.
> > 
> > Tony has already provided link to his patches fixing this in the
> > message you referenced. Is your patch catching any additional instances?
> > 
> > Tony's patch adds comma after some sentinels, which should be dropped.
> 
> My patches are intended just as a reference, please feel free
> to pick from those and apply as needed along with other
> related clean-up.

Tony,

Thanks for clarifying.

Regards,
Sekhar



More information about the linux-arm-kernel mailing list