Age | Commit message (Collapse) | Author | Files | Lines |
|
assuming RAND_MAX is around 1<<31, rand() >> 1 generates large numbers as
random volume data; these likely causes saturated sample values after
applying the volume function -- not a good test
|
|
|
|
|
|
For muted channels, we forgot to increment a pointer, so if one
channel was muted but not the other, sound became distorted in a
Darth Vader like way. To test the difference, start two input
streams and pan one of them hard left (or right).
And hey, if you didn't think it sounded like Darth Vader, it's
your imagination that's broken, not mine! ;-)
BugLink: https://bugs.launchpad.net/bugs/928757
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
If a *_UNLINK_POST hook causes a sink-input/source-output's sink/source
to go away, the subsequent attempt to update the sink/source status will
cause an assert. We deal with this by checking the sink/source status
before trying to update it.
|
|
|
|
|
|
|
|
The practical problem is that some users were left with only one
"LFE on Mono" port, as analog-output was considered a subset of
analog-output-lfe-on-mono. Which was not what they wanted.
BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=40910
BugLink: https://bugs.launchpad.net/bugs/922656
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
Flush the message queue before tearing down, and dest==NULL is valid in case moving failed.
With this my module-loopback finally no longer causes frequent crashes.
|
|
|
|
|
|
English/localized phrases.
The translations are now obviously outdated and wrong, but as the
core meaning has not changed, this is the lesser of two evils.
|
|
* If we don't have "Digital Speakers", we should say "Speakers"
instead of "Analog Speakers", and similar for other ports.
* Change "IEC958" to "S/PDIF" (more well known name)
* Add new ports and mappings for HDMI
* Change "Internal" to "Built-in" for the card name
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
"Front Speaker", "Surround Speaker" seems to be a common enough name
to make it into alsa-utils, so we should probably care about it as
well. In this case, there was a macbook pro whose speakers didn't work
without these controls.
BugLink: http://bugs.launchpad.net/bugs/551441
Reported-by: Jeroen T. Vermeulen <jtv@canonical.com>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
some are comments, some are user-visible doxygen text and documentation
|
|
|
|
|
|
clean_up_db() makes sure that all entries in the database
are valid.
|
|
Fixes a crash: https://bugs.freedesktop.org/show_bug.cgi?id=44522
|
|
|
|
A function that is used for removing matches should not be
called add_matches.
|
|
The original intention of this code was probably that if
adding filter1 succeeded but adding filter2 failed, then
filter1 should be removed so that either both or none of the
filters get added.
|
|
We didn't do anything anyway in case of failures. When we
give NULL as the error, dbus_bus_remove_match() can act
asynchronously, so it becomes faster. Also, the bus daemon
can avoid sending any replies, which reduces the amount of
traffic.
|
|
|
|
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
The recent change to turn off the IEC958 element for analog paths
exposed a bug in AC3 profiles. These were inheriting the analog output
path instead of explicitly selecting the iec958 path.
Thanks to David Henningsson for pointing this out.
|
|
|
|
|
|
The purpose is to make it possible to link to the
enumeration in doxygen.
|
|
The returned error values are all negative, but the documentation did
not explicitly mention this. Thanks to Vincent Penquerc'h for pointing
this out.
|
|
This is needed for the Creative Audigy CA0106 to work. Also makes sure
that the LED for optical out is shut down in analog modes on MacBooks
(these share a port for analog and digital output).
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=44741
|
|
the random floats x should be zero-mean with -(1+eps)<x<(1+eps) for some
small epsilon; previously only negative x were generated
|
|
|
|
|
|
|
|
|
|
|
|
Increase readability of the pa_alsa_profile_set_probe function of
alsa-mixer.c
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
Just like we turn off the "Front Mic" element when we select "Rear Mic",
we should also turn off the "Front Mic Boost" element. And the same for
the other inputs.
Reported-by: Len Owens <len@ovenwerks.net>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
|
|
This option does not exist any more.
|
|
--exit-idle-time and --scache-idle-time were marked as having an
optional argument when the argument is actually mandatory. This causes a
crash when using this argument.
Thanks to Matthijs Kooijman (blathijs on IRC) for pointing this out.
|
|
Use pa_pipe_buf to determine the minimum size for ibuf and obuf, taking into
account the two descriptors that use each of them.
See bug #42715
|
|
This adds an additional check for unavailable PTHREAD_PRIO_INHERIT to the
fallback work done in ca717643ee768307475fc36ea29d920a13db0a8e
See bug #42715
|
|
SO_TIMESTAMP is not POSIX and not available in any platform, so just fail
if the current platform does not have it.
See bug #42715
|
|
Use SCM_* instead of SO_* when checking the type of each cmsghdr.
See bug #42715
|
|
This helps when porting to platforms that lack PIPE_BUF.
See bug #42715
|
|
Clarifies that the latency is returned via an inout parameter and the
return value is an error code or 0 on success.
|
|
This is really quite irrelevant on most modern systems today, and more
so for embedded systems.
|