18 Jul
2017
18 Jul
'17
1:52 p.m.
On Fri, Jul 14, 2017 at 2:28 PM, Takashi Iwai tiwai@suse.de wrote:
On Fri, 14 Jul 2017 14:07:12 +0200,
Thanks for the patch. I have seen it but ignored, so far, as not sure which action is the best. An alternative solution is to use snprintf() blindly, for example.
For mixart, it's even better to drop mgr->shortname[] and longname[] assignment. The shortname is the fixed string, and the longname is used only at copying to card->longname, so we can create a string there from the scratch.
I've done that now, and tried to be a little smarter with the other conversions. I also found related problems in ISA drivers after randconfig testing and fixed those as well.
Sent a 7-patch series now as a replacement.
Arnd