[PATCH 2/3] ARM: CSR: mapping early DEBUG_LL uart

Barry Song 21cnbao at gmail.com
Wed Jul 6 08:29:46 EDT 2011


Hi Russell,
Thanks.

2011/7/6 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> On Wed, Jul 06, 2011 at 07:53:14PM +0800, Barry Song wrote:
>> hi Russell,
>>
>> 2011/7/6 Russell King - ARM Linux <linux at arm.linux.org.uk>:
>> > On Wed, Jul 06, 2011 at 02:47:57AM -0700, Barry Song wrote:
>> >> Signed-off-by: Barry Song <bs14 at csr.com>
>> >
>> > Surely this should be part of the first patch, which won't build without
>> > this one if DEBUG_LL is enabled?
>>
>> i guess that can get fixed by:
>>
>> static void __init sirfsoc_map_io(void)
>> {
>>         sirfsoc_map_lluart();
>> }
>>
>> MACHINE_START(PRIMA2_EVB, "prima2cb")
>>       ...
>>        .map_io         = sirfsoc_map_io,
>>       ...
>> MACHINE_END
>
> That doesn't really change anything.  If you build the first patch without
> the second patch with DEBUG_LL enabled, you'll get a build failure because
> sirfsoc_map_lluart is not defined.
>
> Either omit the .map_io initializer in the first patch and add it in the
> second patch, or combine the two patches together.

real. thanks! to keep this as a separate patch as required in earlier
thread, i will omit the .map_io initializer in the first patch.

>
barry



More information about the linux-arm-kernel mailing list