mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-04 00:03:54 -06:00
trivial patches pull request 20200911
-----BEGIN PGP SIGNATURE----- iQJGBAABCAAwFiEEzS913cjjpNwuT1Fz8ww4vT8vvjwFAl9b0MsSHGxhdXJlbnRA dml2aWVyLmV1AAoJEPMMOL0/L748UH0QAIQYMn0U5eMWREg+DOEEj2n/6h04lTeE K93RsaQ9OsB0cZvwZ08sGoHI6FyfPR8S5yFe5as1ykGcGyO4vNwqxxPeRvqSfgwX 2nkldYwgkOLYsjOD+1G16f/TGJ3MWbUxyn6ta/cSVg+BhbSvMHT3SXTLQ3PB7Lvf 1j3Y3B0BUBAjPh1sNIvBkEV1ZGlV9tqu+XviUf4hfqZOVSTsSb7prtuGVYz9GsH2 RxEcPXQKNWei2eQET4l8otcBpV3dz0D/6sap8nAP8pz0VARn7hJXIn5f8USFHHfg LMWbXrB9I2bjqSmRqhGofxhGMaH2uKQvpv46MyCDel7eMENNj6F+ER2a+7J8Apy8 vVWYg8qVA85rfxQcb7nHuDrLQiD/C1o1uZcV1O/Mwkm7typUcpPn00pQLxhEDf9O xta1/EU8X+7kgTIKWMwpC0HwponOfHiG1TCyzbQ91FiTb/peeOzejTQi88YRNZFV NEg1XIo79Q/no05F/j3nrju2ESzBQTA1F+Jms8aI6Q40nXPaXEjg7IA510os7Py7 uCEtoQ7qEM4lHfuSl67AR2J/NVWlWbZDsjqhxiMsYLR85rkARIf/7fsVIi9ELKOQ AYZdfA/LebovUcp9kDfYHPvDzifGMmv2ppD3mcZWhGuuQTJ06sRMPxQbUF6LBVMc 64FEl8o3wFgv =/CqY -----END PGP SIGNATURE----- Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-for-5.2-pull-request' into staging trivial patches pull request 20200911 # gpg: Signature made Fri 11 Sep 2020 20:32:27 BST # gpg: using RSA key CD2F75DDC8E3A4DC2E4F5173F30C38BD3F2FBE3C # gpg: issuer "laurent@vivier.eu" # gpg: Good signature from "Laurent Vivier <lvivier@redhat.com>" [full] # gpg: aka "Laurent Vivier <laurent@vivier.eu>" [full] # gpg: aka "Laurent Vivier (Red Hat) <lvivier@redhat.com>" [full] # Primary key fingerprint: CD2F 75DD C8E3 A4DC 2E4F 5173 F30C 38BD 3F2F BE3C * remotes/vivier2/tags/trivial-branch-for-5.2-pull-request: target/i386/kvm: Add missing fallthrough comment util/hexdump: Reorder qemu_hexdump() arguments util/hexdump: Convert to take a void pointer argument hw/arm/pxa2xx: Add missing fallthrough comment target/i386/kvm: Rename host_tsx_blacklisted() as host_tsx_broken() test-vmstate: remove unnecessary code in match_interval_mapping_node hw: hyperv: vmbus: Fix 32bit compilation kconfig: fix comment referring to old Makefiles meson.build: tweak sdl-image error message hw/net/e1000e: Remove duplicated write handler for FLSWDATA register hw/net/e1000e: Remove overwritten read handler for STATUS register Makefile: Skip the meson subdir in cscope/TAGS/ctags Makefile: Drop extra phony cscope hw/gpio/max7310: Replace disabled printf() by qemu_log_mask(UNIMP) hw/gpio/omap_gpio: Replace fprintf() by qemu_log_mask(GUEST_ERROR) hw/acpi/tco: Remove unused definitions hw/isa/isa-bus: Replace hw_error() by assert() hw/mips/fuloong2e: Convert pointless error message to an assert() Signed-off-by: Peter Maydell <peter.maydell@linaro.org> # Conflicts: # net/colo-compare.c
This commit is contained in:
commit
842038f55c
23 changed files with 46 additions and 68 deletions
|
@ -15,17 +15,6 @@
|
|||
#include "hw/acpi/tco.h"
|
||||
#include "trace.h"
|
||||
|
||||
//#define DEBUG
|
||||
|
||||
#ifdef DEBUG
|
||||
#define TCO_DEBUG(fmt, ...) \
|
||||
do { \
|
||||
fprintf(stderr, "%s "fmt, __func__, ## __VA_ARGS__); \
|
||||
} while (0)
|
||||
#else
|
||||
#define TCO_DEBUG(fmt, ...) do { } while (0)
|
||||
#endif
|
||||
|
||||
enum {
|
||||
TCO_RLD_DEFAULT = 0x0000,
|
||||
TCO_DAT_IN_DEFAULT = 0x00,
|
||||
|
|
|
@ -444,7 +444,7 @@ static void pxa2xx_mm_write(void *opaque, hwaddr addr,
|
|||
s->mm_regs[addr >> 2] = value;
|
||||
break;
|
||||
}
|
||||
|
||||
/* fallthrough */
|
||||
default:
|
||||
qemu_log_mask(LOG_GUEST_ERROR,
|
||||
"%s: Bad write offset 0x%"HWADDR_PRIx"\n",
|
||||
|
|
|
@ -388,7 +388,7 @@ static void xlnx_dpdma_dump_descriptor(DPDMADescriptor *desc)
|
|||
{
|
||||
if (DEBUG_DPDMA) {
|
||||
qemu_log("DUMP DESCRIPTOR:\n");
|
||||
qemu_hexdump((char *)desc, stdout, "", sizeof(DPDMADescriptor));
|
||||
qemu_hexdump(stdout, "", desc, sizeof(DPDMADescriptor));
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -13,6 +13,7 @@
|
|||
#include "hw/hw.h"
|
||||
#include "hw/irq.h"
|
||||
#include "migration/vmstate.h"
|
||||
#include "qemu/log.h"
|
||||
#include "qemu/module.h"
|
||||
#include "qom/object.h"
|
||||
|
||||
|
@ -71,9 +72,8 @@ static uint8_t max7310_rx(I2CSlave *i2c)
|
|||
return 0xff;
|
||||
|
||||
default:
|
||||
#ifdef VERBOSE
|
||||
printf("%s: unknown register %02x\n", __func__, s->command);
|
||||
#endif
|
||||
qemu_log_mask(LOG_UNIMP, "%s: Unsupported register 0x02%" PRIx8 "\n",
|
||||
__func__, s->command);
|
||||
break;
|
||||
}
|
||||
return 0xff;
|
||||
|
@ -125,9 +125,8 @@ static int max7310_tx(I2CSlave *i2c, uint8_t data)
|
|||
case 0x00: /* Input port - ignore writes */
|
||||
break;
|
||||
default:
|
||||
#ifdef VERBOSE
|
||||
printf("%s: unknown register %02x\n", __func__, s->command);
|
||||
#endif
|
||||
qemu_log_mask(LOG_UNIMP, "%s: Unsupported register 0x02%" PRIx8 "\n",
|
||||
__func__, s->command);
|
||||
return 1;
|
||||
}
|
||||
|
||||
|
|
|
@ -392,8 +392,10 @@ static void omap2_gpio_module_write(void *opaque, hwaddr addr,
|
|||
break;
|
||||
|
||||
case 0x10: /* GPIO_SYSCONFIG */
|
||||
if (((value >> 3) & 3) == 3)
|
||||
fprintf(stderr, "%s: bad IDLEMODE value\n", __func__);
|
||||
if (((value >> 3) & 3) == 3) {
|
||||
qemu_log_mask(LOG_GUEST_ERROR,
|
||||
"%s: Illegal IDLEMODE value: 3\n", __func__);
|
||||
}
|
||||
if (value & 2)
|
||||
omap2_gpio_module_reset(s);
|
||||
s->config[0] = value & 0x1d;
|
||||
|
|
|
@ -380,7 +380,8 @@ static ssize_t gpadl_iter_io(GpadlIter *iter, void *buf, uint32_t len)
|
|||
}
|
||||
}
|
||||
|
||||
p = (void *)(((uintptr_t)iter->map & TARGET_PAGE_MASK) | off_in_page);
|
||||
p = (void *)(uintptr_t)(((uintptr_t)iter->map & TARGET_PAGE_MASK) |
|
||||
off_in_page);
|
||||
if (iter->dir == DMA_DIRECTION_FROM_DEVICE) {
|
||||
memcpy(p, buf, cplen);
|
||||
} else {
|
||||
|
|
|
@ -21,7 +21,6 @@
|
|||
#include "qemu/error-report.h"
|
||||
#include "qemu/module.h"
|
||||
#include "qapi/error.h"
|
||||
#include "hw/hw.h"
|
||||
#include "monitor/monitor.h"
|
||||
#include "hw/sysbus.h"
|
||||
#include "sysemu/sysemu.h"
|
||||
|
@ -85,18 +84,14 @@ void isa_bus_irqs(ISABus *bus, qemu_irq *irqs)
|
|||
qemu_irq isa_get_irq(ISADevice *dev, unsigned isairq)
|
||||
{
|
||||
assert(!dev || ISA_BUS(qdev_get_parent_bus(DEVICE(dev))) == isabus);
|
||||
if (isairq >= ISA_NUM_IRQS) {
|
||||
hw_error("isa irq %d invalid", isairq);
|
||||
}
|
||||
assert(isairq < ISA_NUM_IRQS);
|
||||
return isabus->irqs[isairq];
|
||||
}
|
||||
|
||||
void isa_init_irq(ISADevice *dev, qemu_irq *p, unsigned isairq)
|
||||
{
|
||||
assert(dev->nirqs < ARRAY_SIZE(dev->isairq));
|
||||
if (isairq >= ISA_NUM_IRQS) {
|
||||
hw_error("isa irq %d invalid", isairq);
|
||||
}
|
||||
assert(isairq < ISA_NUM_IRQS);
|
||||
dev->isairq[dev->nirqs] = isairq;
|
||||
*p = isa_get_irq(dev, isairq);
|
||||
dev->nirqs++;
|
||||
|
|
|
@ -240,10 +240,7 @@ static void vt82c686b_southbridge_init(PCIBus *pci_bus, int slot, qemu_irq intc,
|
|||
PCIDevice *dev;
|
||||
|
||||
isa_bus = vt82c686b_isa_init(pci_bus, PCI_DEVFN(slot, 0));
|
||||
if (!isa_bus) {
|
||||
fprintf(stderr, "vt82c686b_init error\n");
|
||||
exit(1);
|
||||
}
|
||||
assert(isa_bus);
|
||||
*p_isa_bus = isa_bus;
|
||||
/* Interrupt controller */
|
||||
/* The 8259 -> IP5 */
|
||||
|
|
|
@ -2916,7 +2916,6 @@ static const readops e1000e_macreg_readops[] = {
|
|||
e1000e_getreg(TSYNCRXCTL),
|
||||
e1000e_getreg(TDH),
|
||||
e1000e_getreg(LEDCTL),
|
||||
e1000e_getreg(STATUS),
|
||||
e1000e_getreg(TCTL),
|
||||
e1000e_getreg(TDBAL),
|
||||
e1000e_getreg(TDLEN),
|
||||
|
@ -3142,7 +3141,6 @@ static const writeops e1000e_macreg_writeops[] = {
|
|||
e1000e_putreg(RXCFGL),
|
||||
e1000e_putreg(TSYNCRXCTL),
|
||||
e1000e_putreg(TSYNCTXCTL),
|
||||
e1000e_putreg(FLSWDATA),
|
||||
e1000e_putreg(EXTCNF_SIZE),
|
||||
e1000e_putreg(EEMNGCTL),
|
||||
e1000e_putreg(RA),
|
||||
|
|
|
@ -357,7 +357,7 @@ static ssize_t etsec_receive(NetClientState *nc,
|
|||
|
||||
#if defined(HEX_DUMP)
|
||||
fprintf(stderr, "%s receive size:%zd\n", nc->name, size);
|
||||
qemu_hexdump((void *)buf, stderr, "", size);
|
||||
qemu_hexdump(stderr, "", buf, size);
|
||||
#endif
|
||||
/* Flush is unnecessary as are already in receiving path */
|
||||
etsec->need_flush = false;
|
||||
|
|
|
@ -269,7 +269,7 @@ static void process_tx_bd(eTSEC *etsec,
|
|||
|
||||
#if defined(HEX_DUMP)
|
||||
qemu_log("eTSEC Send packet size:%d\n", etsec->tx_buffer_len);
|
||||
qemu_hexdump(etsec->tx_buffer, stderr, "", etsec->tx_buffer_len);
|
||||
qemu_hexdump(stderr, "", etsec->tx_buffer, etsec->tx_buffer_len);
|
||||
#endif /* ETSEC_RING_DEBUG */
|
||||
|
||||
if (etsec->first_bd.flags & BD_TX_TOEUN) {
|
||||
|
|
|
@ -1785,7 +1785,7 @@ send_response:
|
|||
}
|
||||
|
||||
#ifdef DEBUG_SD
|
||||
qemu_hexdump((const char *)response, stderr, "Response", rsplen);
|
||||
qemu_hexdump(stderr, "Response", response, rsplen);
|
||||
#endif
|
||||
|
||||
return rsplen;
|
||||
|
|
|
@ -242,7 +242,7 @@ static void usbredir_log_data(USBRedirDevice *dev, const char *desc,
|
|||
if (dev->debug < usbredirparser_debug_data) {
|
||||
return;
|
||||
}
|
||||
qemu_hexdump((char *)data, stderr, desc, len);
|
||||
qemu_hexdump(stderr, desc, data, len);
|
||||
}
|
||||
|
||||
/*
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue