[coreboot-gerrit] Patch set updated for coreboot: mediatek/mt8173: Remove bl31 board parameters passing mechanism

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Mar 17 15:38:25 CET 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14124

-gerrit

commit 5c84d451038705e00ff6557bd755f51204d45d0d
Author: Jimmy Huang <jimmy.huang at mediatek.com>
Date:   Sat Mar 5 10:25:05 2016 +0800

    mediatek/mt8173: Remove bl31 board parameters passing mechanism
    
    As the DA9212 and MT6311 external buck can be controlled by hardware
    since rev-5 board, we don't need to pass any board specific parameter
    to ARM TF.
    
    BRANCH=none
    BUG=none
    TEST=build pass
    
    Change-Id: I43eebe25ab14d3dd84e8bb4286e2bb55c8c3c063
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 9c4dfe61c69042e464b384e2e0edbc55eda23a74
    Original-Change-Id: I541357fee6afb1ff2d771bcb073f7c9a9db52f00
    Original-Signed-off-by: Jimmy Huang <jimmy.huang at mediatek.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/332344
    Original-Commit-Ready: Yidi Lin <yidi.lin at mediatek.com>
    Original-Tested-by: Yidi Lin <yidi.lin at mediatek.com>
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
---
 src/soc/mediatek/mt8173/Makefile.inc               |  3 +--
 src/soc/mediatek/mt8173/bl31_plat_params.c         | 11 +---------
 .../mediatek/mt8173/include/soc/bl31_plat_params.h | 24 ----------------------
 3 files changed, 2 insertions(+), 36 deletions(-)

diff --git a/src/soc/mediatek/mt8173/Makefile.inc b/src/soc/mediatek/mt8173/Makefile.inc
index 217cbc3..82d74e3 100644
--- a/src/soc/mediatek/mt8173/Makefile.inc
+++ b/src/soc/mediatek/mt8173/Makefile.inc
@@ -77,8 +77,7 @@ ramstage-y += usb.c
 
 ramstage-$(CONFIG_ARM64_USE_ARM_TRUSTED_FIRMWARE) += bl31_plat_params.c
 
-BL31_MAKEARGS += PLAT=mt8173 MEDIATEK_MEM_CONFIG=SRAM_192K MEDIATEK_DA9212=1 \
-		 MEDIATEK_MT6311=1
+BL31_MAKEARGS += PLAT=mt8173
 
 ################################################################################
 
diff --git a/src/soc/mediatek/mt8173/bl31_plat_params.c b/src/soc/mediatek/mt8173/bl31_plat_params.c
index 025c202..fc211ef 100644
--- a/src/soc/mediatek/mt8173/bl31_plat_params.c
+++ b/src/soc/mediatek/mt8173/bl31_plat_params.c
@@ -14,17 +14,8 @@
  */
 #include <arm_tf.h>
 #include <assert.h>
-#include <soc/bl31_plat_params.h>
-
-static struct bl31_plat_param *plat_params;
-
-void register_bl31_param(struct bl31_plat_param *param)
-{
-	param->next = (void *)plat_params;
-	plat_params = param;
-}
 
 void *soc_get_bl31_plat_params(bl31_params_t *bl31_params)
 {
-	return (void *)plat_params;
+	return (void *)NULL;
 }
diff --git a/src/soc/mediatek/mt8173/include/soc/bl31_plat_params.h b/src/soc/mediatek/mt8173/include/soc/bl31_plat_params.h
deleted file mode 100644
index c1ea885..0000000
--- a/src/soc/mediatek/mt8173/include/soc/bl31_plat_params.h
+++ /dev/null
@@ -1,24 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright 2015 MediaTek Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-#ifndef __BL31_PLAT_PARAMS_H__
-#define __BL31_PLAT_PARAMS_H__
-
-#if IS_ENABLED(CONFIG_ARM64_USE_ARM_TRUSTED_FIRMWARE)
-#include <arm-trusted-firmware/plat/mediatek/mt8173/include/plat_params.h>
-
-void register_bl31_param(struct bl31_plat_param *param);
-#endif
-
-#endif /* __BL31_PLAT_PARAMS_H__ */



More information about the coreboot-gerrit mailing list