diff options
| author | Andrey Smirnov <andrew.smirnov@gmail.com> | 2019-04-14 17:21:21 -0700 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2019-04-23 11:33:04 +0200 |
| commit | 07290094587d9e1142d530dc164ef2bdb6c47470 (patch) | |
| tree | 0fb0d1ee63cf35a379340c9b06cc6efcf0d769a5 /arch/arm | |
| parent | 598144bc8ef80576e3d54d4cb8d4207b22f64b46 (diff) | |
| download | barebox-07290094587d9e1142d530dc164ef2bdb6c47470.tar.gz barebox-07290094587d9e1142d530dc164ef2bdb6c47470.tar.xz | |
ARM: zii-imx8mq-dev: Drop unnecessary barrier() in switch statement
AArch64 uses PC-relative addressing instead of absolute one for data
lookups, so compiling switch statement into a LUT shouldn't be a
problem regardless if relocation happened or not. Disassembly of PBL
code looks almost exactly the same with or without this workaround, so
it is clearly not needed. Drop it.
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Cc: Heiner Kallweit <hkallweit1@gmail.com>
Cc: Chris Healy <cphealy@gmail.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'arch/arm')
| -rw-r--r-- | arch/arm/boards/zii-imx8mq-dev/lowlevel.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/arch/arm/boards/zii-imx8mq-dev/lowlevel.c b/arch/arm/boards/zii-imx8mq-dev/lowlevel.c index 0fd2ddfca5..33c007e05d 100644 --- a/arch/arm/boards/zii-imx8mq-dev/lowlevel.c +++ b/arch/arm/boards/zii-imx8mq-dev/lowlevel.c @@ -179,13 +179,6 @@ ENTRY_FUNCTION(start_zii_imx8mq_dev, r0, r1, r2) switch (system_type) { default: - /* - * see similar code in - * arch/arm/boards/zii-vf610-dev/lowlevel.c for - * reasoning for placing barrier() below. - */ - barrier(); - if (IS_ENABLED(CONFIG_DEBUG_LL)) { relocate_to_current_adr(); setup_c(); |