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

[David Henningsson]

Hi Takashi and sorry for the slow reply as we were trying to focus on 
getting the PA 6.0 release out.

I think in general you should talk to the udev people first and see what 
they recommend. If discussions stall or so, I'm happy to apply your 
workaround patch, assuming you have tested it and device.description 
then reports something nice (such as e g "Presonus FP10").

On 2015-01-10 04:49, Takashi Sakamoto wrote:
> 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
>

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic