Re: [systemd-devel] [PATCH v3 2/2] Add more firewire properties for sound, to be closer to USB and PCI

2015-04-23 Thread Lennart Poettering
On Wed, 22.04.15 21:05, Adam Goode (ago...@google.com) wrote:

 USB and PCI soundcards have a nice set of ID_* properties. It would
 be handy for firewire soundcards to have the same.
 
 Note that this removes the explicit setting of ID_ID in the firewire
 conditional. Because we are now setting ID_SERIAL, ID_ID will come
 from later in the file.

Applied both. Thanks!

Lennart

-- 
Lennart Poettering, Red Hat
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


Re: [systemd-devel] [PATCH v3 2/2] Add more firewire properties for sound, to be closer to USB and PCI

2015-04-22 Thread systemd github import bot
Patchset imported to github.
Pull request:
https://github.com/systemd-devs/systemd/compare/master...systemd-mailing-devs:1429751139-7651-2-git-send-email-agoode%40google.com

--
Generated by https://github.com/haraldh/mail2git
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


[systemd-devel] [PATCH v3 2/2] Add more firewire properties for sound, to be closer to USB and PCI

2015-04-22 Thread Adam Goode
USB and PCI soundcards have a nice set of ID_* properties. It would
be handy for firewire soundcards to have the same.

Note that this removes the explicit setting of ID_ID in the firewire
conditional. Because we are now setting ID_SERIAL, ID_ID will come
from later in the file.
---
 rules/78-sound-card.rules | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/rules/78-sound-card.rules b/rules/78-sound-card.rules
index bd7a994..04740e8 100644
--- a/rules/78-sound-card.rules
+++ b/rules/78-sound-card.rules
@@ -41,14 +41,17 @@ IMPORT{builtin}=hwdb
 SUBSYSTEMS==usb, IMPORT{builtin}=usb_id
 SUBSYSTEMS==usb, GOTO=skip_pci
 
-SUBSYSTEMS==firewire, ATTRS{vendor_name}==?*, ATTRS{model_name}==?*, \
-  ENV{ID_BUS}=firewire, ENV{ID_VENDOR}=$attr{vendor_name}, 
ENV{ID_MODEL}=$attr{model_name}
-SUBSYSTEMS==firewire, ATTRS{guid}==?*, ENV{ID_ID}=firewire-$attr{guid}
+SUBSYSTEMS==firewire, ATTRS{guid}==?*, \
+  ENV{ID_BUS}=firewire, ENV{ID_SERIAL}=$attr{guid}, 
ENV{ID_SERIAL_SHORT}=$attr{guid}, \
+  ENV{ID_VENDOR_ID}=$attr{vendor}, ENV{ID_MODEL_ID}=$attr{model}, \
+  ENV{ID_VENDOR}=$attr{vendor_name}, ENV{ID_MODEL}=$attr{model_name}
 SUBSYSTEMS==firewire, GOTO=skip_pci
 
 SUBSYSTEMS==pci, ENV{ID_BUS}=pci, ENV{ID_VENDOR_ID}=$attr{vendor}, 
ENV{ID_MODEL_ID}=$attr{device}
 LABEL=skip_pci
 
+# Define ID_ID if ID_BUS and ID_SERIAL are set. This will work for both
+# USB and firewire.
 ENV{ID_SERIAL}==?*, ENV{ID_USB_INTERFACE_NUM}==?*, 
ENV{ID_ID}=$env{ID_BUS}-$env{ID_SERIAL}-$env{ID_USB_INTERFACE_NUM}
 ENV{ID_SERIAL}==?*, ENV{ID_USB_INTERFACE_NUM}==, 
ENV{ID_ID}=$env{ID_BUS}-$env{ID_SERIAL}
 
-- 
2.2.0.rc0.207.ga3a616c

___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel