sound: virtuoso: revert "do not overwrite EEPROM on Xonar D2/D2X"
authorClemens Ladisch <clemens@ladisch.de>
Tue, 17 Feb 2009 08:50:30 +0000 (09:50 +0100)
committerTakashi Iwai <tiwai@suse.de>
Thu, 19 Feb 2009 09:15:39 +0000 (10:15 +0100)
commit6ce6c473a7fd742fdb0db95841e2c4c6b37337c5
tree7cda3d4b0e2bfe87916767f96e78256c1350bbdb
parent2678f60d2bc05a12580b93eb36f089f0e55693e0
sound: virtuoso: revert "do not overwrite EEPROM on Xonar D2/D2X"

This reverts commit 7e86c0e6850504ec9516b953f316a47277825e33 ("do not
overwrite EEPROM on Xonar D2/D2X") because it did not actually help with
the problem.

More user reports show that the overwriting of the EEPROM is not
triggered by using this driver but by installing Linux, and that the
installation of any other operating system (even one without any CMI8788
driver) has the same effect.  In other words, the presence of this
driver does not have any effect on the occurrence of the error.  (So
far, the available evidence seems to point to a BIOS bug.)

Furthermore, it turns out that the EEPROM chip is protected against
stray write commands by the command format and by requiring a separate
write-enable command, so the error scenario in the previous commit (that
SPI writes can be misinterpreted as an EEPROM write command) is not even
theoretically possible.

The mixer control that was removed as a consequence of the previous
commit can only be partially emulated in userspace, which also means it
cannot be seen be the in-kernel OSS API emulation, so it is better to
revert that change.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Cc: <stable@kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/oxygen/virtuoso.c