Subject: Re: CVS commit: [itohy-usb1] src/sys/dev/usb
To: M. Warner Losh <imp@bsdimp.com>
From: Marc Balmer <marc@msys.ch>
List: source-changes
Date: 06/23/2007 09:39:11
* M. Warner Losh wrote:
> In message: <20070622124622.8ff360bd.juan@xtrarom.org>
>             Juan RP <juan@xtrarom.org> writes:
> : On Fri, 22 Jun 2007 10:44:57 +0000 (UTC)
> : ITOH Yasufumi <itohy@netbsd.org> wrote:
> : 
> : > 
> : > Module Name:	src
> : > Committed By:	itohy
> : > Date:		Fri Jun 22 10:44:56 UTC 2007
> : > 
> : > Modified Files:
> : > 	src/sys/dev/usb [itohy-usb1]: if_aue.c if_cdce.c if_cue.c if_kue.c
> : > 	    if_udav.c if_url.c
> : > 
> : > Log Message:
> : > Execute callback functions as tasks on FreeBSD.
> : > (FYI, the code does work on NetBSD, but affects the performance, you
> : > know.)
> : 
> : Is there any reason to keep the #ifdef __OtherBSD__ anymore? even FreeBSD
> : and OpenBSD removed these parts.
> 
> FreeBSD removed most of the #ifdefs because the code simply didn't
> work on the OtherBSDs.  We retained the usb_port.h file for drivers
> that wanted to be portable.  The #ifdefs deeper in the usb code did
> nothing but obscure bugs in our tree, since it hadn't been well
> maintained.

OpenBSD removed this #ifdefs for the same reason.  We do no longer use
the usb_port.h file, but we keep it for reference.

- Marc Balmer