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



_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to