Hi Zoltan, On 2/11/19 4:19 AM, BALATON Zoltan wrote: > At least two machines, the PPC mac99 and MIPS fulong2e, have an ATI > gfx chip by default (Rage 128 Pro and M6/RV100 respectively) and > guests running on these and the PMON2000 firmware of the fulong2e > expect this to be available. Fortunately these are very similar chips > so they can be mostly emulated in the same device model. This patch > adds basic emulation of these ATI VGA chips. > > While this is incomplete and currently only enough to run the MIPS > firmware and get console output with Linux, it allows the fulong2e > board to work more like the real hardware and having it in QEMU in > this state provides a way to experiment with it and allows others to > contribute to improve it (e.g. implement more 2D features to get X > running). It is only enabled for mips64le and ppc now but only the > fulong2e (which currently has no display output at all) is set to use > it by default (in a separate patch). > > Tested with Debian 8.11.0 powerpc on mac99 and the pmon_2e.bin > fulong2e firmware from https://mirrors.cloud.tencent.com/loongson/pmon/ > > Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu> > --- > default-configs/mips64el-softmmu.mak | 1 + > default-configs/ppc-softmmu.mak | 1 + > hw/display/Makefile.objs | 2 + > hw/display/ati.c | 582 > +++++++++++++++++++++++++++++++++++ > hw/display/ati_2d.c | 44 +++ > hw/display/ati_int.h | 67 ++++ > hw/display/ati_regs.h | 452 +++++++++++++++++++++++++++ > hw/display/trace-events | 4 + > 8 files changed, 1153 insertions(+) > create mode 100644 hw/display/ati.c > create mode 100644 hw/display/ati_2d.c > create mode 100644 hw/display/ati_int.h > create mode 100644 hw/display/ati_regs.h > > diff --git a/default-configs/mips64el-softmmu.mak > b/default-configs/mips64el-softmmu.mak > index 9eb1208b58..231f81eca2 100644 > --- a/default-configs/mips64el-softmmu.mak > +++ b/default-configs/mips64el-softmmu.mak > @@ -13,3 +13,4 @@ CONFIG_VT82C686=y > CONFIG_MIPS_BOSTON=y > CONFIG_FITLOADER=y > CONFIG_PCI_EXPRESS_XILINX=y > +CONFIG_ATI_VGA=y > diff --git a/default-configs/ppc-softmmu.mak b/default-configs/ppc-softmmu.mak > index 52acb7cf39..75024b298e 100644 > --- a/default-configs/ppc-softmmu.mak > +++ b/default-configs/ppc-softmmu.mak > @@ -56,6 +56,7 @@ CONFIG_DEC_PCI=y > CONFIG_IDE_MACIO=y > CONFIG_MAC_OLDWORLD=y > CONFIG_MAC_NEWWORLD=y > +CONFIG_ATI_VGA=y > > # For PReP > CONFIG_PREP=y > diff --git a/hw/display/Makefile.objs b/hw/display/Makefile.objs > index 7c4ae9a0fd..e58947e6f3 100644 > --- a/hw/display/Makefile.objs > +++ b/hw/display/Makefile.objs > @@ -53,3 +53,5 @@ virtio-gpu-3d.o-cflags := $(VIRGL_CFLAGS) > virtio-gpu-3d.o-libs += $(VIRGL_LIBS) > obj-$(CONFIG_DPCD) += dpcd.o > obj-$(CONFIG_XLNX_ZYNQMP_ARM) += xlnx_dp.o > + > +obj-$(CONFIG_ATI_VGA) += ati.o ati_2d.o > diff --git a/hw/display/ati.c b/hw/display/ati.c > new file mode 100644 > index 0000000000..18458110b2 > --- /dev/null > +++ b/hw/display/ati.c > @@ -0,0 +1,582 @@ > +/* > + * QEMU ATI SVGA emulation > + * > + * Copyright (c) 2019 BALATON Zoltan > + * > + * This work is licensed under the GNU GPL license version 2 or later. > + */ > + > +/* > + * WARNING: > + * This is very incomplete and only enough to get Linux console output yet. > + * At the moment it's little more than a frame buffer with minimal functions, > + * other more advanced features of the hardware are yet to be implemented. > + * We only aim for Rage 128 Pro (and some RV100) and 2D only at first, > + * No 3D at all yet (maybe after 2D works, but feel free to improve it) > + */ > + > +#include "ati_int.h" > +#include "ati_regs.h" > +#include "vga_regs.h" > +#include "qemu/log.h" > +#include "qemu/error-report.h" > +#include "qapi/error.h" > +#include "hw/hw.h" > +#include "ui/console.h" > +#include "trace.h" > + > +#define PCI_VENDOR_ID_ATI 0x1002 > +/* Rage128 Pro GL */ > +#define PCI_DEVICE_ID_ATI_RAGE128_PF 0x5046 > +/* Radeon RV100 (VE) */ > +#define PCI_DEVICE_ID_ATI_RADEON_QY 0x5159 > + > +enum { VGA_MODE, EXT_MODE }; > + > +static void ati_vga_switch_mode(ATIVGAState *s) > +{ > + DPRINTF("%d -> %d\n", > + s->mode, !!(s->regs.crtc_gen_cntl & CRTC2_EXT_DISP_EN)); > + if (s->regs.crtc_gen_cntl & CRTC2_EXT_DISP_EN) { > + /* Extended mode enabled */ > + s->mode = EXT_MODE; > + if (s->regs.crtc_gen_cntl & CRTC2_EN) { > + /* CRT controller enabled, use CRTC values */ > + uint32_t offs = s->regs.crtc_offset & 0x07ffffff; > + int stride = (s->regs.crtc_pitch & 0x7ff) * 8; > + int bpp = 0; > + int h, v; > + > + if (s->regs.crtc_h_total_disp == 0) { > + s->regs.crtc_h_total_disp = ((640 / 8) - 1) << 16; > + } > + if (s->regs.crtc_v_total_disp == 0) { > + s->regs.crtc_v_total_disp = (480 - 1) << 16; > + } > + h = ((s->regs.crtc_h_total_disp >> 16) + 1) * 8; > + v = (s->regs.crtc_v_total_disp >> 16) + 1; > + switch (s->regs.crtc_gen_cntl & CRTC_PIX_WIDTH_MASK) { > + case CRTC_PIX_WIDTH_4BPP: > + bpp = 4; > + break; > + case CRTC_PIX_WIDTH_8BPP: > + bpp = 8; > + break; > + case CRTC_PIX_WIDTH_15BPP: > + bpp = 15; > + break; > + case CRTC_PIX_WIDTH_16BPP: > + bpp = 16; > + break; > + case CRTC_PIX_WIDTH_24BPP: > + bpp = 24; > + break; > + case CRTC_PIX_WIDTH_32BPP: > + bpp = 32; > + break; > + default: > + qemu_log_mask(LOG_UNIMP, "Unsupported bpp value"); > + } > + assert(bpp != 0); > + DPRINTF("Switching to %dx%d %d %d @ %x\n", h, v, stride, bpp, > offs); > + vbe_ioport_write_index(&s->vga, 0, VBE_DISPI_INDEX_ENABLE); > + vbe_ioport_write_data(&s->vga, 0, VBE_DISPI_DISABLED); > + /* reset VBE regs then set up mode */ > + s->vga.vbe_regs[VBE_DISPI_INDEX_XRES] = h; > + s->vga.vbe_regs[VBE_DISPI_INDEX_YRES] = v; > + s->vga.vbe_regs[VBE_DISPI_INDEX_BPP] = bpp; > + /* enable mode via ioport so it updates vga regs */ > + vbe_ioport_write_index(&s->vga, 0, VBE_DISPI_INDEX_ENABLE); > + vbe_ioport_write_data(&s->vga, 0, VBE_DISPI_ENABLED | > + VBE_DISPI_LFB_ENABLED | VBE_DISPI_NOCLEARMEM | > + (s->regs.dac_cntl & DAC_8BIT_EN ? VBE_DISPI_8BIT_DAC : 0)); > + /* now set offset and stride after enable as that resets these */ > + if (stride) { > + vbe_ioport_write_index(&s->vga, 0, > VBE_DISPI_INDEX_VIRT_WIDTH); > + vbe_ioport_write_data(&s->vga, 0, stride); > + if (offs % stride == 0) { > + vbe_ioport_write_index(&s->vga, 0, > VBE_DISPI_INDEX_Y_OFFSET); > + vbe_ioport_write_data(&s->vga, 0, offs / stride); > + } else { > + /* FIXME what to do with this? */ > + error_report("VGA offset is not multiple of pitch, " > + "expect bad picture"); > + } > + } > + } > + } else { > + /* VGA mode enabled */ > + s->mode = VGA_MODE; > + vbe_ioport_write_index(&s->vga, 0, VBE_DISPI_INDEX_ENABLE); > + vbe_ioport_write_data(&s->vga, 0, VBE_DISPI_DISABLED); > + } > +} > + > +static uint64_t ati_reg_read_offs(uint64_t reg, int offs, unsigned int size) > +{ > + uint64_t val = 0; > + uint32_t mask; > + int i; > + > + for (i = 0; i < size; i++) { > + mask = 0xffUL << (offs + i) * 8; > + val |= reg & mask; > + } > + val >>= offs * 8; > + return val; > +} > + > +static uint64_t ati_mm_read(void *opaque, hwaddr addr, unsigned int size) > +{ > + ATIVGAState *s = opaque; > + uint64_t val = 0; > + > + switch (addr) { > + case MM_INDEX: > + val = s->regs.mm_index; > + break; > + case MM_DATA ... MM_DATA + 3: > + /* indexed access to regs or memory */ > + if (s->regs.mm_index & 0x80000000) { > + if (s->regs.mm_index <= s->vga.vram_size - size) { > + int i = size - 1; > + while (i >= 0) { > + val <<= 8; > + val |= s->vga.vram_ptr[s->regs.mm_index + i--]; > + } > + } > + } else { > + val = ati_mm_read(s, s->regs.mm_index + addr - MM_DATA, size); > + } > + break; > + case BIOS_0_SCRATCH: > + val = s->regs.bios_0_scratch; > + break; > + case CRTC_GEN_CNTL ... CRTC_GEN_CNTL + 3: > + if (addr == CRTC_GEN_CNTL && size == 4) { > + val = s->regs.crtc_gen_cntl; > + } else { > + val = ati_reg_read_offs(s->regs.crtc_gen_cntl, > + addr - CRTC_GEN_CNTL, size); > + } > + break; > + case CRTC_EXT_CNTL ... CRTC_EXT_CNTL + 3: > + if (addr == CRTC_EXT_CNTL && size == 4) { > + val = s->regs.crtc_ext_cntl; > + } else { > + val = ati_reg_read_offs(s->regs.crtc_ext_cntl, > + addr - CRTC_EXT_CNTL, size); > + } > + break; > + case DAC_CNTL: > + val = s->regs.dac_cntl; > + break; > +/* case GPIO_MONID: FIXME hook up DDC I2C here */ > + case PALETTE_INDEX: > + /* FIXME unaligned access */ > + val = vga_ioport_read(&s->vga, VGA_PEL_IR) << 16; > + val |= vga_ioport_read(&s->vga, VGA_PEL_IW) & 0xff; > + break; > + case PALETTE_DATA: > + val = vga_ioport_read(&s->vga, VGA_PEL_D); > + break; > + case CNFG_MEMSIZE: > + val = s->vga.vram_size; > + break; > + case MC_STATUS: > + val = 5; > + break; > + case RBBM_STATUS: > + case GUI_STAT: > + val = 64; /* free CMDFIFO entries */ > + break; > + case CRTC_H_TOTAL_DISP: > + val = s->regs.crtc_h_total_disp; > + break; > + case CRTC_H_SYNC_STRT_WID: > + val = s->regs.crtc_h_sync_strt_wid; > + break; > + case CRTC_V_TOTAL_DISP: > + val = s->regs.crtc_v_total_disp; > + break; > + case CRTC_V_SYNC_STRT_WID: > + val = s->regs.crtc_v_sync_strt_wid; > + break; > + case CRTC_OFFSET: > + val = s->regs.crtc_offset; > + break; > + case CRTC_OFFSET_CNTL: > + val = s->regs.crtc_offset_cntl; > + break; > + case CRTC_PITCH: > + val = s->regs.crtc_pitch; > + break; > + case 0xf00 ... 0xfff: > + val = pci_default_read_config(&s->dev, addr - 0xf00, size); > + break; > + case DST_OFFSET: > + val = s->regs.dst_offset; > + break; > + case DST_PITCH: > + val = s->regs.dst_pitch; > + break; > + case DST_WIDTH: > + val = s->regs.dst_width; > + break; > + case DST_HEIGHT: > + val = s->regs.dst_height; > + break; > + case SRC_X: > + val = s->regs.src_x; > + break; > + case SRC_Y: > + val = s->regs.src_y; > + break; > + case DST_X: > + val = s->regs.dst_x; > + break; > + case DST_Y: > + val = s->regs.dst_y; > + break; > + case DP_GUI_MASTER_CNTL: > + val = s->regs.dp_gui_master_cntl; > + break; > + case DP_BRUSH_BKGD_CLR: > + val = s->regs.dp_brush_bkgd_clr; > + break; > + case DP_BRUSH_FRGD_CLR: > + val = s->regs.dp_brush_frgd_clr; > + break; > + case DP_SRC_FRGD_CLR: > + val = s->regs.dp_src_frgd_clr; > + break; > + case DP_SRC_BKGD_CLR: > + val = s->regs.dp_src_bkgd_clr; > + break; > + case DP_CNTL: > + val = s->regs.dp_cntl; > + break; > + case DP_WRITE_MASK: > + val = s->regs.dp_write_mask; > + break; > + case DEFAULT_OFFSET: > + val = s->regs.default_offset; > + break; > + case DEFAULT_PITCH: > + val = s->regs.default_pitch; > + break; > + case DEFAULT_SC_BOTTOM_RIGHT: > + val = s->regs.default_sc_bottom_right; > + break; > + default: > + break; > + } > + trace_ati_mm_read(size, addr, val); > + > + return val; > +} > + > +static void ati_reg_write_offs(uint32_t *reg, int offs, > + uint64_t data, unsigned int size) > +{ > + int shift, i; > + uint32_t mask; > + > + for (i = 0; i < size; i++) { > + shift = (offs + i) * 8; > + mask = 0xffUL << shift; > + *reg &= ~mask; > + *reg |= (data & 0xff) << shift; > + data >>= 8;
I'd have use a pair of extract32/deposit32 but this is probably easier to singlestep. > + } > +} > + > +static void ati_mm_write(void *opaque, hwaddr addr, > + uint64_t data, unsigned int size) > +{ > + ATIVGAState *s = opaque; > + int i; > + > + trace_ati_mm_write(size, addr, data); > + switch (addr) { > + case MM_INDEX: > + s->regs.mm_index = data; > + break; > + case MM_DATA ... MM_DATA + 3: > + /* indexed access to regs or memory */ > + if (s->regs.mm_index & 0x80000000) { > + if (s->regs.mm_index <= s->vga.vram_size - size) { > + int i = 0; > + while (i < size) { > + s->vga.vram_ptr[s->regs.mm_index + i] = data & 0xff; > + data >>= 8; > + } > + } > + } else { > + ati_mm_write(s, s->regs.mm_index + addr - MM_DATA, data, size); > + } > + break; > + case BIOS_0_SCRATCH: > + s->regs.bios_0_scratch = data; > + break; > + case CRTC_GEN_CNTL ... CRTC_GEN_CNTL + 3: > + if (addr == CRTC_GEN_CNTL && size == 4) { > + s->regs.crtc_gen_cntl = data; > + } else { > + ati_reg_write_offs(&s->regs.crtc_gen_cntl, > + addr - CRTC_GEN_CNTL, data, size); > + } > + break; > + case CRTC_EXT_CNTL ... CRTC_EXT_CNTL + 3: > + if (addr == CRTC_EXT_CNTL && size == 4) { > + s->regs.crtc_ext_cntl = data; > + } else { > + ati_reg_write_offs(&s->regs.crtc_ext_cntl, > + addr - CRTC_EXT_CNTL, data, size); > + } > + if (s->regs.crtc_ext_cntl & CRT_CRTC_DISPLAY_DIS) { > + DPRINTF("Display disabled\n"); > + s->vga.ar_index &= ~0x20; > + } else { > + DPRINTF("Display enabled\n"); > + s->vga.ar_index |= 0x20; > + } > + if (!!(s->regs.crtc_gen_cntl & CRTC2_EXT_DISP_EN) != s->mode) { > + ati_vga_switch_mode(s); > + } > + break; > + case DAC_CNTL: > + s->regs.dac_cntl = data & 0xffffe3ff; > + s->vga.dac_8bit = !!(data & DAC_8BIT_EN); > + break; > +/* case GPIO_MONID: FIXME hook up DDC I2C here */ > + case PALETTE_INDEX ... PALETTE_INDEX + 3: > + if (size == 4) { > + vga_ioport_write(&s->vga, VGA_PEL_IR, (data >> 16) & 0xff); > + vga_ioport_write(&s->vga, VGA_PEL_IW, data & 0xff); > + } else { > + if (addr == PALETTE_INDEX) { > + vga_ioport_write(&s->vga, VGA_PEL_IW, data & 0xff); > + } else { > + vga_ioport_write(&s->vga, VGA_PEL_IR, data & 0xff); > + } > + } > + break; > + case PALETTE_DATA: > + for (i = 0; i < size && i < 3; i++) { > + vga_ioport_write(&s->vga, VGA_PEL_D, data & 0xff); > + data >>= 8; > + } > + break; > + case CRTC_H_TOTAL_DISP: > + s->regs.crtc_h_total_disp = data & 0x07ff07ff; > + break; > + case CRTC_H_SYNC_STRT_WID: > + s->regs.crtc_h_sync_strt_wid = data & 0x17bf1fff; > + break; > + case CRTC_V_TOTAL_DISP: > + s->regs.crtc_v_total_disp = data & 0x0fff0fff; > + break; > + case CRTC_V_SYNC_STRT_WID: > + s->regs.crtc_v_sync_strt_wid = data & 0x9f0fff; > + break; > + case CRTC_OFFSET: > + s->regs.crtc_offset = data & 0xc7ffffff; > + break; > + case CRTC_OFFSET_CNTL: > + s->regs.crtc_offset_cntl = data; /* FIXME */ > + break; > + case CRTC_PITCH: > + s->regs.crtc_pitch = data & 0x07ff07ff; > + break; > + case 0xf00 ... 0xfff: > + /* read-only copy of PCI config space so ignore writes */ > + break; > + case DST_OFFSET: > + s->regs.dst_offset = data & 0xfffffc00; > + break; > + case DST_PITCH: > + s->regs.dst_pitch = data & 0x3fff; > + break; > + case DST_WIDTH: > + s->regs.dst_width = data & 0x3fff; > + ati_2d_blit(s); > + break; > + case DST_HEIGHT: > + s->regs.dst_height = data & 0x3fff; > + break; > + case SRC_X: > + s->regs.src_x = data & 0x3fff; > + break; > + case SRC_Y: > + s->regs.src_y = data & 0x3fff; > + break; > + case DST_X: > + s->regs.dst_x = data & 0x3fff; > + break; > + case DST_Y: > + s->regs.dst_y = data & 0x3fff; > + break; > + case SRC_Y_X: > + s->regs.src_x = data & 0x3fff; > + s->regs.src_y = (data >> 16) & 0x3fff; > + break; > + case DST_Y_X: > + s->regs.dst_x = data & 0x3fff; > + s->regs.dst_y = (data >> 16) & 0x3fff; > + break; > + case DST_HEIGHT_WIDTH: > + s->regs.dst_width = data & 0x3fff; > + s->regs.dst_height = (data >> 16) & 0x3fff; > + ati_2d_blit(s); > + break; > + case DP_GUI_MASTER_CNTL: > + s->regs.dp_gui_master_cntl = data; > + break; > + case DST_WIDTH_X: > + s->regs.dst_x = data & 0x3fff; > + s->regs.dst_width = (data >> 16) & 0x3fff; > + ati_2d_blit(s); > + break; > + case SRC_X_Y: > + s->regs.src_y = data & 0x3fff; > + s->regs.src_x = (data >> 16) & 0x3fff; > + break; > + case DST_X_Y: > + s->regs.dst_y = data & 0x3fff; > + s->regs.dst_x = (data >> 16) & 0x3fff; > + break; > + case DST_HEIGHT_Y: > + s->regs.dst_y = data & 0x3fff; > + s->regs.dst_height = (data >> 16) & 0x3fff; > + break; > + case DP_BRUSH_BKGD_CLR: > + s->regs.dp_brush_bkgd_clr = data; > + break; > + case DP_BRUSH_FRGD_CLR: > + s->regs.dp_brush_frgd_clr = data; > + break; > + case DP_CNTL: > + s->regs.dp_cntl = data; > + break; > + case DP_WRITE_MASK: > + s->regs.dp_write_mask = data; > + break; > + case DEFAULT_OFFSET: > + data &= (s->dev_id == PCI_DEVICE_ID_ATI_RAGE128_PF ? > + 0x03fffc00 : 0xfffffc00); > + s->regs.default_offset = data; > + break; > + case DEFAULT_PITCH: > + if (s->dev_id == PCI_DEVICE_ID_ATI_RAGE128_PF) { > + s->regs.default_pitch = data & 0x103ff; > + } > + break; > + case DEFAULT_SC_BOTTOM_RIGHT: > + s->regs.default_sc_bottom_right = data & 0x3fff3fff; > + break; > + default: > + break; > + } > +} > + > +static const MemoryRegionOps ati_mm_ops = { > + .read = ati_mm_read, > + .write = ati_mm_write, > + .endianness = DEVICE_LITTLE_ENDIAN, > +}; > + > +static void ati_vga_realize(PCIDevice *dev, Error **errp) > +{ > + ATIVGAState *s = ATI_VGA(dev); > + VGACommonState *vga = &s->vga; > + > + if (s->dev_id != PCI_DEVICE_ID_ATI_RAGE128_PF && > + s->dev_id != PCI_DEVICE_ID_ATI_RADEON_QY) { > + error_setg(errp, "Unknown ATI VGA device id, " > + "only 0x5046 and 0x5159 are supported"); > + return; > + } > + if (s->dev_id == PCI_DEVICE_ID_ATI_RADEON_QY && > + s->vga.vram_size_mb < 16) { > + warn_report("Too small video memory for device id"); > + s->vga.vram_size_mb = 16; > + } > + > + /* init vga compat bits */ > + vga_common_init(vga, OBJECT(s)); > + vga_init(vga, OBJECT(s), pci_address_space(dev), > + pci_address_space_io(dev), true); > + vga->con = graphic_console_init(DEVICE(s), 0, s->vga.hw_ops, &s->vga); > + > + /* mmio register space */ > + memory_region_init_io(&s->mm, OBJECT(s), &ati_mm_ops, s, > + "ati.mmregs", 0x4000); > + /* io space is alias to beginning of mmregs */ > + memory_region_init_alias(&s->io, OBJECT(s), "ati.io", &s->mm, 0, 0x100); > + > + pci_register_bar(dev, 0, PCI_BASE_ADDRESS_MEM_PREFETCH, &vga->vram); > + pci_register_bar(dev, 1, PCI_BASE_ADDRESS_SPACE_IO, &s->io); > + pci_register_bar(dev, 2, PCI_BASE_ADDRESS_SPACE_MEMORY, &s->mm); > +} > + > +static void ati_vga_reset(DeviceState *dev) > +{ > + ATIVGAState *s = ATI_VGA(dev); > + PCIDevice *pd = PCI_DEVICE(dev); > + > + pci_set_word(pd->config + PCI_DEVICE_ID, s->dev_id); > + /* reset vga */ > + vga_common_reset(&s->vga); > + s->mode = VGA_MODE; > +} > + > +static void ati_vga_exit(PCIDevice *dev) > +{ > + ATIVGAState *s = ATI_VGA(dev); > + > + graphic_console_close(s->vga.con); > +} > + > +static Property ati_vga_properties[] = { > + DEFINE_PROP_UINT32("vgamem_mb", ATIVGAState, vga.vram_size_mb, 16), > + DEFINE_PROP_UINT16("device_id", ATIVGAState, dev_id, > + PCI_DEVICE_ID_ATI_RAGE128_PF), > + DEFINE_PROP_END_OF_LIST() > +}; > + > +static void ati_vga_class_init(ObjectClass *klass, void *data) > +{ > + DeviceClass *dc = DEVICE_CLASS(klass); > + PCIDeviceClass *k = PCI_DEVICE_CLASS(klass); > + > + dc->reset = ati_vga_reset; > + dc->props = ati_vga_properties; > + dc->hotpluggable = false; > + set_bit(DEVICE_CATEGORY_DISPLAY, dc->categories); > + > + k->class_id = PCI_CLASS_DISPLAY_VGA; > + k->vendor_id = PCI_VENDOR_ID_ATI; > + k->device_id = PCI_DEVICE_ID_ATI_RAGE128_PF; > + k->romfile = "vgabios-stdvga.bin"; > + k->realize = ati_vga_realize; > + k->exit = ati_vga_exit; > +} > + > +static const TypeInfo ati_vga_info = { > + .name = TYPE_ATI_VGA, > + .parent = TYPE_PCI_DEVICE, > + .instance_size = sizeof(ATIVGAState), > + .class_init = ati_vga_class_init, > + .interfaces = (InterfaceInfo[]) { > + { INTERFACE_CONVENTIONAL_PCI_DEVICE }, > + { }, > + }, > +}; > + > +static void ati_vga_register_types(void) > +{ > + type_register_static(&ati_vga_info); > +} > + > +type_init(ati_vga_register_types) > diff --git a/hw/display/ati_2d.c b/hw/display/ati_2d.c > new file mode 100644 > index 0000000000..ca3d296b23 > --- /dev/null > +++ b/hw/display/ati_2d.c > @@ -0,0 +1,44 @@ > +/* > + * QEMU ATI SVGA emulation > + * 2D engine functions > + * > + * Copyright (c) 2019 BALATON Zoltan > + * > + * This work is licensed under the GNU GPL license version 2 or later. > + */ > + > +#include "ati_int.h" > +#include "ati_regs.h" > +#include "qemu/log.h" > + > +/* > + * NOTE: > + * This is 2D _acceleration_ and supposed to be fast. Therefore, don't try to > + * reinvent the wheel (unlikely to get better with a naive implementation > than > + * existing libraries) and avoid (poorly) reimplementing gfx primitives. > + * That is unnecessary and would become a performance problem. Instead, try > to > + * map to and reuse existing optimised facilities (e.g. pixman) wherever > + * possible. > + */ > + > +void ati_2d_blit(ATIVGAState *s) > +{ > + /* FIXME it is really much more complex than this and will need to be */ > + /* rewritten but for now as a start just to get some console output: */ > + if ((s->regs.dp_gui_master_cntl & GMC_ROP3_MASK) == ROP3_SRCCOPY) { > + DisplaySurface *ds = qemu_console_surface(s->vga.con); > + DPRINTF("%p %p, %d %d, (%d,%d) -> (%d,%d) %dx%d\n", ds->image, > + ds->image, surface_stride(ds), surface_bits_per_pixel(ds), > + s->regs.src_x, s->regs.src_y, s->regs.dst_x, s->regs.dst_y, > + s->regs.dst_width, s->regs.dst_height); > + pixman_image_composite(PIXMAN_OP_SRC, ds->image, NULL, ds->image, > + s->regs.src_x, s->regs.src_y, 0, 0, > + s->regs.dst_x, s->regs.dst_y, > + s->regs.dst_width, s->regs.dst_height); > + memory_region_set_dirty(&s->vga.vram, s->vga.vbe_start_addr + > + s->regs.dst_y * surface_stride(ds), > + s->regs.dst_height * surface_stride(ds)); > + } else { > + qemu_log_mask(LOG_UNIMP, "Unimplemented ati_2d blit op"); > + } > +} > diff --git a/hw/display/ati_int.h b/hw/display/ati_int.h > new file mode 100644 > index 0000000000..85d045517c > --- /dev/null > +++ b/hw/display/ati_int.h > @@ -0,0 +1,67 @@ > +/* > + * QEMU ATI SVGA emulation > + * > + * Copyright (c) 2019 BALATON Zoltan > + * > + * This work is licensed under the GNU GPL license version 2 or later. > + */ > + > +#include "qemu/osdep.h" > +#include "hw/pci/pci.h" > +#include "vga_int.h" > + > +#undef DEBUG_ATI > + > +#ifdef DEBUG_ATI > +#define DPRINTF(fmt, ...) printf("%s: " fmt, __func__, ## __VA_ARGS__) > +#else > +#define DPRINTF(fmt, ...) do {} while (0) Please use tracepoints (you already add some!). > +#endif > + > +#define TYPE_ATI_VGA "ati-vga" > +#define ATI_VGA(obj) OBJECT_CHECK(ATIVGAState, (obj), TYPE_ATI_VGA) > + > +typedef struct ATIVGARegs { > + uint32_t mm_index; > + uint32_t bios_0_scratch; > + uint32_t crtc_gen_cntl; > + uint32_t crtc_ext_cntl; > + uint32_t dac_cntl; > + uint32_t crtc_h_total_disp; > + uint32_t crtc_h_sync_strt_wid; > + uint32_t crtc_v_total_disp; > + uint32_t crtc_v_sync_strt_wid; > + uint32_t crtc_offset; > + uint32_t crtc_offset_cntl; > + uint32_t crtc_pitch; > + uint32_t dst_offset; > + uint32_t dst_pitch; > + uint32_t dst_width; > + uint32_t dst_height; > + uint32_t src_x; > + uint32_t src_y; > + uint32_t dst_x; > + uint32_t dst_y; > + uint32_t dp_gui_master_cntl; > + uint32_t dp_brush_bkgd_clr; > + uint32_t dp_brush_frgd_clr; > + uint32_t dp_src_frgd_clr; > + uint32_t dp_src_bkgd_clr; > + uint32_t dp_cntl; > + uint32_t dp_write_mask; > + uint32_t default_offset; > + uint32_t default_pitch; > + uint32_t default_sc_bottom_right; > +} ATIVGARegs; > + > +typedef struct ATIVGAState { > + PCIDevice dev; > + VGACommonState vga; > + uint16_t dev_id; > + uint16_t mode; > + MemoryRegion io; > + MemoryRegion mm; > + ATIVGARegs regs; > +} ATIVGAState; > + > +void ati_2d_blit(ATIVGAState *s); > diff --git a/hw/display/ati_regs.h b/hw/display/ati_regs.h > new file mode 100644 > index 0000000000..dc5ff53ceb > --- /dev/null > +++ b/hw/display/ati_regs.h > @@ -0,0 +1,452 @@ > +/* > + * ATI VGA register definitions > + * > + * based on: > + * linux/include/video/aty128.h > + * Register definitions for ATI Rage128 boards > + * Anthony Tong <at...@uiuc.edu>, 1999 > + * Brad Douglas <b...@neruo.com>, 2000 > + * > + * and linux/include/video/radeon.h > + * > + * This work is licensed under the GNU GPL license version 2. > + */ > + > +/* > + * Register mapping: > + * 0x0000-0x00ff Misc regs also accessible via io and mmio space > + * 0x0100-0x0eff Misc regs only accessible via mmio > + * 0x0f00-0x0fff Read-only copy of PCI config regs > + * 0x1000-0x13ff Concurrent Command Engine (CCE) regs > + * 0x1400-0x1fff GUI (drawing engine) regs > + */ > + > +#ifndef ATI_REGS_H > +#define ATI_REGS_H > + > +#define MM_INDEX 0x0000 > +#define MM_DATA 0x0004 > +#define CLOCK_CNTL_INDEX 0x0008 > +#define CLOCK_CNTL_DATA 0x000c > +#define BIOS_0_SCRATCH 0x0010 > +#define BUS_CNTL 0x0030 > +#define BUS_CNTL1 0x0034 > +#define GEN_INT_CNTL 0x0040 > +#define CRTC_GEN_CNTL 0x0050 > +#define CRTC_EXT_CNTL 0x0054 > +#define DAC_CNTL 0x0058 > +#define GPIO_MONID 0x0068 > +#define I2C_CNTL_1 0x0094 > +#define PALETTE_INDEX 0x00b0 > +#define PALETTE_DATA 0x00b4 > +#define CNFG_CNTL 0x00e0 > +#define GEN_RESET_CNTL 0x00f0 > +#define CNFG_MEMSIZE 0x00f8 > +#define MEM_CNTL 0x0140 > +#define MC_FB_LOCATION 0x0148 > +#define MC_AGP_LOCATION 0x014C > +#define MC_STATUS 0x0150 > +#define MEM_POWER_MISC 0x015c > +#define AGP_BASE 0x0170 > +#define AGP_CNTL 0x0174 > +#define AGP_APER_OFFSET 0x0178 > +#define PCI_GART_PAGE 0x017c > +#define PC_NGUI_MODE 0x0180 > +#define PC_NGUI_CTLSTAT 0x0184 > +#define MPP_TB_CONFIG 0x01C0 > +#define MPP_GP_CONFIG 0x01C8 > +#define VIPH_CONTROL 0x01D0 > +#define CRTC_H_TOTAL_DISP 0x0200 > +#define CRTC_H_SYNC_STRT_WID 0x0204 > +#define CRTC_V_TOTAL_DISP 0x0208 > +#define CRTC_V_SYNC_STRT_WID 0x020c > +#define CRTC_VLINE_CRNT_VLINE 0x0210 > +#define CRTC_CRNT_FRAME 0x0214 > +#define CRTC_GUI_TRIG_VLINE 0x0218 > +#define CRTC_OFFSET 0x0224 > +#define CRTC_OFFSET_CNTL 0x0228 > +#define CRTC_PITCH 0x022c > +#define OVR_CLR 0x0230 > +#define OVR_WID_LEFT_RIGHT 0x0234 > +#define OVR_WID_TOP_BOTTOM 0x0238 > +#define LVDS_GEN_CNTL 0x02d0 > +#define DDA_CONFIG 0x02e0 > +#define DDA_ON_OFF 0x02e4 > +#define VGA_DDA_CONFIG 0x02e8 > +#define VGA_DDA_ON_OFF 0x02ec > +#define CRTC2_H_TOTAL_DISP 0x0300 > +#define CRTC2_H_SYNC_STRT_WID 0x0304 > +#define CRTC2_V_TOTAL_DISP 0x0308 > +#define CRTC2_V_SYNC_STRT_WID 0x030c > +#define CRTC2_VLINE_CRNT_VLINE 0x0310 > +#define CRTC2_CRNT_FRAME 0x0314 > +#define CRTC2_GUI_TRIG_VLINE 0x0318 > +#define CRTC2_OFFSET 0x0324 > +#define CRTC2_OFFSET_CNTL 0x0328 > +#define CRTC2_PITCH 0x032c > +#define DDA2_CONFIG 0x03e0 > +#define DDA2_ON_OFF 0x03e4 > +#define CRTC2_GEN_CNTL 0x03f8 > +#define CRTC2_STATUS 0x03fc > +#define OV0_SCALE_CNTL 0x0420 > +#define SUBPIC_CNTL 0x0540 > +#define PM4_BUFFER_OFFSET 0x0700 > +#define PM4_BUFFER_CNTL 0x0704 > +#define PM4_BUFFER_WM_CNTL 0x0708 > +#define PM4_BUFFER_DL_RPTR_ADDR 0x070c > +#define PM4_BUFFER_DL_RPTR 0x0710 > +#define PM4_BUFFER_DL_WPTR 0x0714 > +#define PM4_VC_FPU_SETUP 0x071c > +#define PM4_FPU_CNTL 0x0720 > +#define PM4_VC_FORMAT 0x0724 > +#define PM4_VC_CNTL 0x0728 > +#define PM4_VC_I01 0x072c > +#define PM4_VC_VLOFF 0x0730 > +#define PM4_VC_VLSIZE 0x0734 > +#define PM4_IW_INDOFF 0x0738 > +#define PM4_IW_INDSIZE 0x073c > +#define PM4_FPU_FPX0 0x0740 > +#define PM4_FPU_FPY0 0x0744 > +#define PM4_FPU_FPX1 0x0748 > +#define PM4_FPU_FPY1 0x074c > +#define PM4_FPU_FPX2 0x0750 > +#define PM4_FPU_FPY2 0x0754 > +#define PM4_FPU_FPY3 0x0758 > +#define PM4_FPU_FPY4 0x075c > +#define PM4_FPU_FPY5 0x0760 > +#define PM4_FPU_FPY6 0x0764 > +#define PM4_FPU_FPR 0x0768 > +#define PM4_FPU_FPG 0x076c > +#define PM4_FPU_FPB 0x0770 > +#define PM4_FPU_FPA 0x0774 > +#define PM4_FPU_INTXY0 0x0780 > +#define PM4_FPU_INTXY1 0x0784 > +#define PM4_FPU_INTXY2 0x0788 > +#define PM4_FPU_INTARGB 0x078c > +#define PM4_FPU_FPTWICEAREA 0x0790 > +#define PM4_FPU_DMAJOR01 0x0794 > +#define PM4_FPU_DMAJOR12 0x0798 > +#define PM4_FPU_DMAJOR02 0x079c > +#define PM4_FPU_STAT 0x07a0 > +#define PM4_STAT 0x07b8 > +#define PM4_TEST_CNTL 0x07d0 > +#define PM4_MICROCODE_ADDR 0x07d4 > +#define PM4_MICROCODE_RADDR 0x07d8 > +#define PM4_MICROCODE_DATAH 0x07dc > +#define PM4_MICROCODE_DATAL 0x07e0 > +#define PM4_CMDFIFO_ADDR 0x07e4 > +#define PM4_CMDFIFO_DATAH 0x07e8 > +#define PM4_CMDFIFO_DATAL 0x07ec > +#define PM4_BUFFER_ADDR 0x07f0 > +#define PM4_BUFFER_DATAH 0x07f4 > +#define PM4_BUFFER_DATAL 0x07f8 > +#define PM4_MICRO_CNTL 0x07fc > +#define CAP0_TRIG_CNTL 0x0950 > +#define CAP1_TRIG_CNTL 0x09c0 > + > +#define RBBM_STATUS 0x0E40 > +#define RBBM_STATUS_alt_1 0x1740 > + > +/* > + * GUI Block Memory Mapped Registers > + * These registers are FIFOed. > + */ > +#define PM4_FIFO_DATA_EVEN 0x1000 > +#define PM4_FIFO_DATA_ODD 0x1004 > + > +#define DST_OFFSET 0x1404 > +#define DST_PITCH 0x1408 > +#define DST_WIDTH 0x140c > +#define DST_HEIGHT 0x1410 > +#define SRC_X 0x1414 > +#define SRC_Y 0x1418 > +#define DST_X 0x141c > +#define DST_Y 0x1420 > +#define SRC_PITCH_OFFSET 0x1428 > +#define DST_PITCH_OFFSET 0x142c > +#define SRC_Y_X 0x1434 > +#define DST_Y_X 0x1438 > +#define DST_HEIGHT_WIDTH 0x143c > +#define DP_GUI_MASTER_CNTL 0x146c > +#define BRUSH_SCALE 0x1470 > +#define BRUSH_Y_X 0x1474 > +#define DP_BRUSH_BKGD_CLR 0x1478 > +#define DP_BRUSH_FRGD_CLR 0x147c > +#define DST_WIDTH_X 0x1588 > +#define DST_HEIGHT_WIDTH_8 0x158c > +#define SRC_X_Y 0x1590 > +#define DST_X_Y 0x1594 > +#define DST_WIDTH_HEIGHT 0x1598 > +#define DST_WIDTH_X_INCY 0x159c > +#define DST_HEIGHT_Y 0x15a0 > +#define DST_X_SUB 0x15a4 > +#define DST_Y_SUB 0x15a8 > +#define SRC_OFFSET 0x15ac > +#define SRC_PITCH 0x15b0 > +#define DST_HEIGHT_WIDTH_BW 0x15b4 > +#define CLR_CMP_CNTL 0x15c0 > +#define CLR_CMP_CLR_SRC 0x15c4 > +#define CLR_CMP_CLR_DST 0x15c8 > +#define CLR_CMP_MASK 0x15cc > +#define DP_SRC_FRGD_CLR 0x15d8 > +#define DP_SRC_BKGD_CLR 0x15dc > +#define DST_BRES_ERR 0x1628 > +#define DST_BRES_INC 0x162c > +#define DST_BRES_DEC 0x1630 > +#define DST_BRES_LNTH 0x1634 > +#define DST_BRES_LNTH_SUB 0x1638 > +#define SC_LEFT 0x1640 > +#define SC_RIGHT 0x1644 > +#define SC_TOP 0x1648 > +#define SC_BOTTOM 0x164c > +#define SRC_SC_RIGHT 0x1654 > +#define SRC_SC_BOTTOM 0x165c > +#define GUI_DEBUG0 0x16a0 > +#define GUI_DEBUG1 0x16a4 > +#define GUI_TIMEOUT 0x16b0 > +#define GUI_TIMEOUT0 0x16b4 > +#define GUI_TIMEOUT1 0x16b8 > +#define GUI_PROBE 0x16bc > +#define DP_CNTL 0x16c0 > +#define DP_DATATYPE 0x16c4 > +#define DP_MIX 0x16c8 > +#define DP_WRITE_MASK 0x16cc > +#define DP_CNTL_XDIR_YDIR_YMAJOR 0x16d0 > +#define DEFAULT_OFFSET 0x16e0 > +#define DEFAULT_PITCH 0x16e4 > +#define DEFAULT_SC_BOTTOM_RIGHT 0x16e8 > +#define SC_TOP_LEFT 0x16ec > +#define SC_BOTTOM_RIGHT 0x16f0 > +#define SRC_SC_BOTTOM_RIGHT 0x16f4 > +#define WAIT_UNTIL 0x1720 > +#define CACHE_CNTL 0x1724 > +#define GUI_STAT 0x1740 > +#define PC_GUI_MODE 0x1744 > +#define PC_GUI_CTLSTAT 0x1748 > +#define PC_DEBUG_MODE 0x1760 > +#define BRES_DST_ERR_DEC 0x1780 > +#define TRAIL_BRES_T12_ERR_DEC 0x1784 > +#define TRAIL_BRES_T12_INC 0x1788 > +#define DP_T12_CNTL 0x178c > +#define DST_BRES_T1_LNTH 0x1790 > +#define DST_BRES_T2_LNTH 0x1794 > +#define SCALE_SRC_HEIGHT_WIDTH 0x1994 > +#define SCALE_OFFSET_0 0x1998 > +#define SCALE_PITCH 0x199c > +#define SCALE_X_INC 0x19a0 > +#define SCALE_Y_INC 0x19a4 > +#define SCALE_HACC 0x19a8 > +#define SCALE_VACC 0x19ac > +#define SCALE_DST_X_Y 0x19b0 > +#define SCALE_DST_HEIGHT_WIDTH 0x19b4 > +#define SCALE_3D_CNTL 0x1a00 > +#define SCALE_3D_DATATYPE 0x1a20 > +#define SETUP_CNTL 0x1bc4 > +#define SOLID_COLOR 0x1bc8 > +#define WINDOW_XY_OFFSET 0x1bcc > +#define DRAW_LINE_POINT 0x1bd0 > +#define SETUP_CNTL_PM4 0x1bd4 > +#define DST_PITCH_OFFSET_C 0x1c80 > +#define DP_GUI_MASTER_CNTL_C 0x1c84 > +#define SC_TOP_LEFT_C 0x1c88 > +#define SC_BOTTOM_RIGHT_C 0x1c8c > + > +#define CLR_CMP_MASK_3D 0x1A28 > +#define MISC_3D_STATE_CNTL_REG 0x1CA0 > +#define MC_SRC1_CNTL 0x19D8 > +#define TEX_CNTL 0x1800 > + > +/* CONSTANTS */ > +#define GUI_ACTIVE 0x80000000 > +#define ENGINE_IDLE 0x0 > + > +#define PLL_WR_EN 0x00000080 > + > +#define CLK_PIN_CNTL 0x0001 > +#define PPLL_CNTL 0x0002 > +#define PPLL_REF_DIV 0x0003 > +#define PPLL_DIV_0 0x0004 > +#define PPLL_DIV_1 0x0005 > +#define PPLL_DIV_2 0x0006 > +#define PPLL_DIV_3 0x0007 > +#define VCLK_ECP_CNTL 0x0008 > +#define HTOTAL_CNTL 0x0009 > +#define X_MPLL_REF_FB_DIV 0x000a > +#define XPLL_CNTL 0x000b > +#define XDLL_CNTL 0x000c > +#define XCLK_CNTL 0x000d > +#define MPLL_CNTL 0x000e > +#define MCLK_CNTL 0x000f > +#define AGP_PLL_CNTL 0x0010 > +#define FCP_CNTL 0x0012 > +#define PLL_TEST_CNTL 0x0013 > +#define P2PLL_CNTL 0x002a > +#define P2PLL_REF_DIV 0x002b > +#define P2PLL_DIV_0 0x002b > +#define POWER_MANAGEMENT 0x002f > + > +#define PPLL_RESET 0x01 > +#define PPLL_ATOMIC_UPDATE_EN 0x10000 > +#define PPLL_VGA_ATOMIC_UPDATE_EN 0x20000 > +#define PPLL_REF_DIV_MASK 0x3FF > +#define PPLL_FB3_DIV_MASK 0x7FF > +#define PPLL_POST3_DIV_MASK 0x70000 > +#define PPLL_ATOMIC_UPDATE_R 0x8000 > +#define PPLL_ATOMIC_UPDATE_W 0x8000 > +#define MEM_CFG_TYPE_MASK 0x3 > +#define XCLK_SRC_SEL_MASK 0x7 > +#define XPLL_FB_DIV_MASK 0xFF00 > +#define X_MPLL_REF_DIV_MASK 0xFF > + > +/* Config control values (CONFIG_CNTL) */ > +#define CFG_VGA_IO_DIS 0x00000400 > + > +/* CRTC control values (CRTC_GEN_CNTL) */ > +#define CRTC_CSYNC_EN 0x00000010 > + > +#define CRTC2_DBL_SCAN_EN 0x00000001 > +#define CRTC2_DISPLAY_DIS 0x00800000 > +#define CRTC2_FIFO_EXTSENSE 0x00200000 > +#define CRTC2_ICON_EN 0x00100000 > +#define CRTC2_CUR_EN 0x00010000 > +#define CRTC2_EXT_DISP_EN 0x01000000 > +#define CRTC2_EN 0x02000000 > +#define CRTC2_DISP_REQ_EN_B 0x04000000 > + > +#define CRTC_PIX_WIDTH_MASK 0x00000700 > +#define CRTC_PIX_WIDTH_4BPP 0x00000100 > +#define CRTC_PIX_WIDTH_8BPP 0x00000200 > +#define CRTC_PIX_WIDTH_15BPP 0x00000300 > +#define CRTC_PIX_WIDTH_16BPP 0x00000400 > +#define CRTC_PIX_WIDTH_24BPP 0x00000500 > +#define CRTC_PIX_WIDTH_32BPP 0x00000600 > + > +/* DAC_CNTL bit constants */ > +#define DAC_8BIT_EN 0x00000100 > +#define DAC_MASK 0xFF000000 > +#define DAC_BLANKING 0x00000004 > +#define DAC_RANGE_CNTL 0x00000003 > +#define DAC_CLK_SEL 0x00000010 > +#define DAC_PALETTE_ACCESS_CNTL 0x00000020 > +#define DAC_PALETTE2_SNOOP_EN 0x00000040 > +#define DAC_PDWN 0x00008000 > + > +/* CRTC_EXT_CNTL */ > +#define CRT_CRTC_DISPLAY_DIS 0x00000400 > +#define CRT_CRTC_ON 0x00008000 > + > +/* GEN_RESET_CNTL bit constants */ > +#define SOFT_RESET_GUI 0x00000001 > +#define SOFT_RESET_VCLK 0x00000100 > +#define SOFT_RESET_PCLK 0x00000200 > +#define SOFT_RESET_ECP 0x00000400 > +#define SOFT_RESET_DISPENG_XCLK 0x00000800 > + > +/* PC_GUI_CTLSTAT bit constants */ > +#define PC_BUSY_INIT 0x10000000 > +#define PC_BUSY_GUI 0x20000000 > +#define PC_BUSY_NGUI 0x40000000 > +#define PC_BUSY 0x80000000 > + > +#define BUS_MASTER_DIS 0x00000040 > +#define PM4_BUFFER_CNTL_NONPM4 0x00000000 > + > +/* DP_DATATYPE bit constants */ > +#define DST_8BPP 0x00000002 > +#define DST_15BPP 0x00000003 > +#define DST_16BPP 0x00000004 > +#define DST_24BPP 0x00000005 > +#define DST_32BPP 0x00000006 > + > +#define BRUSH_SOLIDCOLOR 0x00000d00 > + > +/* DP_GUI_MASTER_CNTL bit constants */ > +#define GMC_SRC_PITCH_OFFSET_DEFAULT 0x00000000 > +#define GMC_DST_PITCH_OFFSET_DEFAULT 0x00000000 > +#define GMC_SRC_CLIP_DEFAULT 0x00000000 > +#define GMC_DST_CLIP_DEFAULT 0x00000000 > +#define GMC_BRUSH_SOLIDCOLOR 0x000000d0 > +#define GMC_SRC_DSTCOLOR 0x00003000 > +#define GMC_BYTE_ORDER_MSB_TO_LSB 0x00000000 > +#define GMC_DP_SRC_RECT 0x02000000 > +#define GMC_3D_FCN_EN_CLR 0x00000000 > +#define GMC_AUX_CLIP_CLEAR 0x20000000 > +#define GMC_DST_CLR_CMP_FCN_CLEAR 0x10000000 > +#define GMC_WRITE_MASK_SET 0x40000000 > +#define GMC_DP_CONVERSION_TEMP_6500 0x00000000 > + > +/* DP_GUI_MASTER_CNTL ROP3 named constants */ > +#define GMC_ROP3_MASK 0x00ff0000 > +#define ROP3_PATCOPY 0x00f00000 > +#define ROP3_SRCCOPY 0x00cc0000 > + > +#define SRC_DSTCOLOR 0x00030000 > + > +/* DP_CNTL bit constants */ > +#define DST_X_RIGHT_TO_LEFT 0x00000000 > +#define DST_X_LEFT_TO_RIGHT 0x00000001 > +#define DST_Y_BOTTOM_TO_TOP 0x00000000 > +#define DST_Y_TOP_TO_BOTTOM 0x00000002 > +#define DST_X_MAJOR 0x00000000 > +#define DST_Y_MAJOR 0x00000004 > +#define DST_X_TILE 0x00000008 > +#define DST_Y_TILE 0x00000010 > +#define DST_LAST_PEL 0x00000020 > +#define DST_TRAIL_X_RIGHT_TO_LEFT 0x00000000 > +#define DST_TRAIL_X_LEFT_TO_RIGHT 0x00000040 > +#define DST_TRAP_FILL_RIGHT_TO_LEFT 0x00000000 > +#define DST_TRAP_FILL_LEFT_TO_RIGHT 0x00000080 > +#define DST_BRES_SIGN 0x00000100 > +#define DST_HOST_BIG_ENDIAN_EN 0x00000200 > +#define DST_POLYLINE_NONLAST 0x00008000 > +#define DST_RASTER_STALL 0x00010000 > +#define DST_POLY_EDGE 0x00040000 > + > +/* DP_MIX bit constants */ > +#define DP_SRC_RECT 0x00000200 > +#define DP_SRC_HOST 0x00000300 > +#define DP_SRC_HOST_BYTEALIGN 0x00000400 > + > +/* LVDS_GEN_CNTL constants */ > +#define LVDS_BL_MOD_LEVEL_MASK 0x0000ff00 > +#define LVDS_BL_MOD_LEVEL_SHIFT 8 > +#define LVDS_BL_MOD_EN 0x00010000 > +#define LVDS_DIGION 0x00040000 > +#define LVDS_BLON 0x00080000 > +#define LVDS_ON 0x00000001 > +#define LVDS_DISPLAY_DIS 0x00000002 > +#define LVDS_PANEL_TYPE_2PIX_PER_CLK 0x00000004 > +#define LVDS_PANEL_24BITS_TFT 0x00000008 > +#define LVDS_FRAME_MOD_NO 0x00000000 > +#define LVDS_FRAME_MOD_2_LEVELS 0x00000010 > +#define LVDS_FRAME_MOD_4_LEVELS 0x00000020 > +#define LVDS_RST_FM 0x00000040 > +#define LVDS_EN 0x00000080 > + > +/* CRTC2_GEN_CNTL constants */ > +#define CRTC2_EN 0x02000000 > + > +/* POWER_MANAGEMENT constants */ > +#define PWR_MGT_ON 0x00000001 > +#define PWR_MGT_MODE_MASK 0x00000006 > +#define PWR_MGT_MODE_PIN 0x00000000 > +#define PWR_MGT_MODE_REGISTER 0x00000002 > +#define PWR_MGT_MODE_TIMER 0x00000004 > +#define PWR_MGT_MODE_PCI 0x00000006 > +#define PWR_MGT_AUTO_PWR_UP_EN 0x00000008 > +#define PWR_MGT_ACTIVITY_PIN_ON 0x00000010 > +#define PWR_MGT_STANDBY_POL 0x00000020 > +#define PWR_MGT_SUSPEND_POL 0x00000040 > +#define PWR_MGT_SELF_REFRESH 0x00000080 > +#define PWR_MGT_ACTIVITY_PIN_EN 0x00000100 > +#define PWR_MGT_KEYBD_SNOOP 0x00000200 > +#define PWR_MGT_TRISTATE_MEM_EN 0x00000800 > +#define PWR_MGT_SELW4MS 0x00001000 > +#define PWR_MGT_SLOWDOWN_MCLK 0x00002000 > + > +#define PMI_PMSCR_REG 0x60 > + > +/* used by ATI bug fix for hardware ROM */ > +#define RAGE128_MPP_TB_CONFIG 0x01c0 > + > +#endif /* ATI_REGS_H */ > diff --git a/hw/display/trace-events b/hw/display/trace-events > index 387c6b8931..6b85cf2f64 100644 > --- a/hw/display/trace-events > +++ b/hw/display/trace-events > @@ -137,3 +137,7 @@ vga_cirrus_write_blt(uint32_t offset, uint32_t val) > "offset 0x%x, val 0x%x" > sii9022_read_reg(uint8_t addr, uint8_t val) "addr 0x%02x, val 0x%02x" > sii9022_write_reg(uint8_t addr, uint8_t val) "addr 0x%02x, val 0x%02x" > sii9022_switch_mode(const char *mode) "mode: %s" > + > +# hw/display/ati*.c > +ati_mm_read(unsigned int size, uint64_t addr, uint64_t val) "%d > 0x%"HWADDR_PRIx" -> 0x%"PRIx64 > +ati_mm_write(unsigned int size, uint64_t addr, uint64_t val) "%d > 0x%"HWADDR_PRIx" <- 0x%"PRIx64 > I don't understand well the display code, but the result works very well, nice work :) Tested-by: Philippe Mathieu-Daudé <phi...@redhat.com>