f1541773af
Currently there are 3 different variants of read_id implementation: 1. opcode only. Found in GD5FxGQ4xF. 2. opcode + 1 addr byte. Found in GD5GxGQ4xA/E 3. opcode + 1 dummy byte. Found in other currently supported chips. Original implementation was for variant 1 and let detect function of chips with variant 2 and 3 to ignore the first byte. This isn't robust: 1. For chips of variant 2, if SPI master doesn't keep MOSI low during read, chip will get a random id offset, and the entire id buffer will shift by that offset, causing detect failure. 2. For chips of variant 1, if it happens to get a devid that equals to manufacture id of variant 2 or 3 chips, it'll get incorrectly detected. This patch reworks detect procedure to address problems above. New logic do detection for all variants separatedly, in 1-2-3 order. Since all current detect methods do exactly the same id matching procedure, unify them into core.c and remove detect method from manufacture_ops. Tested on GD5F1GQ4UAYIG and W25N01GVZEIG. Signed-off-by: Chuanhong Guo <gch981213@gmail.com> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> Link: https://lore.kernel.org/linux-mtd/20200208074439.146296-1-gch981213@gmail.com
116 lines
2.9 KiB
C
116 lines
2.9 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
/*
|
|
* Copyright (c) 2016-2017 Micron Technology, Inc.
|
|
*
|
|
* Authors:
|
|
* Peter Pan <peterpandong@micron.com>
|
|
*/
|
|
|
|
#include <linux/device.h>
|
|
#include <linux/kernel.h>
|
|
#include <linux/mtd/spinand.h>
|
|
|
|
#define SPINAND_MFR_MICRON 0x2c
|
|
|
|
#define MICRON_STATUS_ECC_MASK GENMASK(7, 4)
|
|
#define MICRON_STATUS_ECC_NO_BITFLIPS (0 << 4)
|
|
#define MICRON_STATUS_ECC_1TO3_BITFLIPS (1 << 4)
|
|
#define MICRON_STATUS_ECC_4TO6_BITFLIPS (3 << 4)
|
|
#define MICRON_STATUS_ECC_7TO8_BITFLIPS (5 << 4)
|
|
|
|
static SPINAND_OP_VARIANTS(read_cache_variants,
|
|
SPINAND_PAGE_READ_FROM_CACHE_QUADIO_OP(0, 2, NULL, 0),
|
|
SPINAND_PAGE_READ_FROM_CACHE_X4_OP(0, 1, NULL, 0),
|
|
SPINAND_PAGE_READ_FROM_CACHE_DUALIO_OP(0, 1, NULL, 0),
|
|
SPINAND_PAGE_READ_FROM_CACHE_X2_OP(0, 1, NULL, 0),
|
|
SPINAND_PAGE_READ_FROM_CACHE_OP(true, 0, 1, NULL, 0),
|
|
SPINAND_PAGE_READ_FROM_CACHE_OP(false, 0, 1, NULL, 0));
|
|
|
|
static SPINAND_OP_VARIANTS(write_cache_variants,
|
|
SPINAND_PROG_LOAD_X4(true, 0, NULL, 0),
|
|
SPINAND_PROG_LOAD(true, 0, NULL, 0));
|
|
|
|
static SPINAND_OP_VARIANTS(update_cache_variants,
|
|
SPINAND_PROG_LOAD_X4(false, 0, NULL, 0),
|
|
SPINAND_PROG_LOAD(false, 0, NULL, 0));
|
|
|
|
static int mt29f2g01abagd_ooblayout_ecc(struct mtd_info *mtd, int section,
|
|
struct mtd_oob_region *region)
|
|
{
|
|
if (section)
|
|
return -ERANGE;
|
|
|
|
region->offset = 64;
|
|
region->length = 64;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int mt29f2g01abagd_ooblayout_free(struct mtd_info *mtd, int section,
|
|
struct mtd_oob_region *region)
|
|
{
|
|
if (section)
|
|
return -ERANGE;
|
|
|
|
/* Reserve 2 bytes for the BBM. */
|
|
region->offset = 2;
|
|
region->length = 62;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static const struct mtd_ooblayout_ops mt29f2g01abagd_ooblayout = {
|
|
.ecc = mt29f2g01abagd_ooblayout_ecc,
|
|
.free = mt29f2g01abagd_ooblayout_free,
|
|
};
|
|
|
|
static int mt29f2g01abagd_ecc_get_status(struct spinand_device *spinand,
|
|
u8 status)
|
|
{
|
|
switch (status & MICRON_STATUS_ECC_MASK) {
|
|
case STATUS_ECC_NO_BITFLIPS:
|
|
return 0;
|
|
|
|
case STATUS_ECC_UNCOR_ERROR:
|
|
return -EBADMSG;
|
|
|
|
case MICRON_STATUS_ECC_1TO3_BITFLIPS:
|
|
return 3;
|
|
|
|
case MICRON_STATUS_ECC_4TO6_BITFLIPS:
|
|
return 6;
|
|
|
|
case MICRON_STATUS_ECC_7TO8_BITFLIPS:
|
|
return 8;
|
|
|
|
default:
|
|
break;
|
|
}
|
|
|
|
return -EINVAL;
|
|
}
|
|
|
|
static const struct spinand_info micron_spinand_table[] = {
|
|
SPINAND_INFO("MT29F2G01ABAGD",
|
|
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x24),
|
|
NAND_MEMORG(1, 2048, 128, 64, 2048, 40, 2, 1, 1),
|
|
NAND_ECCREQ(8, 512),
|
|
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
|
|
&write_cache_variants,
|
|
&update_cache_variants),
|
|
0,
|
|
SPINAND_ECCINFO(&mt29f2g01abagd_ooblayout,
|
|
mt29f2g01abagd_ecc_get_status)),
|
|
};
|
|
|
|
static const struct spinand_manufacturer_ops micron_spinand_manuf_ops = {
|
|
};
|
|
|
|
const struct spinand_manufacturer micron_spinand_manufacturer = {
|
|
.id = SPINAND_MFR_MICRON,
|
|
.name = "Micron",
|
|
.chips = micron_spinand_table,
|
|
.nchips = ARRAY_SIZE(micron_spinand_table),
|
|
.ops = µn_spinand_manuf_ops,
|
|
};
|