Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753870AbdLULGE (ORCPT ); Thu, 21 Dec 2017 06:06:04 -0500 Received: from mail.kernel.org ([198.145.29.99]:51096 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753674AbdLULGA (ORCPT ); Thu, 21 Dec 2017 06:06:00 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8BCAB218D8 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=acme@kernel.org Date: Thu, 21 Dec 2017 08:05:56 -0300 From: Arnaldo Carvalho de Melo To: Laura Abbott Cc: Peter Zijlstra , Ingo Molnar , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-kernel@vger.kernel.org Subject: Re: [PATCH] perf: Remove FSF address Message-ID: <20171221110556.GO21971@kernel.org> References: <20171214232845.32259-1-labbott@redhat.com> <20171215135919.GA21971@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171215135919.GA21971@kernel.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1079 Lines: 23 Em Fri, Dec 15, 2017 at 10:59:19AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Thu, Dec 14, 2017 at 03:28:45PM -0800, Laura Abbott escreveu: > > Checkpatch in the kernel now complains about having the FSF address > > in comments. Other tools such as rpmlint are now starting to do the > > same thing. Remove the FSF address to reduce warnings on multiple tools. > > > > Signed-off-by: Laura Abbott > > --- > > For context, I'm updating the packaging for perf in Fedora which > > requires a code/output review. This got flagged during that review. > > Thanks for the patch, applied. Ingo pointed out to me that for the files we copied from the kernel we better wait for the kernel original files to be changed, when we'll notice via tools/perf/check-headers.sh and update the copies. The intel pt decoder still have this check done outside check-headers.sh, we need to do it there, conceptually its the same issue tho. I'll separate the patch that touches things that are tools/ only and apply next week, when I get back from vacation time. - Arnaldo