summaryrefslogtreecommitdiffstats
path: root/meta-openbmc-machines/meta-openpower
diff options
context:
space:
mode:
authorMatt Spinler <spinler@us.ibm.com>2018-05-25 12:34:32 -0500
committerBrad Bishop <bradleyb@fuzziesquirrel.com>2018-05-30 18:21:40 +0000
commitdd8cfa6859a1dff71e39bc88e3989772454b4af5 (patch)
tree416ad3d26d0b8e389fe0da1a52467131e2228a87 /meta-openbmc-machines/meta-openpower
parent713d1c77229ba667d5e299e8263adc06778c0795 (diff)
downloadtalos-openbmc-dd8cfa6859a1dff71e39bc88e3989772454b4af5.tar.gz
talos-openbmc-dd8cfa6859a1dff71e39bc88e3989772454b4af5.zip
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>
Diffstat (limited to 'meta-openbmc-machines/meta-openpower')
-rw-r--r--meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb2
1 files changed, 1 insertions, 1 deletions
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 e1c8aedb7..d05e67fb9 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 @@ FMT_ON = "../${TMPL}:${TGTFMT_ON}.requires/${INSTFMT}"
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"
OpenPOWER on IntegriCloud