cpu/amd/{agesa,pi}: Select NO_FIXED_XIP_ROM_SIZE
AGESA and binaryPI set the whole CACHE_ROM_SIZE to WRPROT during the romstage and do not reference the CONFIG_XIP_ROM_SIZE symbol. Change-Id: I548b9c9066d825c2f03749353b9990b2efddfd9c Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/35825 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
		
				
					committed by
					
						 Patrick Georgi
						Patrick Georgi
					
				
			
			
				
	
			
			
			
						parent
						
							e9649218bf
						
					
				
				
					commit
					e33c50d74c
				
			| @@ -28,20 +28,10 @@ config CPU_AMD_AGESA | ||||
| 	select LAPIC_MONOTONIC_TIMER | ||||
| 	select SPI_FLASH if HAVE_ACPI_RESUME | ||||
| 	select SMM_ASEG | ||||
| 	select NO_FIXED_XIP_ROM_SIZE | ||||
|  | ||||
| if CPU_AMD_AGESA | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
| 	help | ||||
| 	  Overwride the default write through caching size as 1M Bytes. | ||||
| 	  On some AMD platforms, one socket supports 2 or more kinds of | ||||
| 	  processor family, compiling several CPU families agesa code | ||||
| 	  will increase the romstage size. | ||||
| 	  In order to execute romstage in place on the flash ROM, | ||||
| 	  more space is required to be set as write through caching. | ||||
|  | ||||
| config UDELAY_LAPIC_FIXED_FSB | ||||
| 	int | ||||
| 	default 200 | ||||
|   | ||||
| @@ -21,8 +21,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 48 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x80000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -21,8 +21,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 36 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x80000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -21,8 +21,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 48 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -21,10 +21,6 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 40 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
|  | ||||
| config FORCE_AM1_SOCKET_SUPPORT | ||||
| 	bool | ||||
| 	default n | ||||
|   | ||||
| @@ -21,8 +21,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 48 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -21,8 +21,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 48 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -23,8 +23,4 @@ config CPU_ADDR_BITS | ||||
| 	int | ||||
| 	default 40 | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
|  | ||||
| endif | ||||
|   | ||||
| @@ -28,23 +28,13 @@ config CPU_AMD_PI | ||||
| 	select SPI_FLASH if HAVE_ACPI_RESUME | ||||
| 	select CAR_GLOBAL_MIGRATION if BINARYPI_LEGACY_WRAPPER | ||||
| 	select SMM_ASEG | ||||
| 	select NO_FIXED_XIP_ROM_SIZE | ||||
|  | ||||
| if CPU_AMD_PI | ||||
|  | ||||
| config BINARYPI_LEGACY_WRAPPER | ||||
| 	def_bool n | ||||
|  | ||||
| config XIP_ROM_SIZE | ||||
| 	hex | ||||
| 	default 0x100000 | ||||
| 	help | ||||
| 	  Overwride the default write through caching size as 1M Bytes. | ||||
| 	  On some AMD platforms, one socket supports 2 or more kinds of | ||||
| 	  processor family, compiling several CPU families agesa code | ||||
| 	  will increase the romstage size. | ||||
| 	  In order to execute romstage in place on the flash ROM, | ||||
| 	  more space is required to be set as write through caching. | ||||
|  | ||||
| config UDELAY_LAPIC_FIXED_FSB | ||||
| 	int | ||||
| 	default 200 | ||||
|   | ||||
		Reference in New Issue
	
	Block a user