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.
What at mess. ;) Merging hdsp.c into hdspm.c would be the clean approach to this problem, still I don't have time for this in the near future. Maybe in July.
Given that all current variants are misleading in one way or another, we could also leave everything as is. What do you think?
Cheers