From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-arm@nongnu.org, qemu-devel@nongnu.org
Cc: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Guenter Roeck" <linux@roeck-us.net>,
"Paolo Bonzini" <pbonzini@redhat.com>
Subject: [PATCH v2 0/6] arm: drop last bits from deprecated boards
Date: Thu, 3 Oct 2024 15:00:04 +0100 [thread overview]
Message-ID: <20241003140010.1653808-1-peter.maydell@linaro.org> (raw)
This series is the remaining patches not yet applied from my "arm:
Drop deprecated boards" series; this is essentially the device
removals which didn't get review in that series and/or which had some
discussion about whether we should remove them.
To summarise the remaining removals:
* max1110 and max1111 are SSI devices so in theory could be
created by users on the command line. However I found no
evidence of doing this when I did a web search for
"device max1110", and the utility of doing so is unclear
to me since it's not possible for the command line device
to have its ADC inputs or its interrupt line output
wired up to anything.
* max7310 is an I2C GPIO controller, so it has the same
"in theory a user could create this on the command line
but in practice it's pretty useless because there's no
way to wire up the GPIO lines to anything" status.
* the microdrive device can only be used with our PCMCIA
bus emulation, and we no longer have any boards using
that, so I think these are uncontroversially deleteable.
Some boards we still have should in theory have a
PCMCIA controller (strongarm, kzm, sh7750), but we don't
model those controllers and none of those boards are
sufficiently interesting or active that it's likely
anybody will ever add one.
* The ECC code is entirely unused now; this is an
uncontroversial removal of dead code, and I only didn't
put it in my pullreq because it wasn't reviewed yet.
* Consensus on the v1 series was that we could get rid of
-portrait/-rotate even though in theory you could use
them on a non-pxa2xx machine type to achieve the odd
effect of rotating mouse input. I've added a section
to removed-features.rst about this and beefed up the
patch commit message.
Of these, I think we should definitely be dropping microdrive,
pcmcia, ecc, and the portrait/rotate options, and I didn't hear any
dissenting opinions on v1.
For max111x and max7310: these are fairly small (a couple of hundred
lines each) and not too badly behind the curve for code quality/API
usage (in particular they're both QOM devices), so I could fairly
easily be persuaded that they should stay. However I would like in
that case to see:
(a) what's the use case/user ? (e.g. examples of "I'm
actually using this on the command line, like this"
or "I'm using this in the machine model I plan to submit
in the next six months")
(b) some kind of testing of them (which we can write
based on the answer to a.)
thanks
-- PMM
Peter Maydell (6):
hw/adc: Remove MAX111X device
hw/gpio: Remove MAX7310 device
hw/ide: Remove DSCM-1XXXX microdrive device model
hw: Remove PCMCIA subsystem
hw/block: Remove ecc
vl.c: Remove pxa2xx-specific -portrait and -rotate options
docs/about/removed-features.rst | 22 ++
include/hw/adc/max111x.h | 56 ---
include/hw/block/flash.h | 11 -
include/hw/pcmcia.h | 66 ----
include/sysemu/sysemu.h | 1 -
hw/adc/max111x.c | 236 ------------
hw/block/ecc.c | 91 -----
hw/gpio/max7310.c | 217 -----------
hw/ide/microdrive.c | 644 --------------------------------
hw/pcmcia/pcmcia.c | 24 --
system/globals.c | 1 -
system/vl.c | 11 -
ui/input.c | 36 --
hw/Kconfig | 1 -
hw/adc/Kconfig | 3 -
hw/adc/meson.build | 1 -
hw/arm/Kconfig | 1 -
hw/block/Kconfig | 3 -
hw/block/meson.build | 1 -
hw/gpio/Kconfig | 4 -
hw/gpio/meson.build | 1 -
hw/ide/Kconfig | 6 -
hw/ide/meson.build | 1 -
hw/meson.build | 1 -
hw/misc/Kconfig | 1 -
hw/pcmcia/Kconfig | 2 -
hw/pcmcia/meson.build | 1 -
qemu-options.hx | 16 -
28 files changed, 22 insertions(+), 1437 deletions(-)
delete mode 100644 include/hw/adc/max111x.h
delete mode 100644 include/hw/pcmcia.h
delete mode 100644 hw/adc/max111x.c
delete mode 100644 hw/block/ecc.c
delete mode 100644 hw/gpio/max7310.c
delete mode 100644 hw/ide/microdrive.c
delete mode 100644 hw/pcmcia/pcmcia.c
delete mode 100644 hw/pcmcia/Kconfig
delete mode 100644 hw/pcmcia/meson.build
--
2.34.1
next reply other threads:[~2024-10-03 14:01 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-03 14:00 Peter Maydell [this message]
2024-10-03 14:00 ` [PATCH v2 1/6] hw/adc: Remove MAX111X device Peter Maydell
2024-10-03 15:36 ` Philippe Mathieu-Daudé
2024-10-03 14:00 ` [PATCH v2 2/6] hw/gpio: Remove MAX7310 device Peter Maydell
2024-10-03 15:36 ` Philippe Mathieu-Daudé
2024-10-03 14:00 ` [PATCH v2 3/6] hw/ide: Remove DSCM-1XXXX microdrive device model Peter Maydell
2024-10-03 15:36 ` Philippe Mathieu-Daudé
2024-10-03 14:00 ` [PATCH v2 4/6] hw: Remove PCMCIA subsystem Peter Maydell
2024-10-03 14:00 ` [PATCH v2 5/6] hw/block: Remove ecc Peter Maydell
2024-10-03 15:37 ` Philippe Mathieu-Daudé
2024-10-03 14:00 ` [PATCH v2 6/6] vl.c: Remove pxa2xx-specific -portrait and -rotate options Peter Maydell
2024-10-04 13:54 ` [PATCH v2 0/6] arm: drop last bits from deprecated boards Richard Henderson
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=20241003140010.1653808-1-peter.maydell@linaro.org \
--to=peter.maydell@linaro.org \
--cc=linux@roeck-us.net \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@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).