Re: [PATCH 2/2] mtd: nand: remove bf5xx_nand driver

From: Boris Brezillon
Date: Sun Mar 18 2018 - 17:19:13 EST


On Wed, 14 Mar 2018 17:30:23 +0100
Arnd Bergmann <arnd@xxxxxxxx> wrote:

> The blackfin architecture is getting removed, so this driver has
> become obsolete.
>

Applied to nand/next.

Thanks,

Boris

> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> ---
> drivers/mtd/nand/raw/Kconfig | 32 --
> drivers/mtd/nand/raw/Makefile | 1 -
> drivers/mtd/nand/raw/bf5xx_nand.c | 861 --------------------------------------
> 3 files changed, 894 deletions(-)
> delete mode 100644 drivers/mtd/nand/raw/bf5xx_nand.c
>
> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig
> index 605ec8cce67b..0f2b6f348a0a 100644
> --- a/drivers/mtd/nand/raw/Kconfig
> +++ b/drivers/mtd/nand/raw/Kconfig
> @@ -116,38 +116,6 @@ config MTD_NAND_AU1550
> This enables the driver for the NAND flash controller on the
> AMD/Alchemy 1550 SOC.
>
> -config MTD_NAND_BF5XX
> - tristate "Blackfin on-chip NAND Flash Controller driver"
> - depends on BF54x || BF52x
> - help
> - This enables the Blackfin on-chip NAND flash controller
> -
> - No board specific support is done by this driver, each board
> - must advertise a platform_device for the driver to attach.
> -
> - This driver can also be built as a module. If so, the module
> - will be called bf5xx-nand.
> -
> -config MTD_NAND_BF5XX_HWECC
> - bool "BF5XX NAND Hardware ECC"
> - default y
> - depends on MTD_NAND_BF5XX
> - help
> - Enable the use of the BF5XX's internal ECC generator when
> - using NAND.
> -
> -config MTD_NAND_BF5XX_BOOTROM_ECC
> - bool "Use Blackfin BootROM ECC Layout"
> - default n
> - depends on MTD_NAND_BF5XX_HWECC
> - help
> - If you wish to modify NAND pages and allow the Blackfin on-chip
> - BootROM to boot from them, say Y here. This is only necessary
> - if you are booting U-Boot out of NAND and you wish to update
> - U-Boot from Linux' userspace. Otherwise, you should say N here.
> -
> - If unsure, say N.
> -
> config MTD_NAND_S3C2410
> tristate "NAND Flash support for Samsung S3C SoCs"
> depends on ARCH_S3C24XX || ARCH_S3C64XX
> diff --git a/drivers/mtd/nand/raw/Makefile b/drivers/mtd/nand/raw/Makefile
> index f16f59a197a3..165b7ef9e9a1 100644
> --- a/drivers/mtd/nand/raw/Makefile
> +++ b/drivers/mtd/nand/raw/Makefile
> @@ -11,7 +11,6 @@ obj-$(CONFIG_MTD_NAND_DENALI) += denali.o
> obj-$(CONFIG_MTD_NAND_DENALI_PCI) += denali_pci.o
> obj-$(CONFIG_MTD_NAND_DENALI_DT) += denali_dt.o
> obj-$(CONFIG_MTD_NAND_AU1550) += au1550nd.o
> -obj-$(CONFIG_MTD_NAND_BF5XX) += bf5xx_nand.o
> obj-$(CONFIG_MTD_NAND_S3C2410) += s3c2410.o
> obj-$(CONFIG_MTD_NAND_TANGO) += tango_nand.o
> obj-$(CONFIG_MTD_NAND_DAVINCI) += davinci_nand.o
> diff --git a/drivers/mtd/nand/raw/bf5xx_nand.c b/drivers/mtd/nand/raw/bf5xx_nand.c
> deleted file mode 100644
> index 9a1d8d104570..000000000000
> --- a/drivers/mtd/nand/raw/bf5xx_nand.c
> +++ /dev/null
> @@ -1,861 +0,0 @@
> -/*
> - * Copyright 2006-2008 Analog Devices Inc.
> - * http://blackfin.uclinux.org/
> - * Bryan Wu <bryan.wu@xxxxxxxxxx>
> - *
> - * Blackfin BF5xx on-chip NAND flash controller driver
> - *
> - * Derived from s3c2410.c
> - * Copyright (c) 2007 Ben Dooks <ben@xxxxxxxxxxxx>
> - *
> - * Derived from cafe.c
> - * Copyright  2006 Red Hat, Inc.
> - * Copyright  2006 David Woodhouse <dwmw2@xxxxxxxxxxxxx>
> - *
> - * Changelog:
> - * 12-Jun-2007 Bryan Wu: Initial version
> - * 18-Jul-2007 Bryan Wu:
> - * - ECC_HW and ECC_SW supported
> - * - DMA supported in ECC_HW
> - * - YAFFS tested as rootfs in both ECC_HW and ECC_SW
> - *
> - * This program 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.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> - * GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
> -*/
> -
> -#include <linux/module.h>
> -#include <linux/types.h>
> -#include <linux/kernel.h>
> -#include <linux/string.h>
> -#include <linux/ioport.h>
> -#include <linux/platform_device.h>
> -#include <linux/delay.h>
> -#include <linux/dma-mapping.h>
> -#include <linux/err.h>
> -#include <linux/slab.h>
> -#include <linux/io.h>
> -#include <linux/bitops.h>
> -
> -#include <linux/mtd/mtd.h>
> -#include <linux/mtd/rawnand.h>
> -#include <linux/mtd/nand_ecc.h>
> -#include <linux/mtd/partitions.h>
> -
> -#include <asm/blackfin.h>
> -#include <asm/dma.h>
> -#include <asm/cacheflush.h>
> -#include <asm/nand.h>
> -#include <asm/portmux.h>
> -
> -#define DRV_NAME "bf5xx-nand"
> -#define DRV_VERSION "1.2"
> -#define DRV_AUTHOR "Bryan Wu <bryan.wu@xxxxxxxxxx>"
> -#define DRV_DESC "BF5xx on-chip NAND FLash Controller Driver"
> -
> -/* NFC_STAT Masks */
> -#define NBUSY 0x01 /* Not Busy */
> -#define WB_FULL 0x02 /* Write Buffer Full */
> -#define PG_WR_STAT 0x04 /* Page Write Pending */
> -#define PG_RD_STAT 0x08 /* Page Read Pending */
> -#define WB_EMPTY 0x10 /* Write Buffer Empty */
> -
> -/* NFC_IRQSTAT Masks */
> -#define NBUSYIRQ 0x01 /* Not Busy IRQ */
> -#define WB_OVF 0x02 /* Write Buffer Overflow */
> -#define WB_EDGE 0x04 /* Write Buffer Edge Detect */
> -#define RD_RDY 0x08 /* Read Data Ready */
> -#define WR_DONE 0x10 /* Page Write Done */
> -
> -/* NFC_RST Masks */
> -#define ECC_RST 0x01 /* ECC (and NFC counters) Reset */
> -
> -/* NFC_PGCTL Masks */
> -#define PG_RD_START 0x01 /* Page Read Start */
> -#define PG_WR_START 0x02 /* Page Write Start */
> -
> -#ifdef CONFIG_MTD_NAND_BF5XX_HWECC
> -static int hardware_ecc = 1;
> -#else
> -static int hardware_ecc;
> -#endif
> -
> -static const unsigned short bfin_nfc_pin_req[] =
> - {P_NAND_CE,
> - P_NAND_RB,
> - P_NAND_D0,
> - P_NAND_D1,
> - P_NAND_D2,
> - P_NAND_D3,
> - P_NAND_D4,
> - P_NAND_D5,
> - P_NAND_D6,
> - P_NAND_D7,
> - P_NAND_WE,
> - P_NAND_RE,
> - P_NAND_CLE,
> - P_NAND_ALE,
> - 0};
> -
> -#ifdef CONFIG_MTD_NAND_BF5XX_BOOTROM_ECC
> -static int bootrom_ooblayout_ecc(struct mtd_info *mtd, int section,
> - struct mtd_oob_region *oobregion)
> -{
> - if (section > 7)
> - return -ERANGE;
> -
> - oobregion->offset = section * 8;
> - oobregion->length = 3;
> -
> - return 0;
> -}
> -
> -static int bootrom_ooblayout_free(struct mtd_info *mtd, int section,
> - struct mtd_oob_region *oobregion)
> -{
> - if (section > 7)
> - return -ERANGE;
> -
> - oobregion->offset = (section * 8) + 3;
> - oobregion->length = 5;
> -
> - return 0;
> -}
> -
> -static const struct mtd_ooblayout_ops bootrom_ooblayout_ops = {
> - .ecc = bootrom_ooblayout_ecc,
> - .free = bootrom_ooblayout_free,
> -};
> -#endif
> -
> -/*
> - * Data structures for bf5xx nand flash controller driver
> - */
> -
> -/* bf5xx nand info */
> -struct bf5xx_nand_info {
> - /* mtd info */
> - struct nand_hw_control controller;
> - struct nand_chip chip;
> -
> - /* platform info */
> - struct bf5xx_nand_platform *platform;
> -
> - /* device info */
> - struct device *device;
> -
> - /* DMA stuff */
> - struct completion dma_completion;
> -};
> -
> -/*
> - * Conversion functions
> - */
> -static struct bf5xx_nand_info *mtd_to_nand_info(struct mtd_info *mtd)
> -{
> - return container_of(mtd_to_nand(mtd), struct bf5xx_nand_info,
> - chip);
> -}
> -
> -static struct bf5xx_nand_info *to_nand_info(struct platform_device *pdev)
> -{
> - return platform_get_drvdata(pdev);
> -}
> -
> -static struct bf5xx_nand_platform *to_nand_plat(struct platform_device *pdev)
> -{
> - return dev_get_platdata(&pdev->dev);
> -}
> -
> -/*
> - * struct nand_chip interface function pointers
> - */
> -
> -/*
> - * bf5xx_nand_hwcontrol
> - *
> - * Issue command and address cycles to the chip
> - */
> -static void bf5xx_nand_hwcontrol(struct mtd_info *mtd, int cmd,
> - unsigned int ctrl)
> -{
> - if (cmd == NAND_CMD_NONE)
> - return;
> -
> - while (bfin_read_NFC_STAT() & WB_FULL)
> - cpu_relax();
> -
> - if (ctrl & NAND_CLE)
> - bfin_write_NFC_CMD(cmd);
> - else if (ctrl & NAND_ALE)
> - bfin_write_NFC_ADDR(cmd);
> - SSYNC();
> -}
> -
> -/*
> - * bf5xx_nand_devready()
> - *
> - * returns 0 if the nand is busy, 1 if it is ready
> - */
> -static int bf5xx_nand_devready(struct mtd_info *mtd)
> -{
> - unsigned short val = bfin_read_NFC_STAT();
> -
> - if ((val & NBUSY) == NBUSY)
> - return 1;
> - else
> - return 0;
> -}
> -
> -/*
> - * ECC functions
> - * These allow the bf5xx to use the controller's ECC
> - * generator block to ECC the data as it passes through
> - */
> -
> -/*
> - * ECC error correction function
> - */
> -static int bf5xx_nand_correct_data_256(struct mtd_info *mtd, u_char *dat,
> - u_char *read_ecc, u_char *calc_ecc)
> -{
> - struct bf5xx_nand_info *info = mtd_to_nand_info(mtd);
> - u32 syndrome[5];
> - u32 calced, stored;
> - int i;
> - unsigned short failing_bit, failing_byte;
> - u_char data;
> -
> - calced = calc_ecc[0] | (calc_ecc[1] << 8) | (calc_ecc[2] << 16);
> - stored = read_ecc[0] | (read_ecc[1] << 8) | (read_ecc[2] << 16);
> -
> - syndrome[0] = (calced ^ stored);
> -
> - /*
> - * syndrome 0: all zero
> - * No error in data
> - * No action
> - */
> - if (!syndrome[0] || !calced || !stored)
> - return 0;
> -
> - /*
> - * sysdrome 0: only one bit is one
> - * ECC data was incorrect
> - * No action
> - */
> - if (hweight32(syndrome[0]) == 1) {
> - dev_err(info->device, "ECC data was incorrect!\n");
> - return -EBADMSG;
> - }
> -
> - syndrome[1] = (calced & 0x7FF) ^ (stored & 0x7FF);
> - syndrome[2] = (calced & 0x7FF) ^ ((calced >> 11) & 0x7FF);
> - syndrome[3] = (stored & 0x7FF) ^ ((stored >> 11) & 0x7FF);
> - syndrome[4] = syndrome[2] ^ syndrome[3];
> -
> - for (i = 0; i < 5; i++)
> - dev_info(info->device, "syndrome[%d] 0x%08x\n", i, syndrome[i]);
> -
> - dev_info(info->device,
> - "calced[0x%08x], stored[0x%08x]\n",
> - calced, stored);
> -
> - /*
> - * sysdrome 0: exactly 11 bits are one, each parity
> - * and parity' pair is 1 & 0 or 0 & 1.
> - * 1-bit correctable error
> - * Correct the error
> - */
> - if (hweight32(syndrome[0]) == 11 && syndrome[4] == 0x7FF) {
> - dev_info(info->device,
> - "1-bit correctable error, correct it.\n");
> - dev_info(info->device,
> - "syndrome[1] 0x%08x\n", syndrome[1]);
> -
> - failing_bit = syndrome[1] & 0x7;
> - failing_byte = syndrome[1] >> 0x3;
> - data = *(dat + failing_byte);
> - data = data ^ (0x1 << failing_bit);
> - *(dat + failing_byte) = data;
> -
> - return 1;
> - }
> -
> - /*
> - * sysdrome 0: random data
> - * More than 1-bit error, non-correctable error
> - * Discard data, mark bad block
> - */
> - dev_err(info->device,
> - "More than 1-bit error, non-correctable error.\n");
> - dev_err(info->device,
> - "Please discard data, mark bad block\n");
> -
> - return -EBADMSG;
> -}
> -
> -static int bf5xx_nand_correct_data(struct mtd_info *mtd, u_char *dat,
> - u_char *read_ecc, u_char *calc_ecc)
> -{
> - struct nand_chip *chip = mtd_to_nand(mtd);
> - int ret, bitflips = 0;
> -
> - ret = bf5xx_nand_correct_data_256(mtd, dat, read_ecc, calc_ecc);
> - if (ret < 0)
> - return ret;
> -
> - bitflips = ret;
> -
> - /* If ecc size is 512, correct second 256 bytes */
> - if (chip->ecc.size == 512) {
> - dat += 256;
> - read_ecc += 3;
> - calc_ecc += 3;
> - ret = bf5xx_nand_correct_data_256(mtd, dat, read_ecc, calc_ecc);
> - if (ret < 0)
> - return ret;
> -
> - bitflips += ret;
> - }
> -
> - return bitflips;
> -}
> -
> -static void bf5xx_nand_enable_hwecc(struct mtd_info *mtd, int mode)
> -{
> - return;
> -}
> -
> -static int bf5xx_nand_calculate_ecc(struct mtd_info *mtd,
> - const u_char *dat, u_char *ecc_code)
> -{
> - struct bf5xx_nand_info *info = mtd_to_nand_info(mtd);
> - struct nand_chip *chip = mtd_to_nand(mtd);
> - u16 ecc0, ecc1;
> - u32 code[2];
> - u8 *p;
> -
> - /* first 3 bytes ECC code for 256 page size */
> - ecc0 = bfin_read_NFC_ECC0();
> - ecc1 = bfin_read_NFC_ECC1();
> -
> - code[0] = (ecc0 & 0x7ff) | ((ecc1 & 0x7ff) << 11);
> -
> - dev_dbg(info->device, "returning ecc 0x%08x\n", code[0]);
> -
> - p = (u8 *) code;
> - memcpy(ecc_code, p, 3);
> -
> - /* second 3 bytes ECC code for 512 ecc size */
> - if (chip->ecc.size == 512) {
> - ecc0 = bfin_read_NFC_ECC2();
> - ecc1 = bfin_read_NFC_ECC3();
> - code[1] = (ecc0 & 0x7ff) | ((ecc1 & 0x7ff) << 11);
> -
> - /* second 3 bytes in ecc_code for second 256
> - * bytes of 512 page size
> - */
> - p = (u8 *) (code + 1);
> - memcpy((ecc_code + 3), p, 3);
> - dev_dbg(info->device, "returning ecc 0x%08x\n", code[1]);
> - }
> -
> - return 0;
> -}
> -
> -/*
> - * PIO mode for buffer writing and reading
> - */
> -static void bf5xx_nand_read_buf(struct mtd_info *mtd, uint8_t *buf, int len)
> -{
> - int i;
> - unsigned short val;
> -
> - /*
> - * Data reads are requested by first writing to NFC_DATA_RD
> - * and then reading back from NFC_READ.
> - */
> - for (i = 0; i < len; i++) {
> - while (bfin_read_NFC_STAT() & WB_FULL)
> - cpu_relax();
> -
> - /* Contents do not matter */
> - bfin_write_NFC_DATA_RD(0x0000);
> - SSYNC();
> -
> - while ((bfin_read_NFC_IRQSTAT() & RD_RDY) != RD_RDY)
> - cpu_relax();
> -
> - buf[i] = bfin_read_NFC_READ();
> -
> - val = bfin_read_NFC_IRQSTAT();
> - val |= RD_RDY;
> - bfin_write_NFC_IRQSTAT(val);
> - SSYNC();
> - }
> -}
> -
> -static uint8_t bf5xx_nand_read_byte(struct mtd_info *mtd)
> -{
> - uint8_t val;
> -
> - bf5xx_nand_read_buf(mtd, &val, 1);
> -
> - return val;
> -}
> -
> -static void bf5xx_nand_write_buf(struct mtd_info *mtd,
> - const uint8_t *buf, int len)
> -{
> - int i;
> -
> - for (i = 0; i < len; i++) {
> - while (bfin_read_NFC_STAT() & WB_FULL)
> - cpu_relax();
> -
> - bfin_write_NFC_DATA_WR(buf[i]);
> - SSYNC();
> - }
> -}
> -
> -static void bf5xx_nand_read_buf16(struct mtd_info *mtd, uint8_t *buf, int len)
> -{
> - int i;
> - u16 *p = (u16 *) buf;
> - len >>= 1;
> -
> - /*
> - * Data reads are requested by first writing to NFC_DATA_RD
> - * and then reading back from NFC_READ.
> - */
> - bfin_write_NFC_DATA_RD(0x5555);
> -
> - SSYNC();
> -
> - for (i = 0; i < len; i++)
> - p[i] = bfin_read_NFC_READ();
> -}
> -
> -static void bf5xx_nand_write_buf16(struct mtd_info *mtd,
> - const uint8_t *buf, int len)
> -{
> - int i;
> - u16 *p = (u16 *) buf;
> - len >>= 1;
> -
> - for (i = 0; i < len; i++)
> - bfin_write_NFC_DATA_WR(p[i]);
> -
> - SSYNC();
> -}
> -
> -/*
> - * DMA functions for buffer writing and reading
> - */
> -static irqreturn_t bf5xx_nand_dma_irq(int irq, void *dev_id)
> -{
> - struct bf5xx_nand_info *info = dev_id;
> -
> - clear_dma_irqstat(CH_NFC);
> - disable_dma(CH_NFC);
> - complete(&info->dma_completion);
> -
> - return IRQ_HANDLED;
> -}
> -
> -static void bf5xx_nand_dma_rw(struct mtd_info *mtd,
> - uint8_t *buf, int is_read)
> -{
> - struct bf5xx_nand_info *info = mtd_to_nand_info(mtd);
> - struct nand_chip *chip = mtd_to_nand(mtd);
> - unsigned short val;
> -
> - dev_dbg(info->device, " mtd->%p, buf->%p, is_read %d\n",
> - mtd, buf, is_read);
> -
> - /*
> - * Before starting a dma transfer, be sure to invalidate/flush
> - * the cache over the address range of your DMA buffer to
> - * prevent cache coherency problems. Otherwise very subtle bugs
> - * can be introduced to your driver.
> - */
> - if (is_read)
> - invalidate_dcache_range((unsigned int)buf,
> - (unsigned int)(buf + chip->ecc.size));
> - else
> - flush_dcache_range((unsigned int)buf,
> - (unsigned int)(buf + chip->ecc.size));
> -
> - /*
> - * This register must be written before each page is
> - * transferred to generate the correct ECC register
> - * values.
> - */
> - bfin_write_NFC_RST(ECC_RST);
> - SSYNC();
> - while (bfin_read_NFC_RST() & ECC_RST)
> - cpu_relax();
> -
> - disable_dma(CH_NFC);
> - clear_dma_irqstat(CH_NFC);
> -
> - /* setup DMA register with Blackfin DMA API */
> - set_dma_config(CH_NFC, 0x0);
> - set_dma_start_addr(CH_NFC, (unsigned long) buf);
> -
> - /* The DMAs have different size on BF52x and BF54x */
> -#ifdef CONFIG_BF52x
> - set_dma_x_count(CH_NFC, (chip->ecc.size >> 1));
> - set_dma_x_modify(CH_NFC, 2);
> - val = DI_EN | WDSIZE_16;
> -#endif
> -
> -#ifdef CONFIG_BF54x
> - set_dma_x_count(CH_NFC, (chip->ecc.size >> 2));
> - set_dma_x_modify(CH_NFC, 4);
> - val = DI_EN | WDSIZE_32;
> -#endif
> - /* setup write or read operation */
> - if (is_read)
> - val |= WNR;
> - set_dma_config(CH_NFC, val);
> - enable_dma(CH_NFC);
> -
> - /* Start PAGE read/write operation */
> - if (is_read)
> - bfin_write_NFC_PGCTL(PG_RD_START);
> - else
> - bfin_write_NFC_PGCTL(PG_WR_START);
> - wait_for_completion(&info->dma_completion);
> -}
> -
> -static void bf5xx_nand_dma_read_buf(struct mtd_info *mtd,
> - uint8_t *buf, int len)
> -{
> - struct bf5xx_nand_info *info = mtd_to_nand_info(mtd);
> - struct nand_chip *chip = mtd_to_nand(mtd);
> -
> - dev_dbg(info->device, "mtd->%p, buf->%p, int %d\n", mtd, buf, len);
> -
> - if (len == chip->ecc.size)
> - bf5xx_nand_dma_rw(mtd, buf, 1);
> - else
> - bf5xx_nand_read_buf(mtd, buf, len);
> -}
> -
> -static void bf5xx_nand_dma_write_buf(struct mtd_info *mtd,
> - const uint8_t *buf, int len)
> -{
> - struct bf5xx_nand_info *info = mtd_to_nand_info(mtd);
> - struct nand_chip *chip = mtd_to_nand(mtd);
> -
> - dev_dbg(info->device, "mtd->%p, buf->%p, len %d\n", mtd, buf, len);
> -
> - if (len == chip->ecc.size)
> - bf5xx_nand_dma_rw(mtd, (uint8_t *)buf, 0);
> - else
> - bf5xx_nand_write_buf(mtd, buf, len);
> -}
> -
> -static int bf5xx_nand_read_page_raw(struct mtd_info *mtd, struct nand_chip *chip,
> - uint8_t *buf, int oob_required, int page)
> -{
> - nand_read_page_op(chip, page, 0, NULL, 0);
> -
> - bf5xx_nand_read_buf(mtd, buf, mtd->writesize);
> - bf5xx_nand_read_buf(mtd, chip->oob_poi, mtd->oobsize);
> -
> - return 0;
> -}
> -
> -static int bf5xx_nand_write_page_raw(struct mtd_info *mtd,
> - struct nand_chip *chip, const uint8_t *buf, int oob_required,
> - int page)
> -{
> - nand_prog_page_begin_op(chip, page, 0, buf, mtd->writesize);
> - bf5xx_nand_write_buf(mtd, chip->oob_poi, mtd->oobsize);
> -
> - return nand_prog_page_end_op(chip);
> -}
> -
> -/*
> - * System initialization functions
> - */
> -static int bf5xx_nand_dma_init(struct bf5xx_nand_info *info)
> -{
> - int ret;
> -
> - /* Do not use dma */
> - if (!hardware_ecc)
> - return 0;
> -
> - init_completion(&info->dma_completion);
> -
> - /* Request NFC DMA channel */
> - ret = request_dma(CH_NFC, "BF5XX NFC driver");
> - if (ret < 0) {
> - dev_err(info->device, " unable to get DMA channel\n");
> - return ret;
> - }
> -
> -#ifdef CONFIG_BF54x
> - /* Setup DMAC1 channel mux for NFC which shared with SDH */
> - bfin_write_DMAC1_PERIMUX(bfin_read_DMAC1_PERIMUX() & ~1);
> - SSYNC();
> -#endif
> -
> - set_dma_callback(CH_NFC, bf5xx_nand_dma_irq, info);
> -
> - /* Turn off the DMA channel first */
> - disable_dma(CH_NFC);
> - return 0;
> -}
> -
> -static void bf5xx_nand_dma_remove(struct bf5xx_nand_info *info)
> -{
> - /* Free NFC DMA channel */
> - if (hardware_ecc)
> - free_dma(CH_NFC);
> -}
> -
> -/*
> - * BF5XX NFC hardware initialization
> - * - pin mux setup
> - * - clear interrupt status
> - */
> -static int bf5xx_nand_hw_init(struct bf5xx_nand_info *info)
> -{
> - int err = 0;
> - unsigned short val;
> - struct bf5xx_nand_platform *plat = info->platform;
> -
> - /* setup NFC_CTL register */
> - dev_info(info->device,
> - "data_width=%d, wr_dly=%d, rd_dly=%d\n",
> - (plat->data_width ? 16 : 8),
> - plat->wr_dly, plat->rd_dly);
> -
> - val = (1 << NFC_PG_SIZE_OFFSET) |
> - (plat->data_width << NFC_NWIDTH_OFFSET) |
> - (plat->rd_dly << NFC_RDDLY_OFFSET) |
> - (plat->wr_dly << NFC_WRDLY_OFFSET);
> - dev_dbg(info->device, "NFC_CTL is 0x%04x\n", val);
> -
> - bfin_write_NFC_CTL(val);
> - SSYNC();
> -
> - /* clear interrupt status */
> - bfin_write_NFC_IRQMASK(0x0);
> - SSYNC();
> - val = bfin_read_NFC_IRQSTAT();
> - bfin_write_NFC_IRQSTAT(val);
> - SSYNC();
> -
> - /* DMA initialization */
> - if (bf5xx_nand_dma_init(info))
> - err = -ENXIO;
> -
> - return err;
> -}
> -
> -/*
> - * Device management interface
> - */
> -static int bf5xx_nand_add_partition(struct bf5xx_nand_info *info)
> -{
> - struct mtd_info *mtd = nand_to_mtd(&info->chip);
> - struct mtd_partition *parts = info->platform->partitions;
> - int nr = info->platform->nr_partitions;
> -
> - return mtd_device_register(mtd, parts, nr);
> -}
> -
> -static int bf5xx_nand_remove(struct platform_device *pdev)
> -{
> - struct bf5xx_nand_info *info = to_nand_info(pdev);
> -
> - /* first thing we need to do is release all our mtds
> - * and their partitions, then go through freeing the
> - * resources used
> - */
> - nand_release(nand_to_mtd(&info->chip));
> -
> - peripheral_free_list(bfin_nfc_pin_req);
> - bf5xx_nand_dma_remove(info);
> -
> - return 0;
> -}
> -
> -static int bf5xx_nand_scan(struct mtd_info *mtd)
> -{
> - struct nand_chip *chip = mtd_to_nand(mtd);
> - int ret;
> -
> - ret = nand_scan_ident(mtd, 1, NULL);
> - if (ret)
> - return ret;
> -
> - if (hardware_ecc) {
> - /*
> - * for nand with page size > 512B, think it as several sections with 512B
> - */
> - if (likely(mtd->writesize >= 512)) {
> - chip->ecc.size = 512;
> - chip->ecc.bytes = 6;
> - chip->ecc.strength = 2;
> - } else {
> - chip->ecc.size = 256;
> - chip->ecc.bytes = 3;
> - chip->ecc.strength = 1;
> - bfin_write_NFC_CTL(bfin_read_NFC_CTL() & ~(1 << NFC_PG_SIZE_OFFSET));
> - SSYNC();
> - }
> - }
> -
> - return nand_scan_tail(mtd);
> -}
> -
> -/*
> - * bf5xx_nand_probe
> - *
> - * called by device layer when it finds a device matching
> - * one our driver can handled. This code checks to see if
> - * it can allocate all necessary resources then calls the
> - * nand layer to look for devices
> - */
> -static int bf5xx_nand_probe(struct platform_device *pdev)
> -{
> - struct bf5xx_nand_platform *plat = to_nand_plat(pdev);
> - struct bf5xx_nand_info *info = NULL;
> - struct nand_chip *chip = NULL;
> - struct mtd_info *mtd = NULL;
> - int err = 0;
> -
> - dev_dbg(&pdev->dev, "(%p)\n", pdev);
> -
> - if (!plat) {
> - dev_err(&pdev->dev, "no platform specific information\n");
> - return -EINVAL;
> - }
> -
> - if (peripheral_request_list(bfin_nfc_pin_req, DRV_NAME)) {
> - dev_err(&pdev->dev, "requesting Peripherals failed\n");
> - return -EFAULT;
> - }
> -
> - info = devm_kzalloc(&pdev->dev, sizeof(*info), GFP_KERNEL);
> - if (info == NULL) {
> - err = -ENOMEM;
> - goto out_err;
> - }
> -
> - platform_set_drvdata(pdev, info);
> -
> - nand_hw_control_init(&info->controller);
> -
> - info->device = &pdev->dev;
> - info->platform = plat;
> -
> - /* initialise chip data struct */
> - chip = &info->chip;
> - mtd = nand_to_mtd(&info->chip);
> -
> - if (plat->data_width)
> - chip->options |= NAND_BUSWIDTH_16;
> -
> - chip->options |= NAND_CACHEPRG | NAND_SKIP_BBTSCAN;
> -
> - chip->read_buf = (plat->data_width) ?
> - bf5xx_nand_read_buf16 : bf5xx_nand_read_buf;
> - chip->write_buf = (plat->data_width) ?
> - bf5xx_nand_write_buf16 : bf5xx_nand_write_buf;
> -
> - chip->read_byte = bf5xx_nand_read_byte;
> -
> - chip->cmd_ctrl = bf5xx_nand_hwcontrol;
> - chip->dev_ready = bf5xx_nand_devready;
> -
> - nand_set_controller_data(chip, mtd);
> - chip->controller = &info->controller;
> -
> - chip->IO_ADDR_R = (void __iomem *) NFC_READ;
> - chip->IO_ADDR_W = (void __iomem *) NFC_DATA_WR;
> -
> - chip->chip_delay = 0;
> -
> - /* initialise mtd info data struct */
> - mtd->dev.parent = &pdev->dev;
> -
> - /* initialise the hardware */
> - err = bf5xx_nand_hw_init(info);
> - if (err)
> - goto out_err;
> -
> - /* setup hardware ECC data struct */
> - if (hardware_ecc) {
> -#ifdef CONFIG_MTD_NAND_BF5XX_BOOTROM_ECC
> - mtd_set_ooblayout(mtd, &bootrom_ooblayout_ops);
> -#endif
> - chip->read_buf = bf5xx_nand_dma_read_buf;
> - chip->write_buf = bf5xx_nand_dma_write_buf;
> - chip->ecc.calculate = bf5xx_nand_calculate_ecc;
> - chip->ecc.correct = bf5xx_nand_correct_data;
> - chip->ecc.mode = NAND_ECC_HW;
> - chip->ecc.hwctl = bf5xx_nand_enable_hwecc;
> - chip->ecc.read_page_raw = bf5xx_nand_read_page_raw;
> - chip->ecc.write_page_raw = bf5xx_nand_write_page_raw;
> - } else {
> - chip->ecc.mode = NAND_ECC_SOFT;
> - chip->ecc.algo = NAND_ECC_HAMMING;
> - }
> -
> - /* scan hardware nand chip and setup mtd info data struct */
> - if (bf5xx_nand_scan(mtd)) {
> - err = -ENXIO;
> - goto out_err_nand_scan;
> - }
> -
> -#ifdef CONFIG_MTD_NAND_BF5XX_BOOTROM_ECC
> - chip->badblockpos = 63;
> -#endif
> -
> - /* add NAND partition */
> - bf5xx_nand_add_partition(info);
> -
> - dev_dbg(&pdev->dev, "initialised ok\n");
> - return 0;
> -
> -out_err_nand_scan:
> - bf5xx_nand_dma_remove(info);
> -out_err:
> - peripheral_free_list(bfin_nfc_pin_req);
> -
> - return err;
> -}
> -
> -/* driver device registration */
> -static struct platform_driver bf5xx_nand_driver = {
> - .probe = bf5xx_nand_probe,
> - .remove = bf5xx_nand_remove,
> - .driver = {
> - .name = DRV_NAME,
> - },
> -};
> -
> -module_platform_driver(bf5xx_nand_driver);
> -
> -MODULE_LICENSE("GPL");
> -MODULE_AUTHOR(DRV_AUTHOR);
> -MODULE_DESCRIPTION(DRV_DESC);
> -MODULE_ALIAS("platform:" DRV_NAME);



--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com