On 17-07-08 21:38, Rene Herman wrote: > On 17-07-08 21:20, Rene Herman wrote: > >> On 17-07-08 20:40, Takashi Iwai wrote: >> >>> Could you try the patch below? >> >> No, please hang on, it's much simpler. Above commit isn't actually >> itself guilty. Have a patch ready, but am testing a bit. > > I retract that. Your patch should fix things. It does. Acked/Tested-by: Rene Herman > I would like to place something on top to make the init isa/pnp choice > look more generic but I'll submit that tomorrow. This bug was uncovered by !CONFIG_PNP and we had trouble in this driver due to that before so was rewriting the init to be a generic multi-card ALSA init but, actually, not much point. The hardware fundamentally doesn't support more than one card per system due to the fixed chip->mc_base and the bug was in this case not _caused_ but only uncovered by !CONFIG_PNP (it triggers for CONFIG_PNP after supplying "isapnp=0" as a module param) so I'll just wait for the next time this ifdeffed together mess breaks on !CONFIG_PNP to make the init look generic ;-) In fact, in the meantime I'll look at splitting 92x and 93x again. It's much better now that Krzysztof made it use the generic library code but I still hate this driver. It's also poking at ports without being told something is there by either system or user. Trivial remainder of init rewrite that's still applicable: Rene.