romulus: Remove avbus-workaround
This was a workaround for DD1 (pre-production) hardware.
Jordan Keuseman <jkeusema@us.ibm.com> wrote:
> That was an issue back in DD1 time frame where the VRM needed to
> change their signal voltage thresholds for AVS bus to work around a
> problem where the processor was putting out a runt pulse in the final
> bit of the transaction. They fixed that in the silicon so we got rid
> of the work around.
This was removed from Witherspoon when dropping DD1 support from
OpenBMC. We can remove it from Romulus and other platforms too.
(From meta-ibm rev: baedc91099960c0e582a98e465e190ff9c3aa9cc)
Change-Id: I8266c154505d3b4fb268cd7c44aa84f5a4be0301
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
diff --git a/meta-ibm/meta-romulus/recipes-phosphor/chassis/avsbus-control/avsbus-disable@.service b/meta-ibm/meta-romulus/recipes-phosphor/chassis/avsbus-control/avsbus-disable@.service
index 424aa23..38c1e85 100644
--- a/meta-ibm/meta-romulus/recipes-phosphor/chassis/avsbus-control/avsbus-disable@.service
+++ b/meta-ibm/meta-romulus/recipes-phosphor/chassis/avsbus-control/avsbus-disable@.service
@@ -1,7 +1,7 @@
[Unit]
Description=Disable the AVS bus on the VRMs
-Wants=avsbus-workaround@%i.service
-After=avsbus-workaround@%i.service
+Wants=op-power-start@.service
+After=op-power-start@.service
Before=vrm-control@%i.service
Conflicts=obmc-chassis-poweroff@%i.target
ConditionPathExists=!/run/openbmc/chassis@%i-on