Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752506AbbL1Pdj (ORCPT ); Mon, 28 Dec 2015 10:33:39 -0500 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:33960 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752239AbbL1Pdf (ORCPT ); Mon, 28 Dec 2015 10:33:35 -0500 X-Sasl-enc: G5Y2D3FZllM/XOMbG+kBbp7y0e8OByeexlkU9ELYwSdg 1451316814 Date: Mon, 28 Dec 2015 07:33:32 -0800 From: Greg KH To: Andrey Utkin Cc: "linux-kernel@vger.kernel.org" , kernel-janitors , "kernel-mentors@selenic.com" , Linux Media , devel@driverdev.osuosl.org, andrey.od.utkin@gmail.com Subject: Re: On Lindent shortcomings and massive style fixing Message-ID: <20151228153332.GA6159@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1535 Lines: 35 On Mon, Dec 28, 2015 at 04:33:27PM +0200, Andrey Utkin wrote: > After some iterations of checkpatch.pl, on a new developed driver > (tw5864), now I have the following: > > $ grep 'WARNING\|ERROR' /src/checkpatch.tw5864 | sort | uniq -c > 31 ERROR: do not use C99 // comments > 147 WARNING: Block comments use a trailing */ on a separate line > 144 WARNING: Block comments use * on subsequent lines > 435 WARNING: line over 80 characters > > At this point, Lindent was already used, and checkpatch.pl warnings > introduced by Lindent itself were fixed. Usage of "indent > --linux-style" (which behaves differently BTW) doesn't help anymore, > too. > > Could anybody please advise how to sort out these issues > automatically, because they look like perfectly solvable in automated > fashion. Of course manual work would result in more niceness, but I am > not eager to go through hundreds of place of code just to fix "over 80 > characters" issues now. Shouldn't take very long to do so, all of the above can be fixed in less than a day's worth of work manually. Or you can use indent to fix up the line length issues, but watch out for the results, sometimes it's better to refactor the code than to just blindly accept the output of that tool. good luck! greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/