Re: [PATCH 1/2] ALSA: add LaCie Firewire Speakers/Griffin FireWave surround driver

[Clemens Ladisch <clemens@ladisch.de>]

Stefan Richter wrote:
> On Feb 07 Clemens Ladisch wrote:
> > +	static const struct device_info lacie_speakers = {
> > +		.driver_name = "FWSpeakers",
> > +		.short_name = "Firewire Speakers",
> > +		.long_name = "LaCie Firewire Speakers",
> 
> FWIW, it is spelled "LaCie FireWire Speakers" in LaCie's printed material
> and on their web site, whereas their firmware programmers put "LaCie
> Firewire speakers" as model descriptor leaf into the Configuration ROM.

I took the latter as model, but capitalized "Speakers" because it's
a proper noun.  I'll change it to "W".

> Does the precise case of device_info strings matter to userspace?

Only for the driver name.  If I had used a common driver name for both,
the .conf file would have to use the short name to differentiate them.
The long name contains random junk (as far as everybody except
PulseAudio is concerned).


Regards,
Clemens