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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
