From mboxrd@z Thu Jan 1 00:00:00 1970 From: Prabhakar Kushwaha Date: Wed, 21 Mar 2012 22:24:16 +0530 Subject: [U-Boot] [PATCH 1/4][v2] doc:Add documentation for e500 external debugger support In-Reply-To: <4F6A024B.4030200@freescale.com> References: <1332304952-10765-1-git-send-email-prabhakar@freescale.com> <4F6A024B.4030200@freescale.com> Message-ID: <4F6A07B8.7010502@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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