This patchset removes dead code (including a couple of 'template'
headers) for handling UI surfaces that are formats other than
32-bit RGB for the musicpal, tc6393xb and omap_lcdc display devices.
For a long time now the UI layer has guaranteed that the console
surface is always 32 bits per pixel RGB, so the legacy code in these
devices which was handling the possibility that the console surface
was some other format can all be deleted.
Notes:
(1) the patch "hw/display/omap_lcdc: Drop broken bigendian ifdef"
fixes a bug introduced in commit ea644cf343129 in 2016 in a
previous partial start on this cleanup
(2) the omap_lcdc changes are tested only with 'make check' as I
don't have any test images for the OMAP1.
With these plus the other on-list patches, the only remaining
'template' header files in hw/display are pl110 (which needs it to
handle the different guest-side pixel formats) and the
milkymist-vgafb device. Milkymist is in the 'deprecated and will be
deleted in a few releases' bucket so it's not worth doing this
cleanup for it, since it's not blocking any API transition.
(I'll take this task off the BiteSizedTasks list shortly.)
After the musicpal cleanup and the tcx patch I just posted, there
will be no more users of is_surface_bgr(), so that could then be
deleted.
thanks
-- PMM
Peter Maydell (9):
hw/arm/musicpal: Remove dead code for non-32-bit-RGB surfaces
hw/display/tc6393xb: Remove dead code for handling non-32bpp surfaces
hw/display/tc6393xb: Expand out macros in template header
hw/display/tc6393xb: Inline tc6393xb_draw_graphic32() at its callsite
hw/display/omap_lcdc: Expand out macros in template header
hw/display/omap_lcdc: Drop broken bigendian ifdef
hw/display/omap_lcdc: Fix coding style issues in template header
hw/display/omap_lcdc: Inline template header into C file
hw/display/omap_lcdc: Delete unnecessary macro
hw/display/omap_lcd_template.h | 169 ---------------------------------
hw/display/tc6393xb_template.h | 72 --------------
include/ui/console.h | 10 --
hw/arm/musicpal.c | 64 +++++--------
hw/display/omap_lcdc.c | 129 ++++++++++++++++++++++++-
hw/display/tc6393xb.c | 48 ++++------
6 files changed, 165 insertions(+), 327 deletions(-)
delete mode 100644 hw/display/omap_lcd_template.h
delete mode 100644 hw/display/tc6393xb_template.h
--
2.20.1
