[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V1]sm501: dead code removal
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH V1]sm501: dead code removal |
Date: |
Mon, 12 Mar 2018 10:17:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 12/03/2018 06:58, Thomas Huth wrote:
> 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).
It's _almost_ that easy.
First, get_depth_index must be adjusted to reflect that you have removed
the first three elements from the array. It doesn't hurt to add an
assert(surface_bits_per_pixel(surface) == 32)
in the function, too.
Second, the PIXEL_TYPE macro in sm501_template.h now is always uint32_t
and it can be replaced everywhere. Likewise for BPP, which is always 4,
and DEPTH, which is always 32.
Thanks!
Paolo