[PATCH v3 2/6] ARM: davinci: Changed pr_warning() to pr_warn() (part 2)
Sekhar Nori
nsekhar at ti.com
Tue Nov 20 06:27:58 EST 2012
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 <rtivy at 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
More information about the linux-arm-kernel
mailing list