Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752845AbbGSBCf (ORCPT ); Sat, 18 Jul 2015 21:02:35 -0400 Received: from mail-qk0-f176.google.com ([209.85.220.176]:36837 "EHLO mail-qk0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752277AbbGSBCe (ORCPT ); Sat, 18 Jul 2015 21:02:34 -0400 Subject: Re: [PATCH v2 3/4] mtd: pxa3xx_nand: rework flash detection and timing setup To: Antoine Tenart , sebastian.hesselbarth@gmail.com, dwmw2@infradead.org, computersforpeace@gmail.com References: <1436281707-20106-1-git-send-email-antoine.tenart@free-electrons.com> <1436281707-20106-4-git-send-email-antoine.tenart@free-electrons.com> Cc: boris.brezillon@free-electrons.com, zmxu@marvell.com, jszhang@marvell.com, linux-arm-kernel@lists.infradead.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, Robert Jarzmik From: Ezequiel Garcia Organization: VanguardiaSur Message-ID: <55AAF64D.4040201@vanguardiasur.com.ar> Date: Sat, 18 Jul 2015 21:58:53 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 MIME-Version: 1.0 In-Reply-To: <1436281707-20106-4-git-send-email-antoine.tenart@free-electrons.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7921 Lines: 232 On 07/07/2015 12:08 PM, Antoine Tenart wrote: > Rework the pxa3xx_nand driver to allow using functions exported by the > nand framework to detect the flash and the timings. Then setup the > timings using the helpers previously added. > > Signed-off-by: Antoine Tenart > --- > drivers/mtd/nand/pxa3xx_nand.c | 137 ++++++++++++----------------------------- > 1 file changed, 38 insertions(+), 99 deletions(-) > > diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c > index 9a95c24ab2ce..513f8f6069f0 100644 > --- a/drivers/mtd/nand/pxa3xx_nand.c > +++ b/drivers/mtd/nand/pxa3xx_nand.c > @@ -1320,48 +1320,16 @@ static int pxa3xx_nand_waitfunc(struct mtd_info *mtd, struct nand_chip *this) > return NAND_STATUS_READY; > } > > -static int pxa3xx_nand_config_flash(struct pxa3xx_nand_info *info, > - const struct pxa3xx_nand_flash *f) > +static int pxa3xx_nand_config_flash(struct pxa3xx_nand_info *info) > { > - struct platform_device *pdev = info->pdev; > - struct pxa3xx_nand_platform_data *pdata = dev_get_platdata(&pdev->dev); > struct pxa3xx_nand_host *host = info->host[info->cs]; > - uint32_t ndcr = 0x0; /* enable all interrupts */ > - > - if (f->page_size != 2048 && f->page_size != 512) { > - dev_err(&pdev->dev, "Current only support 2048 and 512 size\n"); > - return -EINVAL; > - } > - > - if (f->flash_width != 16 && f->flash_width != 8) { > - dev_err(&pdev->dev, "Only support 8bit and 16 bit!\n"); > - return -EINVAL; > - } > - > - /* calculate flash information */ > - host->read_id_bytes = (f->page_size == 2048) ? 4 : 2; > - > - /* calculate addressing information */ > - host->col_addr_cycles = (f->page_size == 2048) ? 2 : 1; > - > - if (f->num_blocks * f->page_per_block > 65536) > - host->row_addr_cycles = 3; > - else > - host->row_addr_cycles = 2; > - > - ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0; > - ndcr |= (host->col_addr_cycles == 2) ? NDCR_RA_START : 0; > - ndcr |= (f->page_per_block == 64) ? NDCR_PG_PER_BLK : 0; > - ndcr |= (f->page_size == 2048) ? NDCR_PAGE_SZ : 0; > - ndcr |= (f->flash_width == 16) ? NDCR_DWIDTH_M : 0; > - ndcr |= (f->dfc_width == 16) ? NDCR_DWIDTH_C : 0; > - > - ndcr |= NDCR_RD_ID_CNT(host->read_id_bytes); > - ndcr |= NDCR_SPARE_EN; /* enable spare by default */ > + struct mtd_info *mtd = host->mtd; > + struct nand_chip *chip = mtd->priv; > > - info->reg_ndcr = ndcr; > + info->reg_ndcr |= (host->col_addr_cycles == 2) ? NDCR_RA_START : 0; > + info->reg_ndcr |= (chip->page_shift == 6) ? NDCR_PG_PER_BLK : 0; > + info->reg_ndcr |= (mtd->writesize == 2048) ? NDCR_PAGE_SZ : 0; > > - pxa3xx_nand_set_timing(host, f->timing); > return 0; > } > > @@ -1456,19 +1424,31 @@ static void pxa3xx_nand_free_buff(struct pxa3xx_nand_info *info) > } > #endif > > -static int pxa3xx_nand_sensing(struct pxa3xx_nand_info *info) > +static int pxa3xx_nand_sensing(struct pxa3xx_nand_host *host) > { > + struct pxa3xx_nand_info *info = host->info_data; > + struct platform_device *pdev = info->pdev; > + struct pxa3xx_nand_platform_data *pdata = dev_get_platdata(&pdev->dev); > struct mtd_info *mtd; > struct nand_chip *chip; > + const struct nand_sdr_timings *timings; > int ret; > > mtd = info->host[info->cs]->mtd; > chip = mtd->priv; > > + /* configure default flash values */ > + info->reg_ndcr = 0x0; /* enable all interrupts */ > + info->reg_ndcr |= (pdata->enable_arbiter) ? NDCR_ND_ARB_EN : 0; > + info->reg_ndcr |= NDCR_RD_ID_CNT(host->read_id_bytes); > + info->reg_ndcr |= NDCR_SPARE_EN; /* enable spare by default */ > + > /* use the common timing to make a try */ > - ret = pxa3xx_nand_config_flash(info, &builtin_flash_types[0]); > - if (ret) > - return ret; > + timings = onfi_async_timing_mode_to_sdr_timings(0); > + if (IS_ERR(timings)) > + return PTR_ERR(timings); > + > + pxa3xx_nand_set_sdr_timing(host, timings); > > chip->cmdfunc(mtd, NAND_CMD_RESET, 0, 0); > ret = chip->waitfunc(mtd, chip); > @@ -1553,12 +1533,8 @@ static int pxa3xx_nand_scan(struct mtd_info *mtd) > struct pxa3xx_nand_info *info = host->info_data; > struct platform_device *pdev = info->pdev; > struct pxa3xx_nand_platform_data *pdata = dev_get_platdata(&pdev->dev); > - struct nand_flash_dev pxa3xx_flash_ids[2], *def = NULL; > - const struct pxa3xx_nand_flash *f = NULL; > struct nand_chip *chip = mtd->priv; > - uint32_t id = -1; > - uint64_t chipsize; > - int i, ret, num; > + int ret; > uint16_t ecc_strength, ecc_step; > > if (pdata->keep_config && !pxa3xx_nand_detect_config(info)) > @@ -1567,7 +1543,7 @@ static int pxa3xx_nand_scan(struct mtd_info *mtd) > /* Set a default chunk size */ > info->chunk_size = 512; > > - ret = pxa3xx_nand_sensing(info); > + ret = pxa3xx_nand_sensing(host); > if (ret) { > dev_info(&info->pdev->dev, "There is no chip on cs %d!\n", > info->cs); > @@ -1575,64 +1551,26 @@ static int pxa3xx_nand_scan(struct mtd_info *mtd) > return ret; > } > > - chip->cmdfunc(mtd, NAND_CMD_READID, 0, 0); > - id = *((uint16_t *)(info->data_buff)); > - if (id != 0) > - dev_info(&info->pdev->dev, "Detect a flash id %x\n", id); > - else { > - dev_warn(&info->pdev->dev, > - "Read out ID 0, potential timing set wrong!!\n"); > - > - return -EINVAL; > - } > - > - num = ARRAY_SIZE(builtin_flash_types) + pdata->num_flash - 1; > - for (i = 0; i < num; i++) { > - if (i < pdata->num_flash) > - f = pdata->flash + i; > - else > - f = &builtin_flash_types[i - pdata->num_flash + 1]; > - > - /* find the chip in default list */ > - if (f->chip_id == id) > - break; > - } > - > - if (i >= (ARRAY_SIZE(builtin_flash_types) + pdata->num_flash - 1)) { > - dev_err(&info->pdev->dev, "ERROR!! flash not defined!!!\n"); > - > - return -EINVAL; > - } > - > - ret = pxa3xx_nand_config_flash(info, f); > - if (ret) { > - dev_err(&info->pdev->dev, "ERROR! Configure failed\n"); > - return ret; > - } > - > - memset(pxa3xx_flash_ids, 0, sizeof(pxa3xx_flash_ids)); > - > - pxa3xx_flash_ids[0].name = f->name; > - pxa3xx_flash_ids[0].dev_id = (f->chip_id >> 8) & 0xffff; > - pxa3xx_flash_ids[0].pagesize = f->page_size; > - chipsize = (uint64_t)f->num_blocks * f->page_per_block * f->page_size; > - pxa3xx_flash_ids[0].chipsize = chipsize >> 20; > - pxa3xx_flash_ids[0].erasesize = f->page_size * f->page_per_block; > - if (f->flash_width == 16) > - pxa3xx_flash_ids[0].options = NAND_BUSWIDTH_16; > - pxa3xx_flash_ids[1].name = NULL; > - def = pxa3xx_flash_ids; > KEEP_CONFIG: Hm... > - if (info->reg_ndcr & NDCR_DWIDTH_M) > - chip->options |= NAND_BUSWIDTH_16; > - ...can't see why this removal is needed. AFAICS, it removes support for 16-bit devices (only supported when using keep-config). > /* Device detection must be done with ECC disabled */ > if (info->variant == PXA3XX_NAND_VARIANT_ARMADA370) > nand_writel(info, NDECCCTRL, 0x0); > > - if (nand_scan_ident(mtd, 1, def)) > + if (nand_scan_ident(mtd, 1, NULL)) > return -ENODEV; > > + if (!pdata->keep_config) { > + ret = pxa3xx_nand_init_timings(host); > + if (ret) { > + dev_err(&info->pdev->dev, "Failed to set timings: %d\n", ret); > + return ret; > + } > + } > + > + ret = pxa3xx_nand_config_flash(info); > + if (ret) > + return ret; > + > if (pdata->flash_bbt) { > /* > * We'll use a bad block table stored in-flash and don't > @@ -1729,6 +1667,7 @@ static int alloc_nand_resource(struct platform_device *pdev) > host->mtd = mtd; > host->cs = cs; > host->info_data = info; > + host->read_id_bytes = 4; > mtd->priv = host; > mtd->owner = THIS_MODULE; > > -- Ezequiel Garcia, VanguardiaSur www.vanguardiasur.com.ar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/