X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=hardware%2Fopenbench-logic-sniffer%2Fols.c;h=57d7aa0aba31078db7291bf22871458838a5f868;hb=cdbc51d97d027c3889aad43c9d9ed3344cfb5a91;hp=403c2b58aa0ca46c6cb63e0fd4b18b6ced0655ba;hpb=2119ab0364b6a161091a89a7018be14d49bdc7b3;p=libsigrok.git

diff --git a/hardware/openbench-logic-sniffer/ols.c b/hardware/openbench-logic-sniffer/ols.c
index 403c2b58..57d7aa0a 100644
--- a/hardware/openbench-logic-sniffer/ols.c
+++ b/hardware/openbench-logic-sniffer/ols.c
@@ -37,47 +37,49 @@
 #endif
 #include <glib.h>
 #include <sigrok.h>
+#include <sigrok-internal.h>
 
 #ifdef _WIN32
 #define O_NONBLOCK FIONBIO
 #endif
 
-#define NUM_PROBES			32
-#define NUM_TRIGGER_STAGES		4
-#define TRIGGER_TYPES			"01"
-#define SERIAL_SPEED			B115200
-#define CLOCK_RATE			100000000
+#define NUM_PROBES             32
+#define NUM_TRIGGER_STAGES     4
+#define TRIGGER_TYPES          "01"
+#define SERIAL_SPEED           B115200
+#define CLOCK_RATE             MHZ(100)
+#define MIN_NUM_SAMPLES        4
 
 /* Command opcodes */
-#define CMD_RESET			0x00
-#define CMD_ID				0x02
-#define CMD_SET_FLAGS			0x82
-#define CMD_SET_DIVIDER			0x80
-#define CMD_RUN				0x01
-#define CMD_CAPTURE_SIZE		0x81
-#define CMD_SET_TRIGGER_MASK_0		0xc0
-#define CMD_SET_TRIGGER_MASK_1		0xc4
-#define CMD_SET_TRIGGER_MASK_2		0xc8
-#define CMD_SET_TRIGGER_MASK_3		0xcc
-#define CMD_SET_TRIGGER_VALUE_0		0xc1
-#define CMD_SET_TRIGGER_VALUE_1		0xc5
-#define CMD_SET_TRIGGER_VALUE_2		0xc9
-#define CMD_SET_TRIGGER_VALUE_3		0xcd
-#define CMD_SET_TRIGGER_CONFIG_0	0xc2
-#define CMD_SET_TRIGGER_CONFIG_1	0xc6
-#define CMD_SET_TRIGGER_CONFIG_2	0xca
-#define CMD_SET_TRIGGER_CONFIG_3	0xce
+#define CMD_RESET                  0x00
+#define CMD_ID                     0x02
+#define CMD_SET_FLAGS              0x82
+#define CMD_SET_DIVIDER            0x80
+#define CMD_RUN                    0x01
+#define CMD_CAPTURE_SIZE           0x81
+#define CMD_SET_TRIGGER_MASK_0     0xc0
+#define CMD_SET_TRIGGER_MASK_1     0xc4
+#define CMD_SET_TRIGGER_MASK_2     0xc8
+#define CMD_SET_TRIGGER_MASK_3     0xcc
+#define CMD_SET_TRIGGER_VALUE_0    0xc1
+#define CMD_SET_TRIGGER_VALUE_1    0xc5
+#define CMD_SET_TRIGGER_VALUE_2    0xc9
+#define CMD_SET_TRIGGER_VALUE_3    0xcd
+#define CMD_SET_TRIGGER_CONFIG_0   0xc2
+#define CMD_SET_TRIGGER_CONFIG_1   0xc6
+#define CMD_SET_TRIGGER_CONFIG_2   0xca
+#define CMD_SET_TRIGGER_CONFIG_3   0xce
 
 /* Bitmasks for CMD_FLAGS */
-#define FLAG_DEMUX			0x01
-#define FLAG_FILTER			0x02
-#define FLAG_CHANNELGROUP_1		0x04
-#define FLAG_CHANNELGROUP_2		0x08
-#define FLAG_CHANNELGROUP_3		0x10
-#define FLAG_CHANNELGROUP_4		0x20
-#define FLAG_CLOCK_EXTERNAL		0x40
-#define FLAG_CLOCK_INVERTED		0x80
-#define FLAG_RLE			0x0100
+#define FLAG_DEMUX                 0x01
+#define FLAG_FILTER                0x02
+#define FLAG_CHANNELGROUP_1        0x04
+#define FLAG_CHANNELGROUP_2        0x08
+#define FLAG_CHANNELGROUP_3        0x10
+#define FLAG_CHANNELGROUP_4        0x20
+#define FLAG_CLOCK_EXTERNAL        0x40
+#define FLAG_CLOCK_INVERTED        0x80
+#define FLAG_RLE                   0x0100
 
 static int capabilities[] = {
 	HWCAP_LOGIC_ANALYZER,
@@ -461,6 +463,8 @@ static int hw_set_configuration(int device_index, int capability, void *value)
 		break;
 	case HWCAP_LIMIT_SAMPLES:
 		tmp_u64 = value;
+		if (*tmp_u64 < MIN_NUM_SAMPLES)
+			return SIGROK_ERR;
 		limit_samples = *tmp_u64;
 		g_message("ols: sample limit %" PRIu64, limit_samples);
 		ret = SIGROK_OK;
@@ -777,7 +781,7 @@ static void hw_stop_acquisition(int device_index, gpointer session_device_id)
 {
 	struct datafeed_packet packet;
 
-	/* QUICK HACK */
+	/* Avoid compiler warnings. */
 	device_index = device_index;
 
 	packet.type = DF_END;