Message ID | 1323907109.21839.28.camel@pasglop |
---|---|
State | New, archived |
Headers | show |
Am 15.12.2011 00:58, schrieb Benjamin Herrenschmidt: > We rename the mach64 hack to "simple" since that's also applicable > to anything using VGA-style DAC IO ports (set to 8-bit DAC) and we > use it for qemu vga. > > Note that this is keyed on a device-tree "compatible" property that > is currently only set by an upcoming version of SLOF when using the > qemu "pseries" platform. This is on purpose as other qemu ppc platforms > using OpenBIOS aren't properly setting the DAC to 8-bit at the time of > the writing of this patch. > > We can fix OpenBIOS later to do that and add the required property, in > which case it will be matched by this change. Just let me know what's needed for OpenBIOS. Is this just for -vga std as opposed to the default cirrus? Cheers, Andreas
On Thu, 2011-12-15 at 08:49 +0100, Andreas Färber wrote: > Am 15.12.2011 00:58, schrieb Benjamin Herrenschmidt: > > We rename the mach64 hack to "simple" since that's also applicable > > to anything using VGA-style DAC IO ports (set to 8-bit DAC) and we > > use it for qemu vga. > > > > Note that this is keyed on a device-tree "compatible" property that > > is currently only set by an upcoming version of SLOF when using the > > qemu "pseries" platform. This is on purpose as other qemu ppc platforms > > using OpenBIOS aren't properly setting the DAC to 8-bit at the time of > > the writing of this patch. > > > > We can fix OpenBIOS later to do that and add the required property, in > > which case it will be matched by this change. > > Just let me know what's needed for OpenBIOS. > Is this just for -vga std as opposed to the default cirrus? Yes. Cirrus isn't the default on mac99 and on pseries (tho I will eventually add a SLOF driver for it as well). For OpenBIOS I was thinking about just sending you a patch :-) But if you have more time than I do, what is needed is: - Set the 8-bit DAC bit in the VBE enable register when initializing the card (0x20 off the top of my mind but dbl check). Remove your >> 2 in your palette setting. - Implement color! so prom_init can set the initial palette (but that's not strictly necessary). - I assume that the VGA device already has a device_type of "display", can be open()'ed from the client interface and will have the necessary properties to be used by offb (width, height, linebytes, depth, and address if fits in 32-bit (if not, ignore it, offb will pick the largest BAR)). - Stick "qemu,std-vga" into the compatible property of the vga PCI device. Cheers, Ben. -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/video/offb.c b/drivers/video/offb.c index 915acae..da7cf79 100644 --- a/drivers/video/offb.c +++ b/drivers/video/offb.c @@ -41,13 +41,14 @@ /* Supported palette hacks */ enum { cmap_unknown, - cmap_m64, /* ATI Mach64 */ + cmap_simple, /* ATI Mach64 */ cmap_r128, /* ATI Rage128 */ cmap_M3A, /* ATI Rage Mobility M3 Head A */ cmap_M3B, /* ATI Rage Mobility M3 Head B */ cmap_radeon, /* ATI Radeon */ cmap_gxt2000, /* IBM GXT2000 */ cmap_avivo, /* ATI R5xx */ + cmap_qemu, /* qemu vga */ }; struct offb_par { @@ -138,7 +139,7 @@ static int offb_setcolreg(u_int regno, u_int red, u_int green, u_int blue, return 0; switch (par->cmap_type) { - case cmap_m64: + case cmap_simple: writeb(regno, par->cmap_adr); writeb(red, par->cmap_data); writeb(green, par->cmap_data); @@ -208,7 +209,7 @@ static int offb_blank(int blank, struct fb_info *info) if (blank) for (i = 0; i < 256; i++) { switch (par->cmap_type) { - case cmap_m64: + case cmap_simple: writeb(i, par->cmap_adr); for (j = 0; j < 3; j++) writeb(0, par->cmap_data); @@ -350,7 +351,7 @@ static void offb_init_palette_hacks(struct fb_info *info, struct device_node *dp par->cmap_adr = ioremap(base + 0x7ff000, 0x1000) + 0xcc0; par->cmap_data = par->cmap_adr + 1; - par->cmap_type = cmap_m64; + par->cmap_type = cmap_simple; } else if (dp && (of_device_is_compatible(dp, "pci1014,b7") || of_device_is_compatible(dp, "pci1014,21c"))) { par->cmap_adr = offb_map_reg(dp, 0, 0x6000, 0x1000); @@ -371,6 +372,16 @@ static void offb_init_palette_hacks(struct fb_info *info, struct device_node *dp par->cmap_type = cmap_avivo; } of_node_put(pciparent); + } else if (dp && of_device_is_compatible(dp, "qemu,std-vga")) { + const u32 io_of_addr[3] = { 0x01000000, 0x0, 0x0 }; + u64 io_addr = of_translate_address(dp, io_of_addr); + if (io_addr != OF_BAD_ADDR) { + par->cmap_adr = ioremap(io_addr + 0x3c8, 2); + if (par->cmap_adr) { + par->cmap_type = cmap_simple; + par->cmap_data = par->cmap_adr + 1; + } + } } info->fix.visual = (par->cmap_type != cmap_unknown) ? FB_VISUAL_PSEUDOCOLOR : FB_VISUAL_STATIC_PSEUDOCOLOR;
We rename the mach64 hack to "simple" since that's also applicable to anything using VGA-style DAC IO ports (set to 8-bit DAC) and we use it for qemu vga. Note that this is keyed on a device-tree "compatible" property that is currently only set by an upcoming version of SLOF when using the qemu "pseries" platform. This is on purpose as other qemu ppc platforms using OpenBIOS aren't properly setting the DAC to 8-bit at the time of the writing of this patch. We can fix OpenBIOS later to do that and add the required property, in which case it will be matched by this change. Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- drivers/video/offb.c | 19 +++++++++++++++---- 1 files changed, 15 insertions(+), 4 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html