Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S932960AbcLTT7X (ORCPT <rfc822;w@1wt.eu>);
        Tue, 20 Dec 2016 14:59:23 -0500
Received: from smtprelay0179.hostedemail.com ([216.40.44.179]:34601 "EHLO
        smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL)
        by vger.kernel.org with ESMTP id S1752252AbcLTT7W (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 20 Dec 2016 14:59:22 -0500
X-Session-Marker: 6A6F6540706572636865732E636F6D
X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::,RULES_HIT:41:355:379:541:599:800:960:968:973:982:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1431:1437:1515:1516:1518:1534:1542:1593:1594:1711:1730:1747:1777:1792:1801:2197:2199:2393:2553:2559:2562:2828:3138:3139:3140:3141:3142:3353:3622:3653:3865:3867:3870:3871:3872:4321:4605:5007:9121:10004:10400:10450:10455:10848:11026:11232:11233:11473:11658:11914:12043:12262:12294:12438:12555:12679:12740:12760:12895:13439:13851:14181:14659:14721:19904:19999:21063:21080:21221:21365:21451:21525:30054:30070:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:6,LUA_SUMMARY:none
X-HE-Tag: error64_8ba0296670b0d
X-Filterd-Recvd-Size: 2979
Message-ID: <1482263957.1984.29.camel@perches.com>
Subject: Re: [PATCH] checkpatch: stop worrying about include/asm/
From: Joe Perches <joe@perches.com>
To: Paul Bolle <pebolle@tiscali.nl>, Andy Whitcroft <apw@canonical.com>
Cc: linux-kernel@vger.kernel.org
Date: Tue, 20 Dec 2016 11:59:17 -0800
In-Reply-To: <1482260987-21457-1-git-send-email-pebolle@tiscali.nl>
References: <1482260987-21457-1-git-send-email-pebolle@tiscali.nl>
Content-Type: text/plain; charset="ISO-8859-1"
X-Mailer: Evolution 3.22.1-0ubuntu2 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2158
Lines: 59

On Tue, 2016-12-20 at 20:09 +0100, Paul Bolle wrote:
> include/asm/ got removed in v1.1.45. Two decades later it's about time
> to stop worrying whether patches still touch it.
> 
> Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
> ---
> Lightly tested.
> 
>  scripts/checkpatch.pl | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index a8368d1c4348..4975f251ba80 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -2333,11 +2333,6 @@ sub process {
>  				WARN("PATCH_PREFIX",
>  				     "patch prefix '$p1_prefix' exists, appears to be a -p0 patch\n");
>  			}
> -
> -			if ($realfile =~ m@^include/asm/@) {
> -				ERROR("MODIFIED_INCLUDE_ASM",
> -				      "do not modify files in include/asm, change architecture specific files in include/asm-<architecture>\n" . "$here$rawline\n");
> -			}
>  			$found_file = 1;
>  		}

I wonder what Andy really intended with commit

It seems wrong.

commit c1ab33269a84d6056d2ffc728d8bbaa26377d3e3
Author: Andy Whitcroft <apw@shadowen.org>
Date:   Wed Oct 15 22:02:20 2008 -0700

    checkpatch: include/asm checks should be anchored
    
    It is possible to have other include/asm paths within the tree which are
    not subject to the do not edit checks.  Ignore those.
    
    Signed-off-by: Andy Whitcroft <apw@shadowen.org>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 482768cd5790..0e5af8ed107e 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1134,7 +1134,7 @@ sub process {
                        $realfile = $1;
                        $realfile =~ s@^[^/]*/@@;
 
-                       if ($realfile =~ m@include/asm/@) {
+                       if ($realfile =~ m@^include/asm/@) {
                                ERROR("do not modify files in include/asm, change architecture specific files in include/asm-<architecture>\n" . "$here$rawline\n");
                        }