new-driver: Logging helper fixes.
authorUwe Hermann <uwe@hermann-uwe.de>
Wed, 24 Oct 2012 10:11:31 +0000 (12:11 +0200)
committerUwe Hermann <uwe@hermann-uwe.de>
Wed, 24 Oct 2012 10:11:31 +0000 (12:11 +0200)
Keep the sr_err() instead of SR_ERR() name in the #defines, otherwise we
conflict with the SR_ERR integer.

source/drv-api.c
source/drv-protocol.h

index 16f5f6d2e74f9a6870e58ef6ddeab2f32f77fa3a..671149bd6fa4230d1d88de29632df177760540e6 100644 (file)
@@ -56,7 +56,7 @@ static int hw_init(void)
        struct drv_context *drvc;
 
        if (!(drvc = g_try_malloc0(sizeof(struct drv_context)))) {
-               SR_ERR("Driver context malloc failed.");
+               sr_err("Driver context malloc failed.");
                return SR_ERR;
        }
 
@@ -121,7 +121,7 @@ static int hw_info_get(int info_id, const void **data,
        switch (info_id) {
        /* TODO */
        default:
-               SR_ERR("Unknown info_id: %d.", info_id);
+               sr_err("Unknown info_id: %d.", info_id);
                return SR_ERR_ARG;
        }
 
@@ -134,7 +134,7 @@ static int hw_dev_config_set(const struct sr_dev_inst *sdi, int hwcap,
        int ret;
 
        if (sdi->status != SR_ST_ACTIVE) {
-               SR_ERR("Device inactive, can't set config options.");
+               sr_err("Device inactive, can't set config options.");
                return SR_ERR;
        }
 
@@ -142,7 +142,7 @@ static int hw_dev_config_set(const struct sr_dev_inst *sdi, int hwcap,
        switch (hwcap) {
        /* TODO */
        default:
-               SR_ERR("Unknown hardware capability: %d.", hwcap);
+               sr_err("Unknown hardware capability: %d.", hwcap);
                ret = SR_ERR_ARG;
        }
 
index 082ecfcac066d5f4daeef6b2a0100180ae107396..ec14a717f6c34b538957323c3a7ba4225a285807 100644 (file)
 
 /* Message logging helpers with driver-specific prefix string. */
 #define DRIVER_LOG_DOMAIN "${short}: "
-#define SR_LOG(l, s, args...) sr_log(l, DRIVER_LOG_DOMAIN s, ## args)
-#define SR_SPEW(s, args...) sr_spew(DRIVER_LOG_DOMAIN s, ## args)
-#define SR_DBG(s, args...) sr_dbg(DRIVER_LOG_DOMAIN s, ## args)
-#define SR_INFO(s, args...) sr_info(DRIVER_LOG_DOMAIN s, ## args)
-#define SR_WARN(s, args...) sr_warn(DRIVER_LOG_DOMAIN s, ## args)
-#define SR_ERR(s, args...) sr_err(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_log(l, s, args...) sr_log(l, DRIVER_LOG_DOMAIN s, ## args)
+#define sr_spew(s, args...) sr_spew(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_dbg(s, args...) sr_dbg(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_info(s, args...) sr_info(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_warn(s, args...) sr_warn(DRIVER_LOG_DOMAIN s, ## args)
+#define sr_err(s, args...) sr_err(DRIVER_LOG_DOMAIN s, ## args)
 
 /** Private, per-device-instance driver context. */
 struct dev_context {