summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorBora Guvendik <bora.guvendik@intel.com>2018-03-08 16:21:12 -0800
committerPatrick Georgi <pgeorgi@google.com>2018-10-11 10:57:07 +0000
commit39363744cc76a793ab429c039ff5c75e56c3b368 (patch)
tree42f04637af026d40a8d0c7c9648389c4df0a99d8 /src
parent4356e09235b911ad0c66f7467e25f6a88e823009 (diff)
downloadcoreboot-39363744cc76a793ab429c039ff5c75e56c3b368.tar.xz
commonlib/storage: Make pci sdhci code compile in romstage
In romstage malloc is not available, so use CAR_GLOBAL variable instead. BUG=b:78106689 TEST=Boot to OS Change-Id: If9438d0b707c6ffaa61db80bd1d385112bc91cfc Signed-off-by: Bora Guvendik <bora.guvendik@intel.com> Reviewed-on: https://review.coreboot.org/25067 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src')
-rw-r--r--src/commonlib/storage/pci_sdhci.c22
1 files changed, 14 insertions, 8 deletions
diff --git a/src/commonlib/storage/pci_sdhci.c b/src/commonlib/storage/pci_sdhci.c
index 329b652abb..5ecd89b477 100644
--- a/src/commonlib/storage/pci_sdhci.c
+++ b/src/commonlib/storage/pci_sdhci.c
@@ -18,12 +18,13 @@
#define __SIMPLE_DEVICE__ 1
#endif
+#include <arch/early_variables.h>
#include <assert.h>
#include <commonlib/sdhci.h>
#include <device/pci.h>
#include "sd_mmc.h"
+#include <stdint.h>
#include "storage.h"
-#include <string.h>
/* Initialize an SDHCI port */
int sdhci_controller_init(struct sdhci_ctrlr *sdhci_ctrlr, void *ioaddr)
@@ -35,17 +36,22 @@ int sdhci_controller_init(struct sdhci_ctrlr *sdhci_ctrlr, void *ioaddr)
struct sd_mmc_ctrlr *new_mem_sdhci_controller(void *ioaddr)
{
- struct sdhci_ctrlr *sdhci_ctrlr;
+ static bool sdhci_init_done CAR_GLOBAL;
+ static struct sdhci_ctrlr sdhci_ctrlr CAR_GLOBAL;
- sdhci_ctrlr = malloc(sizeof(*sdhci_ctrlr));
- if (sdhci_ctrlr == NULL)
+ if (car_get_var(sdhci_init_done) == true) {
+ sdhc_error("Error: SDHCI is already initialized.\n");
return NULL;
+ }
- if (sdhci_controller_init(sdhci_ctrlr, ioaddr)) {
- free(sdhci_ctrlr);
- sdhci_ctrlr = NULL;
+ if (sdhci_controller_init(car_get_var_ptr(&sdhci_ctrlr), ioaddr)) {
+ sdhc_error("Error: SDHCI initialization failed.\n");
+ return NULL;
}
- return &sdhci_ctrlr->sd_mmc_ctrlr;
+
+ car_set_var(sdhci_init_done, true);
+
+ return car_get_var_ptr(&sdhci_ctrlr.sd_mmc_ctrlr);
}
struct sd_mmc_ctrlr *new_pci_sdhci_controller(uint32_t dev)