Re: Help with RME alsa driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Mon, 9 Feb 2015 15:17:23 +0100,
Tino Mettler wrote:
> 
> On Mon, 2015-02-09 at 15:14 +0100, Takashi Iwai wrote:
> > At Mon, 9 Feb 2015 15:06:14 +0100,
> > Tino Mettler wrote:
> > > 
> > > On Mon, 2015-02-09 at 14:51 +0100, Takashi Iwai wrote:
> 
> > > > What if with --buffer-size=256 or --buffer-size=192, i.e. align buffer
> > > > size and period size?
> > > 
> > > Only 128 works for me.
> > 
> > Then what if you change to --period-size=128 --buffer-size=256,
> > or --period-size=256 --buffer-size=512, etc?
> 
> That values are fine for playback here.

OK, it implies that it can work properly only with periods=2.

The variable number of periods was introduced in the commit
0dca1793063c28dde8f6c49c9c72203fe5cb6efc
   ALSA: hdspm - Add support for RME RayDAT and AIO
    
This seems broken on some boards, then?

The easiest fix would be just to revert to the fixed periods = 2 like
the patch below.  Or, if periods more than two really works on RayDAT
and AIO boards, we can restrict this only for certain models, too.

Adrian, could you check whether periods > 2 really work on any boards?


thanks,

Takashi

---
diff --git a/sound/pci/rme9652/hdspm.c b/sound/pci/rme9652/hdspm.c
index 2c363fdca9fd..f8c80da84d1b 100644
--- a/sound/pci/rme9652/hdspm.c
+++ b/sound/pci/rme9652/hdspm.c
@@ -5835,7 +5835,7 @@ static struct snd_pcm_hardware snd_hdspm_playback_subinfo = {
 	.period_bytes_min = (32 * 4),
 	.period_bytes_max = (8192 * 4) * HDSPM_MAX_CHANNELS,
 	.periods_min = 2,
-	.periods_max = 512,
+	.periods_max = 2,
 	.fifo_size = 0
 };
 
@@ -5860,7 +5860,7 @@ static struct snd_pcm_hardware snd_hdspm_capture_subinfo = {
 	.period_bytes_min = (32 * 4),
 	.period_bytes_max = (8192 * 4) * HDSPM_MAX_CHANNELS,
 	.periods_min = 2,
-	.periods_max = 512,
+	.periods_max = 2,
 	.fifo_size = 0
 };
 

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Alsa-user mailing list
Alsa-user@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-user




[Index of Archives]     [ALSA Devel]     [Linux Audio Users]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]

  Powered by Linux