From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755562AbbCLTTA (ORCPT ); Thu, 12 Mar 2015 15:19:00 -0400 Received: from mail-we0-f169.google.com ([74.125.82.169]:43373 "EHLO mail-we0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752225AbbCLTS4 (ORCPT ); Thu, 12 Mar 2015 15:18:56 -0400 Message-ID: <5501E68D.6010802@gmail.com> Date: Thu, 12 Mar 2015 21:18:37 +0200 From: =?UTF-8?B?R2llZHJpdXMgU3RhdGtldmnEjWl1cw==?= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Dan Carpenter CC: lidza.louina@gmail.com, markh@compro.net, devel@driverdev.osuosl.org, gregkh@linuxfoundation.org, driverdev-devel@linuxdriverproject.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/6] dgnc: convert TRUE/FALSE #defines to 1/0 References: <1426184068-5772-1-git-send-email-giedrius.statkevicius@gmail.com> <1426184068-5772-3-git-send-email-giedrius.statkevicius@gmail.com> <20150312185915.GC10964@mwanda> In-Reply-To: <20150312185915.GC10964@mwanda> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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