At Sun, 28 May 2006 22:32:11 +0300, Paul Mundt wrote: > > Hi Adrian, > > It's getting better, but still a ways to go.. > > On Sun, May 28, 2006 at 07:40:46PM +0100, Adrian McMenamin wrote: > > static int index = -1; > > static char *id; > > static int enable = 1; > > > > module_param(index, int, 0444); > > MODULE_PARM_DESC(index, "Index value for " CARD_NAME " soundcard."); > > module_param(id, charp, 0444); > > MODULE_PARM_DESC(id, "ID string for " CARD_NAME " soundcard."); > > module_param(enable, bool, 0644); > > MODULE_PARM_DESC(enable, "Enable " CARD_NAME " soundcard."); > > > I'm not sure I see what the point of id and enable are. id doesn't appear > to be used at all, and enable seems superfluous. The id should be passed to the second argument of snd_card_new(). enable is referred indeed at open. This option for drivers with a single instance exists usually only for backward compatibility. The sound configuration program sets up often this option unconditionally. Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/alsa-devel
