From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Thu, 22 Mar 2012 14:53:30 -0500 Subject: [U-Boot] [PATCH 3/4][v2] powerpc/85xx:Make debug exception vector accessible In-Reply-To: <4F6B82B8.8070103@freescale.com> References: <1332304982-10835-1-git-send-email-prabhakar@freescale.com> <4F6A3170.30907@freescale.com> <4F6ABE2E.5000502@freescale.com> <4F6B80F5.6070700@freescale.com> <4F6B82B8.8070103@freescale.com> Message-ID: <4F6B833A.60704@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 03/22/2012 02:51 PM, Timur Tabi wrote: > Scott Wood wrote: >>>> For your kind information : in start.S, label label >>>> "create_ccsr_new_tlb", "create_ccsr_old_tlb" uses MAS7 without >>>> CONFIG_ENABLE_36BIT_PHYS #define. >>>> It should be fixed ?? > >> Yes, it should be fixed. That was a fairly recent change and perhaps >> e500v1 has not been tested since then -- Timur, could you look at this? > > On e500v1 parts, CONFIG_SYS_CCSRBAR_PHYS_HIGH will always be 0, so it will > only write 0 to MAS7. Is that still considered a bad thing? The tlbwe > will ignore MAS7. Either it's needed, or we should get rid of CONFIG_ENABLE_36BIT_PHYS entirely. Either way, we should test the results on e500v1 hardware. -Scott