From dd8cfa6859a1dff71e39bc88e3989772454b4af5 Mon Sep 17 00:00:00 2001 From: Matt Spinler Date: Fri, 25 May 2018 12:34:32 -0500 Subject: 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 --- .../meta-ibm/meta-witherspoon/recipes-phosphor/chassis/vrm-control.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor') 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" -- cgit v1.2.3