[mynewt-core] branch master updated: adp5061: mutex should unlock on error (#1263)

[vipulrahane]

This is an automated email from the ASF dual-hosted git repository.

vipulrahane pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git


The following commit(s) were added to refs/heads/master by this push:
     new c3acba6  adp5061: mutex should unlock on error (#1263)
c3acba6 is described below

commit c3acba6d94a6941ebf2200bec7d7470d2ec6a5b1
Author: Vipul Rahane <vrah...@gmail.com>
AuthorDate: Wed Jul 11 22:34:22 2018 -0700

    adp5061: mutex should unlock on error (#1263)
---
 hw/drivers/chg_ctrl/adp5061/src/adp5061.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/hw/drivers/chg_ctrl/adp5061/src/adp5061.c 
b/hw/drivers/chg_ctrl/adp5061/src/adp5061.c
index 03b2d7c..9fe1066 100644
--- a/hw/drivers/chg_ctrl/adp5061/src/adp5061.c
+++ b/hw/drivers/chg_ctrl/adp5061/src/adp5061.c
@@ -217,9 +217,10 @@ adp5061_get_reg(struct adp5061_dev *dev, uint8_t addr, 
uint8_t *value)
             OS_TICKS_PER_SEC / 10, 1);
     *value = payload;
 
-    adp5061_itf_unlock(&dev->a_chg_ctrl.cc_itf);
 
 err:
+    adp5061_itf_unlock(&dev->a_chg_ctrl.cc_itf);
+
     return rc;
 }