Remove wspoon P9 VCS Workaround from boot path
The level of hardware that required it is no longer
supported by the hostboot firmware anyway so systems
with those processors cannot boot up.
A future commit will remove the newly unused files.
Change-Id: Ifb92b697bbf3f7bdfc4fa0a1c6e296f7d9ec3ba6
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
diff --git a/meta-witherspoon/conf/machine/witherspoon.conf b/meta-witherspoon/conf/machine/witherspoon.conf
index 4878278..aead549 100644
--- a/meta-witherspoon/conf/machine/witherspoon.conf
+++ b/meta-witherspoon/conf/machine/witherspoon.conf
@@ -7,7 +7,7 @@
OBMC_POWER_SUPPLY_INSTANCES = "0 1"
-PREFERRED_PROVIDER_virtual/p9-vcs-workaround = 'vcs-pre-post'
+PREFERRED_PROVIDER_virtual/p9-vcs-workaround = ''
require conf/machine/include/ast2500.inc
require conf/machine/include/obmc-bsp-common.inc
diff --git a/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service b/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service
index 99ff9e4..b7fb41e 100644
--- a/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service
+++ b/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/avsbus-enable@.service
@@ -2,7 +2,6 @@
Description=Enable the AVS bus on VRMs
Wants=vrm-control@%i.service
After=vrm-control@%i.service
-Before=vcs_off@%i.service
Conflicts=obmc-chassis-poweroff@%i.target
ConditionPathExists=!/run/openbmc/chassis@%i-on
diff --git a/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/power-workarounds@.service b/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/power-workarounds@.service
index 5e44755..74d8e4b 100644
--- a/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/power-workarounds@.service
+++ b/meta-witherspoon/recipes-phosphor/chassis/avsbus-control/power-workarounds@.service
@@ -1,7 +1,7 @@
[Unit]
Description=Perform power workarounds on VRMs
-Wants=ucd_disable_vcs@%i.service
-After=ucd_disable_vcs@%i.service
+Wants=ir35221-on-unbind@%i.service
+After=ir35221-on-unbind@%i.service
Before=avsbus-disable@%i.service
Conflicts=obmc-chassis-poweroff@%i.target
ConditionPathExists=!/run/openbmc/chassis@%i-on
diff --git a/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb b/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
index e2c09e1..e1c8aed 100644
--- a/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
+++ b/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
@@ -30,19 +30,7 @@
INSTFMT_ON_IRUNBIND = "ir35221-on-unbind@{0}.service"
FMT_ON_IRUNBIND = "../${TMPL_ON_IRUNBIND}:${TGTFMT_ON}.requires/${INSTFMT_ON_IRUNBIND}"
-TGTFMT_OFF = "obmc-chassis-poweroff@{0}.target"
-
-TMPL_OFF_IRBIND = "ir35221-off-bind@.service"
-INSTFMT_OFF_IRBIND = "ir35221-off-bind@{0}.service"
-FMT_OFF_IRBIND = "../${TMPL_OFF_IRBIND}:${TGTFMT_OFF}.requires/${INSTFMT_OFF_IRBIND}"
-
-TMPL_OFF_IRUNBIND = "ir35221-off-unbind@.service"
-INSTFMT_OFF_IRUNBIND = "ir35221-off-unbind@{0}.service"
-FMT_OFF_IRUNBIND = "../${TMPL_OFF_IRUNBIND}:${TGTFMT_OFF}.requires/${INSTFMT_OFF_IRUNBIND}"
-
-SYSTEMD_SERVICE_${PN} += "${TMPL} ${TMPL_ON_IRUNBIND} ${TMPL_ON_IRBIND} ${TMPL_OFF_IRUNBIND} ${TMPL_OFF_IRBIND}"
+SYSTEMD_SERVICE_${PN} += "${TMPL} ${TMPL_ON_IRUNBIND} ${TMPL_ON_IRBIND}"
SYSTEMD_LINK_${PN} += "${@compose_list(d, 'FMT_ON', 'OBMC_CHASSIS_INSTANCES')}"
SYSTEMD_LINK_${PN} += "${@compose_list(d, 'FMT_ON_IRBIND', 'OBMC_CHASSIS_INSTANCES')}"
SYSTEMD_LINK_${PN} += "${@compose_list(d, 'FMT_ON_IRUNBIND', 'OBMC_CHASSIS_INSTANCES')}"
-SYSTEMD_LINK_${PN} += "${@compose_list(d, 'FMT_OFF_IRBIND', 'OBMC_CHASSIS_INSTANCES')}"
-SYSTEMD_LINK_${PN} += "${@compose_list(d, 'FMT_OFF_IRUNBIND', 'OBMC_CHASSIS_INSTANCES')}"
diff --git a/meta-witherspoon/recipes-phosphor/chassis/vrm-control/ir35221-on-bind@.service b/meta-witherspoon/recipes-phosphor/chassis/vrm-control/ir35221-on-bind@.service
index 49d1638..e6adc05 100644
--- a/meta-witherspoon/recipes-phosphor/chassis/vrm-control/ir35221-on-bind@.service
+++ b/meta-witherspoon/recipes-phosphor/chassis/vrm-control/ir35221-on-bind@.service
@@ -1,6 +1,6 @@
[Unit]
Description=Power on bind ir35221 device driver devs
-After=vcs_on@%i.service
+After=avsbus-enable@%i.service
Wants=obmc-host-start-pre@%i.target
Before=obmc-host-start-pre@%i.target
Conflicts=obmc-chassis-poweroff@%i.target
diff --git a/meta-witherspoon/recipes-phosphor/host/p9-host-start.bbappend b/meta-witherspoon/recipes-phosphor/host/p9-host-start.bbappend
index ac2013f..4cac490 100644
--- a/meta-witherspoon/recipes-phosphor/host/p9-host-start.bbappend
+++ b/meta-witherspoon/recipes-phosphor/host/p9-host-start.bbappend
@@ -1,2 +1,3 @@
RDEPENDS_${PN} += 'avsbus-control'
RDEPENDS_${PN} += 'vrm-control'
+RDEPENDS_${PN}_remove += "p9-vcs-workaround"
diff --git a/meta-witherspoon/recipes-phosphor/power/witherspoon-pfault-analysis/witherspoon-pseq-monitor.service b/meta-witherspoon/recipes-phosphor/power/witherspoon-pfault-analysis/witherspoon-pseq-monitor.service
index d0ba9a9..1780baa 100644
--- a/meta-witherspoon/recipes-phosphor/power/witherspoon-pfault-analysis/witherspoon-pseq-monitor.service
+++ b/meta-witherspoon/recipes-phosphor/power/witherspoon-pfault-analysis/witherspoon-pseq-monitor.service
@@ -2,8 +2,7 @@
Description=Witherspoon Power Sequencer Runtime Monitor
Wants=obmc-host-start-pre@0.target
Before=obmc-host-start-pre@0.target
-Wants=vcs_on@0.service
-After=vcs_on@0.service
+After=obmc-power-on@0.target
Conflicts=obmc-chassis-poweroff@0.target
[Service]