[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 05/12] macfb: add trace events for reading and writing the co
From: |
Laurent Vivier |
Subject: |
Re: [PATCH 05/12] macfb: add trace events for reading and writing the control registers |
Date: |
Mon, 4 Oct 2021 10:57:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
Le 02/10/2021 à 13:00, Mark Cave-Ayland a écrit :
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> ---
> hw/display/macfb.c | 8 +++++++-
> hw/display/trace-events | 4 ++++
> 2 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/hw/display/macfb.c b/hw/display/macfb.c
> index e86fbbbb64..62c2727a5b 100644
> --- a/hw/display/macfb.c
> +++ b/hw/display/macfb.c
> @@ -20,6 +20,7 @@
> #include "qapi/error.h"
> #include "hw/qdev-properties.h"
> #include "migration/vmstate.h"
> +#include "trace.h"
>
> #define VIDEO_BASE 0x00001000
> #define DAFB_BASE 0x00800000
> @@ -289,7 +290,10 @@ static uint64_t macfb_ctrl_read(void *opaque,
> hwaddr addr,
> unsigned int size)
> {
> - return 0;
> + uint64_t val = 0;
> +
> + trace_macfb_ctrl_read(addr, val, size);
> + return val;
> }
>
> static void macfb_ctrl_write(void *opaque,
> @@ -312,6 +316,8 @@ static void macfb_ctrl_write(void *opaque,
> }
> break;
> }
> +
> + trace_macfb_ctrl_write(addr, val, size);
> }
>
> static const MemoryRegionOps macfb_ctrl_ops = {
> diff --git a/hw/display/trace-events b/hw/display/trace-events
> index f03f6655bc..be1353e8e7 100644
> --- a/hw/display/trace-events
> +++ b/hw/display/trace-events
> @@ -167,3 +167,7 @@ sm501_disp_ctrl_read(uint32_t addr, uint32_t val)
> "addr=0x%x, val=0x%x"
> sm501_disp_ctrl_write(uint32_t addr, uint32_t val) "addr=0x%x, val=0x%x"
> sm501_2d_engine_read(uint32_t addr, uint32_t val) "addr=0x%x, val=0x%x"
> sm501_2d_engine_write(uint32_t addr, uint32_t val) "addr=0x%x, val=0x%x"
> +
> +# macfb.c
> +macfb_ctrl_read(uint64_t addr, uint64_t value, int size) "addr 0x%"PRIx64 "
> value 0x%"PRIx64 " size %d"
> +macfb_ctrl_write(uint64_t addr, uint64_t value, int size) "addr 0x%"PRIx64 "
> value 0x%"PRIx64 " size %d"
>
As suggested by Philippe: size is unsigned
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
- Re: [PATCH 07/12] macfb: add qdev property to specify display type, (continued)
- [PATCH 05/12] macfb: add trace events for reading and writing the control registers, Mark Cave-Ayland, 2021/10/02
- [PATCH 12/12] q800: wire macfb IRQ to separate video interrupt on VIA2, Mark Cave-Ayland, 2021/10/02
- [PATCH 09/12] macfb: fix up 1-bit pixel encoding, Mark Cave-Ayland, 2021/10/02
- [PATCH 11/12] macfb: add vertical blank interrupt, Mark Cave-Ayland, 2021/10/02
- [PATCH 10/12] macfb: fix 24-bit RGB pixel encoding, Mark Cave-Ayland, 2021/10/02