Open device before adding it to the session

diff --git a/sigrok-cli.c b/sigrok-cli.c
index a37b19d515890f4fd94b81d60e00ed4dba7dbd8a..ade15cd31f137b00226f80fcbd857d7dc6bcfa69 100644 (file)
--- a/sigrok-cli.c
+++ b/sigrok-cli.c
@@ -351,11 +351,8 @@ static void show_dev_detail(void)
        sdi = devices->data;
        print_dev_line(sdi);
 
        sdi = devices->data;
        print_dev_line(sdi);
 

-       /* This properly opens and initializes the device, so we can get
-        * current settings. */
-       sr_session_new();
-       if (sr_session_dev_add(sdi) != SR_OK) {
-               g_critical("Failed to use device.");
+       if (sr_dev_open(sdi) != SR_OK) {
+               g_critical("Failed to open device.");

                return;
        }
 
                return;
        }
 


@@ -563,7 +560,7 @@ static void show_dev_detail(void)
        }
        g_variant_unref(gvar_opts);
 
        }
        g_variant_unref(gvar_opts);
 

-       sr_session_destroy();
+       sr_dev_close(sdi);

 
 }
 
 
 }
 


@@ -1531,15 +1528,14 @@ static void set_options(void)
        }
        sdi = devices->data;
 
        }
        sdi = devices->data;
 

-       sr_session_new();
-       if (sr_session_dev_add(sdi) != SR_OK) {
-               g_critical("Failed to use device.");
+       if (sr_dev_open(sdi) != SR_OK) {
+               g_critical("Failed to open device.");

                return;
        }
 
        set_dev_options(sdi, devargs);
 
                return;
        }
 
        set_dev_options(sdi, devargs);
 

-       sr_session_destroy();
+       sr_dev_close(sdi);

        g_slist_free(devices);
        g_hash_table_destroy(devargs);
 
        g_slist_free(devices);
        g_hash_table_destroy(devargs);
 


@@ -1608,8 +1604,13 @@ static void run_session(void)
        sr_session_new();
        sr_session_datafeed_callback_add(datafeed_in, NULL);
 
        sr_session_new();
        sr_session_datafeed_callback_add(datafeed_in, NULL);
 

+       if (sr_dev_open(sdi) != SR_OK) {
+               g_critical("Failed to open device.");
+               return;
+       }
+

        if (sr_session_dev_add(sdi) != SR_OK) {
        if (sr_session_dev_add(sdi) != SR_OK) {

-               g_critical("Failed to use device.");
+               g_critical("Failed to add device to session.");

                sr_session_destroy();
                return;
        }
                sr_session_destroy();
                return;
        }