On 2015.03.12 20:59, Dan Carpenter wrote: > On Thu, Mar 12, 2015 at 08:14:25PM +0200, Giedrius Statkevičius wrote: >> TRUE/FALSE defined in dgnc_types.h are only used in dgnc_tty.c thus it >> would be better to just use 1/0 to avoid unnecessary extra files and >> includes. >> > > Use true/false. I don't care much about this personally but true/false > is kernel style these days. > > regards, > dan carpenter > After researching more this whole dgnc_Major_Serial_Registered and dgnc_Major_TransparentPrint_Registered seem pretty pointless because they only get set to true or false depending on if those boards are initialised or not and we won't initialise them two or more times - dgnc_init_one() is a probe method and it's never called anywhere else. This seems like a way better solution - redundant stuff will be removed from the struct and a unnecessary header will be removed (types.h is for bools) :) I'll fix this up in v2 but now I'll wait for more comments on other patches (if there will be any) for a bit. -- Thanks, Giedrius _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
