Created attachment 757 [details]
sigrok-cli log file

With our new Saleae Logic Pro 16 (which, just for the record, works perfectly with the official Saleae software), I run into a timeout after initialization, and before the first data is captured.

I could reproduce this both on an Arch linux x64 machine (git snapshot) and a Raspi 4 (libsigrok 0.5.1). The bug report is based on the data from the Arch machine.

libsigrok commit id: fb28e72dfdea0d985e03795f1f058aa4621e7e55
sigrok-cli commit id: 7ce834c1166fcf57ebb0e20bf4e7ba9e856aa6ce

I called sigrok-cli as:
$ LD_LIBRARY_PATH=../libsigrok/.libs ./sigrok-cli -l 5 --driver saleae-logic-pro --continuous

I added some debug output to better track down the issue:
--- cut ---
it diff
diff --git a/src/hardware/saleae-logic-pro/api.c b/src/hardware/saleae-logic-pro/api.c
index 0a2a2728..84afb914 100644
--- a/src/hardware/saleae-logic-pro/api.c
+++ b/src/hardware/saleae-logic-pro/api.c
@@ -383,12 +383,16 @@ static int dev_acquisition_handle(int fd, int revents, void *cb_data)
        struct timeval tv = ALL_ZERO;
 
        (void)fd;
+       sr_dbg("saleae-logic-pro: dev_acquisition_handle()");
 
        libusb_handle_events_timeout(drvc->sr_ctx->libusb_ctx, &tv);
 
        /* Handle timeout */
        if (!revents)
+       {
+               sr_err("saleae-logic-pro: stopping acquisition");
                sr_dev_acquisition_stop(sdi);
+       }
 
        return TRUE;
 }
--- cut ---

The important part of the attached log file is
--- cut ---
cli: Received SR_DF_HEADER.
sr: [00:00.489610] hwdriver: sr_config_get(): key 30000 (samplerate) sdi 0x561e5be9e530 cg NULL -> uint64 10000000
sr: [00:00.489749] saleae-logic-pro: Iterate 0x56931bfc -> 0x14aa0701
sr: [00:00.490221] saleae-logic-pro: Iterate 0x14aa0701 -> 0x493f55c1
Press any key to stop acquisition.
sr: [00:01.491614] saleae-logic-pro: saleae-logic-pro: dev_acquisition_handle()
sr: [00:01.491721] saleae-logic-pro: saleae-logic-pro: stopping acquisition
sr: [00:01.491745] hwdriver: saleae-logic-pro: Stopping acquisition.
--- cut ---

so apparently, dev_acquisition_handle() is called with revents=0, which makes the acquisition fail. Note that this is the first time the function is called.