svn commit: r199002 - in head/sys: dev/fb dev/pci isa

[Jung-uk Kim]

On Friday 06 November 2009 04:47 pm, John Baldwin wrote:
> On Friday 06 November 2009 3:32:26 pm Jung-uk Kim wrote:
> > Author: jkim
> > Date: Fri Nov  6 20:32:26 2009
> > New Revision: 199002
> > URL: http://svn.freebsd.org/changeset/base/199002
> >
> > Log:
> >   Remove duplicate suspend/resume code from vga_pci.c and let
> > vga(4)
>
> register
>
> >   itself to an associated PCI device if it exists.  It is little
> > bit hackish but it should fix build without frame buffer driver
> > since r198964. Fix some style(9) nits in vga_isa.c while we are
> > here.
>
> Hmm, did you consider having vga_isa use an identify routine to
> attach itself as a child of vgapci0? The hack of knowing the first
> thing in the softc is a pointer is really gross and I'd rather avoid
> it.  Just creating a child of vgapci0 will automatically cause
> suspend and resume to work w/o vgapci having to have any special
> knowledge about vga_isa. 

In fact, I did consider it.  However, lack of my new bus foo prevented 
me from doing the right thing AND fixing the build breakage at the 
same.  I'll try little harder today.

BTW, syscons <-> fb <-> vga <-> vgapci <-> etc. relationship is 
amazingly confusing. :-(

Jung-uk Kim