Hi Gerd, On 06/22/2018 03:22 AM, Gerd Hoffmann wrote: > On Thu, Jun 21, 2018 at 02:12:51PM -0300, Philippe Mathieu-Daudé wrote: >> Signed-off-by: Philippe Mathieu-Daudé <[email protected]> >> --- >> hw/display/cirrus_vga.c | 25 ++++++------------------- >> hw/display/trace-events | 2 ++ >> 2 files changed, 8 insertions(+), 19 deletions(-) >> >> diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c >> index 138ae961b9..a826bef6ec 100644 >> --- a/hw/display/cirrus_vga.c >> +++ b/hw/display/cirrus_vga.c >> @@ -2608,11 +2608,8 @@ static uint64_t cirrus_vga_ioport_read(void *opaque, >> hwaddr addr, >> break; >> case 0x3c5: >> val = cirrus_vga_read_sr(c); >> + trace_vga_cirrus_read_reg("SR", s->sr_index, val); >> break; >> -#ifdef DEBUG_VGA_REG >> - printf("vga: read SR%x = 0x%02x\n", s->sr_index, val); >> -#endif > > Indention looks weird.
Actually this is the code around that is weirdly intended, a mix of <tab> and <spaces>. I noticed, except when upgrading generic API, there are very few logical changes (except yours) in this file in the git history. Thus little interest in doing an indentation cleanup previous patch (at this point git history is more useful). > > otherwise the patch looks fine. Do you prefer a previous indent cleanup patch or can I add your Acked-by tag? Thanks, Phil. > > cheers, > Gerd >
