[U-Boot] TI:OMAP: [PATCH 1/7] OMAP3 Beagle Update revision detection
Tom
Tom.Rix at windriver.com
Mon Jan 18 14:07:30 CET 2010
Khasim Syed Mohammed wrote:
> Hi Tom,
>
> On Sun, Jan 17, 2010 at 5:44 AM, Tom <Tom.Rix at windriver.com> wrote:
>> Khasim Syed Mohammed wrote:
>>> From 15fbe5ff9ee2fd2f8da4c16805d6c7ccf7244bae Mon Sep 17 00:00:00 2001
>>> From: Syed Mohammed Khasim <khasim at ti.com>
>>> Date: Fri, 8 Jan 2010 20:13:47 +0530
>>> Subject: [PATCH] OMAP3 Beagle Update revision detection
>>>
>> <snip>
>>
>>> MUX_BEAGLE();
>>>
>>> - if (beagle_revision_c) {
>>> + if (beagle_revision != REVISION_AXBX)
>>> MUX_BEAGLE_C();
>>> - }
>> Because MUX_BEAGLE_C is a multiline macro,
>> it would be good to keep the '{ }'
>>
> I had this before, but it resulted in Checkpatch error, do you still
> want me to keep { } ?,
>
Yes.
Checkpatch is not smart enough to figure out if it is function or
a macro.
Because macros are better behaved in a '{ }', block I prefer that.
Tom
More information about the U-Boot
mailing list