maa: change the linking of maa and make gpio functions match maa_ spec

Signed-off-by: Brendan Le Foll <brendan.le.foll@intel.com>

src/CMakeLists.txt

@@ -5,9 +5,9 @@ include_directories(
 
 set (maa_LIB_HEADERS
   ${PROJECT_SOURCE_DIR}/api/maa.h
+  ${PROJECT_SOURCE_DIR}/api/gpio.h
   ${PROJECT_SOURCE_DIR}/api/i2c.h
   ${PROJECT_SOURCE_DIR}/api/pwm.h
-  ${PROJECT_SOURCE_DIR}/api/gpio.h
   ${PROJECT_SOURCE_DIR}/include/smbus.hpp
   ${PROJECT_SOURCE_DIR}/include/smbus.h
 )
@@ -21,9 +21,9 @@ set (maa_LIB_KERNEL
 
 set (maa_LIB_SRCS
   ${PROJECT_SOURCE_DIR}/src/maa.c
+  ${PROJECT_SOURCE_DIR}/src/gpio/gpio.c
   ${PROJECT_SOURCE_DIR}/src/i2c/i2c.c
   ${PROJECT_SOURCE_DIR}/src/i2c/smbus.c
-  ${PROJECT_SOURCE_DIR}/src/gpio/gpio.c
   ${PROJECT_SOURCE_DIR}/src/pwm/pwm.c
 )
 

src/gpio/gpio.c

@@ -29,60 +29,57 @@
 
 #include "gpio.h"
 
-#ifdef __cplusplus
-extern "C" {
-#endif
-
 static int
-gpio_get_valfp(gpio_t *gpio)
+maa_gpio_get_valfp(gpio_t *gpio)
 {
     char bu[64];
     sprintf(bu, "/sys/class/gpio/gpio%d/value", gpio->pin);
 
-    if((gpio->value_fp = fopen(bu, "r+b")) == NULL) {
+    if ((gpio->value_fp = fopen(bu, "r+b")) == NULL) {
         return 1;
     }
     return 0;
 }
 
-void
-gpio_init(gpio_t *gpio, int pin)
+maa_result_t
+maa_gpio_init(gpio_t *gpio, int pin)
 {
     FILE *export_f;
 
-    if((export_f = fopen("/sys/class/gpio/export", "w")) == NULL) {
+    if ((export_f = fopen("/sys/class/gpio/export", "w")) == NULL) {
         fprintf(stderr, "Failed to open export for writing!\n");
     } else {
         fprintf(export_f, "%d", pin);
         fclose(export_f);
     }
     gpio->pin = pin;
+    return MAA_SUCCESS;
 }
 
 int
-gpio_set(int pin)
+maa_gpio_set(int pin)
 {
     //Stuff
     return 0;
 }
 
 void
-gpio_mode(gpio_t *gpio, gpio_mode_t mode)
+maa_gpio_mode(gpio_t *gpio, gpio_mode_t mode)
 {
     //gpio->pin
 }
 
 void
-gpio_dir(gpio_t *gpio, gpio_dir_t dir)
+maa_gpio_dir(gpio_t *gpio, gpio_dir_t dir)
 {
-    if(gpio->value_fp != NULL) {
+    if (gpio->value_fp != NULL) {
          gpio->value_fp = NULL;
     }
     char filepath[64];
     snprintf(filepath, 64, "/sys/class/gpio/gpio%d/direction", gpio->pin);
 
     FILE *direction;
-    if((direction = fopen(filepath, "w")) == NULL) {
+    if ((direction = fopen(filepath, "w")) == NULL) {
         fprintf(stderr, "Failed to open direction for writing!\n");
     } else {
         fprintf(direction, dir);
@@ -92,10 +89,10 @@ gpio_dir(gpio_t *gpio, gpio_dir_t dir)
 }
 
 int
-gpio_read(gpio_t *gpio)
+maa_gpio_read(gpio_t *gpio)
 {
-    if(gpio->value_fp == NULL) {
-        gpio_get_valfp(gpio);
+    if (gpio->value_fp == NULL) {
+        maa_gpio_get_valfp(gpio);
     }
     fseek(gpio->value_fp, SEEK_SET, 0);
     char buffer[2];
@@ -105,10 +102,10 @@ gpio_read(gpio_t *gpio)
 }
 
 void
-gpio_write(gpio_t *gpio, int value)
+maa_gpio_write(gpio_t *gpio, int value)
 {
-    if(gpio->value_fp == NULL) {
-        gpio_get_valfp(gpio);
+    if (gpio->value_fp == NULL) {
+        maa_gpio_get_valfp(gpio);
     }
     fseek(gpio->value_fp, SEEK_SET, 0);
     fprintf(gpio->value_fp, "%d", value);
@@ -117,18 +114,14 @@ gpio_write(gpio_t *gpio, int value)
 }
 
 void
-gpio_close(gpio_t *gpio)
+maa_gpio_close(gpio_t *gpio)
 {
     FILE *unexport_f;
 
-    if((unexport_f = fopen("/sys/class/gpio/unexport", "w")) == NULL) {
+    if ((unexport_f = fopen("/sys/class/gpio/unexport", "w")) == NULL) {
         fprintf(stderr, "Failed to open unexport for writing!\n");
     } else {
         fprintf(unexport_f, "%d", gpio->pin);
         fclose(unexport_f);
     }
 }
-
-#ifdef __cplusplus
-}
-#endif

src/i2c/i2c.c

@@ -25,16 +25,20 @@
 #include "i2c.h"
 #include "smbus.h"
 
-int
+maa_result_t
 maa_i2c_init(i2c_t* dev)
 {
     // maa allocates the memory for *dev
     dev = malloc(sizeof *dev);
+    if (!dev)
+        return MAA_ERROR_NO_RESOURCES;
+
     // Galileo only has one I2C master which should be /dev/i2c-0
     // reliability is a fickle friend!
     if ((dev->fh = open("/dev/i2c-0", O_RDWR)) < 1) {
         fprintf(stderr, "Failed to open requested i2c port");
     }
+    return MAA_SUCCESS;
 }
 
 void

src/maa.c

@@ -24,7 +24,7 @@
 
 #include "maa.h"
 
-int
+maa_result_t
 maa_get_version()
 {
     return MAA_LIBRARY_VERSION;

src/maa.i

@@ -1,8 +1,11 @@
 %{
     #include "maa.h"
+    #include "gpio.h"
+    #include "pwm.h"
+    #include "i2c.h"
 %}
 
 %include "maa.h"
-%include "i2c.h"
 %include "gpio.h"
 %include "pwm.h"
+%include "i2c.h"

src/pwm/pwm.c

@@ -91,10 +91,13 @@ maa_pwm_get_duty(pwm_t* dev)
     fseek(dev->duty_fp, SEEK_SET, 0);
     return atoi(output);
 }
-int
+
+maa_result_t
 maa_pwm_init(pwm_t* dev, int chipin, int pin)
 {
     dev = malloc(sizeof *dev);
+    if (!dev)
+        return MAA_ERROR_NO_RESOURCES;
     dev->chipid = chipin;
     dev->pin = pin;
 
@@ -104,13 +107,14 @@ maa_pwm_init(pwm_t* dev, int chipin, int pin)
 
     if ((export_f = fopen(buffer, "w")) == NULL) {
         fprintf(stderr, "Failed to open export for writing!\n");
+	return MAA_ERROR_INVALID_HANDLE;
     } else {
         fprintf(export_f, "%d", dev->pin);
         fclose(export_f);
         maa_pwm_setup_duty_fp(dev);
     }
 
-    return 0;
+    return MAA_SUCCESS;
 }
 
 void