On Fri, Apr 12, 2019 at 4:42 PM Stefan Roese sr@denx.de wrote:
Testing has shown that the current DM implementation of a platform / board specific translation offset, as its needed for the SPL on MVEBU platforms is buggy. The translation offset is confingured too late, after the driver bind functions are run. This may result in incorrect address translations. With the current implementation its not possible to configure the offset earlier, as the DM code has not run at all.
This patch now removed the set_/get_translation_offset() calls and moves the translation offset into the GD variable translation_offset. This variable will get used when CONFIG_TRANSLATION_OFFSET is enabled. This option is enabled only for MVEBU on ARM32 platforms, where its currenty needed and configured in the SPL.
Verified to work on Turris Omnia. No freeze at boot time in SPL mode, i2c0 is properly enumerated.
Signed-off-by: Stefan Roese sr@denx.de Cc: Pierre Bourdon delroth@gmail.com Cc: Baruch Siach baruch@tkos.co.il Cc: Simon Glass sjg@chromium.org Cc: Heiko Schocher hs@denx.de Cc: Tom Rini trini@konsulko.com
Tested-by: Pierre Bourdon delroth@gmail.com