wspoon: Change ir35221-on-bind.service dependency
Make this service have a .wants dependency to the
obmc-chassis-poweron@.target instead of .requires. This way
if it fails, the power on will still continue, just without
the failing drivers bound to their devices.
The drivers are only used for surfacing hwmon temps and voltages.
The only ones of those actually used by code are the water
cooled witherspoon VDD VRM temperatures which are used by fan
control, and fan control can deal with missing VRM temps.
Tested: Inject a fail into the bind script, and ensure the service
fails but the power on still continues.
Resolves openbmc/openbmc#3205
Change-Id: Id7551b8dd9e36c105f11e91697035d7ca90bdec3
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
diff --git a/meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb b/meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
index e1c8aed..d05e67f 100644
--- a/meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
+++ b/meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb
@@ -24,7 +24,7 @@
TMPL_ON_IRBIND = "ir35221-on-bind@.service"
INSTFMT_ON_IRBIND = "ir35221-on-bind@{0}.service"
-FMT_ON_IRBIND = "../${TMPL_ON_IRBIND}:${TGTFMT_ON}.requires/${INSTFMT_ON_IRBIND}"
+FMT_ON_IRBIND = "../${TMPL_ON_IRBIND}:${TGTFMT_ON}.wants/${INSTFMT_ON_IRBIND}"
TMPL_ON_IRUNBIND = "ir35221-on-unbind@.service"
INSTFMT_ON_IRUNBIND = "ir35221-on-unbind@{0}.service"