mb/*: drop LPC generic range for port 80
Port 80 (actually 0x80-0x8f) is a fixed I/O range and thus does not have to be set up as generic range. Drop the entries from the devicetrees. Change-Id: I8a54d3c35a321a2d57bd846662f7339eff53e5a8 Signed-off-by: Michael Niewöhner <foss@mniewoehner.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/52237 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
This commit is contained in:
		| @@ -26,9 +26,8 @@ chip soc/intel/skylake | ||||
| 	register "gpe0_dw1" = "GPP_D" | ||||
| 	register "gpe0_dw2" = "GPP_E" | ||||
|  | ||||
| 	register "gen1_dec" = "0x000c0081" | ||||
| 	register "gen2_dec" = "0x000c0681" | ||||
| 	register "gen3_dec" = "0x000c1641" | ||||
| 	register "gen1_dec" = "0x000c0681" | ||||
| 	register "gen2_dec" = "0x000c1641" | ||||
|  | ||||
| 	# Disable DPTF | ||||
| 	register "dptf_enable" = "0" | ||||
|   | ||||
| @@ -114,8 +114,7 @@ chip soc/intel/skylake | ||||
| 		device pci 1f.0 on	    # LPC Interface | ||||
| 			register "gen1_dec" = "0x000c0681" | ||||
| 			register "gen2_dec" = "0x000c1641" | ||||
| 			register "gen3_dec" = "0x000c0081" | ||||
| 			register "gen4_dec" = "0x00040069" | ||||
| 			register "gen3_dec" = "0x00040069" | ||||
| 			register "serirq_mode" = "SERIRQ_CONTINUOUS" | ||||
| 			chip drivers/pc80/tpm | ||||
| 				device pnp 0c31.0 on end | ||||
|   | ||||
| @@ -41,7 +41,6 @@ chip soc/intel/alderlake | ||||
| 	register "gen2_dec" = "0x000c0201" | ||||
| 	# EC memory map range is 0x900-0x9ff | ||||
| 	register "gen3_dec" = "0x00fc0901" | ||||
| 	register "gen4_dec" = "0x000c0081" | ||||
|  | ||||
| 	# This disabled autonomous GPIO power management, otherwise | ||||
| 	# old cr50 FW only supports short pulses; need to clarify | ||||
|   | ||||
| @@ -34,7 +34,6 @@ chip soc/intel/alderlake | ||||
| 	register "gen2_dec" = "0x000c0201" | ||||
| 	# EC memory map range is 0x900-0x9ff | ||||
| 	register "gen3_dec" = "0x00fc0901" | ||||
| 	register "gen4_dec" = "0x000c0081" | ||||
|  | ||||
| 	register "PrmrrSize" = "0" | ||||
|  | ||||
|   | ||||
| @@ -28,7 +28,6 @@ chip soc/intel/skylake | ||||
| 	register "gen1_dec" = "0x007c0a01" # EC 0xa00-0xa7f | ||||
| 	register "gen2_dec" = "0x000c03e1" # COM3 port 0x3e0 - 0x3ef | ||||
| 	register "gen3_dec" = "0x00fc02e1" # COM2/4/5/6 ports 0x2e0 - 0x2ff | ||||
| 	register "gen4_dec" = "0x000c0081" # 0x80 - 0x8f | ||||
|  | ||||
| 	# Disable DPTF | ||||
| 	register "dptf_enable" = "0" | ||||
|   | ||||
| @@ -29,7 +29,6 @@ chip soc/intel/broadwell | ||||
| 		chip soc/intel/broadwell/pch | ||||
| 			# EC host command ranges are in 0x380-0x383 & 0x80-0x8f | ||||
| 			register "gen1_dec" = "0x00000381" | ||||
| 			register "gen2_dec" = "0x000c0081" | ||||
|  | ||||
| 			device pci 13.0 off end # Smart Sound Audio DSP | ||||
| 			device pci 14.0 on  end # USB3 XHCI | ||||
|   | ||||
| @@ -36,7 +36,6 @@ chip soc/intel/skylake | ||||
|  | ||||
| 	# EC host command ranges are in 0x380-0x383 & 0x80-0x8f | ||||
| 	register "gen1_dec" = "0x00000381" | ||||
| 	register "gen2_dec" = "0x000c0081" | ||||
|  | ||||
| 	# Disable DPTF | ||||
| 	register "dptf_enable" = "0" | ||||
|   | ||||
| @@ -15,9 +15,8 @@ chip soc/intel/skylake | ||||
| 	register "gpe0_dw1" = "GPP_D" | ||||
| 	register "gpe0_dw2" = "GPP_E" | ||||
|  | ||||
| 	register "gen1_dec" = "0x000c0081" | ||||
| 	register "gen2_dec" = "0x000c0681" | ||||
| 	register "gen3_dec" = "0x000c1641" | ||||
| 	register "gen1_dec" = "0x000c0681" | ||||
| 	register "gen2_dec" = "0x000c1641" | ||||
|  | ||||
| 	# Disable DPTF | ||||
| 	register "dptf_enable" = "0" | ||||
|   | ||||
| @@ -194,10 +194,9 @@ chip soc/intel/cannonlake | ||||
| 		device pci 1e.2 off end # GSPI #0 | ||||
| 		device pci 1e.3 off end # GSPI #1 | ||||
| 		device pci 1f.0 on # LPC Interface | ||||
| 			register "gen1_dec" = "0x000c0081" | ||||
| 			register "gen2_dec" = "0x00040069" | ||||
| 			register "gen3_dec" = "0x00fc0e01" | ||||
| 			register "gen4_dec" = "0x00fc0f01" | ||||
| 			register "gen1_dec" = "0x00040069" | ||||
| 			register "gen2_dec" = "0x00fc0e01" | ||||
| 			register "gen3_dec" = "0x00fc0f01" | ||||
| 			chip drivers/pc80/tpm | ||||
| 				device pnp 0c31.0 on end | ||||
| 			end | ||||
|   | ||||
| @@ -178,14 +178,12 @@ chip soc/intel/cannonlake | ||||
| 		device pci 1e.3 off end # GSPI #1 | ||||
| 		device pci 1f.0 on	# LPC Interface | ||||
| 			# LPC configuration from lspci -s 1f.0 -xxx | ||||
| 			# Address 0x84: Decode 0x80 - 0x8F (Port 80) | ||||
| 			register "gen1_dec" = "0x000c0081" | ||||
| 			# Address 0x88: Decode 0x68 - 0x6F (EC PM channel) | ||||
| 			register "gen2_dec" = "0x00040069" | ||||
| 			register "gen1_dec" = "0x00040069" | ||||
| 			# Address 0x8C: Decode 0xE00 - 0xEFF (AP/EC command) | ||||
| 			register "gen3_dec" = "0x00fc0E01" | ||||
| 			register "gen2_dec" = "0x00fc0E01" | ||||
| 			# Address 0x90: Decode 0xF00 - 0xFFF (AP/EC debug) | ||||
| 			register "gen4_dec" = "0x00fc0F01" | ||||
| 			register "gen3_dec" = "0x00fc0F01" | ||||
| 			chip drivers/pc80/tpm	# TPM | ||||
| 				device pnp 0c31.0 on  end | ||||
| 			end | ||||
|   | ||||
| @@ -182,10 +182,9 @@ chip soc/intel/cannonlake | ||||
| 		device pci 1e.2 off end # GSPI #0 | ||||
| 		device pci 1e.3 off end # GSPI #1 | ||||
| 		device pci 1f.0 on # LPC Interface | ||||
| 			register "gen1_dec" = "0x000c0081" | ||||
| 			register "gen2_dec" = "0x00040069" | ||||
| 			register "gen3_dec" = "0x00fc0e01" | ||||
| 			register "gen4_dec" = "0x00fc0f01" | ||||
| 			register "gen1_dec" = "0x00040069" | ||||
| 			register "gen2_dec" = "0x00fc0e01" | ||||
| 			register "gen3_dec" = "0x00fc0f01" | ||||
| 			chip drivers/pc80/tpm | ||||
| 				device pnp 0c31.0 on end | ||||
| 			end | ||||
|   | ||||
		Reference in New Issue
	
	Block a user