-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
Andy Green wrote: > Somebody in the thread at some point said: > | Hi Andy: > | > | void (*phase2)(void) = (void > (*)(void))((int)bootloader_second_phase + > | TEXT_BASE); > | where here add TEXT_BASE? i don't understand. > | i see the new patch there is no "+ TEXT_BASE" > > TEXT_BASE is where we copy the qi image to in SDRAM, 0x33000000. Before i can not find where the code that we move the QI from steppingstone 4K to SDRAM. > I got the linker script right, it had to add 0x33000000 to > bootloader_second_phase() so it would run the copy in SDRAM, not the > copy in the steppingstone that starts at 0x0. Because in the old code > bootloader_second_phase had the value like 0x800 or something: it was > compiled to run from steppingstone. All of the code was like that so > actually until the last patches everything only worked because it fitted > in steppingstone 4K region. > > Now the linker script is right, bootloader_second_phase() has a value in > 0x33000xxx range already and works without any magic offsets (the rest > of the code just works with addresses already in the copied region as > well). > > -Andy -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iQEVAwUBSKkwQBFuNemPXNFjAQILDgf+PBRRB74yz2XvlaMjXoYUqZSOGwFasY4Q Zg33XkNvzAo/P7IN915TSAwa9RZlUtGGhDWVj1DdLkQamARiDgPZaDgthlq1mp7X Lnwzuqv0rt9ZAlAAPYueLdGiuyhNJtZVvsGqqlNtPlpI2q4l9sFk10HnwmUQrn73 2zkXT9ZWT1kyZdhZqg5CfPB8JFpoLj7V7Lq4IaV+AzTZl25Jd/NjEIWf22OmojaH eKe3WPg7ZqbhIb8Z0Ao+r9TUTAdLC/auAH0VO2EAM52u9CqyHWZChVb596vKLLy3 2USetB3gR7zxPSywT1gwcpQ21txXD5QJ8oPxi7kPy8a4+Z6CsK9WNg== =t4sq -----END PGP SIGNATURE-----
