Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Mon, 2 Mar 2020 10:12:54 +0100
From:   Miquel Raynal <miquel.raynal@bootlin.com>
To:     Schrempf Frieder <frieder.schrempf@kontron.de>
Cc:     Yoshio Furuyama <ytc-mb-yfuruyama7@kioxia.com>,
        "vigneshr@ti.com" <vigneshr@ti.com>,
        "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 1/2] mtd: spinand: toshiba: Rename function name to
 change suffix and prefix (8Gbit)
Message-ID: <20200302101254.31ca0c83@xps13>
In-Reply-To: <d2837c89-c9b2-fd18-d090-567f2a90cf75@kontron.de>
References: <cover.1582603241.git.ytc-mb-yfuruyama7@kioxia.com>
        <41b30e2d308ec7f252d71970a2ed1c29cd25c0d7.1582603241.git.ytc-mb-yfuruyama7@kioxia.com>
        <d2837c89-c9b2-fd18-d090-567f2a90cf75@kontron.de>
Organization: Bootlin
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hello,

Schrempf Frieder <frieder.schrempf@kontron.de> wrote on Mon, 2 Mar 2020
08:02:25 +0000:

> On 28.02.20 04:11, Yoshio Furuyama wrote:
> > The suffix was changed to classify from "g" to "j" between 1st generation
> > device and 2nd generation device that's new Serial NAND of Kioxia brand.  
> 
> I had to read this sentence multiple times to understand it. Maybe 
> something like this would be better:
> 
>    The suffix was changed from "g" to "j" to classify between 1st
>    generation and 2nd generation serial NAND devices (which now belong to
>    the Kioxia brand).
> 
> > As reference that's
> > 1st generation device of 1Gbit product is "tc58cvg0s3hraig"
> > 2nd generation device of 1Gbit product is "tc58cvg0s3hraij".
> > 
> > The 8Gbit product "TH58CxG3S0HRAIJ" is new line up of Kioxia's serial nand
> > and changed the prefix from tc58 to th58.
> > Thus it was changed argument to the function from "tc58cxgxsx" to
> > "tx58cxgxsxraix".  
> 
> Same here. It is very hard to read. I would write something like this:
> 
>    The 8Gbit type "TH58CxG3S0HRAIJ" is new to Kioxia's serial NAND lineup
>    and the prefix was changed from "TC58" to "TH85".
> 
>    Thus the functions were renamed from tc58cxgxsx_*() to
>    tx58cxgxsxraix_*().
> 
> With an easier to understand commit message:
> 
> Reviewed-by: Frieder Schrempf <frieder.schrempf@kontron.de>

Agreed, the commit log proposal from Frieder looks better.

The rest of the patch is fine by me though.

Thanks,
Miquèl