Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Feb 2009 15:43:07 -0800
From:      Andrew Thompson <thompsa@FreeBSD.org>
To:        Joe Marcus Clarke <marcus@marcuscom.com>
Cc:        usb@freebsd.org, current@freebsd.org
Subject:   Re: CFR: usb switchover patches
Message-ID:  <20090223234307.GA999@citylink.fud.org.nz>
In-Reply-To: <1235432234.91306.9.camel@shumai.marcuscom.com>
References:  <20090220033740.GA903@citylink.fud.org.nz> <1235432234.91306.9.camel@shumai.marcuscom.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Feb 23, 2009 at 06:37:14PM -0500, Joe Marcus Clarke wrote:
> On Thu, 2009-02-19 at 19:37 -0800, Andrew Thompson wrote:
> > Hi,
> > 
> > 
> > I have put together a proposed set of changes for moving USB2 to its
> > permanent location. The layout has some differences to how it is right
> > now so I am looking for feedback.
> > 
> > The changeover requires that the old usb stack be available until 8.0 is
> > branched and moves it from sys/dev/usb to sys/legacy/dev/usb. The reason
> > for this location is to reduce the changes in #includes (using -I
> > compiler hacks). The patch doesnt show userland changes required for
> > usbdevs and friends but they will be done.
> > 
> > Some ports will break. Any that exist solely for the old usb stack can
> > be marked broken (like udesc_dump). I dont know that the fallout will be
> > like for the others, maybe portmgr would be interested in doing a build
> > test.
> > 
> > The change roughly goes
> > 
> >  svn move sys/dev/usb -> sys/legacy/dev/usb
> >  svn move sys/dev/usb2 -> sys/dev/usb (with fixups, see below)
> 
> The headers for usb are no longer installed.  This rebreaks hal as I
> attempt to build both backends.  Can you make sure the old usb headers
> are also installed (/usr/include/dev/legacy/usb)?  Without this, I will
> have to redo the hal build code.

I was hoping I wouldnt have to do this as it breaks kdump since they
both define the same typedefs. kdump builds its tables from trolling
/usr/include.

I will have a look at this.

Andrew



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