From: The Source <thesourcehim@gmail.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: Backported sbxfi driver (UNTESTED!)
Date: Wed, 15 Oct 2008 12:14:57 +0400 [thread overview]
Message-ID: <48F5A681.8050609@gmail.com> (raw)
In-Reply-To: <s5h4p3ez41h.wl%tiwai@suse.de>
[-- Attachment #1: Type: text/plain, Size: 4209 bytes --]
Takashi Iwai пишет:
> At Wed, 15 Oct 2008 11:47:00 +0400,
> The Source wrote:
>
>> Takashi Iwai пишет:
>>
>>> At Wed, 15 Oct 2008 11:16:32 +0400,
>>> The Source wrote:
>>>
>>>
>>>> Takashi Iwai пишет:
>>>>
>>>>
>>>>> At Tue, 14 Oct 2008 15:24:22 +0400,
>>>>> The Source wrote:
>>>>>
>>>>>
>>>>>
>>>>>> By the way, note, that not rate may cause oops but number of channles
>>>>>> for example (wav that causes oops has mono mode and the one that just
>>>>>> plays silently - stereo. Also the oops sample is 16bit, silent one is
>>>>>> 24bit).
>>>>>>
>>>>>>
>>>>>>
>>>>> This is an important information. The driver basically supports only
>>>>> 16bit stereo samples natively. The others are covered by alsa-lib,
>>>>> and this involves with the mmap mode.
>>>>>
>>>>> But I still wonder why 16bit mode.
>>>>>
>>>>> OK, the below is *THE* test I'd like ask you guys to try first.
>>>>>
>>>>> - Build the driver with --with-debug=detect option
>>>>> - Load the driver with debug=2 module option
>>>>> - Prepare a WAV file with 96kHz 16bit stereo, e.g. converted via sox
>>>>> - Raise the Master volume to the max
>>>>> - Run "aplay this.wav"
>>>>>
>>>>> At *each* run, check your kernel message. If you get Oops, copy the
>>>>> whole log immediately.
>>>>>
>>>>> - If you get a DMA error or so in the kernel message, try to define
>>>>> XXX_SYSTEM_TIMER in sbxfi.c, rebuild and reinstall the driver.
>>>>> Run the same test.
>>>>>
>>>>> - If still any serious problem, load the module with debug=3 option,
>>>>> and get the whole kernel message with full register accesses.
>>>>>
>>>>>
>>>>> thanks,
>>>>>
>>>>> Takashi
>>>>>
>>>>>
>>>>>
>>>>>
>>>> I've done what you have asked. I've got no oopses or other things like
>>>> that. But again not sound. I've tried with debug=2 and debug=3.
>>>> Attaching the whole dmesg output.
>>>>
>>>>
>>> Thanks. Judging from your logs, it seems that the timer IRQ isn't
>>> generated as expected, or the pointer (SRCCA register read) doesn't
>>> work, or all broken.
>>>
>>> Could you try with XXX_SYSTEM_TIMER? This eliminates, at least, the
>>> timer issue.
>>>
>>>
>>> Takashi
>>>
>>>
>>>
>> I can't use system timer. changing #undef XXX_SYSTEM_TIMER to #define
>> XXX_SYSTEM_TIMER causes this:
>> In file included from /mnt/e/temp/alsa-driver-unstable/pci/sbxfi/sbxfi.c:2:
>> /mnt/e/temp/alsa-driver-unstable/include/../alsa-kernel/pci/sbxfi/sbxfi.c:
>> In function ‘sbxfi_timer_handle’:
>> /mnt/e/temp/alsa-driver-unstable/include/../alsa-kernel/pci/sbxfi/sbxfi.c:252:
>> error: ‘status’ undeclared (first use in this function)
>> /mnt/e/temp/alsa-driver-unstable/include/../alsa-kernel/pci/sbxfi/sbxfi.c:252:
>> error: (Each undeclared identifier is reported only once
>> /mnt/e/temp/alsa-driver-unstable/include/../alsa-kernel/pci/sbxfi/sbxfi.c:252:
>> error: for each function it appears in.)
>> /mnt/e/temp/alsa-driver-unstable/include/../alsa-kernel/pci/sbxfi/sbxfi.c:252:
>> warning: too many arguments for format
>>
>
> Try the patch below. In case you're using snapshot tarball, apply it
> with -p2 on alsa-kernel directory.
>
>
> Takashi
>
> diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c
> index 9ff5fc4..d6b5b34 100644
> --- a/sound/pci/sbxfi/sbxfi.c
> +++ b/sound/pci/sbxfi/sbxfi.c
> @@ -249,7 +249,7 @@ static void sbxfi_handle_timer_callback(struct sbxfi *chip);
>
> static void sbxfi_timer_handle(unsigned long data)
> {
> - LOG(2, "TIMER ISR\n", status);
> + LOG(2, "TIMER ISR\n");
> sbxfi_handle_timer_callback((struct sbxfi *)data);
> }
>
>
>
>
Ok, I tested again and found that sound is actually played (with and
without system timer), but:
1. The volume is extremely low. I had not only set volume to maximum in
alsamixer but also set my audio 5.1 system volume to maximum to hear a
sound.
2. The sound is very slow and glitchy. It plays much slower that it
should and constant glitches occur.
Attaching dmesg output.
[-- Attachment #2: dmesg.out.bz2 --]
[-- Type: application/x-bzip, Size: 5221 bytes --]
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2008-10-15 8:17 UTC|newest]
Thread overview: 163+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-12 12:53 Backported sbxfi driver (UNTESTED!) Alexey Bazhin
2008-10-13 2:03 ` Takashi Iwai
2008-10-13 12:58 ` Alexey Bazhin
2008-10-13 13:18 ` Takashi Iwai
2008-10-13 14:33 ` The Source
2008-10-13 14:56 ` Takashi Iwai
2008-10-13 15:43 ` Takashi Iwai
2008-10-13 16:02 ` Takashi Iwai
2008-10-13 16:05 ` Takashi Iwai
2008-10-13 19:21 ` Alexey Bazhin
2008-10-14 5:17 ` The Source
2008-10-14 5:41 ` Takashi Iwai
2008-10-14 6:45 ` Takashi Iwai
2008-10-14 7:23 ` The Source
2008-10-14 7:40 ` Takashi Iwai
2008-10-14 7:58 ` The Source
2008-10-14 8:14 ` Takashi Iwai
2008-10-14 8:13 ` Alexey Bazhin
2008-10-14 8:48 ` The Source
2008-10-14 8:55 ` Takashi Iwai
2008-10-14 8:55 ` The Source
2008-10-14 9:07 ` Takashi Iwai
2008-10-14 9:17 ` The Source
2008-10-14 10:52 ` Takashi Iwai
2008-10-14 11:03 ` Takashi Iwai
2008-10-14 14:11 ` The Source
2008-10-14 15:32 ` Takashi Iwai
2008-10-14 16:22 ` Vedran Miletić
2008-10-14 16:25 ` Takashi Iwai
2008-10-14 11:24 ` The Source
2008-10-15 6:38 ` Takashi Iwai
2008-10-15 7:16 ` The Source
2008-10-15 7:36 ` Takashi Iwai
[not found] ` <48F59FF4.20800@gmail.com>
2008-10-15 7:54 ` Takashi Iwai
2008-10-15 8:14 ` The Source [this message]
2008-10-15 9:07 ` Takashi Iwai
2008-10-15 12:34 ` Alexey Bazhin
2008-10-14 20:35 ` Alexey Bazhin
2008-10-15 6:30 ` Takashi Iwai
2008-10-15 8:26 ` Alexey Bazhin
2008-10-15 9:02 ` Takashi Iwai
2008-10-15 18:54 ` Alexey Bazhin
2008-10-16 10:03 ` Takashi Iwai
2008-10-16 10:46 ` Takashi Iwai
2008-10-16 11:09 ` Alexey Bazhin
2008-10-16 11:12 ` Takashi Iwai
-- strict thread matches above, loose matches on Subject: below --
2008-11-10 0:26 Romain
2008-10-15 21:57 Bjoern Olausson
2008-10-16 8:58 ` Takashi Iwai
[not found] ` <5501535d0810161130i7e3fa4c2hbe267ba2cbb15c00@mail.gmail.com>
2008-10-16 20:07 ` Bjoern Olausson
2008-10-17 6:21 ` Takashi Iwai
2008-10-09 18:02 Takashi Iwai
2008-10-09 18:12 ` Brendan Pike
2008-10-09 18:25 ` Ted T. Logian
2008-10-09 18:25 ` William Pitcock
2008-10-09 18:49 ` Ted T. Logian
2008-10-09 19:50 ` James Courtier-Dutton
2008-10-10 6:02 ` Takashi Iwai
2008-10-09 20:03 ` Ted T. Logian
2008-10-09 20:17 ` Ted T. Logian
2008-10-10 6:01 ` Takashi Iwai
2008-10-10 6:26 ` Ted T. Logian
2008-10-10 6:46 ` Takashi Iwai
2008-10-10 18:17 ` William Pitcock
2008-10-11 16:01 ` Takashi Iwai
2008-10-11 17:36 ` William Pitcock
2008-10-11 17:39 ` William Pitcock
2008-10-11 17:41 ` Takashi Iwai
2008-10-11 17:43 ` Takashi Iwai
2008-10-11 18:04 ` William Pitcock
2008-10-12 19:34 ` Takashi Iwai
[not found] ` <e4b753d00810121816i133141d5te6a4d638044f3875@mail.gmail.com>
2008-10-13 2:08 ` Takashi Iwai
2008-10-13 9:51 ` Takashi Iwai
[not found] ` <6d00fe310810100939h6cd61e0cg7817c7f54f6261f6@mail.gmail.com>
2008-10-11 15:47 ` Takashi Iwai
2008-10-11 16:03 ` Ted T. Logian
2008-10-11 17:32 ` Takashi Iwai
2008-10-12 8:41 ` Vedran Miletić
2008-10-12 8:48 ` James Courtier-Dutton
2008-10-12 9:43 ` The Source
2008-10-12 8:44 ` James Courtier-Dutton
2008-10-13 9:52 ` Takashi Iwai
2008-10-11 16:28 ` The Source
2008-10-11 17:34 ` Takashi Iwai
2008-10-16 12:03 ` Jan Wolf
2008-10-16 12:33 ` Takashi Iwai
2008-10-16 12:56 ` Xarteras
2008-10-16 13:36 ` The Source
2008-10-16 13:48 ` Takashi Iwai
2008-10-16 14:15 ` The Source
2008-10-16 14:18 ` Takashi Iwai
2008-10-16 14:41 ` The Source
2008-10-16 14:42 ` Takashi Iwai
2008-10-16 14:51 ` The Source
2008-10-16 15:00 ` The Source
2008-10-16 15:04 ` Takashi Iwai
[not found] ` <48F75892.8000209@gmail.com>
2008-10-16 15:15 ` Takashi Iwai
2008-10-16 17:53 ` Bjoern Olausson
2008-10-16 18:02 ` The Source
2008-10-16 18:18 ` The Source
2008-10-17 6:17 ` Takashi Iwai
2008-10-17 6:26 ` The Source
2008-10-17 6:39 ` Takashi Iwai
2008-10-17 7:16 ` The Source
2008-10-17 8:17 ` The Source
2008-10-17 9:00 ` Xarteras
2008-10-17 9:17 ` The Source
2008-10-17 9:25 ` Takashi Iwai
2008-10-17 9:23 ` Takashi Iwai
2008-10-17 9:50 ` Xarteras
2008-10-17 10:00 ` Tony Vroon
2008-10-17 10:35 ` Xarteras
2008-10-17 15:33 ` Bjoern Olausson
2008-10-17 7:57 ` The Source
2008-10-17 9:35 ` Takashi Iwai
2008-10-17 9:58 ` The Source
2008-10-17 9:59 ` Takashi Iwai
2008-10-17 10:01 ` The Source
2008-10-17 10:04 ` Takashi Iwai
[not found] ` <48F86606.9070108@gmail.com>
2008-10-17 10:27 ` Takashi Iwai
2008-10-17 20:48 ` William Pitcock
2008-10-18 10:06 ` Xarteras
2008-10-18 15:06 ` Takashi Iwai
2008-10-18 17:10 ` William Pitcock
2008-10-18 17:17 ` William Pitcock
2008-10-19 7:59 ` Takashi Iwai
2008-10-19 18:07 ` Bjoern Olausson
2008-10-19 19:47 ` William Pitcock
2008-10-16 21:29 ` Xarteras
2008-10-17 18:16 ` James Courtier-Dutton
2008-10-22 15:24 ` Bjoern Olausson
2008-10-22 15:26 ` Takashi Iwai
2008-10-22 16:07 ` Bjoern Olausson
2008-10-22 16:24 ` Bjoern Olausson
2008-10-22 19:25 ` The Source
2008-10-23 5:40 ` Takashi Iwai
2008-10-23 11:46 ` Bjoern Olausson
2008-10-23 11:53 ` Jason Harvey
2008-10-23 13:56 ` Takashi Iwai
2008-10-23 12:45 ` Jason Harvey
2008-10-23 13:07 ` Bjoern Olausson
2008-10-23 13:21 ` Jason Harvey
2008-10-23 15:54 ` Takashi Iwai
2008-10-23 14:23 ` Jason Harvey
2008-10-23 17:07 ` Takashi Iwai
2008-10-23 17:15 ` Takashi Iwai
2008-10-23 15:37 ` Jason Harvey
2008-10-23 15:40 ` Bjoern Olausson
2008-10-23 17:44 ` Takashi Iwai
2008-10-23 17:49 ` Takashi Iwai
2008-10-23 16:23 ` Jason Harvey
2008-10-23 19:14 ` Takashi Iwai
2008-10-23 17:32 ` Jason Harvey
2008-10-23 21:30 ` Takashi Iwai
2008-10-23 20:17 ` Jason Harvey
2008-10-24 8:40 ` Takashi Iwai
2008-10-24 7:56 ` Jason Harvey
2008-10-24 10:21 ` Takashi Iwai
2008-10-24 8:50 ` Jason Harvey
2008-10-23 17:26 ` Takashi Iwai
2008-10-23 15:31 ` Jason Harvey
2008-10-23 17:41 ` Takashi Iwai
2008-10-23 14:48 ` Jason Harvey
2008-10-23 13:54 ` Takashi Iwai
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=48F5A681.8050609@gmail.com \
--to=thesourcehim@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=tiwai@suse.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox