Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules

Takashi Iwai
2005-08-24T10:35:25Z

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