On Thu, Dec 15, 2011 at 03:30:05, Tony Lindgren wrote: > * Nori, Sekhar <nsek...@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 _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source