Mercurial > ecos

changeset 3148:d5c8fd9bf282

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fix compiler warning, used uninitialized variable in a2fxxx_spi_set_ss().
author vae
date Sun, 01 Apr 2012 19:36:17 +0000
parents 18fc84cf7262
children 3d06557fc66c
files packages/devs/spi/cortexm/a2fxxx/current/ChangeLog packages/devs/spi/cortexm/a2fxxx/current/src/spi_a2fxxx.c
diffstat 2 files changed, 11 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/packages/devs/spi/cortexm/a2fxxx/current/ChangeLog
+++ b/packages/devs/spi/cortexm/a2fxxx/current/ChangeLog
@@ -1,10 +1,14 @@
+2012-01-04  Christophe Coutand  <ecos@hotmail.co.uk>
+	* src/spi_a2fxxx.c:
+	Fix compiler warning, used uninitialized variable in a2fxxx_spi_set_ss().
+
 2011-04-13  Christophe Coutand  <ecos@hotmail.co.uk>
 
-    * cdl/spi_a2fxxx.cdl:
-    * include/spi_a2fxxx.h:
-    * src/spi_a2fxxx.c:
-    New package -- Smartfusion Cortex-M3 SPI driver package.
-    [Bugzilla 1001291]
+	* cdl/spi_a2fxxx.cdl:
+	* include/spi_a2fxxx.h:
+	* src/spi_a2fxxx.c:
+	New package -- Smartfusion Cortex-M3 SPI driver package.
+	[Bugzilla 1001291]
 
 //===========================================================================
 // ####GPLCOPYRIGHTBEGIN####                                                
--- a/packages/devs/spi/cortexm/a2fxxx/current/src/spi_a2fxxx.c
+++ b/packages/devs/spi/cortexm/a2fxxx/current/src/spi_a2fxxx.c
@@ -310,6 +310,7 @@ static inline void a2fxxx_spi_set_ss
             ss = CYGHWR_HAL_A2FXXX_SPI0_SS3;
         } else {
             CYG_ASSERT (ss < 4, "SPI : SPI0 SS out of range.");
+            return;
         }
     }
     else {
@@ -331,6 +332,7 @@ static inline void a2fxxx_spi_set_ss
             ss = CYGHWR_HAL_A2FXXX_SPI1_SS7;
         } else {
             CYG_ASSERT (ss < 8, "SPI : SPI1 SS out of range.");
+            return;
         }
     }