From owner-freebsd-multimedia@FreeBSD.ORG Fri May 18 11:53:36 2007 Return-Path: X-Original-To: freebsd-multimedia@freebsd.org Delivered-To: freebsd-multimedia@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 9319A16A401 for ; Fri, 18 May 2007 11:53:36 +0000 (UTC) (envelope-from rizzo@icir.org) Received: from xorpc.icir.org (xorpc.icir.org [192.150.187.68]) by mx1.freebsd.org (Postfix) with ESMTP id 7F63913C455 for ; Fri, 18 May 2007 11:53:36 +0000 (UTC) (envelope-from rizzo@icir.org) Received: from xorpc.icir.org (localhost [127.0.0.1]) by xorpc.icir.org (8.12.11/8.13.6) with ESMTP id l4IBrYdx043128; Fri, 18 May 2007 04:53:34 -0700 (PDT) (envelope-from rizzo@xorpc.icir.org) Received: (from rizzo@localhost) by xorpc.icir.org (8.12.11/8.12.3/Submit) id l4IBrWpG043127; Fri, 18 May 2007 04:53:32 -0700 (PDT) (envelope-from rizzo) Date: Fri, 18 May 2007 04:53:32 -0700 From: Luigi Rizzo To: Hans Petter Selasky Message-ID: <20070518045331.A41545@xorpc.icir.org> References: <200705181304.17222.hselasky@c2i.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <200705181304.17222.hselasky@c2i.net>; from hselasky@c2i.net on Fri, May 18, 2007 at 01:04:17PM +0200 Cc: freebsd-multimedia@freebsd.org, freebsd-usb@freebsd.org Subject: Re: Linux USB emulation layer now committed to my new USB stack for FreeBSD X-BeenThere: freebsd-multimedia@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Multimedia discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 18 May 2007 11:53:36 -0000 On Fri, May 18, 2007 at 01:04:17PM +0200, Hans Petter Selasky wrote: > Hi, > > If you are interested, the files are: > > http://www.turbocat.net/~hselasky/isdn4bsd/sources/src/sys/dev/usb/usb_compat_linux.c > http://www.turbocat.net/~hselasky/isdn4bsd/sources/src/sys/dev/usb/usb_compat_linux.h > > It is almost finished now. > > And it is not very much code. > > Also I have a preliminary patch for Luigi's webcam driver! But it does not > compile yet. > > NOTE: Before you use a Linux USB endpoint you have to call: > > usb_setup_endpoint() with the buffer size you want. For isoc transfers the > buffer size is ignored. Just set a dummy value. > > Do you have time to fix the rest Luigi? not now, sorry. But seeing the patch that you attach, let me kindly comment once again (i have already told you multiple time) that this approach of ignoring compatibility with existing code (usb stack/api/emulation layer, linux device drivers) and not commenting code at all is really a showstopper for getting your code tested and/or accepted. E.g. see usb_compat_linux.c in the url above - basically the only comment is the copyright - way too little for anyone to understand what is done there. usb_compat_linux.h is just a copy of stuff in the existing linux usb emulation, _but_ without any comments at all (and there was a lot of them in the existing code). Undocumented code is almost useless for us, and nobody has the time to reverse engineer your code and document its architecture. cheers luigi