From mboxrd@z Thu Jan 1 00:00:00 1970 From: Prabhakar Kushwaha Date: Thu, 17 Apr 2014 09:11:55 +0530 Subject: [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section In-Reply-To: <534F1B2A.6020906@freescale.com> References: <1396260094-2328-1-git-send-email-prabhakar@freescale.com> <533CDD4E.4080900@freescale.com> <534F1B2A.6020906@freescale.com> Message-ID: <534F4D83.7010602@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 On 4/17/2014 5:37 AM, York Sun wrote: > On 04/02/2014 09:02 PM, Prabhakar Kushwaha wrote: >> Hi All, >> >> I just want to share my concern with respect to this patch. >> This patch removed hard-coded value for 0x80000 and use CONFIG_SYS_MONITOR_LEN. >> >> There were many Freescale boards config which don?t have correct >> CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I fixed them >> in subsequent patch. >> >> This linker file is being used by many other powerpc boards/platforms. Because >> of hard-coded value of 0x80000 in linker file, there boards are booting till now. >> but after this patch, there is a possibility there boards may not boot because >> of wrong value defined for CONFIG_SYS_MONITOR_LEN by board maintainers. >> >> I will request powerpc board maintainers to check config file for correct value >> of CONFIG_SYS_MONITOR_LEN. >> >> > Prabhakar, > > Where are we for this patch? I don't see any comment. > > I did not get any comments :( Looks like every one is OK with it. regards, Prabhakar