From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mail-pf0-x234.google.com ([2607:f8b0:400e:c00::234])
 by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux))
 id 1ae6GI-0008QE-10
 for linux-mtd@lists.infradead.org; Thu, 10 Mar 2016 19:28:42 +0000
Received: by mail-pf0-x234.google.com with SMTP id 124so76212074pfg.0
 for <linux-mtd@lists.infradead.org>; Thu, 10 Mar 2016 11:28:21 -0800 (PST)
Date: Thu, 10 Mar 2016 11:28:18 -0800
From: Brian Norris <computersforpeace@gmail.com>
To: Joe Perches <joe@perches.com>
Cc: "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
 Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: checkpatch.pl false positive: FSF mailing address
Message-ID: <20160310192818.GA9902@google.com>
References: <20160310190023.GB2545@google.com>
 <1457636880.3433.12.camel@perches.com>
 <CAN8TOE-dOUsuPfm8cp6c3k94XRzyyCb1g7sAaPeOdaxS9zG-uA@mail.gmail.com>
 <1457637955.3433.17.camel@perches.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <1457637955.3433.17.camel@perches.com>
List-Id: Linux MTD discussion mailing list <linux-mtd.lists.infradead.org>
List-Unsubscribe: <http://lists.infradead.org/mailman/options/linux-mtd>,
 <mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe>
List-Archive: <http://lists.infradead.org/pipermail/linux-mtd/>
List-Post: <mailto:linux-mtd@lists.infradead.org>
List-Help: <mailto:linux-mtd-request@lists.infradead.org?subject=help>
List-Subscribe: <http://lists.infradead.org/mailman/listinfo/linux-mtd>,
 <mailto:linux-mtd-request@lists.infradead.org?subject=subscribe>

On Thu, Mar 10, 2016 at 11:25:55AM -0800, Joe Perches wrote:
> On Thu, 2016-03-10 at 11:15 -0800, Brian Norris wrote:
> > On Thu, Mar 10, 2016 at 11:08 AM, Joe Perches <joe@perches.com> wrote:
> 
> Hey again Brian.
> 
> > > Yeah, it's amusing.  It's also likely not something worth
> > > bothering with though as the likelihood of fixing it is low
> > > and eventually the FSF warning will be minimized or removed
> > > as no longer necessary.
> > An easy improvement: don't use "ignore case" for the regex.
> 
> Maybe, but:
> 
> drivers/pcmcia/bfin_cf_pcmcia.c: * if not, write to the free software foundation,
> 
> Of course there's only one of those too.
> 
> Mostly, it's a "I just don't care very much" warning.
> 
> If you care enough to submit a patch removing the "/i"
> from the regex, please do and cc Andrew Morton too.

Maybe I will.

> > > checkpatch is stupid, false positives will always exist.
> 
> I'd almost rather have people understand this.

Then perhaps I'll also patch this note out:

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Regards,
Brian