Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752721AbbFCBdh (ORCPT ); Tue, 2 Jun 2015 21:33:37 -0400 Received: from mailout.micron.com ([137.201.242.129]:17197 "EHLO mailout.micron.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752599AbbFCBd2 (ORCPT ); Tue, 2 Jun 2015 21:33:28 -0400 From: =?utf-8?B?UGV0ZXIgUGFuIOa9mOagiyAocGV0ZXJwYW5kb25nKQ==?= To: =?utf-8?B?UGV0ZXIgUGFuIOa9mOagiyAocGV0ZXJwYW5kb25nKQ==?= , "dwmw2@infradead.org" , Brian Norris , "fransklaver@gmail.com" , "wsa@the-dreams.de" , "zajec5@gmail.com" , "boris.brezillon@free-electrons.com" , "baruch@tkos.co.il" , "ezequiel.garcia@free-electrons.com" , "kdasu.kdev@gmail.com" , "rogerq@ti.com" , "asierra@xes-inc.com" , bpqw CC: "linux-mtd@lists.infradead.org" , "linux-kernel@vger.kernel.org" , =?utf-8?B?UWkgV2FuZyDnjovotbcgKHFpd2FuZyk=?= Subject: RE: [PATCH 1/1] mtd: nand_bbt: separate struct nand_chip from nand_bbt.c Thread-Topic: [PATCH 1/1] mtd: nand_bbt: separate struct nand_chip from nand_bbt.c Thread-Index: AdCO369Kn44//ROMQ26ouDqH5B3pagAAQ31wA68JmgA= Date: Wed, 3 Jun 2015 01:31:27 +0000 Message-ID: <87F60714EC601C4C83DFF1D2E3D390A027B12734@NTXXIAMBX02.xacn.micron.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.167.84.5] X-TM-AS-Product-Ver: SMEX-11.0.0.4179-8.000.1202-21584.005 X-TM-AS-Result: No--6.100100-0.000000-31 X-TM-AS-User-Approved-Sender: Yes X-TM-AS-User-Blocked-Sender: No x-mt-checkinternalsenderrule: True Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t531XeoO024239 Content-Length: 1262 Lines: 30 Ping > > Currently nand_bbt.c is tied with struct nand_chip, and it makes other > NAND family chips hard to use nand_bbt.c. Maybe it's the reason why > onenand has own bbt(onenand_bbt.c). > > Parameterize a few relevant device detail information into a new > nand_bbt struct, and set some hooks for chip specified part. Allocate > and initialize struct nand_bbt in nand_base.c. > > Most of the patch is borrowed from Brian Norris > . > http://git.infradead.org/users/norris/linux- > mtd.git/shortlog/refs/heads/nand-bbt > > Signed-off-by: Peter Pan > Signed-off-by: Brian Norris > --- > drivers/mtd/nand/docg4.c | 8 +- > drivers/mtd/nand/nand_base.c | 145 +++++++++++- > drivers/mtd/nand/nand_bbt.c | 518 +++++++++++++++++------------------ > -------- > include/linux/mtd/bbm.h | 96 +------- > include/linux/mtd/nand.h | 11 +- > include/linux/mtd/nand_bbt.h | 160 +++++++++++++ > 6 files changed, 516 insertions(+), 422 deletions(-) > create mode 100644 include/linux/mtd/nand_bbt.h > ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?