At the moment, this driver supports only playback, while FFADO supports
(only) full-duplex devices.  So, prevent conflicts by not claiming
devices that would be better handled by FFADO.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

While most DICE devices keep TCAT's default category ID of 0x04, Weiss
devices identify themselves with 0x00.
Reported-by: NRolf Anderegg <rolf.anderegg@weiss.ch>
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Remove a wrong typecast that resulted from a copy-and-paste error.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Doing accesses without quadlet alignment is a bad idea because the
firmware's byte-swapping would garble the data; clarify this in the
documentation.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

For easier debugging, add a proc file to show the device's capabilities
and current status.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Output a warning if the wait for the clock change notification times
out.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Instead of relying of some control panel application to configure some
fixed sample rate, allow applications to set it automatically.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

In preparation for sample rate selection support, read the stream
parameters that might change when running at different sample rates.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Reorganize the initialization order so that the driver can receive
notifications earlier.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

In preparation for sample rate selection support, ensure that the driver
knows about the device's clock capabilities.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Avoid a lock inversion between dice->mutex and pcm->open_mutex.
Signed-off-by: NStefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

The pcm field was not actually used.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Since commit f2b3614c (Don't check DMA time-out too shortly), we
need no longer to restrict the period length to less than 10 s.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

When a bus reset happens, the enable register is automatically cleared,
so we do not need to clear it manually when stopping the stream.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Add a flag to snd_fw_transaction() to allow it to abort when a bus reset
happens.  This removes most of the duplicated error handling loops that
were required around calls to the low-level fw_run_transaction().

Also add a flag to suppress error messages; errors are expected when we
attempt to clean up after the device was unplugged.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Instead of reading two consecutive register with two quadlet requests,
use one block read request.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Change the AMDTP streaming code to handle the non-standard stream format
that DICE devices use at sample rates greater than 96 kHz.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

DICE devices do not have a unique specifier ID in their unit directory
(it's always the same as the device vendor's ID), so rely on just the
version ID for driver loading, and use a heuristic in the probe callback
to detect actual DICE devices.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Move the DICE interface symbols into a separate header file, and add
more documentation.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Introduce the helper function amdtp_out_stream_running().  This makes
many checks in amdtp.c clearer and frees the device drivers from having
to track this with a separate variable.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Ensure that misbehaving or aborted userspace programs do not
accidentally keep the lock.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Implement the hwdep locking and notification mechanisms.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

When aborting a PCM stream, the xrun is signaled only if the stream is
running.  When disconnecting a PCM stream, calling snd_card_disconnect()
too early would change the stream into a non-running state and thus
prevent the xrun from being noticed by user space.

To prevent this, move the snd_card_disconnect() call after the xrun.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Allow AMDTP output streams to use blocking mode.

Use it for DICE devices, because the old DICE-II chip will in some cases
not be able to lock to non-blocking streams (erratum E7).
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

The notification bits are not of general interest; log them only when
debugging.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

Instead of forcing a constant 44.1 kHz, read the current sample rate
from the device when opening the PCM device.

Actually changing the sample rate requires some separate controller
application.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

After a bus reset, do not stop the stream completely to avoid having to
reconfigure the device when restarting the stream.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

As a start point for further development, this is an incomplete driver
for DICE devices:
- only playback (so no clock source except the bus clock)
- only 44.1 kHz
- no MIDI
- recovery after bus reset is slow
- hwdep device is created, but not actually implemented

Contains compilation fixes by Stefan Richter.
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>

The recent commit to delay the release of kobject triggered NULL
dereferences of opti9xx drivers.  The cause is that all
snd-opti92x-ad1848, snd-opti92x-cs4231 and snd-opti93x drivers
register the PnP card driver with the very same name, and also
snd-opti92x-ad1848 and -cs4231 drivers register the ISA driver with
the same name, too.  When these drivers are built in, quick
"register-release-and-re-register" actions occur, and this results in
Oops because of the same name is assigned to the kobject.

The fix is simply to assign individual names.  As a bonus, by using
KBUILD_MODNAME, the patch reduces more lines than it adds.

The fix is based on the suggestion by Russell King.
Reported-and-tested-by: NFengguang Wu <fengguang.wu@intel.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Without the dynamic minor assignment, HDMI codec may have less PCM
instances than the number of pins, which eventually leads to Oops.
Reported-by: NStratos Karafotis <stratosk@semaphore.gr>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Yet another entry, just use the existing fixup for this machine, too.
Reported-by: N"Nathanael D. Noblet" <nathanael@gnat.ca>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Gateway LT27 needs a fixup for the inverted digital mic.
Reported-by: N"Nathanael D. Noblet" <nathanael@gnat.ca>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

The Tegra30 I2S driver was writing the AHUB interface parameters to the
playback path register rather than the capture path register. This
caused the capture parameters not to be configured at all, so if
capturing using non-HW-default parameters (e.g. 16-bit stereo rather
than 8-bit mono) the audio would be corrupted.

With this fixed, audio capture from an analog microphone works correctly
on the Cardhu board.

Cc: stable@vger.kernel.org
Signed-off-by: NStephen Warren <swarren@nvidia.com>
Signed-off-by: NMark Brown <broonie@linaro.org>

Add the volume control quirk for avoiding the kernel warning
for the Logitech HD Webcam C525
as in the similar commit 36691e1b
for the Logitech HD Webcam C310.
Reported-by: NMaksim Boyko <maksim.a.boyko@gmail.com>
Tested-by: NMaksim Boyko <maksim.a.boyko@gmail.com>
Cc: <stable@vger.kernel.org> # 3.10.5+
Signed-off-by: NMaksim Boyko <maksim.a.boyko@gmail.com>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

We've added a fake mute control (setting the amp volume to zero) for
CX5051 at commit [3868137e: ALSA: hda - Add a fake mute feature], but
this feature was overlooked in the generic parser implementation.  Now
the driver lacks of mute controls on these codecs.

The fix is just to check both AC_AMPCAP_MUTE and AC_AMPCAP_MIN_MUTE
bits in each place checking the amp capabilities.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=59001
Cc: <stable@vger.kernel.org> [v3.9+]
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Commit aafe77cc (ALSA: usb-audio: add support for many Roland/Yamaha
devices) had several logic errors that prevented create_auto_midi_quirk
from enumerating any MIDI ports.
Reported-by: NKeith A. Milner <maillist@superlative.org>
Signed-off-by: NClemens Ladisch <clemens@ladisch.de>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Patch makes midi output buffer DMA-able by allocating it separately.
Signed-off-by: NTorsten Schenk <torsten.schenk@zoho.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Patch makes pcm buffers DMA-able by allocating each one separately.
Signed-off-by: NTorsten Schenk <torsten.schenk@zoho.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: NTakashi Iwai <tiwai@suse.de>

Correct the pins for a line-in and a headphone on LG LW25 laptop with
ALC880 codec.  Other pins seem fine.
Reported-and-tested-by: NJoonas Saarinen <jonskunator@gmail.com>
Cc: <stable@vger.kernel.org> [v3.9+]
Signed-off-by: NTakashi Iwai <tiwai@suse.de>