[alsa-devel] Backported sbxfi driver (UNTESTED!)

[Bjoern Olausson]

On Wed, Oct 22, 2008 at 17:26, Takashi Iwai <tiwai at suse.de> wrote:
> At Wed, 22 Oct 2008 17:24:09 +0200,
> Bjoern Olausson wrote:
>>
>> With the latest drivers I have random stutterings.
>
> Is it a regression?  I mean, did it ever work better than the latest
> version?
>
I cant remember for shure, sry.

>> I can't say if they
>> occure regular, but they occure often. Sometimes every ~5 seconds,
>> sometimes with more time between, somtimes less...
>>
>> mplayer --->
>> Öffne Audiodecoder: [mp3lib] MPEG layer-2, layer-3
>> AUDIO: 48000 Hz, 2 ch, s16le, 160.0 kbit/10.42% (ratio: 20000->192000)
>> Ausgewählter Audiocodec: [mp3] afm: mp3lib (mp3lib MPEG layer-2, layer-3)
>> <---
>>
>> Master and PCM channels are working.
>
> Do you see the problem except for MPlayer?
>
No problems when using xine as backend or xine allone.

> Does the patch change the behavior?
>
No, mplayer (using the smplayr GUI) behaves like I reportetd, xine
works like bevor.

Just one thing changed:
Running the same file with "mplayer file.avi" instead of "smplayer
file.avi" now results in a crase.
Without the patch I could at least run "mplayer file.avi" Now the
system freezes or just restarts.
No way to get mot info.

I'll revert the patch and check again.

>
> thanks,
>
> Takashi
>
> diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c
> index 0a1c569..ebe2dee 100644
> --- a/sound/pci/sbxfi/sbxfi.c
> +++ b/sound/pci/sbxfi/sbxfi.c
> @@ -1260,7 +1260,7 @@ static struct snd_pcm_hardware sbxfi_pcm_hardware = {
>        .channels_min = 2,
>        .channels_max = 2,
>        .buffer_bytes_max =     SBXFI_MAX_BUFFER,
> -       .period_bytes_min =     64,
> +       .period_bytes_min =     4096,
>        .period_bytes_max =     SBXFI_MAX_BUFFER,
>        .periods_min =          1,
>        .periods_max =          4096,
>