Re: Recursive dependency for SAA7134 in 2.6.15-rc7

Hi Mauro,

Sorry for the delay.

> > Can you please rebase it on Linus' latest (2.6.15-rc7-git4)? So that I
> > can give it a try...
> Updated.

I tried it, it works, but I like it less than the current approach.
Being able to select "Philips SAA7134 DMA audio support" as a module
while it doesn't correspond to real code is a bit confusing. Not being
able to unselect it completely is confusing as well. Thinks go even
worse when only ALSA or OSS has been selected in the general sound
menu, as you end up with a complex layout for no reason.

So I believe that "choice" is an interesting Kconfig feature when used
with boolean options, but with modules I am not convinced, especially
when these modules have different dependencies.

So if I were to decide, I would stick to the current code. But I am
obviously not the one to decide here.

Thanks,
--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages

  • Re: Canalplan Place Guesser
    ... >Is this worth it - would it be more confusing that anything, ... and please jump all over me if it is a feature ... to arrive at Anderton Lift and then moor and delay the departure for 12 ...
    (uk.rec.waterways)
  • Re: Options listing
    ... don't think anyone can deny that the Angband options are a major ... A lot of them are confusing or poorly documented, ... Well, the delay can certainly go, in favor of using the system clock or a regularly-occurring interrupt to control animation speed. ...
    (rec.games.roguelike.angband)