dev_io/dev_io.c: fix declaration of unused client_cfg

The client_cfg variable was unused in some cases (e.g., not AFCv3 board or not with DEVIO_CFG). This caused an error on compilation for legacy boards.

dev_io/dev_io.c: fix declaration of unused client_cfg
The client_cfg variable was unused in some cases (e.g., not AFCv3 board or not with DEVIO_CFG). This caused an error on compilation for legacy boards.
6b51cb07 · Lucas Russo · 7a8e4912 · 6b51cb07
Commit 6b51cb07 authored Feb 01, 2016 by Lucas Russo
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

dev_io.c src/dev_io/dev_io.c +4 -2

No files found.
--- a/src/dev_io/dev_io.c
+++ b/src/dev_io/dev_io.c
@@ -261,7 +261,6 @@ int main (int argc, char *argv[])

    /* Spawn the Configure DEVIO to get the uTCA slot number. This is only
     * available in AFCv3 */
-    bpm_client_t *client_cfg = NULL;
 #if defined (__BOARD_AFCV3__) && (__WITH_DEVIO_CFG__)
    int child_devio_cfg_pid = 0;
    if (llio_type == PCIE_DEV) {
@@ -286,6 +285,7 @@ int main (int argc, char *argv[])

    /* FE DEVIO is expected to have a correct dev_id. So, we don't need to get it
     * from Hardware */
+    bpm_client_t *client_cfg = NULL;
    if (devio_type == BE_DEVIO) {
        /* At this point, the Config DEVIO is ready to receive our commands */
        char devio_config_service_str [DEVIO_SERVICE_LEN];
@@ -456,7 +456,9 @@ err_devio_log_filename_alloc:
    devio_destroy (&devio);
 err_card_slot:
 #if defined (__BOARD_AFCV3__) && (__WITH_DEVIO_CFG__)
-    bpm_client_destroy (&client_cfg);
+    if (client_cfg != NULL) {
+        bpm_client_destroy (&client_cfg);
+    }
 err_client_cfg:
    /* Kill DEVIO Config process */
    kill (child_devio_cfg_pid, DEVIO_KILL_CFG_SIGNAL);