GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mb/google/dedede/var/blipper: Update devicetree and gpio setting 

To reduce power load, set unused GPIOs to NC and close
unused interface in devicetree. GPIOs and
interfaces are as below:
GPIO: GPP_C18/C19/D12/D14/D15/D19/D20/D21/E00/E02/H06/H07
Interface: I2C1/I2C3/I2C5
USB: port2_3/2_4/2_6

BUG=b:185044041
BRANCH=dedede
TEST=Built bios and test, it reduces power load without affecting
device function.

Change-Id: Ib5999f0e129bf3e660fe293eda7af3e8e1426151
Signed-off-by: Zanxi Chen <chenzanxi@huaqin.corp-partner.google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55385
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Reviewed-by: Ben Kao <ben.kao@intel.com>
This commit is contained in:
Zanxi Chen 2021-06-10 20:29:05 +08:00 committed by Patrick Georgi
parent 70d0795df9
commit 1c43d92efc
2 changed files with 43 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/mainboard/google/dedede/variants/blipper/gpio.c
View File

@ -7,8 +7,35 @@
/* Pad configuration in ramstage*/ /* Pad configuration in ramstage*/
static const struct pad_config gpio_table[] = { static const struct pad_config gpio_table[] = {
/* C18 : AP_I2C_EMR_SDA */
PAD_NC(GPP_C18, NONE),
/* C19 : AP_I2C_EMR_SCL */
PAD_NC(GPP_C19, NONE),
/* D12 : WCAM_RST_L */
PAD_NC(GPP_D12, NONE),
/* D13 : EN_PP2800_CAMERA */ /* D13 : EN_PP2800_CAMERA */
PAD_CFG_GPO(GPP_D13, 1, PLTRST), PAD_CFG_GPO(GPP_D13, 1, PLTRST),
/* D14 : EN_PP1200_CAMERA */
PAD_NC(GPP_D14, NONE),
/* D15 : UCAM_RST_L */
PAD_NC(GPP_D15, NONE),
/* D19 : WWAN_WLAN_COEX1 */
PAD_NC(GPP_D19, NONE),
/* D20 : WWAN_WLAN_COEX2 */
PAD_NC(GPP_D20, NONE),
/* D21 : WWAN_WLAN_COEX3 */
PAD_NC(GPP_D21, NONE),
/* E0 : CLK_24M_UCAM */
PAD_NC(GPP_E0, NONE),
/* E2 : CLK_24M_WCAM */
PAD_NC(GPP_E2, NONE),
/* H6 : AP_I2C_CAM_SDA */
PAD_NC(GPP_H6, NONE),
/* H7 : AP_I2C_CAM_SCL */
PAD_NC(GPP_H7, NONE),
}; };
const struct pad_config *variant_override_gpio_table(size_t *num) const struct pad_config *variant_override_gpio_table(size_t *num)

25
src/mainboard/google/dedede/variants/blipper/overridetree.cb
View File

@ -9,9 +9,9 @@ chip soc/intel/jasperlake
#| | for TPM communication | #| | for TPM communication |
#| | before memory is up | #| | before memory is up |
#| I2C0 | Trackpad | #| I2C0 | Trackpad |
#| I2C1 | Digitizer | #| I2C1 | |
#| I2C2 | Touchscreen | #| I2C2 | Touchscreen |
#| I2C3 | Camera | #| I2C3 | |
#| I2C4 | Audio | #| I2C4 | Audio |
#+-------------------+---------------------------+ #+-------------------+---------------------------+
register "common_soc_config" = "{ register "common_soc_config" = "{
@ -24,17 +24,11 @@ chip soc/intel/jasperlake
.rise_time_ns = 247, .rise_time_ns = 247,
.fall_time_ns = 11, .fall_time_ns = 11,
}, },
.i2c[1] = {
.speed = I2C_SPEED_FAST,
},
.i2c[2] = { .i2c[2] = {
.speed = I2C_SPEED_FAST, .speed = I2C_SPEED_FAST,
.rise_time_ns = 175, .rise_time_ns = 175,
.fall_time_ns = 20, .fall_time_ns = 20,
}, },
.i2c[3] = {
.speed = I2C_SPEED_FAST,
},
.i2c[4] = { .i2c[4] = {
.speed = I2C_SPEED_FAST, .speed = I2C_SPEED_FAST,
.rise_time_ns = 99, .rise_time_ns = 99,
@ -43,7 +37,19 @@ chip soc/intel/jasperlake
}" }"
# USB Port Configuration # USB Port Configuration
register "usb2_ports[3]" = "USB2_PORT_EMPTY" # Not Used
register "usb2_ports[4]" = "USB2_PORT_EMPTY" # Not Used
register "usb2_ports[5]" = "USB2_PORT_MID(OC_SKIP)" # UF Camera register "usb2_ports[5]" = "USB2_PORT_MID(OC_SKIP)" # UF Camera
register "usb2_ports[6]" = "USB2_PORT_EMPTY" # Not Used
register "SerialIoI2cMode" = "{
[PchSerialIoIndexI2C0] = PchSerialIoPci,
[PchSerialIoIndexI2C1] = PchSerialIoDisabled,
[PchSerialIoIndexI2C2] = PchSerialIoPci,
[PchSerialIoIndexI2C3] = PchSerialIoDisabled,
[PchSerialIoIndexI2C4] = PchSerialIoPci,
[PchSerialIoIndexI2C5] = PchSerialIoDisabled,
}"
register "power_limits_config" = "{ register "power_limits_config" = "{
.tdp_pl1_override = 6, .tdp_pl1_override = 6,
@ -92,7 +98,6 @@ chip soc/intel/jasperlake
chip drivers/usb/acpi chip drivers/usb/acpi
register "desc" = ""UFCamera"" register "desc" = ""UFCamera""
register "type" = "UPC_TYPE_INTERNAL" register "type" = "UPC_TYPE_INTERNAL"
register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_D13)"
device usb 2.5 on end device usb 2.5 on end
end end
end end
@ -117,6 +122,7 @@ chip soc/intel/jasperlake
device i2c 0x2c on end device i2c 0x2c on end
end end
end # I2C 0 end # I2C 0
device pci 15.1 off end # I2C 1
device pci 15.2 on device pci 15.2 on
chip drivers/i2c/hid chip drivers/i2c/hid
register "generic.hid" = ""GTCH7503"" register "generic.hid" = ""GTCH7503""
@ -154,6 +160,7 @@ chip soc/intel/jasperlake
device i2c 10 on end device i2c 10 on end
end end
end # I2C 2 end # I2C 2
device pci 15.3 off end # I2C 3
device pci 19.0 on device pci 19.0 on
chip drivers/i2c/generic chip drivers/i2c/generic
register "hid" = ""10EC5682"" register "hid" = ""10EC5682""