Various #include file cosmetic fixes.

[libsigrok.git] / src / scpi / scpi_serial.c

diff --git a/src/scpi/scpi_serial.c b/src/scpi/scpi_serial.c
index 4f6e067887c9135298b6512837cdf9f4abb33d09..68a684c2d40fd4758c34f5a6d4b93d29ef76a836 100644 (file)
--- a/src/scpi/scpi_serial.c
+++ b/src/scpi/scpi_serial.c
@@ -18,12 +18,11 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "libsigrok.h"
-#include "libsigrok-internal.h"
-
 #include <glib.h>
 #include <stdlib.h>
 #include <string.h>
+#include "libsigrok.h"
+#include "libsigrok-internal.h"
 
 #define LOG_PREFIX "scpi_serial"
 
@@ -36,7 +35,7 @@ struct scpi_serial {
        size_t read;
 };
 
-static struct {
+static const struct {
        uint16_t vendor_id;
        uint16_t product_id;
        const char *serialcomm;
@@ -54,8 +53,8 @@ static GSList *scpi_serial_scan(struct drv_context *drvc)
        (void)drvc;
 
        for (i = 0; i < ARRAY_SIZE(scpi_serial_usb_ids); i++) {
-               if ((l = sr_serial_find_usb(scpi_serial_usb_ids[i].vendor_id,
-                                           scpi_serial_usb_ids[i].product_id)) == NULL)
+               if (!(l = sr_serial_find_usb(scpi_serial_usb_ids[i].vendor_id,
+                                       scpi_serial_usb_ids[i].product_id)))
                        continue;
                for (r = l; r; r = r->next) {
                        if (scpi_serial_usb_ids[i].serialcomm)
@@ -79,8 +78,7 @@ static int scpi_serial_dev_inst_new(void *priv, struct drv_context *drvc,
        (void)drvc;
        (void)params;
 
-       if (!(sscpi->serial = sr_serial_dev_inst_new(resource, serialcomm)))
-               return SR_ERR;
+       sscpi->serial = sr_serial_dev_inst_new(resource, serialcomm);
 
        return SR_OK;
 }