[coreboot-gerrit] New patch to review for coreboot: drivers/intel/fsp1_1: Don't compile GOP support in romstage

Alexandru Gagniuc (mr.nuke.me@gmail.com) gerrit at coreboot.org
Sat Aug 29 04:25:21 CEST 2015


Alexandru Gagniuc (mr.nuke.me at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11457

-gerrit

commit e7d16a46a1f4870fd18a45b71270c29aef3913c9
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Fri Aug 28 19:23:29 2015 -0400

    drivers/intel/fsp1_1: Don't compile GOP support in romstage
    
    We don't need the code in romstage, and it saves us a few #ifdefs.
    
    Change-Id: I26d867566f07c7d80890cd01bf055be7497130d3
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>
---
 src/drivers/intel/fsp1_1/Makefile.inc | 1 -
 src/drivers/intel/fsp1_1/fsp_gop.c    | 2 --
 src/drivers/intel/fsp1_1/fsp_gop.h    | 3 +--
 3 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/drivers/intel/fsp1_1/Makefile.inc b/src/drivers/intel/fsp1_1/Makefile.inc
index 05dbfe5..f831f9d 100644
--- a/src/drivers/intel/fsp1_1/Makefile.inc
+++ b/src/drivers/intel/fsp1_1/Makefile.inc
@@ -18,7 +18,6 @@
 # Foundation, Inc.
 #
 
-romstage-$(CONFIG_GOP_SUPPORT) += fsp_gop.c
 romstage-y += fsp_util.c
 romstage-y += hob.c
 
diff --git a/src/drivers/intel/fsp1_1/fsp_gop.c b/src/drivers/intel/fsp1_1/fsp_gop.c
index 32a176f..ed1f1b4 100644
--- a/src/drivers/intel/fsp1_1/fsp_gop.c
+++ b/src/drivers/intel/fsp1_1/fsp_gop.c
@@ -61,7 +61,6 @@ const optionrom_vbt_t *fsp_get_vbt(uint32_t *vbt_len)
 	return vbt.data;
 }
 
-#if ENV_RAMSTAGE
 void fsp_gop_framebuffer(struct lb_header *header)
 {
 	struct lb_framebuffer *framebuffer;
@@ -98,4 +97,3 @@ void fsp_gop_framebuffer(struct lb_header *header)
 	framebuffer->tag = LB_TAG_FRAMEBUFFER;
 	framebuffer->size = sizeof(*framebuffer);
 }
-#endif /* ENV_RAMSTAGE */
diff --git a/src/drivers/intel/fsp1_1/fsp_gop.h b/src/drivers/intel/fsp1_1/fsp_gop.h
index fc25901..2999369 100644
--- a/src/drivers/intel/fsp1_1/fsp_gop.h
+++ b/src/drivers/intel/fsp1_1/fsp_gop.h
@@ -27,8 +27,7 @@
 #include <soc/intel/common/gma.h>
 
 const optionrom_vbt_t *fsp_get_vbt(uint32_t *vbt_len);
-#if ENV_RAMSTAGE
 void fsp_gop_framebuffer(struct lb_header *header);
-#endif /* ENV_RAMSTAGE */
+
 #endif /* CONFIG_GOP_SUPPORT */
 #endif /* _FSP_GOP_H_ */



More information about the coreboot-gerrit mailing list