Messages in this thread | | | Date | Mon, 21 Jan 2013 12:14:01 -0500 (EST) | From | Vivien Didelot <> | Subject | Re: [PATCH] HID: add ThingM blink(1) USB LED support |
| |
Hi Jiri,
> hid-blink sounds too generic to me, taking into account that this is > not > 'generic HID blinking' framework, but rather a specific device driver > for > a particular device. > > So something like hid-thingm-blink sounds more reasonable to me.
Ok, I'll name the file hid-thingm-blink1.c instead. Should I keep the Kconfig macro CONFIG_BLINK1 or CONFIG_THINGM_BLINK1 sounds better?
> Linux Kernel is being shipped under GPL v2, without "or any later" > clause > added. So I am afraid I can't merge the code with this license.
No problem, I'll change the clause.
Thanks, Vivien
| |