From patchwork Tue Feb 23 15:54:04 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?b?S8OhcmkgRGF2w63DsHNzb24=?= X-Patchwork-Id: 46072 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 3A305B7D06 for ; Wed, 24 Feb 2010 02:56:53 +1100 (EST) Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1Njx5U-00071q-CP; Tue, 23 Feb 2010 15:54:16 +0000 Received: from mail.mymarel.com ([213.167.134.75] helo=mail.marel.com) by bombadil.infradead.org with esmtps (Exim 4.69 #1 (Red Hat Linux)) id 1Njx5M-00070T-IX for linux-mtd@lists.infradead.org; Tue, 23 Feb 2010 15:54:13 +0000 Received: from [10.100.11.11] (10.100.11.11) by is-hdq-exfront.marel.net (10.100.1.135) with Microsoft SMTP Server id 8.2.234.1; Tue, 23 Feb 2010 15:54:05 +0000 Message-ID: <4B83FA1C.8000902@marel.com> Date: Tue, 23 Feb 2010 15:54:04 +0000 From: =?UTF-8?B?S8OhcmkgRGF2w63DsHNzb24=?= User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.9pre) Gecko/20100217 Lightning/1.0b1 Shredder/3.0.3pre MIME-Version: 1.0 To: "dedekind1@gmail.com" Subject: Re: [PATCH v4 04/11] mtd: Add MPC5121 NAND Flash Controller driver References: <1265377377-29327-5-git-send-email-agust@denx.de> <1266255305-29224-1-git-send-email-agust@denx.de> <1266307868.11659.177.camel@localhost> In-Reply-To: <1266307868.11659.177.camel@localhost> X-CRM114-Version: 20090807-BlameThorstenAndJenny ( TRE 0.7.6 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20100223_105408_902130_AE07026F X-CRM114-Status: GOOD ( 16.18 ) X-Spam-Score: 0.0 (/) X-Spam-Report: SpamAssassin version 3.2.5 on bombadil.infradead.org summary: Content analysis details: (0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- _SUMMARY_ Cc: Piotr Ziecik , Detlev Zundel , "linuxppc-dev@ozlabs.org" , "linux-mtd@lists.infradead.org" , Anatolij Gustschin , David Woodhouse X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-mtd-bounces@lists.infradead.org Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org Any special reason this in MPC5121 Revision 2 only? I did not see anything that contradicted the rev 3 manual in the code. But there is one harmless bug (since the NFC_CMD_FAIL symbol is never used), i.e. git diff The NAND chip is is detected (given that I disable the Revison check), but the /dev7mtd0 is not accessable as block device. Is that normal? The UBIFS also gives up I am assuming because the /dev/mtd0 is not block device. rg kd On 02/16/2010 08:11 AM, Artem Bityutskiy wrote: > On Mon, 2010-02-15 at 18:35 +0100, Anatolij Gustschin wrote: >> Adds NAND Flash Controller driver for MPC5121 Revision 2. >> All device features, except hardware ECC and power management, >> are supported. >> >> Signed-off-by: Piotr Ziecik >> Signed-off-by: Wolfgang Denk >> Signed-off-by: Anatolij Gustschin >> Acked-by: Grant Likely >> Cc: >> Cc: Grant Likely >> Cc: John Rigby >> --- >> Changes since v3: >> - include 'asm/mpc5121.h' header instead of 'asm/mpc5xxx.h'. >> This change is needed because arch patch adding reset >> module definition was reworked and doesn't add mpc5121 >> specific definitions to common header for mpc52xx/mpc5121. >> >> Changes since v2: >> - move the arch bits into separate patch >> (it is the next patch in this series now) >> - use __devinit/__devexit/__devexit_p and __devinitdata >> >> Changes since v1: >> - add logfile with changes since previous version >> >> Changes since the patch version submitted in May 2009: >> >> - move mpc5121_nfc.h to the driver .c as there is only one user >> - remove DRV_VERSION macro >> - replace printk() by dev_*() >> - drop unnecessary .suspend and .resume initializations >> - remove duplicate .name/.owner settings >> - fix mpc5121_nfc_init() to "return of_register_platform_driver(&mpc5121_nfc_driver);" >> - move module_init() to just below the init function >> - remove MODULE_VERSION >> - use "mtd: Add MPC5121 NAND Flash Controller driver" as the subject, >> previously it was "mpc5121: Added NAND Flash Controller driver. >> >> drivers/mtd/nand/Kconfig | 7 + >> drivers/mtd/nand/Makefile | 1 + >> drivers/mtd/nand/mpc5121_nfc.c | 916 ++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 924 insertions(+), 0 deletions(-) >> create mode 100644 drivers/mtd/nand/mpc5121_nfc.c > > Pushed to my l2-mtd-2.6.git / dunno. > diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c index 191bf99..38a99ef 100644 --- a/drivers/mtd/nand/mpc5121_nfc.c +++ b/drivers/mtd/nand/mpc5121_nfc.c @@ -100,7 +100,7 @@ #define NFC_OUTPUT (1 << 3) #define NFC_ID (1 << 4) #define NFC_STATUS (1 << 5) -#define NFC_CMD_FAIL (1 << 15) +#define NFC_CMD_FAIL (1 << 14) #define NFC_INT (1 << 15) /* Bit Definitions: NFC_WRPROT */