Re: [PATCH] staging: comedi: add NI USB-6501 initial support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2014-08-06 09:49, Luca Ellero wrote:
A question only:
apart from renaming file from ni_6501.c to ni_usb6501.c, should I rename
functions/variables as well? For example:

ni6501_auto_attach -> niusb6501_auto_attach
ni6501_private -> niusb6501_private

It's okay to abbreviate the prefix, so ni6501, usb6501 or niusb6501 would all be fine. The main purpose of the prefix is to avoid name clashes with things declared in included header files etc., and to make it easier to spot things in symbol tables, Oops messages, etc.

--
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@xxxxxxxxx>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux