[alsa-devel] [PATCH 21/30] ALSA: oxfw: Change the way to name card

Takashi Sakamoto o-takashi at sakamocchi.jp
Wed Dec 3 15:19:53 CET 2014


Sorry to be late.

On Dec 2 2014 15:41, Takashi Iwai wrote:
>>> You see now that you're breaking user's setup, don't you?
>>> This is another reason why you shouldn't change the short name
>>> string.
>>
>> In this case, it depends on the estimated number of actual users for 
>> this device.
> 
> No.  The number doesn't matter unless it's proved to be really no
> regression.
> 
>> At least, snd-firewire-speakers cannot handle this device 
>> correctly, while there're no bug reports, for 4 years. I think this is 
>> an evidence of few users.
> 
> Did you get reports that the name hw:Speaker is bad, too?
> If not, why do you insist to "fix" this at all?  It's just because you
> feel so -- which is no objective measurement, either.

Not. Just depending on my feeling.

>> ...although havind said so, it's not my favorite to spend more time to 
>> such a small things, instead of more important things. I'll keep the 
>> naming rule what it was in next patchset.
> 
> There is no more important things than fixing a regression.
> 
> Just fix it for the next patchset, or patches won't be accepted.
> That's so simple.
> 
> I'm giving a strong word here because you might miss the point: in
> such a rewrite, the primary item you have to care is to avoid
> regressions.  Supporting other devices, the cleanness of the code, or
> even the correctness of the code, is hence in the second place.

I'm sure.

This weekend, I'll post the next patchset. I hope it's the last one for
OXFW 970/971.


Regards

Takashi Sakamoto
o-takashi at sakamocchi.jp



More information about the Alsa-devel mailing list