From 57fb5e703ba8ab824d1849d7436abd64684caf20 Mon Sep 17 00:00:00 2001
From: barthess <barthess@35acf78f-673a-0410-8e92-d51de3d6d3f4>
Date: Sat, 2 Apr 2011 09:33:46 +0000
Subject: I2C. Additional locks added to avoiding system hangups. Some mistypes
 in comments fixed.

git-svn-id: svn://svn.code.sf.net/p/chibios/svn/branches/i2c_dev@2865 35acf78f-673a-0410-8e92-d51de3d6d3f4
---
 os/hal/platforms/STM32/i2c_lld.c | 4 ++--
 os/hal/src/i2c.c                 | 9 ++++++++-
 2 files changed, 10 insertions(+), 3 deletions(-)

(limited to 'os/hal')

diff --git a/os/hal/platforms/STM32/i2c_lld.c b/os/hal/platforms/STM32/i2c_lld.c
index 9dc3a66bc..1ac7e4309 100644
--- a/os/hal/platforms/STM32/i2c_lld.c
+++ b/os/hal/platforms/STM32/i2c_lld.c
@@ -215,7 +215,7 @@ static void i2c_serve_event_interrupt(I2CDriver *i2cp) {
 
     if (i2cp->id_slave_config->id_callback != NULL)
       i2cp->id_slave_config->id_callback(i2cp, i2cp->id_slave_config);
-    else /* No callback function set - generate stop */
+    else /* No callback function set. Generate stop */
       i2c_lld_master_stop(i2cp);
 
     return;
@@ -497,7 +497,7 @@ void i2c_lld_master_start(I2CDriver *i2cp){
   while (i2cp->id_i2c->CR1 & I2C_CR1_START);
 
   /* enable interrupts from I2C hardware. They will disable in driver state
-  machine after the tranafer finish.*/
+  machine after the transfer finish.*/
   i2cp->id_i2c->CR2 |= I2C_CR2_ITEVTEN | I2C_CR2_ITBUFEN;
 }
 
diff --git a/os/hal/src/i2c.c b/os/hal/src/i2c.c
index 11d4fccfa..ad9a5d0ac 100644
--- a/os/hal/src/i2c.c
+++ b/os/hal/src/i2c.c
@@ -138,7 +138,9 @@ void i2cMasterStart(I2CDriver *i2cp){
 
   chDbgCheck((i2cp != NULL), "i2cMasterTransmit");
 
+  chSysLock();
   i2c_lld_master_start(i2cp);
+  chSysUnlock();
 }
 
 /**
@@ -149,8 +151,9 @@ void i2cMasterStart(I2CDriver *i2cp){
 void i2cMasterStop(I2CDriver *i2cp){
 
   chDbgCheck((i2cp != NULL), "i2cMasterTransmit");
-
+  chSysLock();
   i2c_lld_master_stop(i2cp);
+  chSysUnlock();
 }
 
 /**
@@ -168,7 +171,9 @@ void i2cMasterTransmit(I2CDriver *i2cp, I2CSlaveConfig *i2cscfg) {
               "i2cMasterTransmit(), #1",
               "not active");
 
+  chSysLock();
   i2c_lld_master_transmit(i2cp, i2cscfg);
+  chSysUnlock();
 }
 
 
@@ -186,7 +191,9 @@ void i2cMasterReceive(I2CDriver *i2cp, I2CSlaveConfig *i2cscfg) {
               "i2cMasterReceive(), #1",
               "not active");
 
+  chSysLock();
   i2c_lld_master_receive(i2cp, i2cscfg);
+  chSysUnlock();
 }
 
 
-- 
cgit v1.2.3