From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dominik Brodowski Date: Tue, 13 Dec 2005 19:39:57 +0000 Subject: Re: grey- and blacklisting drivers [Was: Re: Using the "best available" driver] Message-Id: <20051213193957.GA3841@isilmar.linta.de> List-Id: References: <20051207181524.71dc2d41.zaitcev@redhat.com> In-Reply-To: <20051207181524.71dc2d41.zaitcev@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org Hi, On Tue, Dec 13, 2005 at 05:55:04AM +0100, Kay Sievers wrote: > New version, which does not do the dirty "default parameters" trick. It > depends on a regular module parameter, if "bind_mode" control at module > load or boot prompt is needed. It uses "auto" and "manual" as the values > and the sysfs file in the driver directory is called "bind_mode". Great, many thanks! > index 036c485..3f31c2a 100644 > --- a/drivers/usb/media/ov511.c > +++ b/drivers/usb/media/ov511.c > @@ -84,6 +84,10 @@ > * (See ov511.txt for detailed descriptions of these) > **********************************************************************/ > > +static char bind_mode[8]; > +module_param_string(bind_mode, bind_mode, sizeof(bind_mode), 0); > + The only thing which worries me is that we add quite some bytes to the kernel here... What do you think about this? #define module_param_bind_mode(driver) \ module_param_named(dont_bind, &driver->bind_mode, uint, 0); \ MODULE_PARM_DESC(bind_mode, "Do not bind this driver to " \ "devices automatically."); Then we'd need to add module_param_named(&ov511_driver.driver); to ov511.c and get the same functionality. Thanks, Dominik ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://ads.osdn.com/?ad_idv37&alloc_id865&op=click _______________________________________________ Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net Linux-hotplug-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel