At Tue, 07 Jun 2011 12:39:33 +0200, Takashi Iwai wrote:
At Tue, 7 Jun 2011 11:11:32 +0200, Adrian Knoth wrote:
On Tue, Jun 07, 2011 at 10:44:18AM +0200, Jörn Nettingsmeier wrote:
hi adi, *!
Hi!
here's a trivial patch to change the driver name of hdspm.c to avoid irritating users who don't have the MADI version (just happened on the LAU list).
static struct pci_driver driver = {
- .name = "RME Hammerfall DSP MADI",
- .name = "RME Hammerfall HDSPe Series",
Are braces allowed in driver names? I guess so.
"HDSPe" is also confusing, only referring to the PCIe versions. Since we also support the old PCI versions, I suggest to name it
"RME Hammerfall HDSP(e) Series"
On the other hand, the 'H' in "HDSP" stands for "Hammerfall", so
"RME HDSP(e) Series"
would be more appropriate. Luckily, we don't collide with the hdsp.c driver name, but still, it's misleading, because there are cards that match "RME HDSP(e) Series" (like the Multiface/Digiface) but are handled by hdsp.c instead.
It's allowed but better to avoid, since this entry appears as a file name in /sys. We've chosen generic names for the pci name entries of sound drivers, but these look messy. Maybe we should choose just the module name as pci name.
OK, now I fixed the issues in all sound/pci/* drivers in sound git tree. Now they simply use KBUILD_MODNAME in the name field.
thanks,
Takashi