From e3011451cccece7668f95e59dfb6f61c878b7e0a Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Wed, 23 May 2018 21:27:33 -0700 Subject: mb/google/poppy/variants/nami: Perform PL2 setting in variant_devtree_udpate This change moves PL2 override to variant_devtree_update for two reasons: 1. This function was added to basically override devtree settings in variant specific code. So, it would be a good idea to perform all the overrides in a single place. 2. Adding a device for performing nami_enable would require changes to devicetree and special handling for calling this device enable. Thus, nami_enable was never getting called. BUG=b:80148703 Change-Id: Ifa24a7b6e99cad2368b3d656a757f26297373121 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/26499 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- .../google/poppy/variants/nami/Makefile.inc | 1 - .../google/poppy/variants/nami/mainboard.c | 21 +++++++++- src/mainboard/google/poppy/variants/nami/pl2.c | 48 ---------------------- 3 files changed, 19 insertions(+), 51 deletions(-) delete mode 100644 src/mainboard/google/poppy/variants/nami/pl2.c (limited to 'src/mainboard') diff --git a/src/mainboard/google/poppy/variants/nami/Makefile.inc b/src/mainboard/google/poppy/variants/nami/Makefile.inc index e9e90f5080..618011caef 100644 --- a/src/mainboard/google/poppy/variants/nami/Makefile.inc +++ b/src/mainboard/google/poppy/variants/nami/Makefile.inc @@ -33,7 +33,6 @@ romstage-y += memory.c ramstage-y += gpio.c ramstage-y += nhlt.c -ramstage-y += pl2.c ramstage-y += mainboard.c # Add OEM ID table diff --git a/src/mainboard/google/poppy/variants/nami/mainboard.c b/src/mainboard/google/poppy/variants/nami/mainboard.c index 6ff296022b..492167946f 100644 --- a/src/mainboard/google/poppy/variants/nami/mainboard.c +++ b/src/mainboard/google/poppy/variants/nami/mainboard.c @@ -23,11 +23,23 @@ #include #include #include +#include #include #include #include #include +#define PL2_I7_SKU 44 +#define PL2_DEFAULT 29 + +static uint32_t get_pl2(void) +{ + if (cpuid_eax(1) == CPUID_KABYLAKE_Y0) + return PL2_I7_SKU; + + return PL2_DEFAULT; +} + uint32_t variant_board_sku(void) { static uint32_t sku_id = SKU_UNKNOWN; @@ -42,16 +54,21 @@ uint32_t variant_board_sku(void) return sku_id; } +/* Override dev tree settings per board */ void variant_devtree_update(void) { - /* Override dev tree settings per board */ uint32_t sku_id = variant_board_sku(); struct device *root = SA_DEV_ROOT; config_t *cfg = root->chip_info; + + /* Update PL2 based on SKU. */ + cfg->tdp_pl2_override = get_pl2(); + switch (sku_id) { case SKU_1_VAYNE: case SKU_2_VAYNE: - cfg->usb2_ports[5].enable = 0;//rear camera + /* Disable unused port USB port */ + cfg->usb2_ports[5].enable = 0; break; default: break; diff --git a/src/mainboard/google/poppy/variants/nami/pl2.c b/src/mainboard/google/poppy/variants/nami/pl2.c deleted file mode 100644 index 6744f2c407..0000000000 --- a/src/mainboard/google/poppy/variants/nami/pl2.c +++ /dev/null @@ -1,48 +0,0 @@ -/* - * This file is part of the coreboot project. - * - * Copyright 2017 Google Inc. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; version 2 of the License. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#include -#include -#include -#include - -#define PL2_I7_SKU 44 -#define PL2_DEFAULT 29 - -static uint32_t nami_get_pl2(void) -{ - struct cpuid_result cpuidr; - - cpuidr = cpuid(1); - if (cpuidr.eax == CPUID_KABYLAKE_Y0) - return PL2_I7_SKU; - - return PL2_DEFAULT; -} - -static void nami_enable(device_t dev) -{ - struct device *root = SA_DEV_ROOT; - config_t *conf = root->chip_info; - - if (!conf) - return; - - conf->tdp_pl2_override = nami_get_pl2(); -} - -struct chip_operations nami_ops = { - .enable_dev = nami_enable, -}; -- cgit v1.2.3