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

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

 



On 06/08/2014 11:29, Ian Abbott wrote:
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.


Thank you Ian.
I'll stick to the actual names then.
Regards
Luca


--
Luca Ellero

E-mail: luca.ellero@xxxxxxxxxxxxxxxx
Internet: www.brickedbrain.com

_______________________________________________
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