Message ID | 20211002110007.30825-6-mark.cave-ayland@ilande.co.uk |
---|---|
State | New |
Headers | show |
Series | macfb: fixes for booting MacOS | expand |
On 10/2/21 13:00, Mark Cave-Ayland wrote: > 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(-) > @@ -289,7 +290,10 @@ static uint64_t macfb_ctrl_read(void *opaque, > hwaddr addr, > unsigned int size) > { > +# 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" > 'size' is unsigned, otherwise: Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
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>
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"
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(-)