Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/87183?usp=email )
Change subject: WIP: mainboard/fatcat: Temporarily disable barrel charger power limit optimization ......................................................................
WIP: mainboard/fatcat: Temporarily disable barrel charger power limit optimization
This commit temporarily disables the logic that prevents CPU power limit optimization when a barrel charger is attached. This is being done to reduce the boot time for fatcat SKU.
We don't wish to keep barrel charger detection logic which is time consuming.
TEST=Able to reduce boot time by 62ms.
Change-Id: I613417be5a59790b8a5e6055957a2f518f4c42df Signed-off-by: Subrata Banik subratabanik@google.com --- M src/mainboard/google/fatcat/variants/baseboard/fatcat/ramstage.c 1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/87183/1
diff --git a/src/mainboard/google/fatcat/variants/baseboard/fatcat/ramstage.c b/src/mainboard/google/fatcat/variants/baseboard/fatcat/ramstage.c index 21f8ccd..3e6405e4 100644 --- a/src/mainboard/google/fatcat/variants/baseboard/fatcat/ramstage.c +++ b/src/mainboard/google/fatcat/variants/baseboard/fatcat/ramstage.c @@ -52,10 +52,11 @@
void baseboard_devtree_update(void) { +#if 0 /* Don't optimize the power limit if booting with barrel attached */ if (CONFIG(BOARD_GOOGLE_MODEL_FATCAT) && google_chromeec_is_barrel_charger_present()) return; - +#endif if (!google_chromeec_is_battery_present()) variant_update_cpu_power_limits(power_optimized_limits, ARRAY_SIZE(power_optimized_limits));