At Sun, 27 Nov 2011 17:42:29 +0100, Daniel Mack wrote:
On 11/27/2011 05:10 PM, John F Leach wrote:
Entries in this list should be ordered by the USB vendor/product ID, which is not the case judging from the next block:
/* Roland SH-201 */ USB_DEVICE(0x0582, 0x00ad), .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {
Otherwise looks good.
Thanks for sending patches!
Daniel
Thanks for the feedback Daniel; I submitted a revised patch with the correct list order and fixed the wrapped line for the driver_info.
Nice. The only thing worth mentioning here is that your "change log" shouldn't be placed in the mail subject (which will be the patch headline eventually). Next time, just put things you don't want to be part of the actual patch itself underneath the "---" line below the diffstat.
But I think this time, Takashi can fix it up when commiting? :)
Yes, I can fix it by myself. But, the patch doesn't pass checkpatch.pl test, so I postpone for now.
John, please fix warnings scripts/checkaptch.pl gives as much as possible. Some warnings can be ignored (e.g. over 80 chars), but space issues should be fixed at least.
thanks,
Takashi