* [PULL v2 00/61] Misc HW & UI patches for 2024-09-12
@ 2024-09-12 6:52 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 44/61] hw/sensor/tmp105: Coding style fixes Philippe Mathieu-Daudé
` (13 more replies)
0 siblings, 14 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
v2:
- Fill Pierrick's commit description suggested by Eric Blake
- Include TMP105 fixes from Guenter
The following changes since commit a4eb31c678400472de0b4915b9154a7c20d8332f:
Merge tag 'pull-testing-gdbstub-oct-100924-1' of https://gitlab.com/stsquad/qemu into staging (2024-09-11 13:17:29 +0100)
are available in the Git repository at:
https://github.com/philmd/qemu.git tags/hw-misc-20240912
for you to fetch changes up to bd480a2baab659abe90da878bc955670691f53a8:
ui: remove break after g_assert_not_reached() (2024-09-12 08:44:48 +0200)
----------------------------------------------------------------
Misc HW & UI patches
- Remove deprecated SH4 SHIX machine TC58128 NAND EEPROM (Phil)
- Remove deprecated CRIS target (Phil)
- Remove deprecated RISC-V 'any' CPU type (Phil)
- Add fifo8_peek_buf() to correctly handle FIFO wraparound (Mark)
- Minor cleanups in Designware PCIe, PL011 and loongson IPI models (Phil)
- Fixes in TI TMP105 temperature (Guenter)
- Convert Sun ESCC and ADB mouses to QemuInputHandler (Mark)
- Prevent heap overflow in VIRTIO sound device (Volker)
- Cleanups around g_assert_not_reached() call (Pierrick)
- Add Clément as VT-d reviewer (Clément)
- Prevent stuck modifier keys and unexpected text input on Windows (Volker)
- Explicitly set SDL2 swap interval when OpenGL is enabled (Gert)
----------------------------------------------------------------
CLEMENT MATHIEU--DRIF (1):
MAINTAINERS: Add myself as a reviewer of VT-d
Gert Wollny (1):
ui/sdl2: set swap interval explicitly when OpenGL is enabled
Guenter Roeck (2):
hw/sensor/tmp105: Coding style fixes
hw/sensor/tmp105: Lower 4 bit of limit registers are always 0
Mark Cave-Ayland (11):
fifo8: rename fifo8_peekpop_buf() to fifo8_peekpop_bufptr()
fifo8: introduce head variable for fifo8_peekpop_bufptr()
fifo8: add skip parameter to fifo8_peekpop_bufptr()
fifo8: replace fifo8_pop_bufptr() with fifo8_peekpop_bufptr() in
fifo8_pop_buf()
fifo8: rename fifo8_pop_buf() to fifo8_peekpop_buf()
fifo8: honour do_pop argument in fifo8_peekpop_buf()
fifo8: add fifo8_peek_buf() function
fifo8: introduce fifo8_peek() function
tests/unit: add test-fifo unit test
hw/char/escc: convert Sun mouse to use QemuInputHandler
hw/input/adb-mouse: convert to use QemuInputHandler
Philippe Mathieu-Daudé (35):
hw/pci-host/designware: Declare CPU QOM types using DEFINE_TYPES()
macro
hw/pci-host/designware: Add 'host_mem' variable for clarity
hw/intc/loongson_ipi: Remove unused headers
hw/sh4: Remove the deprecated SHIX machine
hw/block: Remove TC58128 NAND EEPROM
hw/sh4: Remove sh7750_register_io_device() helper
tests/tcg: Remove CRIS libc test files
tests/tcg: Remove CRIS bare test files
buildsys: Remove CRIS cross container
linux-user: Remove support for CRIS target
hw/cris: Remove the axis-dev88 machine
hw/cris: Remove image loader helper
hw/intc: Remove TYPE_ETRAX_FS_PIC device
hw/char: Remove TYPE_ETRAX_FS_SERIAL device
hw/net: Remove TYPE_ETRAX_FS_ETH device
hw/dma: Remove ETRAX_FS DMA device
hw/timer: Remove TYPE_ETRAX_FS_TIMER device
system: Remove support for CRIS target
target/cris: Remove the deprecated CRIS target
seccomp: Remove check for CRIS host
target/riscv: Remove the deprecated 'any' CPU type
hw/char/pl011: Remove unused 'readbuff' field
hw/char/pl011: Move pl011_put_fifo() earlier
hw/char/pl011: Move pl011_loopback_enabled|tx() around
hw/char/pl011: Split RX/TX path of pl011_reset_fifo()
hw/char/pl011: Extract pl011_write_txdata() from pl011_write()
hw/char/pl011: Extract pl011_read_rxdata() from pl011_read()
hw/char/pl011: Warn when using disabled transmitter
hw/char/pl011: Rename RX FIFO methods
tests/unit: Strengthen FIFO8 tests
tests/unit: Expand test_fifo8_peek_buf_wrap() coverage
tests/unit: Comment FIFO8 tests
hw/sensor/tmp105: Use registerfields API
hw/sensor/tmp105: Pass 'oneshot' argument to tmp105_alarm_update()
hw/sensor/tmp105: OS (one-shot) bit in config register always returns
0
Pierrick Bouvier (8):
hw/char: replace assert(0) with g_assert_not_reached()
hw/core: replace assert(0) with g_assert_not_reached()
hw/watchdog: replace assert(0) with g_assert_not_reached()
hw/gpio: remove break after g_assert_not_reached()
hw/misc: remove break after g_assert_not_reached()
hw/pci-host: remove break after g_assert_not_reached()
system: replace assert(0) with g_assert_not_reached()
ui: remove break after g_assert_not_reached()
Volker Rümelin (3):
hw/audio/virtio-sound: fix heap buffer overflow
ui/sdl2: release all modifiers
ui/sdl2: ignore GUI keys in SDL_TEXTINPUT handler
MAINTAINERS | 24 +-
docs/about/deprecated.rst | 27 -
docs/about/emulation.rst | 4 -
docs/about/removed-features.rst | 20 +
docs/user/main.rst | 4 -
configure | 4 -
configs/devices/cris-softmmu/default.mak | 4 -
configs/devices/sh4-softmmu/default.mak | 1 -
configs/targets/cris-linux-user.mak | 1 -
configs/targets/cris-softmmu.mak | 1 -
qapi/machine.json | 2 +-
hw/cris/boot.h | 16 -
include/exec/poison.h | 1 -
include/hw/char/escc.h | 3 +
include/hw/char/pl011.h | 1 -
include/hw/cris/etraxfs.h | 54 -
include/hw/cris/etraxfs_dma.h | 36 -
include/hw/pci-host/designware.h | 2 -
include/hw/sh4/sh.h | 19 -
include/qemu/fifo8.h | 25 +
include/sysemu/arch_init.h | 1 -
include/ui/sdl2.h | 2 +
include/user/abitypes.h | 7 -
linux-user/cris/sockbits.h | 1 -
linux-user/cris/syscall_nr.h | 367 --
linux-user/cris/target_cpu.h | 45 -
linux-user/cris/target_elf.h | 14 -
linux-user/cris/target_errno_defs.h | 7 -
linux-user/cris/target_fcntl.h | 11 -
linux-user/cris/target_mman.h | 13 -
linux-user/cris/target_prctl.h | 1 -
linux-user/cris/target_proc.h | 1 -
linux-user/cris/target_resource.h | 1 -
linux-user/cris/target_signal.h | 9 -
linux-user/cris/target_structs.h | 1 -
linux-user/cris/target_syscall.h | 46 -
linux-user/cris/termbits.h | 225 --
linux-user/syscall_defs.h | 7 +-
target/cris/cpu-param.h | 16 -
target/cris/cpu-qom.h | 32 -
target/cris/cpu.h | 286 --
target/cris/crisv10-decode.h | 112 -
target/cris/crisv32-decode.h | 133 -
target/cris/helper.h | 23 -
target/cris/mmu.h | 22 -
target/cris/opcode-cris.h | 355 --
target/riscv/cpu-qom.h | 1 -
tests/tcg/cris/libc/crisutils.h | 76 -
tests/tcg/cris/libc/sys.h | 18 -
hw/audio/virtio-snd.c | 24 -
hw/block/tc58128.c | 211 --
hw/char/avr_usart.c | 2 +-
hw/char/escc.c | 92 +-
hw/char/etraxfs_ser.c | 267 --
hw/char/pl011.c | 209 +-
hw/core/numa.c | 2 +-
hw/cris/axis_dev88.c | 351 --
hw/cris/boot.c | 102 -
hw/dma/etraxfs_dma.c | 781 ----
hw/gpio/nrf51_gpio.c | 1 -
hw/input/adb-mouse.c | 63 +-
hw/intc/etraxfs_pic.c | 172 -
hw/intc/loongson_ipi.c | 9 -
hw/misc/imx6_ccm.c | 1 -
hw/misc/mac_via.c | 2 -
hw/net/etraxfs_eth.c | 688 ----
hw/pci-host/designware.c | 44 +-
hw/pci-host/gt64120.c | 2 -
hw/sensor/tmp105.c | 66 +-
hw/sh4/sh7750.c | 57 +-
hw/sh4/shix.c | 86 -
hw/timer/etraxfs_timer.c | 407 ---
hw/watchdog/watchdog.c | 2 +-
linux-user/cris/cpu_loop.c | 95 -
linux-user/cris/signal.c | 194 -
linux-user/elfload.c | 15 -
linux-user/syscall.c | 10 +-
system/qemu-seccomp.c | 4 +-
system/rtc.c | 2 +-
target/cris/cpu.c | 323 --
target/cris/gdbstub.c | 127 -
target/cris/helper.c | 287 --
target/cris/machine.c | 93 -
target/cris/mmu.c | 356 --
target/cris/op_helper.c | 580 ---
target/cris/translate.c | 3252 -----------------
target/riscv/cpu.c | 28 -
tests/qtest/machine-none-test.c | 1 -
tests/tcg/cris/bare/sys.c | 63 -
tests/tcg/cris/libc/check_abs.c | 40 -
tests/tcg/cris/libc/check_addc.c | 58 -
tests/tcg/cris/libc/check_addcm.c | 85 -
tests/tcg/cris/libc/check_addo.c | 125 -
tests/tcg/cris/libc/check_addoq.c | 44 -
tests/tcg/cris/libc/check_bound.c | 142 -
tests/tcg/cris/libc/check_ftag.c | 37 -
.../cris/libc/check_gcctorture_pr28634-1.c | 15 -
.../tcg/cris/libc/check_gcctorture_pr28634.c | 15 -
.../tcg/cris/libc/check_glibc_kernelversion.c | 116 -
tests/tcg/cris/libc/check_hello.c | 7 -
tests/tcg/cris/libc/check_int64.c | 47 -
tests/tcg/cris/libc/check_lz.c | 49 -
tests/tcg/cris/libc/check_mapbrk.c | 39 -
tests/tcg/cris/libc/check_mmap1.c | 48 -
tests/tcg/cris/libc/check_mmap2.c | 48 -
tests/tcg/cris/libc/check_mmap3.c | 33 -
tests/tcg/cris/libc/check_moveq.c | 51 -
tests/tcg/cris/libc/check_openpf1.c | 38 -
tests/tcg/cris/libc/check_openpf2.c | 16 -
tests/tcg/cris/libc/check_openpf3.c | 49 -
tests/tcg/cris/libc/check_openpf5.c | 56 -
tests/tcg/cris/libc/check_settls1.c | 45 -
tests/tcg/cris/libc/check_sigalrm.c | 26 -
tests/tcg/cris/libc/check_stat1.c | 16 -
tests/tcg/cris/libc/check_stat2.c | 20 -
tests/tcg/cris/libc/check_stat3.c | 25 -
tests/tcg/cris/libc/check_stat4.c | 27 -
tests/tcg/cris/libc/check_swap.c | 76 -
tests/tcg/cris/libc/check_time2.c | 18 -
tests/unit/test-fifo.c | 449 +++
ui/qemu-pixman.c | 1 -
ui/sdl2-input.c | 5 +
ui/sdl2.c | 19 +-
util/fifo8.c | 42 +-
fpu/softfloat-specialize.c.inc | 4 +-
target/cris/translate_v10.c.inc | 1262 -------
.gitlab-ci.d/buildtest.yml | 2 +-
.gitlab-ci.d/container-cross.yml | 5 -
.gitlab-ci.d/crossbuild-template.yml | 4 +-
hw/Kconfig | 1 -
hw/audio/trace-events | 1 -
hw/block/Kconfig | 3 -
hw/block/meson.build | 1 -
hw/char/meson.build | 1 -
hw/char/trace-events | 4 +-
hw/cris/Kconfig | 11 -
hw/cris/meson.build | 5 -
hw/dma/meson.build | 1 -
hw/intc/meson.build | 1 -
hw/meson.build | 1 -
hw/net/meson.build | 1 -
hw/net/trace-events | 5 -
hw/sh4/Kconfig | 7 -
hw/sh4/meson.build | 1 -
hw/timer/meson.build | 1 -
scripts/coverity-scan/COMPONENTS.md | 3 -
scripts/probe-gdb-support.py | 1 -
target/Kconfig | 1 -
target/cris/Kconfig | 2 -
target/cris/meson.build | 17 -
target/meson.build | 1 -
tests/data/qobject/qdict.txt | 6 -
tests/docker/Makefile.include | 1 -
.../dockerfiles/fedora-cris-cross.docker | 14 -
tests/tcg/cris/.gdbinit | 11 -
tests/tcg/cris/Makefile.target | 62 -
tests/tcg/cris/README | 1 -
tests/tcg/cris/bare/check_addcv17.s | 65 -
tests/tcg/cris/bare/check_addi.s | 57 -
tests/tcg/cris/bare/check_addiv32.s | 62 -
tests/tcg/cris/bare/check_addm.s | 96 -
tests/tcg/cris/bare/check_addq.s | 47 -
tests/tcg/cris/bare/check_addr.s | 96 -
tests/tcg/cris/bare/check_addxc.s | 91 -
tests/tcg/cris/bare/check_addxm.s | 106 -
tests/tcg/cris/bare/check_addxr.s | 96 -
tests/tcg/cris/bare/check_andc.s | 80 -
tests/tcg/cris/bare/check_andm.s | 90 -
tests/tcg/cris/bare/check_andq.s | 46 -
tests/tcg/cris/bare/check_andr.s | 95 -
tests/tcg/cris/bare/check_asr.s | 230 --
tests/tcg/cris/bare/check_ba.s | 93 -
tests/tcg/cris/bare/check_bas.s | 102 -
tests/tcg/cris/bare/check_bcc.s | 197 -
tests/tcg/cris/bare/check_boundc.s | 101 -
tests/tcg/cris/bare/check_boundr.s | 125 -
tests/tcg/cris/bare/check_btst.s | 96 -
tests/tcg/cris/bare/check_clearfv32.s | 19 -
tests/tcg/cris/bare/check_clrjmp1.s | 36 -
tests/tcg/cris/bare/check_cmp-2.s | 15 -
tests/tcg/cris/bare/check_cmpc.s | 86 -
tests/tcg/cris/bare/check_cmpm.s | 96 -
tests/tcg/cris/bare/check_cmpq.s | 75 -
tests/tcg/cris/bare/check_cmpr.s | 102 -
tests/tcg/cris/bare/check_cmpxc.s | 92 -
tests/tcg/cris/bare/check_cmpxm.s | 106 -
tests/tcg/cris/bare/check_dstep.s | 42 -
tests/tcg/cris/bare/check_jsr.s | 85 -
tests/tcg/cris/bare/check_lapc.s | 78 -
tests/tcg/cris/bare/check_lsl.s | 217 --
tests/tcg/cris/bare/check_lsr.s | 218 --
tests/tcg/cris/bare/check_mcp.s | 49 -
tests/tcg/cris/bare/check_movdelsr1.s | 33 -
tests/tcg/cris/bare/check_movecr.s | 37 -
tests/tcg/cris/bare/check_movei.s | 50 -
tests/tcg/cris/bare/check_movemr.s | 78 -
tests/tcg/cris/bare/check_movemrv32.s | 96 -
tests/tcg/cris/bare/check_mover.s | 28 -
tests/tcg/cris/bare/check_moverm.s | 45 -
tests/tcg/cris/bare/check_movmp.s | 131 -
tests/tcg/cris/bare/check_movpmv32.s | 35 -
tests/tcg/cris/bare/check_movpr.s | 28 -
tests/tcg/cris/bare/check_movprv32.s | 21 -
tests/tcg/cris/bare/check_movscr.s | 29 -
tests/tcg/cris/bare/check_movsm.s | 44 -
tests/tcg/cris/bare/check_movsr.s | 46 -
tests/tcg/cris/bare/check_movucr.s | 33 -
tests/tcg/cris/bare/check_movum.s | 40 -
tests/tcg/cris/bare/check_movur.s | 45 -
tests/tcg/cris/bare/check_mulv32.s | 51 -
tests/tcg/cris/bare/check_mulx.s | 257 --
tests/tcg/cris/bare/check_neg.s | 104 -
tests/tcg/cris/bare/check_not.s | 31 -
tests/tcg/cris/bare/check_orc.s | 71 -
tests/tcg/cris/bare/check_orm.s | 75 -
tests/tcg/cris/bare/check_orq.s | 41 -
tests/tcg/cris/bare/check_orr.s | 84 -
tests/tcg/cris/bare/check_ret.s | 25 -
tests/tcg/cris/bare/check_scc.s | 95 -
tests/tcg/cris/bare/check_subc.s | 87 -
tests/tcg/cris/bare/check_subm.s | 96 -
tests/tcg/cris/bare/check_subq.s | 52 -
tests/tcg/cris/bare/check_subr.s | 102 -
tests/tcg/cris/bare/check_xarith.s | 72 -
tests/tcg/cris/bare/crt.s | 13 -
tests/tcg/cris/bare/testutils.inc | 117 -
tests/unit/meson.build | 1 +
227 files changed, 865 insertions(+), 19137 deletions(-)
delete mode 100644 configs/devices/cris-softmmu/default.mak
delete mode 100644 configs/targets/cris-linux-user.mak
delete mode 100644 configs/targets/cris-softmmu.mak
delete mode 100644 hw/cris/boot.h
delete mode 100644 include/hw/cris/etraxfs.h
delete mode 100644 include/hw/cris/etraxfs_dma.h
delete mode 100644 linux-user/cris/sockbits.h
delete mode 100644 linux-user/cris/syscall_nr.h
delete mode 100644 linux-user/cris/target_cpu.h
delete mode 100644 linux-user/cris/target_elf.h
delete mode 100644 linux-user/cris/target_errno_defs.h
delete mode 100644 linux-user/cris/target_fcntl.h
delete mode 100644 linux-user/cris/target_mman.h
delete mode 100644 linux-user/cris/target_prctl.h
delete mode 100644 linux-user/cris/target_proc.h
delete mode 100644 linux-user/cris/target_resource.h
delete mode 100644 linux-user/cris/target_signal.h
delete mode 100644 linux-user/cris/target_structs.h
delete mode 100644 linux-user/cris/target_syscall.h
delete mode 100644 linux-user/cris/termbits.h
delete mode 100644 target/cris/cpu-param.h
delete mode 100644 target/cris/cpu-qom.h
delete mode 100644 target/cris/cpu.h
delete mode 100644 target/cris/crisv10-decode.h
delete mode 100644 target/cris/crisv32-decode.h
delete mode 100644 target/cris/helper.h
delete mode 100644 target/cris/mmu.h
delete mode 100644 target/cris/opcode-cris.h
delete mode 100644 tests/tcg/cris/libc/crisutils.h
delete mode 100644 tests/tcg/cris/libc/sys.h
delete mode 100644 hw/block/tc58128.c
delete mode 100644 hw/char/etraxfs_ser.c
delete mode 100644 hw/cris/axis_dev88.c
delete mode 100644 hw/cris/boot.c
delete mode 100644 hw/dma/etraxfs_dma.c
delete mode 100644 hw/intc/etraxfs_pic.c
delete mode 100644 hw/net/etraxfs_eth.c
delete mode 100644 hw/sh4/shix.c
delete mode 100644 hw/timer/etraxfs_timer.c
delete mode 100644 linux-user/cris/cpu_loop.c
delete mode 100644 linux-user/cris/signal.c
delete mode 100644 target/cris/cpu.c
delete mode 100644 target/cris/gdbstub.c
delete mode 100644 target/cris/helper.c
delete mode 100644 target/cris/machine.c
delete mode 100644 target/cris/mmu.c
delete mode 100644 target/cris/op_helper.c
delete mode 100644 target/cris/translate.c
delete mode 100644 tests/tcg/cris/bare/sys.c
delete mode 100644 tests/tcg/cris/libc/check_abs.c
delete mode 100644 tests/tcg/cris/libc/check_addc.c
delete mode 100644 tests/tcg/cris/libc/check_addcm.c
delete mode 100644 tests/tcg/cris/libc/check_addo.c
delete mode 100644 tests/tcg/cris/libc/check_addoq.c
delete mode 100644 tests/tcg/cris/libc/check_bound.c
delete mode 100644 tests/tcg/cris/libc/check_ftag.c
delete mode 100644 tests/tcg/cris/libc/check_gcctorture_pr28634-1.c
delete mode 100644 tests/tcg/cris/libc/check_gcctorture_pr28634.c
delete mode 100644 tests/tcg/cris/libc/check_glibc_kernelversion.c
delete mode 100644 tests/tcg/cris/libc/check_hello.c
delete mode 100644 tests/tcg/cris/libc/check_int64.c
delete mode 100644 tests/tcg/cris/libc/check_lz.c
delete mode 100644 tests/tcg/cris/libc/check_mapbrk.c
delete mode 100644 tests/tcg/cris/libc/check_mmap1.c
delete mode 100644 tests/tcg/cris/libc/check_mmap2.c
delete mode 100644 tests/tcg/cris/libc/check_mmap3.c
delete mode 100644 tests/tcg/cris/libc/check_moveq.c
delete mode 100644 tests/tcg/cris/libc/check_openpf1.c
delete mode 100644 tests/tcg/cris/libc/check_openpf2.c
delete mode 100644 tests/tcg/cris/libc/check_openpf3.c
delete mode 100644 tests/tcg/cris/libc/check_openpf5.c
delete mode 100644 tests/tcg/cris/libc/check_settls1.c
delete mode 100644 tests/tcg/cris/libc/check_sigalrm.c
delete mode 100644 tests/tcg/cris/libc/check_stat1.c
delete mode 100644 tests/tcg/cris/libc/check_stat2.c
delete mode 100644 tests/tcg/cris/libc/check_stat3.c
delete mode 100644 tests/tcg/cris/libc/check_stat4.c
delete mode 100644 tests/tcg/cris/libc/check_swap.c
delete mode 100644 tests/tcg/cris/libc/check_time2.c
create mode 100644 tests/unit/test-fifo.c
delete mode 100644 target/cris/translate_v10.c.inc
delete mode 100644 hw/cris/Kconfig
delete mode 100644 hw/cris/meson.build
delete mode 100644 target/cris/Kconfig
delete mode 100644 target/cris/meson.build
delete mode 100644 tests/docker/dockerfiles/fedora-cris-cross.docker
delete mode 100644 tests/tcg/cris/.gdbinit
delete mode 100644 tests/tcg/cris/Makefile.target
delete mode 100644 tests/tcg/cris/README
delete mode 100644 tests/tcg/cris/bare/check_addcv17.s
delete mode 100644 tests/tcg/cris/bare/check_addi.s
delete mode 100644 tests/tcg/cris/bare/check_addiv32.s
delete mode 100644 tests/tcg/cris/bare/check_addm.s
delete mode 100644 tests/tcg/cris/bare/check_addq.s
delete mode 100644 tests/tcg/cris/bare/check_addr.s
delete mode 100644 tests/tcg/cris/bare/check_addxc.s
delete mode 100644 tests/tcg/cris/bare/check_addxm.s
delete mode 100644 tests/tcg/cris/bare/check_addxr.s
delete mode 100644 tests/tcg/cris/bare/check_andc.s
delete mode 100644 tests/tcg/cris/bare/check_andm.s
delete mode 100644 tests/tcg/cris/bare/check_andq.s
delete mode 100644 tests/tcg/cris/bare/check_andr.s
delete mode 100644 tests/tcg/cris/bare/check_asr.s
delete mode 100644 tests/tcg/cris/bare/check_ba.s
delete mode 100644 tests/tcg/cris/bare/check_bas.s
delete mode 100644 tests/tcg/cris/bare/check_bcc.s
delete mode 100644 tests/tcg/cris/bare/check_boundc.s
delete mode 100644 tests/tcg/cris/bare/check_boundr.s
delete mode 100644 tests/tcg/cris/bare/check_btst.s
delete mode 100644 tests/tcg/cris/bare/check_clearfv32.s
delete mode 100644 tests/tcg/cris/bare/check_clrjmp1.s
delete mode 100644 tests/tcg/cris/bare/check_cmp-2.s
delete mode 100644 tests/tcg/cris/bare/check_cmpc.s
delete mode 100644 tests/tcg/cris/bare/check_cmpm.s
delete mode 100644 tests/tcg/cris/bare/check_cmpq.s
delete mode 100644 tests/tcg/cris/bare/check_cmpr.s
delete mode 100644 tests/tcg/cris/bare/check_cmpxc.s
delete mode 100644 tests/tcg/cris/bare/check_cmpxm.s
delete mode 100644 tests/tcg/cris/bare/check_dstep.s
delete mode 100644 tests/tcg/cris/bare/check_jsr.s
delete mode 100644 tests/tcg/cris/bare/check_lapc.s
delete mode 100644 tests/tcg/cris/bare/check_lsl.s
delete mode 100644 tests/tcg/cris/bare/check_lsr.s
delete mode 100644 tests/tcg/cris/bare/check_mcp.s
delete mode 100644 tests/tcg/cris/bare/check_movdelsr1.s
delete mode 100644 tests/tcg/cris/bare/check_movecr.s
delete mode 100644 tests/tcg/cris/bare/check_movei.s
delete mode 100644 tests/tcg/cris/bare/check_movemr.s
delete mode 100644 tests/tcg/cris/bare/check_movemrv32.s
delete mode 100644 tests/tcg/cris/bare/check_mover.s
delete mode 100644 tests/tcg/cris/bare/check_moverm.s
delete mode 100644 tests/tcg/cris/bare/check_movmp.s
delete mode 100644 tests/tcg/cris/bare/check_movpmv32.s
delete mode 100644 tests/tcg/cris/bare/check_movpr.s
delete mode 100644 tests/tcg/cris/bare/check_movprv32.s
delete mode 100644 tests/tcg/cris/bare/check_movscr.s
delete mode 100644 tests/tcg/cris/bare/check_movsm.s
delete mode 100644 tests/tcg/cris/bare/check_movsr.s
delete mode 100644 tests/tcg/cris/bare/check_movucr.s
delete mode 100644 tests/tcg/cris/bare/check_movum.s
delete mode 100644 tests/tcg/cris/bare/check_movur.s
delete mode 100644 tests/tcg/cris/bare/check_mulv32.s
delete mode 100644 tests/tcg/cris/bare/check_mulx.s
delete mode 100644 tests/tcg/cris/bare/check_neg.s
delete mode 100644 tests/tcg/cris/bare/check_not.s
delete mode 100644 tests/tcg/cris/bare/check_orc.s
delete mode 100644 tests/tcg/cris/bare/check_orm.s
delete mode 100644 tests/tcg/cris/bare/check_orq.s
delete mode 100644 tests/tcg/cris/bare/check_orr.s
delete mode 100644 tests/tcg/cris/bare/check_ret.s
delete mode 100644 tests/tcg/cris/bare/check_scc.s
delete mode 100644 tests/tcg/cris/bare/check_subc.s
delete mode 100644 tests/tcg/cris/bare/check_subm.s
delete mode 100644 tests/tcg/cris/bare/check_subq.s
delete mode 100644 tests/tcg/cris/bare/check_subr.s
delete mode 100644 tests/tcg/cris/bare/check_xarith.s
delete mode 100644 tests/tcg/cris/bare/crt.s
delete mode 100644 tests/tcg/cris/bare/testutils.inc
--
2.45.2
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PULL v2 44/61] hw/sensor/tmp105: Coding style fixes
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 45/61] hw/sensor/tmp105: Use registerfields API Philippe Mathieu-Daudé
` (12 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Guenter Roeck <linux@roeck-us.net>
Coding style asks for no space between variable and "++". The next patch
in this series will change one of those assignments. Instead of changing
just one with that patch, change all of them for consistency.
While at it, also fix other coding style problems reported by checkpatch.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20240906154911.86803-2-philmd@linaro.org>
---
hw/sensor/tmp105.c | 44 ++++++++++++++++++++++++--------------------
1 file changed, 24 insertions(+), 20 deletions(-)
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index a8730d0b7f..ad97c9684c 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -29,16 +29,17 @@
static void tmp105_interrupt_update(TMP105State *s)
{
- qemu_set_irq(s->pin, s->alarm ^ ((~s->config >> 2) & 1)); /* POL */
+ qemu_set_irq(s->pin, s->alarm ^ ((~s->config >> 2) & 1)); /* POL */
}
static void tmp105_alarm_update(TMP105State *s)
{
- if ((s->config >> 0) & 1) { /* SD */
- if ((s->config >> 7) & 1) /* OS */
- s->config &= ~(1 << 7); /* OS */
- else
+ if ((s->config >> 0) & 1) { /* SD */
+ if ((s->config >> 7) & 1) { /* OS */
+ s->config &= ~(1 << 7); /* OS */
+ } else {
return;
+ }
}
if (s->config >> 1 & 1) {
@@ -89,7 +90,8 @@ static void tmp105_get_temperature(Object *obj, Visitor *v, const char *name,
visit_type_int(v, name, &value, errp);
}
-/* Units are 0.001 centigrades relative to 0 C. s->temperature is 8.8
+/*
+ * Units are 0.001 centigrades relative to 0 C. s->temperature is 8.8
* fixed point, so units are 1/256 centigrades. A simple ratio will do.
*/
static void tmp105_set_temperature(Object *obj, Visitor *v, const char *name,
@@ -118,30 +120,30 @@ static void tmp105_read(TMP105State *s)
{
s->len = 0;
- if ((s->config >> 1) & 1) { /* TM */
+ if ((s->config >> 1) & 1) { /* TM */
s->alarm = 0;
tmp105_interrupt_update(s);
}
switch (s->pointer & 3) {
case TMP105_REG_TEMPERATURE:
- s->buf[s->len ++] = (((uint16_t) s->temperature) >> 8);
- s->buf[s->len ++] = (((uint16_t) s->temperature) >> 0) &
- (0xf0 << ((~s->config >> 5) & 3)); /* R */
+ s->buf[s->len++] = (((uint16_t) s->temperature) >> 8);
+ s->buf[s->len++] = (((uint16_t) s->temperature) >> 0) &
+ (0xf0 << ((~s->config >> 5) & 3)); /* R */
break;
case TMP105_REG_CONFIG:
- s->buf[s->len ++] = s->config;
+ s->buf[s->len++] = s->config;
break;
case TMP105_REG_T_LOW:
- s->buf[s->len ++] = ((uint16_t) s->limit[0]) >> 8;
- s->buf[s->len ++] = ((uint16_t) s->limit[0]) >> 0;
+ s->buf[s->len++] = ((uint16_t) s->limit[0]) >> 8;
+ s->buf[s->len++] = ((uint16_t) s->limit[0]) >> 0;
break;
case TMP105_REG_T_HIGH:
- s->buf[s->len ++] = ((uint16_t) s->limit[1]) >> 8;
- s->buf[s->len ++] = ((uint16_t) s->limit[1]) >> 0;
+ s->buf[s->len++] = ((uint16_t) s->limit[1]) >> 8;
+ s->buf[s->len++] = ((uint16_t) s->limit[1]) >> 0;
break;
}
}
@@ -153,18 +155,20 @@ static void tmp105_write(TMP105State *s)
break;
case TMP105_REG_CONFIG:
- if (s->buf[0] & ~s->config & (1 << 0)) /* SD */
+ if (s->buf[0] & ~s->config & (1 << 0)) { /* SD */
printf("%s: TMP105 shutdown\n", __func__);
+ }
s->config = s->buf[0];
- s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
+ s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
tmp105_alarm_update(s);
break;
case TMP105_REG_T_LOW:
case TMP105_REG_T_HIGH:
- if (s->len >= 3)
+ if (s->len >= 3) {
s->limit[s->pointer & 1] = (int16_t)
((((uint16_t) s->buf[0]) << 8) | s->buf[1]);
+ }
tmp105_alarm_update(s);
break;
}
@@ -175,7 +179,7 @@ static uint8_t tmp105_rx(I2CSlave *i2c)
TMP105State *s = TMP105(i2c);
if (s->len < 2) {
- return s->buf[s->len ++];
+ return s->buf[s->len++];
} else {
return 0xff;
}
@@ -215,7 +219,7 @@ static int tmp105_post_load(void *opaque, int version_id)
{
TMP105State *s = opaque;
- s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
+ s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
tmp105_interrupt_update(s);
return 0;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 45/61] hw/sensor/tmp105: Use registerfields API
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 44/61] hw/sensor/tmp105: Coding style fixes Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 46/61] hw/sensor/tmp105: Pass 'oneshot' argument to tmp105_alarm_update() Philippe Mathieu-Daudé
` (11 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
To improve readability, use the registerfields API.
Define the register bits with FIELD(), and use the
FIELD_EX8() and FIELD_DP8() macros. Remove the
abbreviations in comments.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Message-Id: <20240906154911.86803-3-philmd@linaro.org>
---
hw/sensor/tmp105.c | 30 +++++++++++++++++++-----------
1 file changed, 19 insertions(+), 11 deletions(-)
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index ad97c9684c..150d09b278 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -26,23 +26,31 @@
#include "qapi/error.h"
#include "qapi/visitor.h"
#include "qemu/module.h"
+#include "hw/registerfields.h"
+
+FIELD(CONFIG, SHUTDOWN_MODE, 0, 1)
+FIELD(CONFIG, THERMOSTAT_MODE, 1, 1)
+FIELD(CONFIG, POLARITY, 2, 1)
+FIELD(CONFIG, FAULT_QUEUE, 3, 2)
+FIELD(CONFIG, CONVERTER_RESOLUTION, 5, 2)
+FIELD(CONFIG, ONE_SHOT, 7, 1)
static void tmp105_interrupt_update(TMP105State *s)
{
- qemu_set_irq(s->pin, s->alarm ^ ((~s->config >> 2) & 1)); /* POL */
+ qemu_set_irq(s->pin, s->alarm ^ FIELD_EX8(~s->config, CONFIG, POLARITY));
}
static void tmp105_alarm_update(TMP105State *s)
{
- if ((s->config >> 0) & 1) { /* SD */
- if ((s->config >> 7) & 1) { /* OS */
- s->config &= ~(1 << 7); /* OS */
+ if (FIELD_EX8(s->config, CONFIG, SHUTDOWN_MODE)) {
+ if (FIELD_EX8(s->config, CONFIG, ONE_SHOT)) {
+ s->config = FIELD_DP8(s->config, CONFIG, ONE_SHOT, 0);
} else {
return;
}
}
- if (s->config >> 1 & 1) {
+ if (FIELD_EX8(s->config, CONFIG, THERMOSTAT_MODE)) {
/*
* TM == 1 : Interrupt mode. We signal Alert when the
* temperature rises above T_high, and expect the guest to clear
@@ -120,7 +128,7 @@ static void tmp105_read(TMP105State *s)
{
s->len = 0;
- if ((s->config >> 1) & 1) { /* TM */
+ if (FIELD_EX8(s->config, CONFIG, THERMOSTAT_MODE)) {
s->alarm = 0;
tmp105_interrupt_update(s);
}
@@ -129,7 +137,7 @@ static void tmp105_read(TMP105State *s)
case TMP105_REG_TEMPERATURE:
s->buf[s->len++] = (((uint16_t) s->temperature) >> 8);
s->buf[s->len++] = (((uint16_t) s->temperature) >> 0) &
- (0xf0 << ((~s->config >> 5) & 3)); /* R */
+ (0xf0 << (FIELD_EX8(~s->config, CONFIG, CONVERTER_RESOLUTION)));
break;
case TMP105_REG_CONFIG:
@@ -155,11 +163,11 @@ static void tmp105_write(TMP105State *s)
break;
case TMP105_REG_CONFIG:
- if (s->buf[0] & ~s->config & (1 << 0)) { /* SD */
+ if (FIELD_EX8(s->buf[0] & ~s->config, CONFIG, SHUTDOWN_MODE)) {
printf("%s: TMP105 shutdown\n", __func__);
}
s->config = s->buf[0];
- s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
+ s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
tmp105_alarm_update(s);
break;
@@ -219,7 +227,7 @@ static int tmp105_post_load(void *opaque, int version_id)
{
TMP105State *s = opaque;
- s->faults = tmp105_faultq[(s->config >> 3) & 3]; /* F */
+ s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
tmp105_interrupt_update(s);
return 0;
@@ -277,7 +285,7 @@ static void tmp105_reset(I2CSlave *i2c)
s->temperature = 0;
s->pointer = 0;
s->config = 0;
- s->faults = tmp105_faultq[(s->config >> 3) & 3];
+ s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
s->alarm = 0;
s->detect_falling = false;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 46/61] hw/sensor/tmp105: Pass 'oneshot' argument to tmp105_alarm_update()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 44/61] hw/sensor/tmp105: Coding style fixes Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 45/61] hw/sensor/tmp105: Use registerfields API Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 47/61] hw/sensor/tmp105: OS (one-shot) bit in config register always returns 0 Philippe Mathieu-Daudé
` (10 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
The next commit will clear the ONE_SHOT bit in the WRITE
path (to keep the READ path trivial). As a preliminary step,
pass the 'oneshot' value as argument to tmp105_alarm_update().
No logical change intended.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Message-Id: <20240906154911.86803-4-philmd@linaro.org>
---
hw/sensor/tmp105.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index 150d09b278..6740200aea 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -40,10 +40,10 @@ static void tmp105_interrupt_update(TMP105State *s)
qemu_set_irq(s->pin, s->alarm ^ FIELD_EX8(~s->config, CONFIG, POLARITY));
}
-static void tmp105_alarm_update(TMP105State *s)
+static void tmp105_alarm_update(TMP105State *s, bool one_shot)
{
if (FIELD_EX8(s->config, CONFIG, SHUTDOWN_MODE)) {
- if (FIELD_EX8(s->config, CONFIG, ONE_SHOT)) {
+ if (one_shot) {
s->config = FIELD_DP8(s->config, CONFIG, ONE_SHOT, 0);
} else {
return;
@@ -119,7 +119,7 @@ static void tmp105_set_temperature(Object *obj, Visitor *v, const char *name,
s->temperature = (int16_t) (temp * 256 / 1000);
- tmp105_alarm_update(s);
+ tmp105_alarm_update(s, false);
}
static const int tmp105_faultq[4] = { 1, 2, 4, 6 };
@@ -168,7 +168,7 @@ static void tmp105_write(TMP105State *s)
}
s->config = s->buf[0];
s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
- tmp105_alarm_update(s);
+ tmp105_alarm_update(s, FIELD_EX8(s->buf[0], CONFIG, ONE_SHOT));
break;
case TMP105_REG_T_LOW:
@@ -177,7 +177,7 @@ static void tmp105_write(TMP105State *s)
s->limit[s->pointer & 1] = (int16_t)
((((uint16_t) s->buf[0]) << 8) | s->buf[1]);
}
- tmp105_alarm_update(s);
+ tmp105_alarm_update(s, false);
break;
}
}
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 47/61] hw/sensor/tmp105: OS (one-shot) bit in config register always returns 0
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (2 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 46/61] hw/sensor/tmp105: Pass 'oneshot' argument to tmp105_alarm_update() Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 48/61] hw/sensor/tmp105: Lower 4 bit of limit registers are always 0 Philippe Mathieu-Daudé
` (9 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
Per datasheet, "ONE-SHOT (OS)", the OS bit always returns 0 when reading
the configuration register.
Clear the ONE_SHOT bit in the WRITE path. Now than the READ path is
simpler, we can also simplify tmp105_alarm_update().
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20240906154911.86803-5-philmd@linaro.org>
---
hw/sensor/tmp105.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index 6740200aea..f5101af919 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -42,12 +42,8 @@ static void tmp105_interrupt_update(TMP105State *s)
static void tmp105_alarm_update(TMP105State *s, bool one_shot)
{
- if (FIELD_EX8(s->config, CONFIG, SHUTDOWN_MODE)) {
- if (one_shot) {
- s->config = FIELD_DP8(s->config, CONFIG, ONE_SHOT, 0);
- } else {
- return;
- }
+ if (FIELD_EX8(s->config, CONFIG, SHUTDOWN_MODE) && !one_shot) {
+ return;
}
if (FIELD_EX8(s->config, CONFIG, THERMOSTAT_MODE)) {
@@ -166,7 +162,7 @@ static void tmp105_write(TMP105State *s)
if (FIELD_EX8(s->buf[0] & ~s->config, CONFIG, SHUTDOWN_MODE)) {
printf("%s: TMP105 shutdown\n", __func__);
}
- s->config = s->buf[0];
+ s->config = FIELD_DP8(s->buf[0], CONFIG, ONE_SHOT, 0);
s->faults = tmp105_faultq[FIELD_EX8(s->config, CONFIG, FAULT_QUEUE)];
tmp105_alarm_update(s, FIELD_EX8(s->buf[0], CONFIG, ONE_SHOT));
break;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 48/61] hw/sensor/tmp105: Lower 4 bit of limit registers are always 0
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (3 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 47/61] hw/sensor/tmp105: OS (one-shot) bit in config register always returns 0 Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 51/61] hw/char: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
` (8 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Guenter Roeck <linux@roeck-us.net>
Per datasheet, "HIGH AND LOW LIMIT REGISTERS", the lower 4 bit
of the limit registers are unused and always report 0.
The lower 4 bit should not be used for temperature comparisons,
so mask the unused bits before storing the limits.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20240906154911.86803-6-philmd@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/sensor/tmp105.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/sensor/tmp105.c b/hw/sensor/tmp105.c
index f5101af919..9d7b911f59 100644
--- a/hw/sensor/tmp105.c
+++ b/hw/sensor/tmp105.c
@@ -171,7 +171,7 @@ static void tmp105_write(TMP105State *s)
case TMP105_REG_T_HIGH:
if (s->len >= 3) {
s->limit[s->pointer & 1] = (int16_t)
- ((((uint16_t) s->buf[0]) << 8) | s->buf[1]);
+ ((((uint16_t) s->buf[0]) << 8) | (s->buf[1] & 0xf0));
}
tmp105_alarm_update(s, false);
break;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 51/61] hw/char: replace assert(0) with g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (4 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 48/61] hw/sensor/tmp105: Lower 4 bit of limit registers are always 0 Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 52/61] hw/core: " Philippe Mathieu-Daudé
` (7 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
assert(0);
| }
| ^
Solve that by unifying the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-5-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/char/avr_usart.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/char/avr_usart.c b/hw/char/avr_usart.c
index 5bcf9db0b7..e738a2ca97 100644
--- a/hw/char/avr_usart.c
+++ b/hw/char/avr_usart.c
@@ -86,7 +86,7 @@ static void update_char_mask(AVRUsartState *usart)
usart->char_mask = 0b11111111;
break;
default:
- assert(0);
+ g_assert_not_reached();
}
}
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 52/61] hw/core: replace assert(0) with g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (5 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 51/61] hw/char: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 53/61] hw/watchdog: " Philippe Mathieu-Daudé
` (6 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
assert(0);
| }
| ^
Solve that by unifying the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-6-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/core/numa.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/core/numa.c b/hw/core/numa.c
index fb81c1ed51..1b5f44baea 100644
--- a/hw/core/numa.c
+++ b/hw/core/numa.c
@@ -380,7 +380,7 @@ void parse_numa_hmat_lb(NumaState *numa_state, NumaHmatLBOptions *node,
}
lb_data.data = node->bandwidth;
} else {
- assert(0);
+ g_assert_not_reached();
}
g_array_append_val(hmat_lb->list, lb_data);
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 53/61] hw/watchdog: replace assert(0) with g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (6 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 52/61] hw/core: " Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 54/61] hw/gpio: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
` (5 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
assert(0);
| }
| ^
Solve that by unifying the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard W.M. Jones <rjones@redhat.com>
Message-ID: <20240910221606.1817478-8-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/watchdog/watchdog.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/watchdog/watchdog.c b/hw/watchdog/watchdog.c
index 955046161b..d0ce3c4ac5 100644
--- a/hw/watchdog/watchdog.c
+++ b/hw/watchdog/watchdog.c
@@ -85,7 +85,7 @@ void watchdog_perform_action(void)
break;
default:
- assert(0);
+ g_assert_not_reached();
}
}
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 54/61] hw/gpio: remove break after g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (7 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 53/61] hw/watchdog: " Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 55/61] hw/misc: " Philippe Mathieu-Daudé
` (4 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
g_assert_not_reached();
break;
| ^^^^^
Solve that by removing the unreachable 'break' statement, unifying
the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-28-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/gpio/nrf51_gpio.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/gpio/nrf51_gpio.c b/hw/gpio/nrf51_gpio.c
index ffc7dff796..f259be651e 100644
--- a/hw/gpio/nrf51_gpio.c
+++ b/hw/gpio/nrf51_gpio.c
@@ -40,7 +40,6 @@ static bool is_connected(uint32_t config, uint32_t level)
break;
default:
g_assert_not_reached();
- break;
}
return state;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 55/61] hw/misc: remove break after g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (8 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 54/61] hw/gpio: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 56/61] hw/pci-host: " Philippe Mathieu-Daudé
` (3 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
g_assert_not_reached();
break;
| ^^^^^
Solve that by removing the unreachable 'break' statement, unifying
the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-29-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/misc/imx6_ccm.c | 1 -
hw/misc/mac_via.c | 2 --
2 files changed, 3 deletions(-)
diff --git a/hw/misc/imx6_ccm.c b/hw/misc/imx6_ccm.c
index b1def7f05b..fd5d7ce482 100644
--- a/hw/misc/imx6_ccm.c
+++ b/hw/misc/imx6_ccm.c
@@ -301,7 +301,6 @@ static uint64_t imx6_analog_get_periph_clk(IMX6CCMState *dev)
default:
/* We should never get there */
g_assert_not_reached();
- break;
}
trace_imx6_analog_get_periph_clk(freq);
diff --git a/hw/misc/mac_via.c b/hw/misc/mac_via.c
index 652395b84f..af2b2b1af3 100644
--- a/hw/misc/mac_via.c
+++ b/hw/misc/mac_via.c
@@ -495,7 +495,6 @@ static void via1_rtc_update(MOS6522Q800VIA1State *v1s)
break;
default:
g_assert_not_reached();
- break;
}
return;
}
@@ -556,7 +555,6 @@ static void via1_rtc_update(MOS6522Q800VIA1State *v1s)
break;
default:
g_assert_not_reached();
- break;
}
return;
}
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 56/61] hw/pci-host: remove break after g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (9 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 55/61] hw/misc: " Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 57/61] system: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
` (2 subsequent siblings)
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
g_assert_not_reached();
break;
| ^^^^^
Solve that by removing the unreachable 'break' statement, unifying
the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-31-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/pci-host/gt64120.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/pci-host/gt64120.c b/hw/pci-host/gt64120.c
index 33607dfbec..5855741662 100644
--- a/hw/pci-host/gt64120.c
+++ b/hw/pci-host/gt64120.c
@@ -689,7 +689,6 @@ static void gt64120_writel(void *opaque, hwaddr addr,
case GT_PCI0_CFGDATA:
/* Mapped via in gt64120_pci_mapping() */
g_assert_not_reached();
- break;
/* Interrupts */
case GT_INTRCAUSE:
@@ -933,7 +932,6 @@ static uint64_t gt64120_readl(void *opaque,
case GT_PCI0_CFGDATA:
/* Mapped via in gt64120_pci_mapping() */
g_assert_not_reached();
- break;
case GT_PCI0_CMD:
case GT_PCI0_TOR:
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 57/61] system: replace assert(0) with g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (10 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 56/61] hw/pci-host: " Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 61/61] ui: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
2024-09-13 14:30 ` [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Peter Maydell
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
assert(0);
| }
| ^
Solve that by unifying the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-11-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
system/rtc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/system/rtc.c b/system/rtc.c
index dc44576686..216d2aee3a 100644
--- a/system/rtc.c
+++ b/system/rtc.c
@@ -62,7 +62,7 @@ static time_t qemu_ref_timedate(QEMUClockType clock)
}
break;
default:
- assert(0);
+ g_assert_not_reached();
}
return value;
}
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PULL v2 61/61] ui: remove break after g_assert_not_reached()
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (11 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 57/61] system: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
@ 2024-09-12 6:52 ` Philippe Mathieu-Daudé
2024-09-13 14:30 ` [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Peter Maydell
13 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-12 6:52 UTC (permalink / raw)
To: qemu-devel
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Use of assert(false) can trip spurious control flow warnings from
some versions of GCC (i.e. using -fsanitize=thread with gcc-12):
error: control reaches end of non-void function [-Werror=return-type]
default:
g_assert_not_reached();
break;
| ^^^^^
Solve that by removing the unreachable 'break' statement, unifying
the code base on g_assert_not_reached() instead.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20240910221606.1817478-37-pierrick.bouvier@linaro.org>
[PMD: Add description suggested by Eric Blake]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
ui/qemu-pixman.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/ui/qemu-pixman.c b/ui/qemu-pixman.c
index 5ca55dd199..6cada8b45e 100644
--- a/ui/qemu-pixman.c
+++ b/ui/qemu-pixman.c
@@ -49,7 +49,6 @@ PixelFormat qemu_pixelformat_from_pixman(pixman_format_code_t format)
break;
default:
g_assert_not_reached();
- break;
}
pf.amax = (1 << pf.abits) - 1;
--
2.45.2
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PULL v2 00/61] Misc HW & UI patches for 2024-09-12
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
` (12 preceding siblings ...)
2024-09-12 6:52 ` [PULL v2 61/61] ui: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
@ 2024-09-13 14:30 ` Peter Maydell
2024-09-13 17:04 ` Philippe Mathieu-Daudé
13 siblings, 1 reply; 16+ messages in thread
From: Peter Maydell @ 2024-09-13 14:30 UTC (permalink / raw)
To: Philippe Mathieu-Daudé; +Cc: qemu-devel
On Thu, 12 Sept 2024 at 07:53, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> v2:
> - Fill Pierrick's commit description suggested by Eric Blake
> - Include TMP105 fixes from Guenter
>
> The following changes since commit a4eb31c678400472de0b4915b9154a7c20d8332f:
>
> Merge tag 'pull-testing-gdbstub-oct-100924-1' of https://gitlab.com/stsquad/qemu into staging (2024-09-11 13:17:29 +0100)
>
> are available in the Git repository at:
>
> https://github.com/philmd/qemu.git tags/hw-misc-20240912
>
> for you to fetch changes up to bd480a2baab659abe90da878bc955670691f53a8:
>
> ui: remove break after g_assert_not_reached() (2024-09-12 08:44:48 +0200)
>
> ----------------------------------------------------------------
> Misc HW & UI patches
>
> - Remove deprecated SH4 SHIX machine TC58128 NAND EEPROM (Phil)
> - Remove deprecated CRIS target (Phil)
> - Remove deprecated RISC-V 'any' CPU type (Phil)
> - Add fifo8_peek_buf() to correctly handle FIFO wraparound (Mark)
> - Minor cleanups in Designware PCIe, PL011 and loongson IPI models (Phil)
> - Fixes in TI TMP105 temperature (Guenter)
> - Convert Sun ESCC and ADB mouses to QemuInputHandler (Mark)
> - Prevent heap overflow in VIRTIO sound device (Volker)
> - Cleanups around g_assert_not_reached() call (Pierrick)
> - Add Clément as VT-d reviewer (Clément)
> - Prevent stuck modifier keys and unexpected text input on Windows (Volker)
> - Explicitly set SDL2 swap interval when OpenGL is enabled (Gert)
Fails tests on some CI configs:
https://gitlab.com/qemu-project/qemu/-/jobs/7820098438
ERROR:../tests/qtest/tmp105-test.c:103:send_and_receive: assertion
failed (i2c_get16(i2cdev, TMP105_REG_T_LOW) == 0x1234): (0x00001230 ==
0x00001234)
(test program exited with status code -6)
https://gitlab.com/qemu-project/qemu/-/jobs/7820098228
ERROR:../tests/qtest/bcm2835-i2c-test.c:84:test_i2c_read_write:
assertion failed (i2cdata == 0xad): (160 == 173)
(test program exited with status code -6)
-- PMM
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PULL v2 00/61] Misc HW & UI patches for 2024-09-12
2024-09-13 14:30 ` [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Peter Maydell
@ 2024-09-13 17:04 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-09-13 17:04 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel, Guenter Roeck
On 13/9/24 16:30, Peter Maydell wrote:
> On Thu, 12 Sept 2024 at 07:53, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>>
>> v2:
>> - Fill Pierrick's commit description suggested by Eric Blake
>> - Include TMP105 fixes from Guenter
>>
>> The following changes since commit a4eb31c678400472de0b4915b9154a7c20d8332f:
>>
>> Merge tag 'pull-testing-gdbstub-oct-100924-1' of https://gitlab.com/stsquad/qemu into staging (2024-09-11 13:17:29 +0100)
>>
>> are available in the Git repository at:
>>
>> https://github.com/philmd/qemu.git tags/hw-misc-20240912
>>
>> for you to fetch changes up to bd480a2baab659abe90da878bc955670691f53a8:
>>
>> ui: remove break after g_assert_not_reached() (2024-09-12 08:44:48 +0200)
>>
>> ----------------------------------------------------------------
>> Misc HW & UI patches
>>
>> - Remove deprecated SH4 SHIX machine TC58128 NAND EEPROM (Phil)
>> - Remove deprecated CRIS target (Phil)
>> - Remove deprecated RISC-V 'any' CPU type (Phil)
>> - Add fifo8_peek_buf() to correctly handle FIFO wraparound (Mark)
>> - Minor cleanups in Designware PCIe, PL011 and loongson IPI models (Phil)
>> - Fixes in TI TMP105 temperature (Guenter)
>> - Convert Sun ESCC and ADB mouses to QemuInputHandler (Mark)
>> - Prevent heap overflow in VIRTIO sound device (Volker)
>> - Cleanups around g_assert_not_reached() call (Pierrick)
>> - Add Clément as VT-d reviewer (Clément)
>> - Prevent stuck modifier keys and unexpected text input on Windows (Volker)
>> - Explicitly set SDL2 swap interval when OpenGL is enabled (Gert)
>
> Fails tests on some CI configs:
>
> https://gitlab.com/qemu-project/qemu/-/jobs/7820098438
>
> ERROR:../tests/qtest/tmp105-test.c:103:send_and_receive: assertion
> failed (i2c_get16(i2cdev, TMP105_REG_T_LOW) == 0x1234): (0x00001230 ==
> 0x00001234)
> (test program exited with status code -6)
>
> https://gitlab.com/qemu-project/qemu/-/jobs/7820098228
>
> ERROR:../tests/qtest/bcm2835-i2c-test.c:84:test_i2c_read_write:
> assertion failed (i2cdata == 0xad): (160 == 173)
> (test program exited with status code -6)
Sorry, I neglected to test the tmp105 patches on CI before posting v2.
Guenter I'll squash on patch #48 (lower 4 bits):
-- >8 --
diff --git a/tests/qtest/bcm2835-i2c-test.c b/tests/qtest/bcm2835-i2c-test.c
index 513ecce61d..1599194926 100644
--- a/tests/qtest/bcm2835-i2c-test.c
+++ b/tests/qtest/bcm2835-i2c-test.c
@@ -81,7 +81,7 @@ static void test_i2c_read_write(gconstpointer data)
g_assert_cmpint(i2cdata, ==, 0xde);
i2cdata = readl(base_addr + BCM2835_I2C_FIFO);
- g_assert_cmpint(i2cdata, ==, 0xad);
+ g_assert_cmpint(i2cdata, ==, 0xa0);
/* Clear flags */
writel(base_addr + BCM2835_I2C_S, BCM2835_I2C_S_DONE |
BCM2835_I2C_S_ERR |
diff --git a/tests/qtest/tmp105-test.c b/tests/qtest/tmp105-test.c
index 3678646df5..85ad4eed85 100644
--- a/tests/qtest/tmp105-test.c
+++ b/tests/qtest/tmp105-test.c
@@ -100,9 +100,9 @@ static void send_and_receive(void *obj, void *data,
QGuestAllocator *alloc)
g_assert_cmphex(value, ==, 0x14f0);
i2c_set16(i2cdev, TMP105_REG_T_LOW, 0x1234);
- g_assert_cmphex(i2c_get16(i2cdev, TMP105_REG_T_LOW), ==, 0x1234);
+ g_assert_cmphex(i2c_get16(i2cdev, TMP105_REG_T_LOW), ==, 0x1230);
i2c_set16(i2cdev, TMP105_REG_T_HIGH, 0x4231);
- g_assert_cmphex(i2c_get16(i2cdev, TMP105_REG_T_HIGH), ==, 0x4231);
+ g_assert_cmphex(i2c_get16(i2cdev, TMP105_REG_T_HIGH), ==, 0x4230);
}
---
Regards,
Phil.
^ permalink raw reply related [flat|nested] 16+ messages in thread
end of thread, other threads:[~2024-09-13 17:05 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-12 6:52 [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 44/61] hw/sensor/tmp105: Coding style fixes Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 45/61] hw/sensor/tmp105: Use registerfields API Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 46/61] hw/sensor/tmp105: Pass 'oneshot' argument to tmp105_alarm_update() Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 47/61] hw/sensor/tmp105: OS (one-shot) bit in config register always returns 0 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 48/61] hw/sensor/tmp105: Lower 4 bit of limit registers are always 0 Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 51/61] hw/char: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 52/61] hw/core: " Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 53/61] hw/watchdog: " Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 54/61] hw/gpio: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 55/61] hw/misc: " Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 56/61] hw/pci-host: " Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 57/61] system: replace assert(0) with g_assert_not_reached() Philippe Mathieu-Daudé
2024-09-12 6:52 ` [PULL v2 61/61] ui: remove break after g_assert_not_reached() Philippe Mathieu-Daudé
2024-09-13 14:30 ` [PULL v2 00/61] Misc HW & UI patches for 2024-09-12 Peter Maydell
2024-09-13 17:04 ` Philippe Mathieu-Daudé
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).