summary refs log tree commit diff stats
path: root/hw/core/qdev-properties-system.c
diff options
context:
space:
mode:
authorPaul Burton <paul.burton@imgtec.com>2016-07-22 10:55:40 +0100
committerLeon Alrae <leon.alrae@imgtec.com>2016-07-28 11:24:00 +0100
commit7f81dbb9a0e89b5306c1337e0cd0e1cea8a03f6d (patch)
treedec2a2d4cd1a42443ee3f84ce2c213f9102cfe7e /hw/core/qdev-properties-system.c
parent21a21b853a1bb606358af61e738abfb9aecbd720 (diff)
downloadfocaccia-qemu-7f81dbb9a0e89b5306c1337e0cd0e1cea8a03f6d.tar.gz
focaccia-qemu-7f81dbb9a0e89b5306c1337e0cd0e1cea8a03f6d.zip
hw/mips_malta: Fix YAMON API print routine
The print routine provided as part of the in-built bootloader had a bug
in that it attempted to use a jump instruction as part of a loop, but
the target has its upper bits zeroed leading to control flow
transferring to 0xb0000814 rather than the intended 0xbfc00814. Fix this
by using a branch instruction instead, which seems more fit for purpose.

A simple way to test this is to build a Linux kernel with EVA enabled &
attempt to boot it in QEMU. It will attempt to print a message
indicating the configuration mismatch but QEMU would previously
incorrectly jump & wind up printing a continuous stream of the letter E.

Signed-off-by: Paul Burton <paul.burton@imgtec.com>
Cc: Aurelien Jarno <aurelien@aurel32.net>
Cc: Leon Alrae <leon.alrae@imgtec.com>
Reviewed-by: Aurelien Jarno <aurelien@aurel32.net>
Reviewed-by: Leon Alrae <leon.alrae@imgtec.com>
Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
Diffstat (limited to 'hw/core/qdev-properties-system.c')
0 files changed, 0 insertions, 0 deletions