Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 09 Nov 2009 12:24:27 -0600
From:      Robert Noland <rnoland@FreeBSD.org>
To:        Jung-uk Kim <jkim@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, John Baldwin <jhb@freebsd.org>
Subject:   Re: svn commit: r199002 - in head/sys: dev/fb dev/pci isa
Message-ID:  <1257791067.27939.25.camel@balrog.2hip.net>
In-Reply-To: <200911091322.40212.jkim@FreeBSD.org>
References:  <200911062032.nA6KWRXb027876@svn.freebsd.org> <200911061647.00983.jhb@freebsd.org> <200911091322.40212.jkim@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 2009-11-09 at 13:22 -0500, Jung-uk Kim wrote:
> 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. :-(

Agreed, this is one of the things that has kept me from working on
kms...

robert.

> Jung-uk Kim
-- 
Robert Noland <rnoland@FreeBSD.org>
FreeBSD




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1257791067.27939.25.camel>