Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751896AbdIXIUK (ORCPT ); Sun, 24 Sep 2017 04:20:10 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:59682 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751355AbdIXIUI (ORCPT ); Sun, 24 Sep 2017 04:20:08 -0400 Date: Sun, 24 Sep 2017 10:20:06 +0200 From: Boris Brezillon To: Marek Vasut Cc: Pavel Machek , dwmw2@infradead.org, computersforpeace@gmail.com, richard@nod.at, cyrille.pitchen@wedev4u.fr, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, trivial@kernel.org Subject: Re: [PATCH] Fix C++ kernel in include/linux/mtd/mtd.h Message-ID: <20170924102006.5aa42982@bbrezillon> In-Reply-To: References: <20170923201314.GA14742@amd> X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 972 Lines: 36 On Sat, 23 Sep 2017 23:10:26 +0200 Marek Vasut wrote: > On 09/23/2017 10:13 PM, Pavel Machek wrote: > > C++ comments look wrong in kernel tree. Fix one. > > AFAIR they are now (sadly) accepted, but this patch works for me. Works for me too. Here is my Acked-by: Boris Brezillon > Would be nice if trivial could pick it. Hm, I'd recommend not doing that, otherwise we might end up with a conflict if a patch modify something in this area. > > > Signed-off-by: Pavel Machek > > > > diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h > > index 6cd0f6b..849543f1 100644 > > --- a/include/linux/mtd/mtd.h > > +++ b/include/linux/mtd/mtd.h > > @@ -267,7 +267,7 @@ struct mtd_info { > > */ > > unsigned int bitflip_threshold; > > > > - // Kernel-only stuff starts here. > > + /* Kernel-only stuff starts here. */ > > const char *name; > > int index; > > > > > >