From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37247) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wlnf2-0006pp-Qi for qemu-devel@nongnu.org; Sat, 17 May 2014 19:05:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wlnet-00046T-Os for qemu-devel@nongnu.org; Sat, 17 May 2014 19:05:00 -0400 Received: from mail-we0-x22f.google.com ([2a00:1450:400c:c03::22f]:39114) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wlnet-00046F-Iy for qemu-devel@nongnu.org; Sat, 17 May 2014 19:04:51 -0400 Received: by mail-we0-f175.google.com with SMTP id t61so4070789wes.34 for ; Sat, 17 May 2014 16:04:50 -0700 (PDT) From: =?UTF-8?q?Marc=20Mar=C3=AD?= Date: Sun, 18 May 2014 01:03:41 +0200 Message-Id: <1400367823-32610-15-git-send-email-marc.mari.barcelo@gmail.com> In-Reply-To: <1400367823-32610-1-git-send-email-marc.mari.barcelo@gmail.com> References: <1400367823-32610-1-git-send-email-marc.mari.barcelo@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH v3 14/16] lan9118: Convert debug printfs to QEMU_DPRINTF List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?q?Marc=20Mar=C3=AD?= , Stefan Hajnoczi , Peter Crosthwaite , =?UTF-8?q?Andreas=20F=C3=A4rber?= Modify debug macros to have the same format through the codebase and use regular ifs instead of ifdef. As the debug printf is always put in code, some formats had to be changed to avoid warnings treated as errors at compile time. Signed-off-by: Marc Marí --- hw/net/lan9118.c | 29 +++++++++++++++++++---------- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git a/hw/net/lan9118.c b/hw/net/lan9118.c index e528290..1781b16 100644 --- a/hw/net/lan9118.c +++ b/hw/net/lan9118.c @@ -21,16 +21,23 @@ //#define DEBUG_LAN9118 #ifdef DEBUG_LAN9118 -#define DPRINTF(fmt, ...) \ -do { printf("lan9118: " fmt , ## __VA_ARGS__); } while (0) -#define BADF(fmt, ...) \ -do { hw_error("lan9118: error: " fmt , ## __VA_ARGS__);} while (0) +#define DEBUG_LAN9118_ENABLED 1 #else -#define DPRINTF(fmt, ...) do {} while(0) -#define BADF(fmt, ...) \ -do { fprintf(stderr, "lan9118: error: " fmt , ## __VA_ARGS__);} while (0) +#define DEBUG_LAN9118_ENABLED 0 #endif +#define DPRINTF(fmt, ...) \ + QEMU_DPRINTF(DEBUG_LAN9118_ENABLED, "lan9118", fmt, ## __VA_ARGS__) + +#define BADF(fmt, ...) \ + do { \ + if(DEBUG_LAN9118_ENABLED) { \ + hw_error("lan9118: error: " fmt , ## __VA_ARGS__); \ + } else { \ + fprintf(stderr, "lan9118: error: " fmt , ## __VA_ARGS__); \ + } \ + } while (0) + #define CSR_ID_REV 0x50 #define CSR_IRQ_CFG 0x54 #define CSR_INT_STS 0x58 @@ -1031,7 +1038,7 @@ static void lan9118_writel(void *opaque, hwaddr offset, s->int_sts |= val & SW_INT; break; case CSR_FIFO_INT: - DPRINTF("FIFO INT levels %08x\n", val); + DPRINTF("FIFO INT levels %08" PRIx64 "\n", val); s->fifo_int = val; break; case CSR_RX_CFG: @@ -1112,9 +1119,11 @@ static void lan9118_writel(void *opaque, hwaddr offset, if (val & 0x80000000) { if (val & 0x40000000) { s->mac_data = do_mac_read(s, val & 0xf); - DPRINTF("MAC read %d = 0x%08x\n", val & 0xf, s->mac_data); + DPRINTF("MAC read %" PRId64 " = 0x%08x\n", + (val & 0xf), s->mac_data); } else { - DPRINTF("MAC write %d = 0x%08x\n", val & 0xf, s->mac_data); + DPRINTF("MAC write %" PRId64 " = 0x%08x\n", + (val & 0xf), s->mac_data); do_mac_write(s, val & 0xf, s->mac_data); } } -- 1.7.10.4