Re: s3 product

[Michael <macallan%netbsd.org@localhost>]

Hello,

On Nov 26, 2008, at 1:20 AM, KIYOHARA Takashi wrote:

> > 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.

Then Xorg won't be able to access PCI IO space on your machine.

> 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'.

But Xorg doesn't know about it and 0x0 is not a safe offset on most  
framebuffer drivers anyway since it would collide with video memory.

> 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.

Defining PCI_MAGIC_IO_RANGE should get you a whole lot further.

have fun
Michael