ALSA: cmipci: Allow disabling MPU port via module option
authorTakashi Iwai <tiwai@suse.de>
Tue, 17 Dec 2019 08:14:48 +0000 (09:14 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 17 Dec 2019 08:16:15 +0000 (09:16 +0100)
commitd8cac6207ec1abd3c85cff2625dd3898d59c4df5
tree9014795d017b10f29952945d7d2f48a813f51073
parent651bbb9d5161ae7170bc19fec893b8bf05fc590f
ALSA: cmipci: Allow disabling MPU port via module option

Patrick May reported that his sound card with CMI8378 chip causes a
crash / reboot when accessing the MIDI port that isn't actually
present on the board.  Moreover, despite of the documentation,
passing mpu_port=0 doesn't disable the MIDI port on this board.

It implies that the chip is a newer revision and the MPU401 port is
integrated and mapped on the PCI register.  For this chip model, the
driver enables the MPU port unconditionally, so far.

Although fixing the unexpected reboot would be the best solution, it's
not so trivial to identify the cause.  So, as a plan B, this patch
extends the existing mpu_port option usage to allow disabling the port
by specifying the value 0, just like we applied for fm_port option in
commit 2f24d159d5ac ("[ALSA] cmipci - Allow to disable integrated FM
port").  As default, the MPU port is still enabled, but user can pass
mpu_port=0 to disable it.

Reported-and-tested-by: Patrick May <dusthillresident@gmail.com>
Link: https://lore.kernel.org/r/20191217081448.1144-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Documentation/sound/alsa-configuration.rst
sound/pci/cmipci.c