[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 8/8] i.MX: Standardize i.MX GPT debug
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH v3 8/8] i.MX: Standardize i.MX GPT debug |
Date: |
Sat, 24 Oct 2015 18:58:31 -0700 |
On Sat, Oct 24, 2015 at 1:15 PM, Jean-Christophe Dubois
<address@hidden> wrote:
> The goal is to have debug code always compiled during build.
>
> We standardize all debug output on the following format:
>
> [QOM_TYPE_NAME]reporting_function: debug message
>
> We also replace IPRINTF with qemu_log_mask(). The qemu_log_mask() output
> is following the same format as the above debug.
>
> Signed-off-by: Jean-Christophe Dubois <address@hidden>
Reviewed-by: Peter Crosthwaite <address@hidden>
> ---
>
> Changes since v1:
> * standardize qemu_log_mask on same model.
>
> Changes since v2:
> * use HWADDR_PRIx for address formating
> * remove intermediate "reg" variable to be on same logic as other files
>
> hw/timer/imx_gpt.c | 56
> ++++++++++++++++++++++--------------------------------
> 1 file changed, 23 insertions(+), 33 deletions(-)
>
> diff --git a/hw/timer/imx_gpt.c b/hw/timer/imx_gpt.c
> index 4bac67d..7257f42 100644
> --- a/hw/timer/imx_gpt.c
> +++ b/hw/timer/imx_gpt.c
> @@ -16,11 +16,17 @@
> #include "hw/misc/imx_ccm.h"
> #include "qemu/main-loop.h"
>
> -/*
> - * Define to 1 for debug messages
> - */
> -#define DEBUG_TIMER 0
> -#if DEBUG_TIMER
> +#ifndef DEBUG_IMX_GPT
> +#define DEBUG_IMX_GPT 0
> +#endif
> +
> +#define DPRINTF(fmt, args...) \
> + do { \
> + if (DEBUG_IMX_GPT) { \
> + fprintf(stderr, "[%s]%s: " fmt , TYPE_IMX_GPT, \
> + __func__, ##args); \
> + } \
> + } while (0)
>
> static char const *imx_gpt_reg_name(uint32_t reg)
> {
> @@ -50,24 +56,6 @@ static char const *imx_gpt_reg_name(uint32_t reg)
> }
> }
>
> -# define DPRINTF(fmt, args...) \
> - do { printf("%s: " fmt , __func__, ##args); } while (0)
> -#else
> -# define DPRINTF(fmt, args...) do {} while (0)
> -#endif
> -
> -/*
> - * Define to 1 for messages about attempts to
> - * access unimplemented registers or similar.
> - */
> -#define DEBUG_IMPLEMENTATION 1
> -#if DEBUG_IMPLEMENTATION
> -# define IPRINTF(fmt, args...) \
> - do { fprintf(stderr, "%s: " fmt, __func__, ##args); } while (0)
> -#else
> -# define IPRINTF(fmt, args...) do {} while (0)
> -#endif
> -
> static const VMStateDescription vmstate_imx_timer_gpt = {
> .name = TYPE_IMX_GPT,
> .version_id = 3,
> @@ -224,9 +212,8 @@ static uint64_t imx_gpt_read(void *opaque, hwaddr offset,
> unsigned size)
> {
> IMXGPTState *s = IMX_GPT(opaque);
> uint32_t reg_value = 0;
> - uint32_t reg = offset >> 2;
>
> - switch (reg) {
> + switch (offset >> 2) {
> case 0: /* Control Register */
> reg_value = s->cr;
> break;
> @@ -256,12 +243,14 @@ static uint64_t imx_gpt_read(void *opaque, hwaddr
> offset, unsigned size)
> break;
>
> case 7: /* input Capture Register 1 */
> - qemu_log_mask(LOG_UNIMP, "icr1 feature is not implemented\n");
> + qemu_log_mask(LOG_UNIMP, "[%s]%s: icr1 feature is not implemented\n",
> + TYPE_IMX_GPT, __func__);
> reg_value = s->icr1;
> break;
>
> case 8: /* input Capture Register 2 */
> - qemu_log_mask(LOG_UNIMP, "icr2 feature is not implemented\n");
> + qemu_log_mask(LOG_UNIMP, "[%s]%s: icr2 feature is not implemented\n",
> + TYPE_IMX_GPT, __func__);
> reg_value = s->icr2;
> break;
>
> @@ -271,11 +260,12 @@ static uint64_t imx_gpt_read(void *opaque, hwaddr
> offset, unsigned size)
> break;
>
> default:
> - IPRINTF("Bad offset %x\n", reg);
> + qemu_log_mask(LOG_GUEST_ERROR, "[%s]%s: Bad register at offset 0x%"
> + HWADDR_PRIx "\n", TYPE_IMX_GPT, __func__, offset);
> break;
> }
>
> - DPRINTF("(%s) = 0x%08x\n", imx_gpt_reg_name(reg), reg_value);
> + DPRINTF("(%s) = 0x%08x\n", imx_gpt_reg_name(offset >> 2), reg_value);
>
> return reg_value;
> }
> @@ -322,12 +312,11 @@ static void imx_gpt_write(void *opaque, hwaddr offset,
> uint64_t value,
> {
> IMXGPTState *s = IMX_GPT(opaque);
> uint32_t oldreg;
> - uint32_t reg = offset >> 2;
>
> - DPRINTF("(%s, value = 0x%08x)\n", imx_gpt_reg_name(reg),
> + DPRINTF("(%s, value = 0x%08x)\n", imx_gpt_reg_name(offset >> 2),
> (uint32_t)value);
>
> - switch (reg) {
> + switch (offset >> 2) {
> case 0:
> oldreg = s->cr;
> s->cr = value & ~0x7c14;
> @@ -403,7 +392,8 @@ static void imx_gpt_write(void *opaque, hwaddr offset,
> uint64_t value,
> break;
>
> default:
> - IPRINTF("Bad offset %x\n", reg);
> + qemu_log_mask(LOG_GUEST_ERROR, "[%s]%s: Bad register at offset 0x%"
> + HWADDR_PRIx "\n", TYPE_IMX_GPT, __func__, offset);
> break;
> }
> }
> --
> 2.5.0
>
- Re: [Qemu-devel] [PATCH v3 3/8] i.MX: Standardize i.MX I2C debug, (continued)
- [Qemu-devel] [PATCH v3 7/8] i.MX: Standardize i.MX EPIT debug, Jean-Christophe Dubois, 2015/10/24
- [Qemu-devel] [PATCH v3 4/8] i.MX: Standardize i.MX AVIC debug, Jean-Christophe Dubois, 2015/10/24
- [Qemu-devel] [PATCH v3 5/8] i.MX: Standardize i.MX CCM debug, Jean-Christophe Dubois, 2015/10/24
- [Qemu-devel] [PATCH v3 6/8] i.MX: Standardize i.MX FEC debug, Jean-Christophe Dubois, 2015/10/24
- [Qemu-devel] [PATCH v3 8/8] i.MX: Standardize i.MX GPT debug, Jean-Christophe Dubois, 2015/10/24
- Re: [Qemu-devel] [PATCH v3 8/8] i.MX: Standardize i.MX GPT debug,
Peter Crosthwaite <=