Kconfig: Move defaults for CBFS_SIZE
We want the question for CBFS size to be next to the rom size in the mainboard directory, but that doesn't seem to work for how people want to set the defaults. Instead of having the list of exceptions to the size, just set the defaults at the end of kconfig. - Move the defaults for chipsets not setting HAVE_INTEL_FIRMWARE into the chipset Kconfigs (gm45, nehalem, sandybridge, x4x) - Override the default for HAVE_INTEL_FIRMWARE on skylake. - Move the HAVE_INTEL_FIRMWARE default setting into the firmware Kconfig file - Move the location of the default CBFS_SIZE=ROM_SIZE to the end of the top level kconfig file, while leaving the question where it is. Test=rebuild Kconfig files before and after the change, verify that they are how they were intended to be. Note: the Skylake boards actually changed value, because they were picking up the 0x100000 from HAVE_INTEL_FIRMWARE instead of the 0x200000 desired. This was due to the SOC_INTEL_SKYLAKE being after the HAVE_INTEL_FIRMWARE default. Affected boards were: Google chell, glados, & lars and Intel kunimitsu. Change-Id: I2963a7a7eab037955558d401f5573533674a664f Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/13645 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
parent
a3e4833e5d
commit
59ff3400b0
18
src/Kconfig
18
src/Kconfig
|
@ -376,17 +376,9 @@ menu "Mainboard"
|
||||||
|
|
||||||
source "src/mainboard/Kconfig"
|
source "src/mainboard/Kconfig"
|
||||||
|
|
||||||
|
# defaults for CBFS_SIZE are set at the end of the file.
|
||||||
config CBFS_SIZE
|
config CBFS_SIZE
|
||||||
hex "Size of CBFS filesystem in ROM"
|
hex "Size of CBFS filesystem in ROM"
|
||||||
default 0x100000 if HAVE_INTEL_FIRMWARE || \
|
|
||||||
NORTHBRIDGE_INTEL_X4X || \
|
|
||||||
NORTHBRIDGE_INTEL_GM45 || NORTHBRIDGE_INTEL_SANDYBRIDGE_MRC || \
|
|
||||||
NORTHBRIDGE_INTEL_IVYBRIDGE_MRC || NORTHBRIDGE_INTEL_IVYBRIDGE || \
|
|
||||||
NORTHBRIDGE_INTEL_SANDYBRIDGE || \
|
|
||||||
NORTHBRIDGE_INTEL_NEHALEM || SOC_INTEL_BRASWELL || \
|
|
||||||
SOC_INTEL_BROADWELL
|
|
||||||
default 0x200000 if SOC_INTEL_SKYLAKE
|
|
||||||
default ROM_SIZE
|
|
||||||
help
|
help
|
||||||
This is the part of the ROM actually managed by CBFS, located at the
|
This is the part of the ROM actually managed by CBFS, located at the
|
||||||
end of the ROM (passed through cbfstool -o) on x86 and at at the start
|
end of the ROM (passed through cbfstool -o) on x86 and at at the start
|
||||||
|
@ -1157,3 +1149,11 @@ config MAX_REBOOT_CNT
|
||||||
Internal option that sets the maximum number of bootblock executions allowed
|
Internal option that sets the maximum number of bootblock executions allowed
|
||||||
with the normal image enabled before assuming the normal image is defective
|
with the normal image enabled before assuming the normal image is defective
|
||||||
and switching to the fallback image.
|
and switching to the fallback image.
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default ROM_SIZE
|
||||||
|
help
|
||||||
|
This is the part of the ROM actually managed by CBFS. Set it to be
|
||||||
|
equal to the full rom size if that hasn't been overridden by the
|
||||||
|
chipset or mainboard.
|
||||||
|
|
|
@ -27,6 +27,10 @@ config NORTHBRIDGE_SPECIFIC_OPTIONS # dummy
|
||||||
select INTEL_EDID
|
select INTEL_EDID
|
||||||
select INTEL_GMA_ACPI
|
select INTEL_GMA_ACPI
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x100000
|
||||||
|
|
||||||
config BOOTBLOCK_NORTHBRIDGE_INIT
|
config BOOTBLOCK_NORTHBRIDGE_INIT
|
||||||
string
|
string
|
||||||
default "northbridge/intel/gm45/bootblock.c"
|
default "northbridge/intel/gm45/bootblock.c"
|
||||||
|
|
|
@ -25,6 +25,10 @@ config NORTHBRIDGE_INTEL_NEHALEM
|
||||||
|
|
||||||
if NORTHBRIDGE_INTEL_NEHALEM
|
if NORTHBRIDGE_INTEL_NEHALEM
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x100000
|
||||||
|
|
||||||
config VGA_BIOS_ID
|
config VGA_BIOS_ID
|
||||||
string
|
string
|
||||||
default "8086,0046"
|
default "8086,0046"
|
||||||
|
|
|
@ -45,6 +45,10 @@ config NORTHBRIDGE_INTEL_IVYBRIDGE
|
||||||
|
|
||||||
if NORTHBRIDGE_INTEL_SANDYBRIDGE_MRC || NORTHBRIDGE_INTEL_IVYBRIDGE_MRC || NORTHBRIDGE_INTEL_IVYBRIDGE || NORTHBRIDGE_INTEL_SANDYBRIDGE
|
if NORTHBRIDGE_INTEL_SANDYBRIDGE_MRC || NORTHBRIDGE_INTEL_IVYBRIDGE_MRC || NORTHBRIDGE_INTEL_IVYBRIDGE || NORTHBRIDGE_INTEL_SANDYBRIDGE
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x100000
|
||||||
|
|
||||||
config VGA_BIOS_ID
|
config VGA_BIOS_ID
|
||||||
string
|
string
|
||||||
default "8086,0106"
|
default "8086,0106"
|
||||||
|
|
|
@ -28,6 +28,10 @@ config NORTHBRIDGE_SPECIFIC_OPTIONS # dummy
|
||||||
select INTEL_GMA_ACPI
|
select INTEL_GMA_ACPI
|
||||||
select EARLY_CBMEM_INIT
|
select EARLY_CBMEM_INIT
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x100000
|
||||||
|
|
||||||
config BOOTBLOCK_NORTHBRIDGE_INIT
|
config BOOTBLOCK_NORTHBRIDGE_INIT
|
||||||
string
|
string
|
||||||
default "northbridge/intel/x4x/bootblock.c"
|
default "northbridge/intel/x4x/bootblock.c"
|
||||||
|
|
|
@ -64,6 +64,10 @@ config BOOTBLOCK_SOUTHBRIDGE_INIT
|
||||||
string
|
string
|
||||||
default "soc/intel/skylake/bootblock/pch.c"
|
default "soc/intel/skylake/bootblock/pch.c"
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x200000
|
||||||
|
|
||||||
config CPU_ADDR_BITS
|
config CPU_ADDR_BITS
|
||||||
int
|
int
|
||||||
default 36
|
default 36
|
||||||
|
|
|
@ -146,4 +146,10 @@ config LOCK_MANAGEMENT_ENGINE
|
||||||
|
|
||||||
If unsure, say N.
|
If unsure, say N.
|
||||||
|
|
||||||
|
config CBFS_SIZE
|
||||||
|
hex
|
||||||
|
default 0x100000
|
||||||
|
help
|
||||||
|
Reduce CBFS size to give room to the IFD blobs.
|
||||||
|
|
||||||
endif #INTEL_FIRMWARE
|
endif #INTEL_FIRMWARE
|
||||||
|
|
Loading…
Reference in New Issue