Sergei Shtylyov <sshtyl...@ru.mvista.com> writes:

> Hello.
>
> Kevin Hilman wrote:
>
>> Signed-off-by: Kevin Hilman <khil...@deeprootsystems.com>
>>   
>
> [...]
>
>> diff --git a/arch/arm/mach-davinci/include/mach/debug-macro.S 
>> b/arch/arm/mach-davinci/include/mach/debug-macro.S
>> index 69837a6..642f000 100644
>> --- a/arch/arm/mach-davinci/include/mach/debug-macro.S
>> +++ b/arch/arm/mach-davinci/include/mach/debug-macro.S
>> @@ -26,7 +26,7 @@
>>              movne   \rx, #0xfe000000        @ virtual base
>>  #if defined(CONFIG_ARCH_DAVINCI_DA8XX) && defined(CONFIG_ARCH_DAVINCI_DMx)
>>  #error Cannot enable DaVinci and DA8XX platforms concurrently
>> -#elif defined(CONFIG_MACH_DAVINCI_DA8XX_EVM)
>> +#elif defined(CONFIG_ARCH_DAVINCI_DA8XX)
>>   
>
>   Er, debug UART depends on the board, not SoC. 

True, but so far both da8xx EVMs use the same UART, so I removed
the board-specific parts making it SoC specific.

If we need to eventually make this board-specific, I'd rather not do it
with #ifdefs.
 
But I get your point, and you're right.  For this series, I should
just do a da8xx-->da830 change and save the other change for a later
patch.  Will update.

BTW, this this boot for you?  I don't have a da830 EVM.

Thanks,

Kevin

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to