Andrea Palmieri / X_NUCLEO_COMMON

Fork of X_NUCLEO_COMMON lib adding 16-bit SPI support

Fork of X_NUCLEO_COMMON by ST

Files at this revision

API Documentation at this revision

Revision 3:bc5fc631e9c5, committed 2015-04-29

Comitter:
Wolfgang Betz
Date:
Wed Apr 29 16:14:03 2015 +0200
Parent:
2:c1cac955d4bd
Child:
4:33ee0cf483de
Commit message:
Added some important safety checks

Changed in this revision

DevI2C/DevI2C.h Show annotated file Show diff for this revision Revisions of this file 
--- a/DevI2C/DevI2C.h	Tue Apr 14 15:32:06 2015 +0200
+++ b/DevI2C/DevI2C.h	Wed Apr 29 16:14:03 2015 +0200
@@ -64,7 +64,9 @@
 	 * @param  RegisterAddr specifies the internal address register 
 	 *         where to start writing to (must be correctly masked).
 	 * @param  NumByteToWrite number of bytes to be written.
-	 * @retval 0 if ok, -1 if an I2C error has occured
+	 * @retval 0 if ok, 
+	 *         -1 if an I2C error has occured, or
+	 *         -2 on temporary buffer overflow (i.e. NumByteToWrite was too high)
 	 * @note   On some devices if NumByteToWrite is greater
 	 *         than one, the RegisterAddr must be masked correctly!
 	 */
@@ -72,8 +74,10 @@
 		      uint16_t NumByteToWrite)
 	{
 		int ret;
-		uint8_t tmp[32];
+		uint8_t tmp[TEMP_BUF_SIZE];
 	
+		if(NumByteToWrite >= TEMP_BUF_SIZE) return -2;
+		
 		/* First, send device address. Then, send data and STOP condition */
 		tmp[0] = RegisterAddr;
 		memcpy(tmp+1, pBuffer, NumByteToWrite);
@@ -110,6 +114,9 @@
 		if(ret) return -1;
 		return 0;
 	}
+
+ private:
+	static const unsigned int TEMP_BUF_SIZE = 32;
 };
 
 #endif /* __DEV_I2C_H */