
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