Re: s3 product

[KIYOHARA Takashi <kiyohara%kk.iij4u.or.jp@localhost>]

Hi! Tim and Michael,


From: KIYOHARA Takashi <kiyohara%kk.iij4u.or.jp@localhost>
Date: Tue, 25 Nov 2008 18:25:08 +0900 (JST)

> I have look at main-board just today.  I found the RAMDAC (S3 86C716 SDAC)
> maybe.  Also I try to probe RAMDAC on my prep.  It failed, because don't
> understand.

The s3_drv and my check routines found RAMDAC (S3 SDAC).  It causes fail in
it because of failing in the mmap of the I/O port(0x3c0-0x3df).
Also

> However, I have the possibility of understanding the method of using
> vga_drv now.  When trying before, nothing was displayed on the screen
> though xterm and xclock ran.  However, there is a possibility to be able
> to solve now.  ;-)

Maybe vga_drv works OK.  However, I look the message 'Sync out of range'
(free translation ;-) on my monitor inconveniently.  X-<   Maybe, vga_drv
supports only 'Modes "320x240"'.
Our powerpc machines use vga(4) driver.  It is not support macro
PCI_MAGIC_IO_RANGE.

Index: vga_pci.c
===================================================================
RCS file: /cvsroot/src/sys/dev/pci/vga_pci.c,v
retrieving revision 1.44
diff -u -r1.44 vga_pci.c
--- vga_pci.c   3 Aug 2008 02:12:22 -0000       1.44
+++ vga_pci.c   26 Nov 2008 04:24:53 -0000
@@ -350,6 +350,17 @@
                return (bus_space_mmap(vc->hdl.vh_memt, IOM_BEGIN,
                    (offset - IOM_BEGIN), prot, 0));
 
+       /* allow mapping of IO space */
+#ifdef PCI_MAGIC_IO_RANGE
+       if (offset >= PCI_MAGIC_IO_RANGE &&
+           (offset < PCI_MAGIC_IO_RANGE + 0x10000))
+               return (bus_space_mmap(vc->hdl.vh_iot,
+                   offset - PCI_MAGIC_IO_RANGE, 0, prot, 0));
+#else
+       if (offset < 0x10000)
+               return (bus_space_mmap(vc->hdl.vh_iot, offset, 0, prot, 0));
+#endif /* PCI_MAGIC_IO_RANGE */
+
        /* Range not found. */
        return (-1);
 }


It might be unnecessary since '#else'.

Moreover, I noticed macro AVOID{INT10,_VGAHW_SAVEFONTS} concerning bebox
was unnecessary.  The 'options WSFB_FAKE_VGA_FB' is necessary for us
instead of it.
I will make the patch again tonight.  Before support for Vision 864 of S3. 

Thanks,
--
kiyohara