qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V1]sm501: dead code removal


From: Thomas Huth
Subject: Re: [Qemu-devel] [PATCH V1]sm501: dead code removal
Date: Mon, 12 Mar 2018 06:58:50 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 11.03.2018 17:40, Aishwarya Kadlag wrote:
> From: Aishwarya Kadlag <address@hidden>
> 
> Remove support for DEPTH != 32 values from hw/display/*_template.h 
> files and other files that include them. Only DEPTH == 32 case is 
> supported.
> 
> Signed-off-by: Aishwarya Kadlag <address@hidden>
> ---
> hw/display/sw501_1.c              |   36 ------------------------------------ 
> hw/display/sw501_template_1.h     |    4 +++- 
> 2 files changed, 3 insertions(+), 37 deletions(-)
> 
> 
> diff -u sm501.c sm501_1.c > sm501.patch -s

Patches should be applicable from the the top directory. It's best if
you create the patches with "git format-patch", see:

https://wiki.qemu.org/Contribute/SubmitAPatch#Use_git_format-patch

> --- sm501.c   2018-03-11 17:46:33.621452968 +0530
> +++ sm501_1.c 2018-03-11 17:53:26.933445566 +0530
> @@ -1358,22 +1358,6 @@
>                                  int width, const uint8_t *palette,
>                                  int c_x, int c_y);
>  
> -#define DEPTH 8
> -#include "sm501_template.h"
> -
> -#define DEPTH 15
> -#include "sm501_template.h"
> -
> -#define BGR_FORMAT
> -#define DEPTH 15
> -#include "sm501_template.h"
> -
> -#define DEPTH 16
> -#include "sm501_template.h"
> -
> -#define BGR_FORMAT
> -#define DEPTH 16
> -#include "sm501_template.h"
>  
>  #define DEPTH 32
>  #include "sm501_template.h"
> @@ -1383,43 +1367,23 @@
>  #include "sm501_template.h"
>  
>  static draw_line_func *draw_line8_funcs[] = {
> -    draw_line8_8,
> -    draw_line8_15,
> -    draw_line8_16,
>      draw_line8_32,
>      draw_line8_32bgr,
> -    draw_line8_15bgr,
> -    draw_line8_16bgr,
>  };
>  
>  static draw_line_func *draw_line16_funcs[] = {
> -    draw_line16_8,
> -    draw_line16_15,
> -    draw_line16_16,
>      draw_line16_32,
>      draw_line16_32bgr,
> -    draw_line16_15bgr,
> -    draw_line16_16bgr,
>  };
>  
>  static draw_line_func *draw_line32_funcs[] = {
> -    draw_line32_8,
> -    draw_line32_15,
> -    draw_line32_16,
>      draw_line32_32,
>      draw_line32_32bgr,
> -    draw_line32_15bgr,
> -    draw_line32_16bgr,
>  };
>  
>  static draw_hwc_line_func *draw_hwc_line_funcs[] = {
> -    draw_hwc_line_8,
> -    draw_hwc_line_15,
> -    draw_hwc_line_16,
>      draw_hwc_line_32,
>      draw_hwc_line_32bgr,
> -    draw_hwc_line_15bgr,
> -    draw_hwc_line_16bgr,
>  };
>  
>  static inline int get_depth_index(DisplaySurface *surface)
> 
> 
> 
> diff -u sm501_template.h sm501_template_1.h > sm501_template.patch -s
> --- sm501_template.h    2018-03-11 17:25:36.816653718 +0530
> +++ sm501_template_1.h  2018-03-11 17:25:16.828654076 +0530
> @@ -22,13 +22,15 @@
>   * THE SOFTWARE.
>   */
>  
> +/*
>  #if DEPTH == 8
>  #define BPP 1
>  #define PIXEL_TYPE uint8_t
>  #elif DEPTH == 15 || DEPTH == 16
>  #define BPP 2
>  #define PIXEL_TYPE uint16_t
> -#elif DEPTH == 32
> +*/
> +#if DEPTH == 32
>  #define BPP 4
>  #define PIXEL_TYPE uint32_t
>  #else
> 

I don't know the hw/display code very well, but I think it's not that
easy... Putting Gerd on CC:, maybe he can describe the details of that
BiteSizeTask a little bit better.

(When sending patches, please also always use the get_maintainers.pl
script to get a set of people who should be put on CC: or the patch
might get lost in the high traffic of the qemu-devel mailing list).

 Thomas



reply via email to

[Prev in Thread] Current Thread [Next in Thread]