> -----Original Message----- > From: Tony Lindgren [mailto:[email protected]] > Sent: Saturday, December 17, 2011 6:58 AM > To: Hiremath, Vaibhav > Cc: [email protected]; Hilman, Kevin > Subject: Re: [PATCH-V2] arm:omap: cleanup & split omap2/3/4_check_revision > function > > * Vaibhav Hiremath <[email protected]> [111216 03:32]: > > This patch doesn't change functionality or behavior of the code > > execution; it barely cleans up the code and splits into SoC > > specific implementation for Rev ID and feature detection. > > > > Signed-off-by: Vaibhav Hiremath <[email protected]> > > --- > > Changes from V1: > > - Incorporated suggessions from Tony to split the function > > Thanks for working on this, the end result looks much better. > > Can you please split it into two or more patches as this patch > is pretty much unreadable unfortunately? > > Maybe add empty check_revision functions first or something > like that so the diff stays readable? > > Also.. > > > +static void __init omap3_cpuinfo(const char *cpu_rev) > > +{ > > + const char *cpu_name; > > ..the _cpuinfo functions can now be void as the cpu_rev > is already set globally? > Thanks Tony,
I will incorporate all the changes (split the patch) and submit the patch series now. Thanks, Vaibhav > Regards, > > Tony -- 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
