121 lines
3.6 KiB
Diff
121 lines
3.6 KiB
Diff
From 6eced6b4b6dca9f62ea3d13a40c5991231df3c5c Mon Sep 17 00:00:00 2001
|
|
From: Takashi Iwai <tiwai@suse.de>
|
|
Date: Thu, 18 Oct 2018 09:38:38 +0200
|
|
Subject: [PATCH 1/6] conf/ucm/Dell-WD15-Dock: Fix incorrect device names
|
|
|
|
The device name string for Dell WD15 (and its variants) dock is set as
|
|
"WD15Dock", while the actual device name to be used is "Dock".
|
|
|
|
Bugzilla: https://bugzilla.suse.com/show_bug.cgi?id=1112292
|
|
Fixes: 8ebb40c96970 ("conf/ucm: Add a UCM profile for Dell WD15 Dock USB-audio")
|
|
Signed-off-by: Takashi Iwai <tiwai@suse.de>
|
|
---
|
|
src/conf/ucm/Dell-WD15-Dock/HiFi.conf | 6 +++---
|
|
1 file changed, 3 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/src/conf/ucm/Dell-WD15-Dock/HiFi.conf b/src/conf/ucm/Dell-WD15-Dock/HiFi.conf
|
|
index b07f3440..1b6bd212 100644
|
|
--- a/src/conf/ucm/Dell-WD15-Dock/HiFi.conf
|
|
+++ b/src/conf/ucm/Dell-WD15-Dock/HiFi.conf
|
|
@@ -3,7 +3,7 @@ SectionDevice."Headphone" {
|
|
|
|
Value {
|
|
PlaybackChannels "2"
|
|
- PlaybackPCM "hw:WD15Dock,0"
|
|
+ PlaybackPCM "hw:Dock,0"
|
|
}
|
|
}
|
|
|
|
@@ -12,7 +12,7 @@ SectionDevice."LineOut" {
|
|
|
|
Value {
|
|
PlaybackChannels "2"
|
|
- PlaybackPCM "hw:WD15Dock,1"
|
|
+ PlaybackPCM "hw:Dock,1"
|
|
}
|
|
}
|
|
|
|
@@ -21,6 +21,6 @@ SectionDevice."Mic" {
|
|
|
|
Value {
|
|
CaptureChannels "2"
|
|
- CapturePCM "hw:WD15Dock,0"
|
|
+ CapturePCM "hw:Dock,0"
|
|
}
|
|
}
|
|
--
|
|
2.13.6
|
|
|
|
|
|
From b420056604f06117c967b65d43d01536c5ffcbc9 Mon Sep 17 00:00:00 2001
|
|
From: Timo Wischer <twischer@de.adit-jv.com>
|
|
Date: Thu, 18 Oct 2018 13:33:24 +0200
|
|
Subject: [PATCH 6/6] pcm: interval: Interpret (x x+1] correctly and return x+1
|
|
|
|
Without this change an interval of (x x+1] will be interpreted as an
|
|
empty interval but the right value would be x+1.
|
|
This leads to a failing snd_pcm_hw_params() call which returns -EINVAL.
|
|
|
|
An example issue log is given in the following:
|
|
snd_pcm_hw_params failed with err -22 (Invalid argument)
|
|
ACCESS: MMAP_NONINTERLEAVED
|
|
FORMAT: S16_LE
|
|
SUBFORMAT: STD
|
|
SAMPLE_BITS: 16
|
|
FRAME_BITS: 16
|
|
CHANNELS: 1
|
|
RATE: 16000
|
|
PERIOD_TIME: (15999 16000]
|
|
PERIOD_SIZE: (255 256]
|
|
PERIOD_BYTES: (510 512]
|
|
PERIODS: [2 3)
|
|
BUFFER_TIME: 32000
|
|
BUFFER_SIZE: 512
|
|
BUFFER_BYTES: 1024
|
|
|
|
In case of (x x+1) we have to interpret it anyway as a single value of x to
|
|
compensate rounding issues.
|
|
For example the period size will result in an interval of (352 353) when
|
|
the period time is 16ms and the sample rate 22050 Hz
|
|
(16ms * 22,05 kHz = 352,8 frames). But 352 has to be chosen to allow a
|
|
buffer size of 705 (32ms * 22,05 kHz = 705,6 frames) which has to be >= 2x
|
|
period size to avoid Xruns. The buffer size will not end up with an
|
|
interval of (705 706) simular to the period size because
|
|
snd_pcm_rate_hw_refine_cchange() calls snd_interval_floor() for the buffer
|
|
size. Therefore this value will be interpreted as an integer interval
|
|
instead of a real interval further on.
|
|
|
|
This issue seems to exist since the change of 9bb985c38 ("pcm:
|
|
snd_interval_refine_first/last: exclude value only if also excluded
|
|
before")
|
|
|
|
Signed-off-by: Timo Wischer <twischer@de.adit-jv.com>
|
|
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
|
|
---
|
|
src/pcm/interval_inline.h | 4 +++-
|
|
1 file changed, 3 insertions(+), 1 deletion(-)
|
|
|
|
diff --git a/src/pcm/interval_inline.h b/src/pcm/interval_inline.h
|
|
index a68e292b..d9a30b2a 100644
|
|
--- a/src/pcm/interval_inline.h
|
|
+++ b/src/pcm/interval_inline.h
|
|
@@ -51,12 +51,14 @@ INTERVAL_INLINE int snd_interval_single(const snd_interval_t *i)
|
|
{
|
|
assert(!snd_interval_empty(i));
|
|
return (i->min == i->max ||
|
|
- (i->min + 1 == i->max && i->openmax));
|
|
+ (i->min + 1 == i->max && (i->openmin || i->openmax)));
|
|
}
|
|
|
|
INTERVAL_INLINE int snd_interval_value(const snd_interval_t *i)
|
|
{
|
|
assert(snd_interval_single(i));
|
|
+ if (i->openmin && !i->openmax)
|
|
+ return i->max;
|
|
return i->min;
|
|
}
|
|
|
|
--
|
|
2.13.6
|
|
|