diff mbox series

[PATCH-for-5.2,1/4] hw/display/cirrus_vga: Remove debugging code commented out

Message ID 20201103112558.2554390-2-philmd@redhat.com
State Accepted
Commit 0d5528612badad84821c5d93c82253f86987659d
Headers show
Series misc: Trivial format string fixes | expand

Commit Message

Philippe Mathieu-Daudé Nov. 3, 2020, 11:25 a.m. UTC
Commit ec87f206d70 ("cirrus: replace debug printf with trace points")
forgot to remove this code once replaced. Do it now.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 hw/display/cirrus_vga.c | 18 ------------------
 1 file changed, 18 deletions(-)

Comments

Dr. David Alan Gilbert Nov. 3, 2020, 12:42 p.m. UTC | #1
* Philippe Mathieu-Daudé (philmd@redhat.com) wrote:
> Commit ec87f206d70 ("cirrus: replace debug printf with trace points")

> forgot to remove this code once replaced. Do it now.

> 

> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>


Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>


> ---

>  hw/display/cirrus_vga.c | 18 ------------------

>  1 file changed, 18 deletions(-)

> 

> diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c

> index 722b9e7004c..e14096deb46 100644

> --- a/hw/display/cirrus_vga.c

> +++ b/hw/display/cirrus_vga.c

> @@ -2532,9 +2532,6 @@ static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,

>  	case 0x3c5:

>  	    val = cirrus_vga_read_sr(c);

>              break;

> -#ifdef DEBUG_VGA_REG

> -	    printf("vga: read SR%x = 0x%02x\n", s->sr_index, val);

> -#endif

>  	    break;

>  	case 0x3c6:

>  	    val = cirrus_read_hidden_dac(c);

> @@ -2560,9 +2557,6 @@ static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,

>  	    break;

>  	case 0x3cf:

>  	    val = cirrus_vga_read_gr(c, s->gr_index);

> -#ifdef DEBUG_VGA_REG

> -	    printf("vga: read GR%x = 0x%02x\n", s->gr_index, val);

> -#endif

>  	    break;

>  	case 0x3b4:

>  	case 0x3d4:

> @@ -2571,9 +2565,6 @@ static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,

>  	case 0x3b5:

>  	case 0x3d5:

>              val = cirrus_vga_read_cr(c, s->cr_index);

> -#ifdef DEBUG_VGA_REG

> -	    printf("vga: read CR%x = 0x%02x\n", s->cr_index, val);

> -#endif

>  	    break;

>  	case 0x3ba:

>  	case 0x3da:

> @@ -2645,9 +2636,6 @@ static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,

>  	s->sr_index = val;

>  	break;

>      case 0x3c5:

> -#ifdef DEBUG_VGA_REG

> -	printf("vga: write SR%x = 0x%02" PRIu64 "\n", s->sr_index, val);

> -#endif

>  	cirrus_vga_write_sr(c, val);

>          break;

>      case 0x3c6:

> @@ -2670,9 +2658,6 @@ static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,

>  	s->gr_index = val;

>  	break;

>      case 0x3cf:

> -#ifdef DEBUG_VGA_REG

> -	printf("vga: write GR%x = 0x%02" PRIu64 "\n", s->gr_index, val);

> -#endif

>  	cirrus_vga_write_gr(c, s->gr_index, val);

>  	break;

>      case 0x3b4:

> @@ -2681,9 +2666,6 @@ static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,

>  	break;

>      case 0x3b5:

>      case 0x3d5:

> -#ifdef DEBUG_VGA_REG

> -	printf("vga: write CR%x = 0x%02"PRIu64"\n", s->cr_index, val);

> -#endif

>  	cirrus_vga_write_cr(c, val);

>  	break;

>      case 0x3ba:

> -- 

> 2.26.2

> 

-- 
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
diff mbox series

Patch

diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c
index 722b9e7004c..e14096deb46 100644
--- a/hw/display/cirrus_vga.c
+++ b/hw/display/cirrus_vga.c
@@ -2532,9 +2532,6 @@  static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,
 	case 0x3c5:
 	    val = cirrus_vga_read_sr(c);
             break;
-#ifdef DEBUG_VGA_REG
-	    printf("vga: read SR%x = 0x%02x\n", s->sr_index, val);
-#endif
 	    break;
 	case 0x3c6:
 	    val = cirrus_read_hidden_dac(c);
@@ -2560,9 +2557,6 @@  static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,
 	    break;
 	case 0x3cf:
 	    val = cirrus_vga_read_gr(c, s->gr_index);
-#ifdef DEBUG_VGA_REG
-	    printf("vga: read GR%x = 0x%02x\n", s->gr_index, val);
-#endif
 	    break;
 	case 0x3b4:
 	case 0x3d4:
@@ -2571,9 +2565,6 @@  static uint64_t cirrus_vga_ioport_read(void *opaque, hwaddr addr,
 	case 0x3b5:
 	case 0x3d5:
             val = cirrus_vga_read_cr(c, s->cr_index);
-#ifdef DEBUG_VGA_REG
-	    printf("vga: read CR%x = 0x%02x\n", s->cr_index, val);
-#endif
 	    break;
 	case 0x3ba:
 	case 0x3da:
@@ -2645,9 +2636,6 @@  static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,
 	s->sr_index = val;
 	break;
     case 0x3c5:
-#ifdef DEBUG_VGA_REG
-	printf("vga: write SR%x = 0x%02" PRIu64 "\n", s->sr_index, val);
-#endif
 	cirrus_vga_write_sr(c, val);
         break;
     case 0x3c6:
@@ -2670,9 +2658,6 @@  static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,
 	s->gr_index = val;
 	break;
     case 0x3cf:
-#ifdef DEBUG_VGA_REG
-	printf("vga: write GR%x = 0x%02" PRIu64 "\n", s->gr_index, val);
-#endif
 	cirrus_vga_write_gr(c, s->gr_index, val);
 	break;
     case 0x3b4:
@@ -2681,9 +2666,6 @@  static void cirrus_vga_ioport_write(void *opaque, hwaddr addr, uint64_t val,
 	break;
     case 0x3b5:
     case 0x3d5:
-#ifdef DEBUG_VGA_REG
-	printf("vga: write CR%x = 0x%02"PRIu64"\n", s->cr_index, val);
-#endif
 	cirrus_vga_write_cr(c, val);
 	break;
     case 0x3ba: