mb/google/herobrine: Implement mainboard_needs_pcie_init

Implement mainboard_needs_pcie_init() for herobrine in order to
determine if we need to initialize the pcie links.  When the SKU id is
unknown or unprovisioned (for example at the beginning of the factory
flow), we should still initialize PCIe. Otherwise the devices with
NVMe will fail to boot.

BUG=b:254281839
BRANCH=None
TEST=emerge-herobrine coreboot

Change-Id: I8972424f0c5d082165c185ab52a638e8b134064c
Signed-off-by: Shelley Chen <shchen@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69689
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
This commit is contained in:
Shelley Chen 2022-11-16 16:59:27 -08:00 committed by Felix Held
parent ce4dc66319
commit b5af064f54
2 changed files with 27 additions and 0 deletions

View File

@ -8,10 +8,12 @@
#include <delay.h> #include <delay.h>
#include <device/device.h> #include <device/device.h>
#include <device/mmio.h> #include <device/mmio.h>
#include <ec/google/chromeec/ec.h>
#include <edid.h> #include <edid.h>
#include <soc/clock.h> #include <soc/clock.h>
#include <soc/display/mdssreg.h> #include <soc/display/mdssreg.h>
#include <soc/display/edp_ctrl.h> #include <soc/display/edp_ctrl.h>
#include <soc/pcie.h>
#include <soc/qupv3_config_common.h> #include <soc/qupv3_config_common.h>
#include <soc/qup_se_handlers_common.h> #include <soc/qup_se_handlers_common.h>
#include <soc/qcom_qup_se.h> #include <soc/qcom_qup_se.h>
@ -82,6 +84,30 @@ static void display_startup(void)
} }
} }
/*
* Determine if board need to perform PCIe initialization. On Herobrine,
* resistor strapping will be such that bit 0 will be assigned 2 (high Z) if it
* is an NVMe enabled platform.
*/
bool mainboard_needs_pcie_init(void)
{
uint32_t sku = sku_id();
if (sku == CROS_SKU_UNKNOWN) {
printk(BIOS_WARNING, "Unknown SKU (%#x); assuming PCIe", sku);
return true;
} else if (sku == CROS_SKU_UNPROVISIONED) {
printk(BIOS_WARNING, "Unprovisioned SKU (%#x); assuming PCIe", sku);
return true;
}
if ((sku % 3) == 2)
return true;
/* Otherwise, eMMC */
return false;
}
static void mainboard_init(struct device *dev) static void mainboard_init(struct device *dev)
{ {
/* Configure clock for eMMC */ /* Configure clock for eMMC */

View File

@ -221,5 +221,6 @@ void gcom_pcie_power_on_ep(void);
void gcom_pcie_get_config(struct qcom_pcie_cntlr_t *host_cfg); void gcom_pcie_get_config(struct qcom_pcie_cntlr_t *host_cfg);
void qcom_pci_domain_read_resources(struct device *dev); void qcom_pci_domain_read_resources(struct device *dev);
void qcom_setup_pcie_host(struct device *dev); void qcom_setup_pcie_host(struct device *dev);
bool mainboard_needs_pcie_init(void);
#endif #endif