mb/google/reef: provide override GPIO table in coral
Allow overriding specific GPIOs by SKU ID. Override two GPIO settings for nasher to save the power consumption when the system in S0ix. Change as below: AVS_DMIC_CLK_A1: IGNORE -> Tx1RXDCRx0. AVS_DMIC_CLK_B1: IGNORE -> Tx1RXDCRx0. BUG=b:69025557 BRANCH=master TEST=compile/verify the power consumption change from ~150mW to ~100mW on clamshell SKU and from ~200mW to ~100mW for convertible SKU. Change-Id: I9e0674f206426fddb3947273754774b310106334 Signed-off-by: Chris Wang <chriswang@ami.corp-partner.google.com> Reviewed-on: https://review.coreboot.org/22752 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
parent
ec10fbb90e
commit
97ac471a71
|
@ -29,6 +29,14 @@
|
|||
#include <variant/ec.h>
|
||||
#include <variant/gpio.h>
|
||||
|
||||
/* override specific gpio by sku id */
|
||||
const struct pad_config __attribute__((weak))
|
||||
*variant_sku_gpio_table(size_t *num)
|
||||
{
|
||||
*num = 0;
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static void mainboard_init(void *chip_info)
|
||||
{
|
||||
int boardid;
|
||||
|
@ -41,6 +49,9 @@ static void mainboard_init(void *chip_info)
|
|||
pads = variant_gpio_table(&num);
|
||||
gpio_configure_pads(pads, num);
|
||||
|
||||
pads = variant_sku_gpio_table(&num);
|
||||
gpio_configure_pads(pads, num);
|
||||
|
||||
mainboard_ec_init();
|
||||
|
||||
variant_board_ec_set_skuid();
|
||||
|
|
|
@ -29,6 +29,7 @@ uint8_t sku_strapping_value(void);
|
|||
const struct pad_config *variant_gpio_table(size_t *num);
|
||||
const struct pad_config *variant_early_gpio_table(size_t *num);
|
||||
const struct pad_config *variant_sleep_gpio_table(u8 slp_typ, size_t *num);
|
||||
const struct pad_config *variant_sku_gpio_table(size_t *num);
|
||||
|
||||
/* Baseboard default swizzle. Can be reused if swizzle is same. */
|
||||
extern const struct lpddr4_swizzle_cfg baseboard_lpddr4_swizzle;
|
||||
|
|
|
@ -19,6 +19,7 @@
|
|||
#include <soc/cpu.h>
|
||||
#include <soc/intel/apollolake/chip.h>
|
||||
#include <soc/intel/common/vbt.h>
|
||||
#include <soc/gpio.h>
|
||||
|
||||
enum {
|
||||
SKU_0_ASTRONAUT = 0,
|
||||
|
@ -122,3 +123,34 @@ const char *mainboard_vbt_filename(void)
|
|||
break;
|
||||
}
|
||||
}
|
||||
|
||||
static const struct pad_config nasher_gpio_tables[] = {
|
||||
/* AVS_DMIC_CLK_A1 */
|
||||
PAD_CFG_NF_IOSSTATE(GPIO_79, NATIVE, DEEP, NF1, Tx1RXDCRx0),
|
||||
/* AVS_DMIC_CLK_B1 */
|
||||
PAD_CFG_NF_IOSSTATE(GPIO_80, NATIVE, DEEP, NF1, Tx1RXDCRx0),
|
||||
};
|
||||
|
||||
const struct pad_config *variant_sku_gpio_table(size_t *num)
|
||||
{
|
||||
int sku_id = variant_board_sku();
|
||||
const struct pad_config *board_gpio_tables;
|
||||
|
||||
switch (sku_id) {
|
||||
case SKU_160_NASHER:
|
||||
case SKU_161_NASHER:
|
||||
case SKU_162_NASHER:
|
||||
case SKU_163_NASHER360:
|
||||
case SKU_164_NASHER360:
|
||||
case SKU_165_NASHER360:
|
||||
case SKU_166_NASHER360:
|
||||
*num = ARRAY_SIZE(nasher_gpio_tables);
|
||||
board_gpio_tables = nasher_gpio_tables;
|
||||
break;
|
||||
default:
|
||||
*num = 0;
|
||||
board_gpio_tables = NULL;
|
||||
break;
|
||||
}
|
||||
return board_gpio_tables;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue