From owner-freebsd-hackers@FreeBSD.ORG  Thu Feb 28 22:14:59 2008
Return-Path: <owner-freebsd-hackers@FreeBSD.ORG>
Delivered-To: freebsd-hackers@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 79F4C1065677
	for <freebsd-hackers@freebsd.org>; Thu, 28 Feb 2008 22:14:59 +0000 (UTC)
	(envelope-from yuri@rawbw.com)
Received: from shell.rawbw.com (shell.rawbw.com [198.144.192.42])
	by mx1.freebsd.org (Postfix) with ESMTP id 5D9A48FC2F
	for <freebsd-hackers@freebsd.org>; Thu, 28 Feb 2008 22:14:59 +0000 (UTC)
	(envelope-from yuri@rawbw.com)
Received: from eagle.syrec.org (ip224.carlyle.sfo.ygnition.net
	[24.219.144.224]) (authenticated bits=0)
	by shell.rawbw.com (8.13.6/8.13.6) with ESMTP id m1SMExhp086251;
	Thu, 28 Feb 2008 14:14:59 -0800 (PST)
Message-ID: <47C73262.1020805@rawbw.com>
Date: Thu, 28 Feb 2008 14:14:58 -0800
From: Yuri <yuri@rawbw.com>
User-Agent: Thunderbird 2.0.0.9 (X11/20080122)
MIME-Version: 1.0
To: freebsd-hackers@freebsd.org
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: "J.R. Oldroyd" <fbsd@opal.com>
Subject: umass: should the device specific information be moved from C code
 to the text file?
X-BeenThere: freebsd-hackers@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: yuri@rawbw.com
List-Id: Technical Discussions relating to FreeBSD
	<freebsd-hackers.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>, 
	<mailto:freebsd-hackers-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-hackers>
List-Post: <mailto:freebsd-hackers@freebsd.org>
List-Help: <mailto:freebsd-hackers-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
	<mailto:freebsd-hackers-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 28 Feb 2008 22:14:59 -0000

Currently all devices supported by umass driver are listed in
* sys/dev/usb/usbdevs
  - which has vendor table (vendor-id/name)
  - and product table (ref-to-vendor-id/product-id/name)
and
* sys/dev/usb/umass.c
  - which has some per-device flags.

The problem with this way is that some people might want to disable 
umass for particular devices.
For example some people prefer to use gphoto2 with usb-cameras and 
gphoto2 needs the device to be ugen.

I suggest device specific information should be moved from 
sys/dev/usb/usbdevs and sys/dev/usb/umass.c
to some text file (ex. /etc/umass-devices) which would be read by umass 
driver when it's loaded.

This way users will be able to easily remove/add entries without the 
need to recompile the module.

Yuri