All of lore.kernel.org
 help / color / mirror / Atom feed
From: Raymond <rayau@netvigator.com>
To: Erik <assie2@zonnet.nl>
Cc: Alien <alien999999999@users.sourceforge.net>,
	Igor Kovalenko <garrison@mail.ru>,
	openvortex-dev@nongnu.org, alsa-devel@lists.sourceforge.net,
	Manuel Jander <manuel.jander@gmail.com>
Subject: Re: problems with snd_au8830 module under mandriva 2005
Date: Fri, 02 Sep 2005 11:39:22 +0800	[thread overview]
Message-ID: <4317C96A.9020008@netvigator.com> (raw)


There are 3 bug reports for AMD64 in ALSA BTS.
1047 - au8810
1138 - au8820
1025 - au8830

au8810/au8820/au8830 driver is tightly coupled in ALSA au88x0

1) EQ, SPDIF of au8810/au8830 are not present in au8820,
2) WT of au8820/au8830 are not present in au8810,
3) SoftModem of au8810 are not present in au8820/au8830.

Please correct me if I am wrong, I think au8810 do not have any 3D 
effect hardware.

The essential part of the patch is

void __iomem *mmio
#define    hwread(x,y) readl((x)+(y))
#define    hwwrite(x,y,z) writel((z),(x)+(y))

There are 32bits and 64bits data type difference need to be fixed.

1) DMA, SRC, Mixer, Equalizer, Capturing
2) MPU401, Gameport, SPDIF
3) 3D, XTALK

Please clearly describe your volume bar problem.

The volume issue may be relate to the equalizer, hardware mixer, xtalk 
or codec initialization.

The EQ part in diff11.diff is not tested since EQ is not present in au8820.

Please post the output

lspci -vvn

cat /proc/asound/au8830/codec97#0/ac97#0-0+regs

I am not sure Alien's au8810 patch is as same as Igor Kovalenko 's 
au8820 patch. They seem to have their own patches.


The error is properly forget to change the functions declared in 
au88x0_xtalk.h

If the code in au88x0_xtalk.c is arranged in correct order, it is 
possible to delete all those function declaration in au88x0_xtalk.h

It's diffcult to say the XTALK part in diff1.diff is right or wrong.

Any hint on how to test the xtalk canceller ?

I think any patch to the stable ALSA driver in alsa-kernel is to fix 
bugs instead of introducing more confusion.

Any side effect if keeping the XTALK part unchanged ?

or

Implement a kcontrol to test the XTALK



It's great to have an user with au8830 and AMD64 to test all features in 
au8830, this allow nearly all implemented features in au88x0 to be tested.

http://savannah.nongnu.org/patch/download.php?item_id=3948


You should cc your reply to openvortex-dev@nongnu.org so that the other 
developers know the problem.


 > this is what i get when compiling the kernel
 > but maybe i have to delete the kernel source tree
 > because i can remember that i had allready patched the alsa part before
 > a long time ago
 > it was in june i posted the error and not long after that alien from
 > nongnu list contacted me and told me about a patch/.diff
 > i have a working alsa driver with that diff but the only problem still
 > is that the volume bar is acting strange so i think i have allready done
 > this patch
 > but i was glad i could compile it under 64 bit kernel
 >
 > do you know if this is the same patch if not i will clean all sources
 > and compile the kernel again.
 >
 > Erik
 >
 >
 >
 > CC [M]  sound/pci/au88x0/au8830.o
 > In file included from sound/pci/au88x0/au8830.c:17:
 > sound/pci/au88x0/au88x0_xtalk.c:645: error: conflicting types for
 > 'vortex_XtalkHw_SetSampleRate'
 > sound/pci/au88x0/au88x0_xtalk.h:51: error: previous declaration of
 > 'vortex_XtalkHw_SetSampleRate' was here
 > sound/pci/au88x0/au88x0_xtalk.c:645: error: conflicting types for
 > 'vortex_XtalkHw_SetSampleRate'
 > sound/pci/au88x0/au88x0_xtalk.h:51: error: previous declaration of
 > 'vortex_XtalkHw_SetSampleRate' was here
 > sound/pci/au88x0/au88x0_xtalk.h:51: warning:
 > 'vortex_XtalkHw_SetSampleRate' declared `static' but never defined
 > sound/pci/au88x0/au88x0_xtalk.c:645: warning:
 > 'vortex_XtalkHw_SetSampleRate' defined but not used
 > make[3]: *** [sound/pci/au88x0/au8830.o] Error 1
 > make[2]: *** [sound/pci/au88x0] Error 2
 > make[1]: *** [sound/pci] Error 2
 > make: *** [sound] Error 2
 >
 >


-------------------------------------------------------
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

             reply	other threads:[~2005-09-02  3:39 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-02  3:39 Raymond [this message]
  -- strict thread matches above, loose matches on Subject: below --
2005-09-02  3:51 problems with snd_au8830 module under mandriva 2005 Raymond
2005-09-04 19:34 ` Alien
2005-09-07 22:15   ` Erik
2005-09-08  7:07     ` Alien
2005-09-10  0:40       ` Raymond
2005-09-10  7:03         ` Alien
2005-09-14  3:32       ` Erik
2005-09-14  4:42         ` Raymond
2005-09-16 14:16         ` Raymond
2005-10-12  0:30           ` Erik
     [not found] <1118256592.28940.0.camel@localhost>
2005-08-28  3:32 ` Raymond

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4317C96A.9020008@netvigator.com \
    --to=rayau@netvigator.com \
    --cc=alien999999999@users.sourceforge.net \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=assie2@zonnet.nl \
    --cc=garrison@mail.ru \
    --cc=manuel.jander@gmail.com \
    --cc=openvortex-dev@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.