[U-Boot] [PATCH 1/4][v2] doc:Add documentation for e500 external debugger support

Prabhakar Kushwaha prabhakar at freescale.com
Wed Mar 21 17:54:16 CET 2012


Hi Scott,
  Thanks for reviewing it. Please find my reply in-lined.

On Wednesday 21 March 2012 10:01 PM, Scott Wood wrote:
> On 03/20/2012 11:42 PM, Prabhakar Kushwaha wrote:
>> diff --git a/README b/README
>> index 8964672..48e50ed 100644
>> --- a/README
>> +++ b/README
>> @@ -3347,6 +3347,20 @@ Low Level (hardware related) configuration options:
>>   		be used if available. These functions may be faster under some
>>   		conditions but may increase the binary size.
>>
>> +E500 (MPC85xx) External Debug Support
>> +-------------------------------------
>> +-CONFIG_E500
>> +		Enables BOOKE e500 family.It includes e500, e500v2 e500mc,
>> +		e5500 processor etc.
> This is not a debug-specific option.  What's wrong with putting both of
> these options under the existing "85xx CPU Options" section?

Yes. Both CONFIG_E500 and  CONFIG_SYS_PPC_E500_DEBUG_TLB can be put 
under "85xx CPU Options".
I made separate section thinking of  CONFIG_SYS_PPC_E500_DEBUG_TLB as it 
required for overcoming e500 v1/v2 debugging restriction.

i will send  the updated patch.

--Prabhakar





More information about the U-Boot mailing list