From daf1fd4e605304c233d1bdb382d60ea5cc4e3110 Mon Sep 17 00:00:00 2001 From: Brandon Bergren Date: Tue, 8 May 2018 18:48:11 +0000 Subject: [PATCH] Fix power sequencing on Talos II. We need to wait for PGOOD before attempting to run the AVSBus services. Additionally, we want to hold off on reaching the obmc-power-start target until the AVSBus is set up properly, to keep things like fan control from interfering. By declaring these dependencies explicitly, the power sequencing on the BMC side should no longer be sensitive to the time it takes for the FPGA to turn on the voltage regulators, and it should avoid attempting to program the CPU VRMs before they are ready to be programmed. Signed-off-by: Brandon Bergren --- .../recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service | 2 ++ .../recipes-phosphor/chassis/avsbus-control/avsbus-workaround@.service | 2 ++ 2 files changed, 4 insertions(+) diff --git a/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service b/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service index b7fb41e2..4b0b19e8 100644 --- a/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service +++ b/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service @@ -2,6 +2,8 @@ Description=Enable the AVS bus on VRMs Wants=vrm-control@%i.service After=vrm-control@%i.service +Wants=obmc-power-start@%i.target +Before=obmc-power-start@%i.target Conflicts=obmc-chassis-poweroff@%i.target ConditionPathExists=!/run/openbmc/chassis@%i-on diff --git a/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-workaround@.service b/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-workaround@.service index 156ef03c..e7e00c04 100644 --- a/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-workaround@.service +++ b/meta-openbmc-machines/meta-openpower/meta-rcs/meta-talos/recipes-phosphor/chassis/avsbus-control/avsbus-workaround@.service @@ -2,6 +2,8 @@ Description=Perform AVS bus workaround on VRMs Wants=op-power-start@.service After=op-power-start@.service +Wants=op-wait-power-on@%i.service +After=op-wait-power-on@%i.service Before=avsbus-disable@%i.service Conflicts=obmc-chassis-poweroff@%i.target ConditionPathExists=!/run/openbmc/chassis@%i-on -- 2.11.0