From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules Date: Wed, 24 Aug 2005 12:35:25 +0200 Message-ID: References: <430B3EA0.1000903@netvigator.com> <200508232340.46629.alien999999999@users.sourceforge.net> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <200508232340.46629.alien999999999@users.sourceforge.net> Sender: alsa-devel-admin@lists.sourceforge.net Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Alien Cc: alsa-devel@lists.sourceforge.net, Raymond , Manuel Jander List-Id: alsa-devel@alsa-project.org At Tue, 23 Aug 2005 23:40:44 +0200, Alien wrote: > > Op dinsdag 23 augustus 2005 19:18, schreef Takashi Iwai: > > At Tue, 23 Aug 2005 23:20:00 +0800, > > > > Raymond wrote: > > > Takashi Iwai wrote: > > > > At Sat, 20 Aug 2005 08:45:47 +0800, > > > > > > > > Raymond wrote: > > > >>I am still using Kernel 2.4 which is not affected by the missing of > > > >>these two patches. > > > >> > > > >>Is the missing of "snd_card_set_dev(card, &pci->dev);" affected only > > > >>some distrubtion of Kernel 2.6 (32-bits platform) ? > > > >> > > > >>https://bugtrack.alsa-project.org/alsa-bug/view.php?id=299 > > > > > > > > Sure, it's only for 2.6, and it can't be critical. > > > > It just adds the sysfs entry for udev/hal. It must work fine without > > > > it. > > > > > > > >>The reporter of bug 216 using Kernel 2.4 and the reporter of bug 299 > > > >>using Kernel 2.6 > > > > > > > > Then this will fix that bug very unlikely. > > > > > > > > > > > > Takashi > > > > > > For au88x0 on AMD64 > > > > > > One of the au8810 user still experience hangs when using GAIM. > > > > > > Igor V. Kovalenko reported his patch is OK for his au8820 > > > > > > http://savannah.nongnu.org/patch/?func=detailitem&item_id=3948#comment43 > > > > > > There are no feed back from the reporters of ALSA BTS and another au8830 > > > user in the alsa.user mailing list. > > > > > > https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1047 au8810 > > > https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1138 au8820 > > > https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1025 au8830 > > > > > > Would you related this three bugs report ? > > > > > > They can try Igor V. Kovalenko's patch at the openvortex project. > > > > The current code can't work obviously on 64bit architecture because > > of the following definition: > > > > struct snd_vortex { > > ... > > unsigned long __iomem *mmio; > > ... > > }; > > > > It must be u32 if you want to be architecture independent. > > that doesn't mean much, you need to see how it handles adding to it, with all > the typecasts... It's definitely a part of the problems. You can find that the driver doesn't cast in hwread/hwwrite at all in the current CVS code (I don't mention about the modified codes). Most likely there are other problems incompatible with 64bit architectures, too. Takashi ------------------------------------------------------- SF.Net email is Sponsored by the Better Software Conference & EXPO September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf