From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753209AbcJCRih (ORCPT ); Mon, 3 Oct 2016 13:38:37 -0400 Received: from down.free-electrons.com ([37.187.137.238]:43265 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752028AbcJCRia (ORCPT ); Mon, 3 Oct 2016 13:38:30 -0400 Date: Mon, 3 Oct 2016 19:38:26 +0200 From: Boris Brezillon To: Brian Norris Cc: Richard Weinberger , "linux-mtd@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PULL] mtd: nand: material for 4.9 Message-ID: <20161003193826.414116cf@bbrezillon> In-Reply-To: <20161003165533.GA66959@google.com> References: <20160925205326.67b5a9bd@bbrezillon> <20161003165533.GA66959@google.com> X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; 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 Hi Brian, On Mon, 3 Oct 2016 09:55:33 -0700 Brian Norris wrote: > Hi Boris, > > On Sun, Sep 25, 2016 at 08:53:26PM +0200, Boris Brezillon wrote: > > Hi Brian, > > > > Here is my PR for 4.9. Let me know if you see any problem. > > Also, please let us know ahead of time if you can't take this PR (I know you > > are busy lately, and I guess I could send this PR directly to Linus or ask > > Richard to take it in its UBIFS/UBI tree). > > I made some time for this one. We'll see about the pairing one > (hopefully trivial? but I'm not sure if there's value in merging without > users). Well, I already posted UBI patches using this API, and that'd be easier if the interface is already available (that would solve cross-tree dependencies). > > > Thanks, > > > > Boris > > > > The following changes since commit 29b4817d4018df78086157ea3a55c1d9424a7cfc: > > > > Linux 4.8-rc1 (2016-08-07 18:18:00 -0700) > > > > are available in the git repository at: > > > > git@github.com:linux-nand/linux.git tags/for-4.9 > > > > for you to fetch changes up to d44154f969a44269a9288c274c1c2fd9e85df8a5: > > > > mtd: nand: Provide nand_cleanup() function to free NAND related resources (2016-09-23 09:35:16 +0200) > > Thanks. I only added one trivial fixup that checkpatch.pl's spelling.txt > (what??) caught: > > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c > index 7a00245ebada..e5718e5ecf92 100644 > --- a/drivers/mtd/nand/nand_base.c > +++ b/drivers/mtd/nand/nand_base.c > @@ -746,7 +746,7 @@ static void nand_command_lp(struct mtd_info *mtd, unsigned int command, > chip->cmd_ctrl(mtd, column, ctrl); > ctrl &= ~NAND_CTRL_CHANGE; > > - /* Only ouput a single addr cycle for 8bits opcodes. */ > + /* Only output a single addr cycle for 8bits opcodes. */ > if (!nand_opcode_8bits(command)) > chip->cmd_ctrl(mtd, column >> 8, ctrl); > } > > > > ---------------------------------------------------------------- > > Notable core changes: > > - add the infrastructure to automate NAND timings configuration > > - provide a generic DT property to maximize ECC strength > > > > The rest is just a bunch of minor drivers and core fixes/cleanup > > patches. > > > > ---------------------------------------------------------------- > > I added a small note in the merge commit about the BBT changes you made. > That's an especially fragile piece of code I think (and used by every > NAND driver), but it looked OK to me. Oh, I completely forgot about this important change. Thanks for fixing the typo and adding more information in the merge message. Boris