Fix return convention errors

Author: Matthew West

source/common.c

@@ -248,7 +248,6 @@ BBIO_err lookup_uart_by_name(const char *input_name, char *dt)
             return BBIO_OK;
         }
     }
-    fprintf(stderr, "return 0 lookup_uart_by_name");
     return BBIO_INVARG;
 }
 
@@ -347,8 +346,10 @@ BBIO_err get_adc_ain(const char *key, unsigned int *ain)
 
 BBIO_err get_uart_device_tree_name(const char *name, char *dt)
 {
-    if (!lookup_uart_by_name(name, dt)) {
-        return BBIO_INVARG;
+    BBIO_err err;
+    err = lookup_uart_by_name(name, dt);
+    if (err != BBIO_OK) {
+        return err;
     }
 
     return BBIO_OK;

source/py_uart.c

@@ -42,18 +42,21 @@ static PyObject *py_setup_uart(PyObject *self, PyObject *args)
 {
     char dt[FILENAME_BUFFER_SIZE];
     char *channel;
+    BBIO_err err;
 
     if (!PyArg_ParseTuple(args, "s", &channel)) {
         PyErr_SetString(PyExc_ValueError, "Invalid UART channel.");
         return NULL;
     }
 
-    if (!get_uart_device_tree_name(channel, dt)) {
+    err = get_uart_device_tree_name(channel, dt);
+    if (err != BBIO_OK) {
         PyErr_SetString(PyExc_ValueError, "Invalid UART channel.");
         return NULL;
     }
 
-    if (!uart_setup(dt)) {
+    err = uart_setup(dt);
+    if (err != BBIO_OK) {
         PyErr_SetString(PyExc_RuntimeError, "Unable to export UART channel.");
         return NULL;        
     }