ALSA: bebob: correct duplicated entries with TerraTec OUI
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Mon, 5 Jul 2021 11:14:55 +0000 (20:14 +0900)
committerTakashi Iwai <tiwai@suse.de>
Mon, 5 Jul 2021 17:31:12 +0000 (19:31 +0200)
ALSA bebob driver has duplicated entries for modalias of
'ieee1394:ven00000AACmo00000002sp0000A02Dver00010001' since entries for
two devices below have the same parameters:

 * Acoustic Reality eAR Master One, Eroica, Figaro, and Ciaccona
 * TerraTec Aureon 7.1 FireWire

I relied on FFADO revision 737 to add the former entry, on the other hand,
the latter is based on message posted by actual user with information of
sysfs node:

 * https://sourceforge.net/p/ffado/mailman/ffado-user/thread/5743F969.2080204%40marcobaldo.ch/

It appears that they have OUI of Terratec Electronic GmbH (0x000aac) and
the same model ID, thus suffice to say that they have something common
in their internals.

Although it's not going to make a big difference, this commit arranges
the entries.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Link: https://lore.kernel.org/r/20210705111455.63788-1-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/bebob/bebob.c

index 42980da45fbf3567c8e2d98f082305a2b8e52f40..06a7ced218e292942729bcca84710e9ff2625acc 100644 (file)
@@ -403,8 +403,6 @@ static const struct ieee1394_device_id bebob_id_table[] = {
        SND_BEBOB_DEV_ENTRY(VEN_APOGEE, 0x01eeee, &spec_normal),
        /* ESI, Quatafire610 */
        SND_BEBOB_DEV_ENTRY(VEN_ESI, 0x00010064, &spec_normal),
-       // AcousticReality, eARMasterOne. Terratec OEM.
-       SND_BEBOB_DEV_ENTRY(VEN_TERRATEC, 0x00000002, &spec_normal),
        /* CME, MatrixKFW */
        SND_BEBOB_DEV_ENTRY(VEN_CME, 0x00030000, &spec_normal),
        // Phonic Helix Board 12 FireWire MkII.
@@ -434,7 +432,8 @@ static const struct ieee1394_device_id bebob_id_table[] = {
        SND_BEBOB_DEV_ENTRY(VEN_TERRATEC, 0x00000007, &yamaha_terratec_spec),
        /* TerraTec Electronic GmbH, EWS MIC2/MIC8 */
        SND_BEBOB_DEV_ENTRY(VEN_TERRATEC, 0x00000005, &spec_normal),
-       /* Terratec Electronic GmbH, Aureon 7.1 Firewire */
+       // Terratec Electronic GmbH, Aureon 7.1 Firewire.
+       // AcousticReality, eAR Master One, Eroica, Figaro, and Ciaccona. Perhaps Terratec OEM.
        SND_BEBOB_DEV_ENTRY(VEN_TERRATEC, 0x00000002, &spec_normal),
        /* Yamaha, GO44 */
        SND_BEBOB_DEV_ENTRY(VEN_YAMAHA, 0x0010000b, &yamaha_terratec_spec),