arm: vf610: add NAND flash support
Subsystems (2)
v4 cover arm: vf610: add NAND flash support
This patch set adds NAND Flash Controller (NFC) support for Freescale Vybrid ARM SoCs (vf610).
The driver is based on Bill Pringlemeirs prelineary patch sent in January 2014 to the MTD mailing list:
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/226623.html
Changes in v4
- Drop preparation patches 1 and 2 since they are already merged upstream
- Add memcpy accessor function vf610_nfc_memcpy
- Remove disabled code
Changes in v3
- Further optimizations in vf610_nfc_send_command(s) to avoid performance hit by additional barriers introduced through using writel/readl
- Removed unnecessary barrier and added comment
- Use void __iomem * as memory base data type
- Set page to -1 on read error to allow reread of pages by the higher layers
- Minor style fixes
Changes in v2:
- Renamed the driver from fsl_nfc to vf610_nfc
- Use writel/readl in register access functions
- Optimized some register accesses by read/write value only once in code
- Use CONFIG_SYS_NAND_SELF_INIT and fix board_nand_init implementation
- Removed uncommented code/fixed comments
- Add CONFIG_USE_ARCH_MEMCPY for improved NAND performance
- Use hweight32 for improved count_written_bits performance
- Implement page_read/page_write rather than reuse MTD stacks version
Stefan Agner (2):
mtd: nand: add Freescale vf610_nfc driver
arm: vf610: add NAND support for vf610twr
board/freescale/vf610twr/vf610twr.c | 47 ++- configs/vf610twr_defconfig | 2 +- configs/vf610twr_nand_defconfig | 3 + drivers/mtd/nand/Makefile | 1 + drivers/mtd/nand/vf610_nfc.c | 721 ++++++++++++++++++++++++++++++++++++ include/configs/vf610twr.h | 46 ++- 6 files changed, 815 insertions(+), 5 deletions(-) create mode 100644 configs/vf610twr_nand_defconfig create mode 100644 drivers/mtd/nand/vf610_nfc.c
Signature · 1 line
2.1.0
PATCH v4 1/2 mtd: nand: add Freescale vf610_nfc driver
This adds initial support for Freescale NFC (NAND Flash Controller) found in ARM Vybrid SoC's, Power Architecture MPC5125 and others. The driver is called vf610_nfc since this is the first supported and tested hardware platform supported by the driver.
--- drivers/mtd/nand/Makefile | 1 + drivers/mtd/nand/vf610_nfc.c | 721 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 722 insertions(+) create mode 100644 drivers/mtd/nand/vf610_nfc.c
drivers/mtd/nand/Makefile +1 -0
@@ -51,6 +51,7 @@ obj-$(CONFIG_NAND_KB9202) += kb9202_nand.o obj-$(CONFIG_NAND_KIRKWOOD) += kirkwood_nand.o obj-$(CONFIG_NAND_KMETER1) += kmeter1_nand.o obj-$(CONFIG_NAND_MPC5121_NFC) += mpc5121_nfc.o +obj-$(CONFIG_NAND_VF610_NFC) += vf610_nfc.o obj-$(CONFIG_NAND_MXC) += mxc_nand.o obj-$(CONFIG_NAND_MXS) += mxs_nand.o obj-$(CONFIG_NAND_NDFC) += ndfc.o
drivers/mtd/nand/vf610_nfc.c new file +721 -0
721 lines not on this page · raw message
Signature · 1 line
2.1.0
PATCH v4 2/2 arm: vf610: add NAND support for vf610twr
In reply to Stefan AgnerThis adds NAND support for the Vybrid tower system (TWR-VF65GS10) provided by the vf610_nfc driver. Full 16-Bit bus width is supported. Also an aditional config vf610twr_nand is introduced which gets the environment from NAND. However, booting U-Boot from NAND is not yet possible due to missing boot configuration block (BCB).
--- board/freescale/vf610twr/vf610twr.c | 47 ++++++++++++++++++++++++++++++++++--- configs/vf610twr_defconfig | 2 +- configs/vf610twr_nand_defconfig | 3 +++ include/configs/vf610twr.h | 46 +++++++++++++++++++++++++++++++++++- 4 files changed, 93 insertions(+), 5 deletions(-) create mode 100644 configs/vf610twr_nand_defconfig
board/freescale/vf610twr/vf610twr.c +44 -3
@@ -278,6 +278,39 @@ static void setup_iomux_i2c(void)imx_iomux_v3_setup_multiple_pads (i2c0_pads,ARRAY_SIZE (i2c0_pads)); } +#ifdef CONFIG_NAND_VF610_NFC +static void setup_iomux_nfc (void ) +{ +static const iomux_v3_cfg_t nfc_pads[] = { +VF610_PAD_PTD31__NF_IO15 , +VF610_PAD_PTD30__NF_IO14 , +VF610_PAD_PTD29__NF_IO13 , +VF610_PAD_PTD28__NF_IO12 , +VF610_PAD_PTD27__NF_IO11 , +VF610_PAD_PTD26__NF_IO10 , +VF610_PAD_PTD25__NF_IO9 , +VF610_PAD_PTD24__NF_IO8 , +VF610_PAD_PTD23__NF_IO7 , +VF610_PAD_PTD22__NF_IO6 , +VF610_PAD_PTD21__NF_IO5 , +VF610_PAD_PTD20__NF_IO4 , +VF610_PAD_PTD19__NF_IO3 , +VF610_PAD_PTD18__NF_IO2 , +VF610_PAD_PTD17__NF_IO1 , +VF610_PAD_PTD16__NF_IO0 , +VF610_PAD_PTB24__NF_WE_B , +VF610_PAD_PTB25__NF_CE0_B , +VF610_PAD_PTB27__NF_RE_B , +VF610_PAD_PTC26__NF_RB_B , +VF610_PAD_PTC27__NF_ALE , +VF610_PAD_PTC28__NF_CLE + }; + +imx_iomux_v3_setup_multiple_pads (nfc_pads,ARRAY_SIZE (nfc_pads)); +} +#endif + +static void setup_iomux_qspi (void ) {static const iomux_v3_cfg_t qspi0_pads[] = {
@@ -354,6 +387,8 @@ static void clock_init(void) CCM_CCGR7_SDHC1_CTRL_MASK);clrsetbits_le32 (&ccm->ccgr9,CCM_REG_CTRL_MASK ,CCM_CCGR9_FEC0_CTRL_MASK |CCM_CCGR9_FEC1_CTRL_MASK ); +clrsetbits_le32 (&ccm->ccgr10,CCM_REG_CTRL_MASK , +CCM_CCGR10_NFC_CTRL_MASK );clrsetbits_le32 (&anadig->pll2_ctrl,ANADIG_PLL2_CTRL_POWERDOWN ,ANADIG_PLL2_CTRL_ENABLE |ANADIG_PLL2_CTRL_DIV_SELECT );
@@ -373,14 +408,17 @@ static void clock_init(void)CCM_CACRR_IPG_CLK_DIV (1 ) |CCM_CACRR_BUS_CLK_DIV (2 ) | CCM_CACRR_ARM_CLK_DIV(0));clrsetbits_le32 (&ccm->cscmr1,CCM_REG_CTRL_MASK , -CCM_CSCMR1_ESDHC1_CLK_SEL (3 ) |CCM_CSCMR1_QSPI0_CLK_SEL (3 )); +CCM_CSCMR1_ESDHC1_CLK_SEL (3 ) |CCM_CSCMR1_QSPI0_CLK_SEL (3 ) | +CCM_CSCMR1_NFC_CLK_SEL (0 ));clrsetbits_le32 (&ccm->cscdr1,CCM_REG_CTRL_MASK ,CCM_CSCDR1_RMII_CLK_EN );clrsetbits_le32 (&ccm->cscdr2,CCM_REG_CTRL_MASK , -CCM_CSCDR2_ESDHC1_EN |CCM_CSCDR2_ESDHC1_CLK_DIV (0 )); +CCM_CSCDR2_ESDHC1_EN |CCM_CSCDR2_ESDHC1_CLK_DIV (0 ) | +CCM_CSCDR2_NFC_EN );clrsetbits_le32 (&ccm->cscdr3,CCM_REG_CTRL_MASK ,CCM_CSCDR3_QSPI0_EN |CCM_CSCDR3_QSPI0_DIV (1 ) | -CCM_CSCDR3_QSPI0_X2_DIV (1 ) |CCM_CSCDR3_QSPI0_X4_DIV (3 )); +CCM_CSCDR3_QSPI0_X2_DIV (1 ) |CCM_CSCDR3_QSPI0_X4_DIV (3 ) | +CCM_CSCDR3_NFC_PRE_DIV (5 ));clrsetbits_le32 (&ccm->cscmr2,CCM_REG_CTRL_MASK ,CCM_CSCMR2_RMII_CLK_SEL (0 )); }
@@ -411,6 +449,9 @@ int board_early_init_f(void)setup_iomux_enet ();setup_iomux_i2c ();setup_iomux_qspi (); +#ifdef CONFIG_NAND_VF610_NFC +setup_iomux_nfc (); +#endif return 0 ; }
configs/vf610twr_defconfig +1 -1
@@ -1,3 +1,3 @@ -CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=board/freescale/vf610twr/imximage.cfg" +CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=board/freescale/vf610twr/imximage.cfg,ENV_IS_IN_MMC" CONFIG_ARM=y CONFIG_TARGET_VF610TWR=y
configs/vf610twr_nand_defconfig new file +3 -0
@@ -0,0 +1,3 @@ +CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=board/freescale/vf610twr/imximage.cfg,ENV_IS_IN_NAND" +CONFIG_ARM=y +CONFIG_TARGET_VF610TWR=y
include/configs/vf610twr.h +45 -1
68 lines not on this page · raw message
Signature · 1 line
2.1.0
PATCH v4 1/2 mtd: nand: add Freescale vf610_nfc driver
In reply to Stefan AgnerQuoted 6 lines · stefan at agner.ch
This adds initial support for Freescale NFC (NAND Flash Controller) found in ARM Vybrid SoC's, Power Architecture MPC5125 and others. The driver is called
vf610_nfcsince this is the first supported and tested hardware platform supported by the driver.Signed-off-by Stefan Agner <[email protected]>
PATCH v4 1/2 mtd: nand: add Freescale vf610_nfc driver
Quoted 11 lines · Bill Pringlemeir
Quoted 6 lines · stefan at agner.ch
This adds initial support for Freescale NFC (NAND Flash Controller) found in ARM Vybrid SoC's, Power Architecture MPC5125 and others. The driver is called
vf610_nfcsince this is the first supported and tested hardware platform supported by the driver.Signed-off-by Stefan Agner <[email protected]>Acked-by Bill Pringlemeir <[email protected]>
Thanks. I will merge it into u-boot-imx.
Best regards,
Stefano Babic
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================
PATCH v4 1/2 mtd: nand: add Freescale vf610_nfc driver
In reply to Stefan AgnerHi Stefan,
Quoted 7 lines · Stefan Agner
This adds initial support for Freescale NFC (NAND Flash Controller) found in ARM Vybrid SoC's, Power Architecture MPC5125 and others. The driver is called
vf610_nfcsince this is the first supported and tested hardware platform supported by the driver.Signed-off-by Stefan Agner <[email protected]>---
By applying I have seen some other minor issues. checkpatch reports some warnings, can you fix them ?
WARNING: line over 80 characters
#200: FILE: drivers/mtd/nand/vf610_nfc.c:157:
+#define mtd_to_nfc(_mtd) (struct vf610_nfc *)((struct nand_chip *)_mtd->priv)->priv;
CHECK: Alignment should match open parenthesis
#367: FILE: drivers/mtd/nand/vf610_nfc.c:324:
+ vf610_nfc_set_field(mtd, NFC_COL_ADDR, COL_ADDR_MASK,
+ COL_ADDR_SHIFT, column);
CHECK: Alignment should match open parenthesis
#371: FILE: drivers/mtd/nand/vf610_nfc.c:328:
+ vf610_nfc_set_field(mtd, NFC_ROW_ADDR, ROW_ADDR_MASK,
+ ROW_ADDR_SHIFT, page);
CHECK: Alignment should match open parenthesis
#388: FILE: drivers/mtd/nand/vf610_nfc.c:345:
+ vf610_nfc_send_commands(nfc->regs, NAND_CMD_SEQIN,
+ command, PROGRAM_PAGE_CMD_CODE);
CHECK: Alignment should match open parenthesis
#409: FILE: drivers/mtd/nand/vf610_nfc.c:366:
+ vf610_nfc_send_commands(nfc->regs, NAND_CMD_READ0,
+ NAND_CMD_READSTART, READ_PAGE_CMD_CODE);
CHECK: Alignment should match open parenthesis
#417: FILE: drivers/mtd/nand/vf610_nfc.c:374:
+ vf610_nfc_send_commands(nfc->regs, command,
+ NAND_CMD_ERASE2, ERASE_CMD_CODE);
WARNING: line over 80 characters
#428: FILE: drivers/mtd/nand/vf610_nfc.c:385:
+ vf610_nfc_send_command(nfc->regs, command, STATUS_READ_CMD_CODE);
WARNING: line over 80 characters
#460: FILE: drivers/mtd/nand/vf610_nfc.c:417:
+ vf610_nfc_memcpy(buf, nfc->regs + NFC_MAIN_AREA(0) + c, l);
CHECK: Alignment should match open parenthesis
#668: FILE: drivers/mtd/nand/vf610_nfc.c:625:
+ vf610_nfc_set_field(mtd, NFC_FLASH_CONFIG,
+ CONFIG_ECC_MODE_MASK,
CHECK: Alignment should match open parenthesis
#690: FILE: drivers/mtd/nand/vf610_nfc.c:647:
+ vf610_nfc_set_field(mtd, NFC_FLASH_CONFIG, CONFIG_PAGE_CNT_MASK,
+ CONFIG_PAGE_CNT_SHIFT, 1);
CHECK: Alignment should match open parenthesis
#694: FILE: drivers/mtd/nand/vf610_nfc.c:651:
+ vf610_nfc_set_field(mtd, NFC_FLASH_CONFIG,
+ CONFIG_ECC_SRAM_ADDR_MASK,
CHECK: Alignment should match open parenthesis
#737: FILE: drivers/mtd/nand/vf610_nfc.c:694:
+ vf610_nfc_set_field(mtd, NFC_FLASH_CONFIG,
+ CONFIG_ECC_MODE_MASK,
Quoted 170 lines
drivers/mtd/nand/Makefile | 1 + drivers/mtd/nand/vf610_nfc.c | 721 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 722 insertions(+) create mode 100644 drivers/mtd/nand/vf610_nfc.cdrivers/mtd/nand/Makefile +1
-0@@ -51,6 +51,7 @@ obj-$(CONFIG_NAND_KB9202) += kb9202_nand.o obj-$(CONFIG_NAND_KIRKWOOD) += kirkwood_nand.o obj-$(CONFIG_NAND_KMETER1) += kmeter1_nand.o obj-$(CONFIG_NAND_MPC5121_NFC) += mpc5121_nfc.o +obj-$(CONFIG_NAND_VF610_NFC) += vf610_nfc.o obj-$(CONFIG_NAND_MXC) += mxc_nand.o obj-$(CONFIG_NAND_MXS) += mxs_nand.o obj-$(CONFIG_NAND_NDFC) += ndfc.odrivers/mtd/nand/vf610_nfc.c new file +147
-0@@ -0,0 +1,721 @@ +/* +* Copyright 2009-2014 Freescale Semiconductor, Inc. and others +* +* Description: MPC5125, VF610, MCF54418 and Kinetis K70 Nand driver. +* Ported to U-Boot by Stefan Agner +* Based on RFC driver posted on Kernel Mailing list by Bill Pringlemeir +* Jason ported to M54418TWR and MVFA5. +* Authors: Stefan Agner <[email protected]> +* Bill Pringlemeir <[email protected]> +* Shaohui Xie <[email protected]> +* Jason Jin <[email protected]> +* +* Based on original driver mpc5121_nfc.c. +* +* This is free software; you can redistribute it and/or modify it +* under the terms of the GNU General Public License as published by +* the Free Software Foundation; either version 2 of the License, or +* (at your option) any later version. +* +* Limitations: +* - Untested on MPC5125 and M54418. +* - DMA not used. +* - 2K pages or less. +* - Only 2K page w. 64+OOB and hardware ECC. +*/ + +#include <common.h> +#include <malloc.h> + +#include <linux/mtd/mtd.h> +#include <linux/mtd/nand.h> +#include <linux/mtd/partitions.h> + +#include <nand.h> +#include <errno.h> +#include <asm/io.h> + +/* Register Offsets */ +#define NFC_FLASH_CMD1 0x3F00 +#define NFC_FLASH_CMD2 0x3F04 +#define NFC_COL_ADDR 0x3F08 +#define NFC_ROW_ADDR 0x3F0c +#define NFC_ROW_ADDR_INC 0x3F14 +#define NFC_FLASH_STATUS1 0x3F18 +#define NFC_FLASH_STATUS2 0x3F1c +#define NFC_CACHE_SWAP 0x3F28 +#define NFC_SECTOR_SIZE 0x3F2c +#define NFC_FLASH_CONFIG 0x3F30 +#define NFC_IRQ_STATUS 0x3F38 + +/* Addresses for NFC MAIN RAM BUFFER areas */ +#define NFC_MAIN_AREA (n) ((n) * 0x1000) + +#define PAGE_2K 0x0800 +#define OOB_64 0x0040 + +/* +* NFC_CMD2[CODE] values. See section: +* - 31.4.7 Flash Command Code Description, Vybrid manual +* - 23.8.6 Flash Command Sequencer, MPC5125 manual +* +* Briefly these are bitmasks of controller cycles. +*/ +#define READ_PAGE_CMD_CODE 0x7EE0 +#define PROGRAM_PAGE_CMD_CODE 0x7FC0 +#define ERASE_CMD_CODE 0x4EC0 +#define READ_ID_CMD_CODE 0x4804 +#define RESET_CMD_CODE 0x4040 +#define STATUS_READ_CMD_CODE 0x4068 + +/* NFC ECC mode define */ +#define ECC_BYPASS 0 +#define ECC_45_BYTE 6 + +/*** Register Mask and bit definitions */ + +/* NFC_FLASH_CMD1 Field */ +#define CMD_BYTE2_MASK 0xFF000000 +#define CMD_BYTE2_SHIFT 24 + +/* NFC_FLASH_CM2 Field */ +#define CMD_BYTE1_MASK 0xFF000000 +#define CMD_BYTE1_SHIFT 24 +#define CMD_CODE_MASK 0x00FFFF00 +#define CMD_CODE_SHIFT 8 +#define BUFNO_MASK 0x00000006 +#define BUFNO_SHIFT 1 +#define START_BIT (1<<0) + +/* NFC_COL_ADDR Field */ +#define COL_ADDR_MASK 0x0000FFFF +#define COL_ADDR_SHIFT 0 + +/* NFC_ROW_ADDR Field */ +#define ROW_ADDR_MASK 0x00FFFFFF +#define ROW_ADDR_SHIFT 0 +#define ROW_ADDR_CHIP_SEL_RB_MASK 0xF0000000 +#define ROW_ADDR_CHIP_SEL_RB_SHIFT 28 +#define ROW_ADDR_CHIP_SEL_MASK 0x0F000000 +#define ROW_ADDR_CHIP_SEL_SHIFT 24 + +/* NFC_FLASH_STATUS2 Field */ +#define STATUS_BYTE1_MASK 0x000000FF + +/* NFC_FLASH_CONFIG Field */ +#define CONFIG_ECC_SRAM_ADDR_MASK 0x7FC00000 +#define CONFIG_ECC_SRAM_ADDR_SHIFT 22 +#define CONFIG_ECC_SRAM_REQ_BIT (1<<21) +#define CONFIG_DMA_REQ_BIT (1<<20) +#define CONFIG_ECC_MODE_MASK 0x000E0000 +#define CONFIG_ECC_MODE_SHIFT 17 +#define CONFIG_FAST_FLASH_BIT (1<<16) +#define CONFIG_16BIT (1<<7) +#define CONFIG_BOOT_MODE_BIT (1<<6) +#define CONFIG_ADDR_AUTO_INCR_BIT (1<<5) +#define CONFIG_BUFNO_AUTO_INCR_BIT (1<<4) +#define CONFIG_PAGE_CNT_MASK 0xF +#define CONFIG_PAGE_CNT_SHIFT 0 + +/* NFC_IRQ_STATUS Field */ +#define IDLE_IRQ_BIT (1<<29) +#define IDLE_EN_BIT (1<<20) +#define CMD_DONE_CLEAR_BIT (1<<18) +#define IDLE_CLEAR_BIT (1<<17) + +#define NFC_TIMEOUT (1000) + +/* ECC status placed at end of buffers. */ +#define ECC_SRAM_ADDR ((PAGE_2K+256-8) >> 3) +#define ECC_STATUS_MASK 0x80 +#define ECC_ERR_COUNT 0x3F + +/* +* ECC status is stored at NFC_CFG[ECCADD] +4 for little-endian +* and +7 for big-endian SOC. +*/ +#ifdef CONFIG_VF610 +#define ECC_OFFSET 4 +#else +#define ECC_OFFSET 7 +#endif + +struct vf610_nfc { +struct mtd_info *mtd; +struct nand_chip chip; +void __iomem *regs; +uint column;
drivers/mtd/nand/vf610_nfc.c line 157
Quoted 10 lines
drivers/mtd/nand/vf610_nfc.c new file +10
-0+int spareonly; +int page; +/* Status and ID are in alternate locations. */ +int alt_buf; +#define ALT_BUF_ID 1 +#define ALT_BUF_STAT 2 +struct clk *clk; +}; + +#define mtd_to_nfc (_mtd) (struct vf610_nfc *)((struct nand_chip *)_mtd->priv)->priv;
The ending semicolon should be not part of the macro.
Best regards,
Stefano Babic
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================