From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33597) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VDbnC-0004nJ-U6 for qemu-devel@nongnu.org; Sun, 25 Aug 2013 10:59:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VDbnB-0006eU-UV for qemu-devel@nongnu.org; Sun, 25 Aug 2013 10:59:50 -0400 Received: from 1.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.d.1.0.0.b.8.0.1.0.0.2.ip6.arpa ([2001:8b0:1d0::1]:59464 helo=mnementh.archaic.org.uk) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VDbnB-0006cn-NO for qemu-devel@nongnu.org; Sun, 25 Aug 2013 10:59:49 -0400 From: Peter Maydell Date: Sun, 25 Aug 2013 15:59:28 +0100 Message-Id: <1377442777-11066-1-git-send-email-peter.maydell@linaro.org> Subject: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Stefan Hajnoczi , "Michael S. Tsirkin" , patches@linaro.org, Anthony Liguori , Richard Henderson The bswap.h header includes a set of "legacy unaligned functions" that (since commit c732a52d3 at the beginning of this year) are just wrappers for underlying {ld,st} functions. The legacy functions aren't used in many places, so just replace all their uses with uses of the new-style {ld,st} functions; this lets us remove the legacy wrappers altogether. Since we know the {ld,st}* routines are definitely functions, we can in the process remove some casts which were left over from when the legacy unaligned functions were previously macros. The patchset is divided up by function being removed, rather than by which device/subsystem is being fixed; I think this way round is easier to review since you only have to keep one substitution in your head when reading a patch. Peter Maydell (9): bswap.h: Remove cpu_to_le16wu() bswap.h: Remove cpu_to_le32wu() bswap.h: Remove le16_to_cpupu() bswap.h: Remove le32_to_cpupu() bswap.h: Remove be32_to_cpupu() bswap.h: Remove cpu_to_be16wu() bswap.h: Remove cpu_to_be32wu() bswap.h: Remove cpu_to_be64wu() bswap.h: Remove cpu_to_32wu() block/qcow2-cluster.c | 2 +- hw/acpi/core.c | 3 +-- hw/block/cdrom.c | 10 +++++----- hw/display/vga_template.h | 14 ++++++++------ hw/ide/atapi.c | 16 +++++++-------- hw/net/e1000.c | 22 +++++++++------------ hw/net/ne2000.c | 4 ++-- hw/pci/pcie_aer.c | 4 ++-- include/hw/pci/pci.h | 8 ++++---- include/qemu/bswap.h | 47 --------------------------------------------- 10 files changed, 40 insertions(+), 90 deletions(-) -- 1.7.9.5