[PATCH 087/108] x86: apl: Add a check for reading the FSP-S config
Simon Glass
sjg at chromium.org
Mon Jan 27 06:06:34 CET 2020
This currently fails as two required items are commented out. Put them
back in and add a check that reading the config succeeds.
Both of these items need a little more work to put them in an appropriate
driver at some point.
Signed-off-by: Simon Glass <sjg at chromium.org>
---
arch/x86/cpu/apollolake/fsp_s.c | 6 ++++--
arch/x86/dts/chromebook_coral.dts | 12 ++++++------
2 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/arch/x86/cpu/apollolake/fsp_s.c b/arch/x86/cpu/apollolake/fsp_s.c
index e7e0c30873..d1aad7a094 100644
--- a/arch/x86/cpu/apollolake/fsp_s.c
+++ b/arch/x86/cpu/apollolake/fsp_s.c
@@ -350,8 +350,10 @@ int fsps_update_config(struct udevice *dev, ulong rom_offset,
apl = malloc(sizeof(*apl));
if (!apl)
- return log_msg_ret("config", -ENOMEM);
- get_config(dev, apl);
+ return log_msg_ret("alloc", -ENOMEM);
+ ret = get_config(dev, apl);
+ if (ret)
+ return log_msg_ret("config", ret);
cfg->ish_enable = 0;
cfg->enable_sata = 0;
diff --git a/arch/x86/dts/chromebook_coral.dts b/arch/x86/dts/chromebook_coral.dts
index 7ba38d7791..3c6f14310f 100644
--- a/arch/x86/dts/chromebook_coral.dts
+++ b/arch/x86/dts/chromebook_coral.dts
@@ -497,13 +497,13 @@
* If the Board has PERST_0 signal, assign the GPIO
* If the Board does not have PERST_0, assign GPIO_PRT0_UDEF
*
- * This are not used yet, so comment them out for now.
- *
- * prt0-gpio = <GPIO_122>;
- *
- * GPIO for SD card detect
- * sdcard-cd-gpio = <GPIO_177>;
+ * This should be converted to real GPIOs and ideally stored in the
+ * device they relate to.
*/
+ prt0-gpio = <GPIO_122>;
+
+ /* GPIO for SD card detect */
+ sdcard-cd-gpio = <GPIO_177>;
/*
* Order is emmc-tx-data-cntl1, emmc-tx-data-cntl2,
--
2.25.0.341.g760bfbb309-goog
More information about the U-Boot
mailing list