mirror of https://gitee.com/openkylin/qemu.git
exynos4210_fimd.c: fix display resize bug introduced after console revamp
In exynos4210 display update function, we were acquiring DisplaySurface pointer before calling screen resize function, not paying attention that resize procedure can replace current DisplaySurface with newly allocated one. Right thing to do is to initialize DisplaySurface AFTER a call to resize function. Signed-off-by: Igor Mitsyanko <i.mitsyanko@gmail.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
parent
24a6e7f4d9
commit
522fccbe71
|
@ -1242,7 +1242,7 @@ static void exynos4210_update_resolution(Exynos4210fimdState *s)
|
|||
static void exynos4210_fimd_update(void *opaque)
|
||||
{
|
||||
Exynos4210fimdState *s = (Exynos4210fimdState *)opaque;
|
||||
DisplaySurface *surface = qemu_console_surface(s->console);
|
||||
DisplaySurface *surface;
|
||||
Exynos4210fimdWindow *w;
|
||||
int i, line;
|
||||
hwaddr fb_line_addr, inc_size;
|
||||
|
@ -1255,11 +1255,12 @@ static void exynos4210_fimd_update(void *opaque)
|
|||
const int global_height = ((s->vidtcon[2] >> FIMD_VIDTCON2_VER_SHIFT) &
|
||||
FIMD_VIDTCON2_SIZE_MASK) + 1;
|
||||
|
||||
if (!s || !s->console || !surface_bits_per_pixel(surface) ||
|
||||
!s->enabled) {
|
||||
if (!s || !s->console || !s->enabled ||
|
||||
surface_bits_per_pixel(qemu_console_surface(s->console)) == 0) {
|
||||
return;
|
||||
}
|
||||
exynos4210_update_resolution(s);
|
||||
surface = qemu_console_surface(s->console);
|
||||
|
||||
for (i = 0; i < NUM_OF_WINDOWS; i++) {
|
||||
w = &s->window[i];
|
||||
|
|
Loading…
Reference in New Issue