[PATCH 3/6] dgnc: convert TRUE/FALSE #defines to 1/0

Giedrius Statkevičius giedrius.statkevicius at gmail.com
Thu Mar 12 19:18:37 UTC 2015


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


More information about the devel mailing list