convert fprintf() to syslog() for #178

Author: pdp7

source/common.c

@@ -554,6 +554,7 @@ int uboot_overlay_enabled(void) {
     file = popen(cmd, "r");
     if (file == NULL) {
        fprintf(stderr, "error: uboot_overlay_enabled() failed to run cmd=%s\n", cmd);
+       syslog(LOG_ERR, "libadafruit-bbio: error: uboot_overlay_enabled() failed to run cmd=%s\n", cmd);
        return -1;
     }
     uboot_overlay = fgetc(file);
@@ -577,6 +578,7 @@ int pocketbeagle(void) {
     file = popen(cmd, "r");
     if (file == NULL) {
        fprintf(stderr, "error: pocketbeagle() failed to run cmd=%s\n", cmd);
+       syslog(LOG_ERR, "libadafruit-bbio: error: pocketbeagle() failed to run cmd=%s\n", cmd);
        return -1;
     }
     pocketbeagle = fgetc(file);
@@ -604,6 +606,7 @@ int beaglebone_blue(void) {
     file = popen(cmd, "r");
     if (file == NULL) {
        fprintf(stderr, "error: beaglebone_blue() failed to run cmd=%s\n", cmd);
+       syslog(LOG_ERR, "libadafruit-bbio: error: beaglebone_blue() failed to run cmd=%s\n", cmd);
        return -1;
     }
     blue = fgetc(file);

source/event_gpio.c

@@ -196,7 +196,7 @@ int open_value_file(unsigned int gpio)
     if ((gpio >= USR_LED_GPIO_MIN) && (gpio <=  USR_LED_GPIO_MAX)) {
         snprintf(filename, sizeof(filename), "/sys/class/leds/beaglebone:green:usr%d/brightness", gpio -  USR_LED_GPIO_MIN);
     } else if (beaglebone_blue()) {
-        fprintf(stderr, "gpio open_value_file: beaglebone_blue() is true\n");
+        syslog(LOG_DEBUG, "libadafruit-bbio: gpio open_value_file: beaglebone_blue() is true\n");
         switch(gpio) {
             case USR_LED_RED:
                 snprintf(filename, sizeof(filename), "/sys/class/leds/red/brightness");
@@ -224,10 +224,10 @@ int open_value_file(unsigned int gpio)
                 break;
         }
     } else {
-        fprintf(stderr, "gpio open_value_file: default gpio path\n");
+        syslog(LOG_DEBUG, "libadafruit-bbio: gpio open_value_file: default gpio path\n");
         snprintf(filename, sizeof(filename), "/sys/class/gpio/gpio%d/value", gpio);
     }
-    fprintf(stderr, "gpio open_value_file: filename=%s\n", filename);
+    syslog(LOG_DEBUG, "libadafruit-bbio: gpio open_value_file: filename=%s\n", filename);
 
     // if(gpio == USR_LED_RED) {     // red LED
     //     snprintf(filename, sizeof(filename), "/sys/class/leds/red/brightness");
@@ -302,7 +302,6 @@ BBIO_err gpio_set_direction(unsigned int gpio, unsigned int in_flag)
              )
            )
         {
-            fprintf(stderr, "gpio_set_direction: %u not applicable to the USR LED\n", gpio);
             syslog(LOG_DEBUG, "gpio_set_direction: %u not applicable to the USR LED", gpio);
             return BBIO_OK; // direction is not applicable to the USR LED pins
         }
@@ -370,25 +369,6 @@ BBIO_err gpio_set_value(unsigned int gpio, unsigned int value)
     char filename[MAX_FILENAME];
     char vstr[10];
 
-    if(uboot_overlay_enabled()) {
-      //syslog(LOG_DEBUG, "libadafruit-bbio: uboot_overlay_enabled() is true");
-      fprintf(stderr, "gpio_set_value: uboot_overlay_enabled() is true\n");
-    } else {   
-      fprintf(stderr, "gpio_set_value: uboot_overlay_enabled() is FASLE\n");
-    }
-
-    if(pocketbeagle()) {
-      fprintf(stderr, "gpio_set_value: pocketbeagle() is true\n");
-    } else {   
-      fprintf(stderr, "gpio_set_value: pocketbeagle() is FASLE\n");
-    }
-
-    if(beaglebone_blue()) {
-      fprintf(stderr, "gpio_set_value: beaglebone_blue() is true\n");
-    } else {
-      fprintf(stderr, "gpio_set_value: beaglebone_blue() is FALSE\n");
-    }
-
     if ((gpio >= USR_LED_GPIO_MIN) && (gpio <=  USR_LED_GPIO_MAX)) {
 
         char *usr_led_trigger[] = { "heartbeat", "mmc0", "cpu0", "mmc1" };