On Thu, Apr 19, 2012 at 11:12 AM, Laxman Dewangan <ldewangan@xxxxxxxxxx> wrote: > On Wednesday 18 April 2012 04:05 AM, Stephen Warren wrote: >> >> On 04/17/2012 12:45 AM, Laxman Dewangan wrote: >>> >>> Add the device info for ALS and proximity sensor isl29018. >> >> >> a) Perhaps the vendor prefix should be "isil" not "isl". Olof, can you >> please comment here? >> > > Any comments on this? I think "isil" is good as it is the symbol in nasdaq > also: Intersil Corporation (ISIL). > If no objection than I can send my next version of patch ;) > > Adding LinusW, Mark and Grant if they have any comment here. Like most of the kernel people I don't care much about that as long as the string is unique ... your pick is likely the best. Linus Walleij _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel