summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLiju-Clr Chen <liju-clr.chen@mediatek.com>2022-11-21 18:36:36 +0800
committerFelix Held <felix-coreboot@felixheld.de>2022-11-23 16:31:29 +0000
commite1ee23f29d9b0ca86bc29b540efa9290d7b40cb0 (patch)
tree499bd97d7372029149c707b5ac63d54b2e50bba4
parent977673894ff486af2ff74ba5585ca6039701eb8f (diff)
soc/mediatek: Add error handling for dptx_get_edid()
Skip eDP initialization when we failed to get EDID. This prevents the PLL assertion in dp_intf_config() if the display could not be initialized properly. BUG=b:233720142 TEST=boot to depthcharge on MT8188 EVB. Change-Id: I0fd672b175feb9b813c1d9ec4140e4273079ff07 Signed-off-by: Liju-Clr Chen <liju-clr.chen@mediatek.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/69858 Reviewed-by: Rex-BC Chen <rex-bc.chen@mediatek.com> Reviewed-by: Yidi Lin <yidilin@google.com> Reviewed-by: Yu-Ping Wu <yupingso@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/soc/mediatek/common/dp/dptx.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/src/soc/mediatek/common/dp/dptx.c b/src/soc/mediatek/common/dp/dptx.c
index f2c0905cc4..08d288acf8 100644
--- a/src/soc/mediatek/common/dp/dptx.c
+++ b/src/soc/mediatek/common/dp/dptx.c
@@ -1111,7 +1111,10 @@ int mtk_edp_init(struct edid *edid)
dptx_check_sinkcap(&mtk_edp);
- dptx_get_edid(&mtk_edp, edid);
+ if (dptx_get_edid(&mtk_edp, edid) != 0) {
+ printk(BIOS_ERR, "Failed to get EDID\n");
+ return -1;
+ }
dptx_set_trainingstart(&mtk_edp);
dp_intf_config(edid);