[pulseaudio-discuss] [RFC] pulseaudio: udev: use ID_MODEL/ID_VENDOR to give friendly name for FireWire devices

[Takashi Sakamoto]

Hi all,

ALSA in Linux kernel 3.16 or later supports some FireWire sound devices
newly, approx 120 models. And it's nice that PulseAudio 6.0 may support
multichannel profiles to them. I'm really appreciate David's work for
it, thank you.

But there's still an issue about card naming. For any FireWire devices,
udev's database gives the name of IEEE 1394 Phy/Link chipset to
ID_XXX_FROM_DATABASE. For example:

$ udevadm info
--path=/devices/pci0000:00/0000:00:1e.0/0000:0c:06.0/fw1/fw1.0/sound/card0
...
E: ID_BUS=firewire
...
E: ID_MODEL=ImpactTwin
E: ID_MODEL_FROM_DATABASE=TSB43AB23 IEEE-1394a-2000 Controller
(PHY/Link) (Motherboard)
...
E: ID_VENDOR=TC Electronic
E: ID_VENDOR_FROM_DATABASE=Texas Instruments

Then PulseAudio gives these names to the card because
ID_XXX_FROM_DATABASE is prior to ID_VENDOR/ID_MODEL.

$ pactl list cards
Card #1
 Name: alsa_card.firewire-0x0001660409c03252
 Driver: module-alsa-card.c
 Owner Module: 6
 Properties:
  alsa.card = "0"
  alsa.card_name = "Impact Twin"
  ...
  device.vendor.name = "Texas Instruments"
  device.product.name = "TSB43AB23 IEEE-1394a-2000 Controller (PHY/Link)
(Motherboard)"

The attached patch is for this issue just in PulseAudio side. This patch
just adds some execution paths just for FireWire devices.

I have another option, to work in udev (systemd), as usb subsystem does.

http://cgit.freedesktop.org/systemd/systemd/tree/src/udev/udev-builtin-usb_id.c#n214

But this work involves developers in FireWire subsystem, therefore will
be an excess of my effort.

It's my glad to get your comments about this issue.


Regards

Takashi Sakamoto
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-udev-use-ID_MODEL-ID_VENDOR-to-give-friendly-name-fo.patch
Type: text/x-diff
Size: 3290 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20150110/4045ea2d/attachment.patch>