[alsa-devel] [PATCH] ALSA: update the Kconfig files to be easier to use

Takashi Iwai tiwai at suse.de
Sun Feb 24 10:24:52 CET 2008


At Sat, 23 Feb 2008 12:04:01 -0500,
Andrew Paprocki wrote:
> 
> To respond better, I'd actually prefer not putting 'default y', but I
> figured it was more important to stay compatible with existing
> configs. If you feel that isn't a big deal, then it can be removed.
> The default in these cases should do no harm because it doesn't
> actually enable any code.

I think it's more important not to break the old config than avoiding
default y.

I'll apply the patch after a bit more testing (I'm off from the main
workstation right now)...


thanks,

Takashi

> On Sat, Feb 23, 2008 at 10:52 AM, Andrew Paprocki <andrew at ishiboo.com> wrote:
> > Yes, I realized that and I made the new toplevel options default y so
> >  that it would mimic the existing config. The main reason for doing
> >  this is so that I can quickly 'n' to shut off things like "ISA
> >  devices" with one key. I believe I got them all, but please verify.
> >
> >  Thanks,
> >  -Andrew
> >
> >
> >
> >  On Sat, Feb 23, 2008 at 5:49 AM, Takashi Iwai <tiwai at suse.de> wrote:
> >  > At Sat, 23 Feb 2008 01:06:46 -0400,
> >  >
> >  > Andrew Paprocki wrote:
> >  >  >
> >  >  > This update to the Kconfig files allows the ability to select/deselect
> >  >  > entire submenus, starting with the root "Sound device support" on the main
> >  >  > kernel configuration page. Inside sound support, ALSA/OSS can be easily
> >  >  > enabled/disabled. Inside ALSA, entire classes of devices can be easily
> >  >  > enabled/disabled at the ALSA menu (e.g. ISA, PCI, USB devices) without
> >  >  > having to manually disable individual cards within the submenus.
> >  >  > Modified menus: main, isa, pci, pcmcia, soc, spi, usb
> >  >
> >  >  A problem I found at the last time from a similar patch is that the
> >  >  replacement with menuconfig throws away the old configuration unless
> >  >  you have "default y".  And people are picky about "default y"...
> >  >
> >  >  I need to check this again, anyway.
> >  >
> >  >
> >  >  thanks,
> >  >
> >  >  Takashi
> >  >
> >  >
> >
> 


More information about the Alsa-devel mailing list