From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: intel8x0 has stopped working. Date: Mon, 09 Feb 2004 11:56:56 +0100 Sender: alsa-devel-admin@lists.sourceforge.net Message-ID: References: <401E8BAE.6010802@superbug.demon.co.uk> <401EA82D.3060702@superbug.demon.co.uk> <40217A56.2020808@superbug.demon.co.uk> <4022C373.7060601@superbug.demon.co.uk> <4023F396.7030407@superbug.demon.co.uk> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=US-ASCII Return-path: Received: from Cantor.suse.de (ns.suse.de [195.135.220.2]) by alsa.alsa-project.org (8.9.3/8.9.3/SuSE Linux 8.9.3-0.1) with ESMTP id MAA00841 for ; Mon, 9 Feb 2004 12:04:25 +0100 In-Reply-To: <4023F396.7030407@superbug.demon.co.uk> Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: James Courtier-Dutton Cc: ALSA development List-Id: alsa-devel@alsa-project.org At Fri, 06 Feb 2004 20:05:42 +0000, James Courtier-Dutton wrote: > > Takashi Iwai wrote: > > At Thu, 05 Feb 2004 22:28:03 +0000, > > James Courtier-Dutton wrote: > > > >>I checked out a new anonymouse cvs of alsa-driver/alsa-kernel, applied > >>your patch, and attach the output from doing modprobe snd-intel8x0. > >> > >>I can only see printf's in the patch, so I don't see how it could have > >>fixes the VRA. > > > > > > no, it doesn't fix yet. > > > > > >>Feb 5 22:22:53 new kernel: determined VRA rates: 0xfe > >>Feb 5 22:22:53 new kernel: determined VRA rates: 0xfe > >>Feb 5 22:22:53 new kernel: determined SPDIF rates: 0xe0 > >>Feb 5 22:22:53 new kernel: determined SDAC rates: 0x80 > >>Feb 5 22:22:53 new kernel: determined LDAC rates: 0x80 > > > > > > here's the problem. surround and LFE DAC sample rates are not > > detected properly. > > i don't figure out yet why... > > > > does the attached patch have influence? check the messages above. > > (it doesn't contain the patch to ac97_pcm.c. it's not needed now.) > > > > > > Takashi > > > > > I checked out a new cvs of alsa-driver, alsa-kernel > Added your patch, and the rates are detected correctly now, and the > rates change on the hardware correctly now. > Thanks thanks for the confirmation. i'll apply the patch to cvs. > By looking at your code, it looks like you are making sure the rate for > the Front, Surround and Center/LFE all change together, which seems to > work, although, the alc650 datasheet spec does not seem to mention that > this has to happen. Why have 3 ac97 registers, if they all have to > change together? we're fooled :) sometime it is explicitly written that the surround and LFE rates are simply the mirror of the front/center rate. that's why the fix came into my mind. Takashi ------------------------------------------------------- The SF.Net email is sponsored by EclipseCon 2004 Premiere Conference on Open Tools Development and Integration See the breadth of Eclipse activity. February 3-5 in Anaheim, CA. http://www.eclipsecon.org/osdn