qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Cc: Gerd Hoffmann <kraxel@redhat.com>,
	qemu-ppc@nongnu.org, Greg Kurz <groug@kaod.org>,
	David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH 0/3] hw/display/sm501: Tidy up template header
Date: Fri, 12 Feb 2021 18:06:50 +0000	[thread overview]
Message-ID: <20210212180653.27588-1-peter.maydell@linaro.org> (raw)

For a long time now the UI layer has guaranteed that the console
surface is always 32 bits per pixel RGB, but some older display device
models still have the code to handle other formats. This patchset
cleans up that dead code for the sm501 device, which allows us
to remove the multiply-included sm501_template.h header entirely.

There have been a few attempts at doing this cleanup on this
device in the past by various people; rather than trying to
resurrect those old patchsets and get them to apply to the current
code in master, I just started from scratch.

Tested with AROS ISO image on sam460ex.

thanks
-- PMM

Peter Maydell (3):
  hw/display/sm501: Remove dead code for non-32-bit RGB surfaces
  hw/display/sm501: Expand out macros in template header
  hw/display/sm501: Inline template header into C file

 hw/display/sm501_template.h | 131 ----------------------------
 hw/display/sm501.c          | 166 ++++++++++++++++++------------------
 2 files changed, 83 insertions(+), 214 deletions(-)
 delete mode 100644 hw/display/sm501_template.h

-- 
2.20.1



             reply	other threads:[~2021-02-12 18:08 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-12 18:06 Peter Maydell [this message]
2021-02-12 18:06 ` [PATCH 1/3] hw/display/sm501: Remove dead code for non-32-bit RGB surfaces Peter Maydell
2021-02-12 18:06 ` [PATCH 2/3] hw/display/sm501: Expand out macros in template header Peter Maydell
2021-02-12 18:06 ` [PATCH 3/3] hw/display/sm501: Inline template header into C file Peter Maydell
2021-02-12 20:28 ` [PATCH 0/3] hw/display/sm501: Tidy up template header BALATON Zoltan
2021-02-15  6:12   ` David Gibson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210212180653.27588-1-peter.maydell@linaro.org \
    --to=peter.maydell@linaro.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=groug@kaod.org \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).