From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id 892876E487 for ; Sun, 16 May 2021 02:51:16 +0000 (UTC) Date: Sat, 15 May 2021 19:51:15 -0700 Message-ID: <87y2cfie70.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" In-Reply-To: References: <20210514064953.1644250-1-alan.previn.teres.alexis@intel.com> <20210514064953.1644250-4-alan.previn.teres.alexis@intel.com> <1f62b09d-dd6a-063c-253d-e0db2ff4a73b@intel.com> <87a6ovsi58.wl-ashutosh.dixit@intel.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Subject: Re: [igt-dev] [PATCH i-g-t 03/16] Add basic PXP testing of buffer and context alloc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: "Teres Alexis, Alan Previn" Cc: "igt-dev@lists.freedesktop.org" List-ID: On Sat, 15 May 2021 16:58:24 -0700, Teres Alexis, Alan Previn wrote: > > I have been using checkpatch.pl on last few revs with zero issues. One caveat though and I just realize this now: > - when I run checkpatch "--no-tree -f my_modified_file.c", I don't get any warnings. > - when I run but running on the patches I get warnings about "exceeding 100 columns". > > What is the official hard rule? Are we at 120, 100 or 80? IGT follows the kernel coding style. The kernel originally had a 80 char width limit which was recently increased to 100 that is why checkpatch now complains about 100. So yes it's either 80 or 100 however you want to do it. But the official hard limit is now 100 I'd think. So I'd fix it to be within one of those widths. > > ...alan > > -----Original Message----- > From: Dixit, Ashutosh > Sent: Saturday, May 15, 2021 4:16 PM > To: Teres Alexis, Alan Previn > Cc: igt-dev@lists.freedesktop.org > Subject: Re: [igt-dev] [PATCH i-g-t 03/16] Add basic PXP testing of buffer and context alloc > > On Fri, 14 May 2021 02:40:26 -0700, Teres Alexis, Alan Previn wrote: > > > > > > Okay - i will fix them all and push a new rev. > > Mostly coding style looks good now but an easy way to check for style/indendation issues in the code is to run checkpatch from the kernel on the IGT patches and fix all issues which are not kernel specific (and also ignore those which don't make sense): > > https://github.com/torvalds/linux/blob/master/scripts/checkpatch.pl > > > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev