On Tue, May 15, 2012 at 03:54:11, Hilman, Kevin wrote: > Vaibhav Hiremath <[email protected]> writes: > > > From: Kevin Hilman <[email protected]> > > > > There are certain fields inside 'struct dpll_data' which are > > included under ARCH_OMAP3 and ARCH_OMAP4 option, which makes it > > difficult to use it for new devices like, am33xx, ti81xx, etc... > > > > So remove the ifdef completely, this will add few fields to the struct > > unused, but it improves readability and maintainability of the code. > > > > Signed-off-by: Kevin Hilman <[email protected]> > > Signed-off-by: Vaibhav Hiremath <[email protected]> > > Cc: Tony Lindgren <[email protected]> > > Cc: Paul Walmsley <[email protected]> > > Cc: Santosh Shilimkar <[email protected]> > > Cc: R Sricharan <[email protected]> > > --- > > Since Kevin had provided this idea and code change, > > making this patch under his authorship. > > Be sure to give yourself credit for writing the changelog. Normally, if > you make changes/additions to patch, you can add a line just before your > signoff. Something like: > > [[email protected]: wrote detailed changelog] >
Ok... Paul, Can you please add above statement, when you merge it into your repo? Thanks, Vaibhav -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
