>Takashi Iwai wrote >the patch looks good except for the module options for spdif. >since ac97_codec is a generic module, it's better to avoid >such device specific options. isn't it detectable?
If the Yamaha YMF753 spec provided a way to autodetect which pin was being used for S/PDIF ouput, I'd use it. The alternative to providing a module parameter via insmod would be to provide a mixer control, where one would have the choice of disabling S/PDIF output, or selecting pin 43 or 48. My instincts told me that it made more sense having this as a module parameter, as once one has determined which pin is being used, why would one ever change that value. I felt the same about the parameter for adding the delay after DAC startup in order to avoid missing audio. I could make both items mixer controls if both Jaroslav and Takashi feel it is better. Could you both please respond? Regards, David Shust ------------------------------------------------------------ Free, BeOS-friendly email accounts: http://BeMail.org/ BeOS News and Community: http://www.BeGroovy.com/ --------------------------------------------------------------------- Express yourself with a super cool email address from BigMailBox.com. Hundreds of choices. It's free! http://www.bigmailbox.com --------------------------------------------------------------------- ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Alsa-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/alsa-devel
