[Skiboot] [RFC PATCH 1/2] config: introduce CONFIG_FSP in Makefiles

Cédric Le Goater clg at fr.ibm.com
Wed Apr 13 22:06:23 AEST 2016


Here is a quick patch to remove support of FSP based systems in
skiboot. The goal is to reduce the size of skiboot which is limited to
1MB on OpenPOWER systems.

To activate support, one needs to add CONFIG_FSP=1 to the make command
line.

Signed-off-by: Cédric Le Goater <clg at fr.ibm.com>
---
 Makefile.main          |    4 ++++
 hw/Makefile.inc        |    2 ++
 platforms/Makefile.inc |    4 ++++
 3 files changed, 10 insertions(+)

Index: skiboot.git/hw/Makefile.inc
===================================================================
--- skiboot.git.orig/hw/Makefile.inc
+++ skiboot.git/hw/Makefile.inc
@@ -9,7 +9,9 @@ HW_OBJS += phb3.o sfc-ctrl.o fake-rtc.o
 HW_OBJS += dts.o lpc-rtc.o npu.o npu-hw-procedures.o
 HW=hw/built-in.o
 
+ifeq ($(CONFIG_FSP),1)
 include $(SRC)/hw/fsp/Makefile.inc
+endif
 include $(SRC)/hw/ec/Makefile.inc
 include $(SRC)/hw/ast-bmc/Makefile.inc
 include $(SRC)/hw/ipmi/Makefile.inc
Index: skiboot.git/platforms/Makefile.inc
===================================================================
--- skiboot.git.orig/platforms/Makefile.inc
+++ skiboot.git/platforms/Makefile.inc
@@ -1,9 +1,13 @@
+# -*-Makefile-*-
+
 PLATDIR = platforms
 
 SUBDIRS += $(PLATDIR)
 PLATFORMS = $(PLATDIR)/built-in.o
 
+ifeq ($(CONFIG_FSP),1)
 include $(SRC)/$(PLATDIR)/ibm-fsp/Makefile.inc
+endif
 include $(SRC)/$(PLATDIR)/rhesus/Makefile.inc
 include $(SRC)/$(PLATDIR)/astbmc/Makefile.inc
 include $(SRC)/$(PLATDIR)/mambo/Makefile.inc
Index: skiboot.git/Makefile.main
===================================================================
--- skiboot.git.orig/Makefile.main
+++ skiboot.git/Makefile.main
@@ -65,6 +65,10 @@ CFLAGS += -Wl,--oformat,elf64-powerpc
 CFLAGS += $(call try-cflag,$(CC),-mabi=elfv1)
 CFLAGS += $(call try-cflag,$(CC),-std=gnu11)
 
+ifeq ($(CONFIG_FSP),1)
+CFLAGS += -DCONFIG_FSP
+endif
+
 ifeq ($(SKIBOOT_GCOV),1)
 CFLAGS += -fprofile-arcs -ftest-coverage -DSKIBOOT_GCOV=1
 endif



More information about the Skiboot mailing list