From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Wilcox Subject: Re: Checkpatch False positive Date: Mon, 20 Aug 2018 13:22:50 -0700 Message-ID: <20180820202250.GA6345@bombadil.infradead.org> References: <8cf91e5fb848eecd98a2f9493387906b3a5f1506.camel@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andy Whitcroft , Joe Perches , netdev , Linux Docs To: Jeff Kirsher Return-path: Received: from bombadil.infradead.org ([198.137.202.133]:54182 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726099AbeHTXjz (ORCPT ); Mon, 20 Aug 2018 19:39:55 -0400 Content-Disposition: inline In-Reply-To: <8cf91e5fb848eecd98a2f9493387906b3a5f1506.camel@intel.com> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Aug 20, 2018 at 01:21:30PM -0700, Jeff Kirsher wrote: > Checkpatch.pl should probably exclude *.rst files from the SPDX License > Identifier check, since they are documentational files, like *.txt. They're still copyrighted work, so they should still have SPDX tags to identify their license. checkpatch has: } elsif ($realfile =~ /\.rst$/) { $comment = '..'; so we're explicitly including rst files in the checking for now.