On 11/15/2012 3:16 PM, Sergei Shtylyov wrote: > Hello. > > On 15-11-2012 5:53, Tivy, Robert wrote: > >>>> Also, while modifying those pr_warning() calls I changed hardcoded >>>> function names to use '"%s:", __func__' instead > >>>> Signed-off-by: Robert Tivy <rt...@ti.com> >>>> --- >>>> Clean up files that will be otherwise modified in subsequent patch. > >>>> Applies to v3.7-rc2 tag (commit >>>> 6f0c0580b70c89094b3422ba81118c7b959c7556) of Linus' mainline kernel >>> at git.kernel.org. > >>>> arch/arm/mach-davinci/board-omapl138-hawk.c | 30 ++++++++++------ >>> ----------- >>>> 1 file changed, 11 insertions(+), 19 deletions(-) > >>> Taksing of separation of board and SoC specific changes, this patch > > I meant "talking". :-/ > >>> shouldn't have been separated from patch 1 at all -- since it's two >>> boards built around the same chip, OMAP-L138... > >> The 4 patches that are of the same nature ("Changed pr_warning() to >> pr_warn() (part #)") were split as 4 separate patches on request by >> Sekhar, for the purpose of making it easier to merge later. > > To quote Sekhar: "My main motivation was to keep board and soc file > changes from mixing together." You just went too far.
Since this has already gone back and forth a couple of times (probably due to me not being entirely clear on what I want), lets proceed this way: Merge patches 1/6 and 2/6 into a patch titled: ARM: davinci: da850 board: change pr_warning() to pr_warn() Re-headline 3/6 as: ARM: davinci: devices-da8xx.c: change pr_warning() to pr_warn() Re-headline 4/6 as: ARM: davinci: psc.c: change pr_warning() to pr_warn() I hope that's acceptable to everyone involved. Thanks, Sekhar _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source