Juno AArch32: Remove duplicate definition of bl2 platform API
authorSoby Mathew <soby.mathew@arm.com>
Wed, 15 Nov 2017 12:05:28 +0000 (12:05 +0000)
committerSoby Mathew <soby.mathew@arm.com>
Wed, 29 Nov 2017 14:37:29 +0000 (14:37 +0000)
The bl2_early_platform_setup() and bl2_platform_setup() were
redefined for Juno AArch32 eventhough CSS platform layer had
same definition for them. The CSS definitions definitions were
previously restricted to EL3_PAYLOAD_BASE builds and this is now
modified to include the Juno AArch32 builds as well thus
allowing us to remove the duplicate definitions in Juno platform
layer.

Change-Id: Ibd1d8c1428cc1d51ac0ba90f19f5208ff3278ab5
Signed-off-by: Soby Mathew <soby.mathew@arm.com>
plat/arm/board/juno/juno_bl2_setup.c
plat/arm/css/common/css_bl2_setup.c

index 2771e0f3794d1b0c648ac03438f85eb8e741b5ea..cedef66c9a0a585135726ff3da0d8fb09677c8ed 100644 (file)
@@ -29,33 +29,4 @@ int bl2_plat_handle_post_image_load(unsigned int image_id)
 
        return err;
 }
-
-#if !CSS_USE_SCMI_SDS_DRIVER
-/*
- * We need to override some of the platform functions when booting SP_MIN
- * on Juno AArch32. These needs to be done only for SCPI/BOM SCP systems as
- * in case of SDS, the structures remain in memory and doesn't need to be
- * overwritten.
- */
-
-static unsigned int scp_boot_config;
-
-void bl2_early_platform_setup(meminfo_t *mem_layout)
-{
-       arm_bl2_early_platform_setup(mem_layout);
-
-       /* Save SCP Boot config before it gets overwritten by SCP_BL2 loading */
-       VERBOSE("BL2: Saving SCP Boot config = 0x%x\n", scp_boot_config);
-       scp_boot_config = mmio_read_32(SCP_BOOT_CFG_ADDR);
-}
-
-void bl2_platform_setup(void)
-{
-       arm_bl2_platform_setup();
-
-       mmio_write_32(SCP_BOOT_CFG_ADDR, scp_boot_config);
-       VERBOSE("BL2: Restored SCP Boot config = 0x%x\n", scp_boot_config);
-}
-#endif
-
 #endif /* JUNO_AARCH32_EL3_RUNTIME */
index 9b4800e340b87753c0f229cbb87f7257b91b0102..b4aafd464ae303263f1674a6f93ffd6fc7f66c88 100644 (file)
@@ -49,13 +49,13 @@ int bl2_plat_handle_scp_bl2(image_info_t *scp_bl2_image_info)
 }
 
 #if !CSS_USE_SCMI_SDS_DRIVER
-# ifdef EL3_PAYLOAD_BASE
+# if defined(EL3_PAYLOAD_BASE) || JUNO_AARCH32_EL3_RUNTIME
 
 /*
  * We need to override some of the platform functions when booting an EL3
- * payload. These needs to be done only for SCPI/BOM SCP systems as
- * in case of SDS, the structures remain in memory and doesn't need to be
- * overwritten.
+ * payload or SP_MIN on Juno AArch32. This needs to be done only for
+ * SCPI/BOM SCP systems as in case of SDS, the structures remain in memory and
+ * don't need to be overwritten.
  */
 
 static unsigned int scp_boot_config;