[U-Boot] [PATCH v2] tegra: nand: make ONFI detection work

Add the missing bits to the Tegra NAND driver to make ONFI detection work properly.
Also add it to the Tegra default config, as it seems to be a reasonable thing to have it available on all boards that use any kind of NAND.
Signed-off-by: Lucas Stach dev@lynxeye.de --- v2: use puts instead of printf --- drivers/mtd/nand/tegra_nand.c | 36 ++++++++++++++++++++++++++++++++++++ include/configs/tegra20-common.h | 1 + 2 Dateien geändert, 37 Zeilen hinzugefügt(+)
diff --git a/drivers/mtd/nand/tegra_nand.c b/drivers/mtd/nand/tegra_nand.c index 5408c51..4d94cc6 100644 --- a/drivers/mtd/nand/tegra_nand.c +++ b/drivers/mtd/nand/tegra_nand.c @@ -219,6 +219,34 @@ static uint8_t read_byte(struct mtd_info *mtd) }
/** + * Read len bytes from the chip into a buffer + * + * @param mtd MTD device structure + * @param buf buffer to store data to + * @param len number of bytes to read + * + * Read function for 8bit bus-width + */ +static void read_buf(struct mtd_info *mtd, uint8_t *buf, int len) +{ + int i, s; + unsigned int reg; + struct nand_chip *chip = mtd->priv; + struct nand_drv *info = (struct nand_drv *)chip->priv; + + for (i = 0; i < len; i += 4) { + s = (len - i) > 4 ? 4 : len - i; + writel(CMD_PIO | CMD_RX | CMD_A_VALID | CMD_CE0 | + ((s - 1) << CMD_TRANS_SIZE_SHIFT) | CMD_GO, + &info->reg->command); + if (!nand_waitfor_cmd_completion(info->reg)) + puts("Command timeout during read_buf\n"); + reg = readl(&info->reg->resp); + memcpy(buf + i, ®, s); + } +} + +/** * Check NAND status to see if it is ready or not * * @param mtd MTD device structure @@ -317,6 +345,7 @@ static void nand_command(struct mtd_info *mtd, unsigned int command, switch (command) { case NAND_CMD_READID: writel(NAND_CMD_READID, &info->reg->cmd_reg1); + writel(column & 0xFF, &info->reg->addr_reg1); writel(CMD_GO | CMD_CLE | CMD_ALE | CMD_PIO | CMD_RX | ((4 - 1) << CMD_TRANS_SIZE_SHIFT) @@ -324,6 +353,12 @@ static void nand_command(struct mtd_info *mtd, unsigned int command, &info->reg->command); info->pio_byte_index = 0; break; + case NAND_CMD_PARAM: + writel(NAND_CMD_PARAM, &info->reg->cmd_reg1); + writel(column & 0xFF, &info->reg->addr_reg1); + writel(CMD_GO | CMD_CLE | CMD_ALE | CMD_CE0, + &info->reg->command); + break; case NAND_CMD_READ0: writel(NAND_CMD_READ0, &info->reg->cmd_reg1); writel(NAND_CMD_READSTART, &info->reg->cmd_reg2); @@ -976,6 +1011,7 @@ int tegra_nand_init(struct nand_chip *nand, int devnum) nand->options = LP_OPTIONS; nand->cmdfunc = nand_command; nand->read_byte = read_byte; + nand->read_buf = read_buf; nand->ecc.read_page = nand_read_page_hwecc; nand->ecc.write_page = nand_write_page_hwecc; nand->ecc.read_page_raw = nand_read_page_raw; diff --git a/include/configs/tegra20-common.h b/include/configs/tegra20-common.h index 744dc59..9e60020 100644 --- a/include/configs/tegra20-common.h +++ b/include/configs/tegra20-common.h @@ -204,5 +204,6 @@ #define CONFIG_SPL_LDSCRIPT "$(CPUDIR)/tegra20/u-boot-spl.lds"
#define CONFIG_SYS_NAND_SELF_INIT +#define CONFIG_SYS_NAND_ONFI_DETECTION
#endif /* __TEGRA20_COMMON_H */

Scott, can I have your Acked-by for this, so Tom can take it through the Tegra tree?
Thanks, Lucas
Am Sonntag, den 07.10.2012, 23:29 +0200 schrieb Lucas Stach:
Add the missing bits to the Tegra NAND driver to make ONFI detection work properly.
Also add it to the Tegra default config, as it seems to be a reasonable thing to have it available on all boards that use any kind of NAND.
Signed-off-by: Lucas Stach dev@lynxeye.de
v2: use puts instead of printf
drivers/mtd/nand/tegra_nand.c | 36 ++++++++++++++++++++++++++++++++++++ include/configs/tegra20-common.h | 1 + 2 Dateien geändert, 37 Zeilen hinzugefügt(+)
diff --git a/drivers/mtd/nand/tegra_nand.c b/drivers/mtd/nand/tegra_nand.c index 5408c51..4d94cc6 100644 --- a/drivers/mtd/nand/tegra_nand.c +++ b/drivers/mtd/nand/tegra_nand.c @@ -219,6 +219,34 @@ static uint8_t read_byte(struct mtd_info *mtd) }
/**
- Read len bytes from the chip into a buffer
- @param mtd MTD device structure
- @param buf buffer to store data to
- @param len number of bytes to read
- Read function for 8bit bus-width
- */
+static void read_buf(struct mtd_info *mtd, uint8_t *buf, int len) +{
- int i, s;
- unsigned int reg;
- struct nand_chip *chip = mtd->priv;
- struct nand_drv *info = (struct nand_drv *)chip->priv;
- for (i = 0; i < len; i += 4) {
s = (len - i) > 4 ? 4 : len - i;
writel(CMD_PIO | CMD_RX | CMD_A_VALID | CMD_CE0 |
((s - 1) << CMD_TRANS_SIZE_SHIFT) | CMD_GO,
&info->reg->command);
if (!nand_waitfor_cmd_completion(info->reg))
puts("Command timeout during read_buf\n");
reg = readl(&info->reg->resp);
memcpy(buf + i, ®, s);
- }
+}
+/**
- Check NAND status to see if it is ready or not
- @param mtd MTD device structure
@@ -317,6 +345,7 @@ static void nand_command(struct mtd_info *mtd, unsigned int command, switch (command) { case NAND_CMD_READID: writel(NAND_CMD_READID, &info->reg->cmd_reg1);
writel(CMD_GO | CMD_CLE | CMD_ALE | CMD_PIO | CMD_RX | ((4 - 1) << CMD_TRANS_SIZE_SHIFT)writel(column & 0xFF, &info->reg->addr_reg1);
@@ -324,6 +353,12 @@ static void nand_command(struct mtd_info *mtd, unsigned int command, &info->reg->command); info->pio_byte_index = 0; break;
- case NAND_CMD_PARAM:
writel(NAND_CMD_PARAM, &info->reg->cmd_reg1);
writel(column & 0xFF, &info->reg->addr_reg1);
writel(CMD_GO | CMD_CLE | CMD_ALE | CMD_CE0,
&info->reg->command);
case NAND_CMD_READ0: writel(NAND_CMD_READ0, &info->reg->cmd_reg1); writel(NAND_CMD_READSTART, &info->reg->cmd_reg2);break;
@@ -976,6 +1011,7 @@ int tegra_nand_init(struct nand_chip *nand, int devnum) nand->options = LP_OPTIONS; nand->cmdfunc = nand_command; nand->read_byte = read_byte;
- nand->read_buf = read_buf; nand->ecc.read_page = nand_read_page_hwecc; nand->ecc.write_page = nand_write_page_hwecc; nand->ecc.read_page_raw = nand_read_page_raw;
diff --git a/include/configs/tegra20-common.h b/include/configs/tegra20-common.h index 744dc59..9e60020 100644 --- a/include/configs/tegra20-common.h +++ b/include/configs/tegra20-common.h @@ -204,5 +204,6 @@ #define CONFIG_SPL_LDSCRIPT "$(CPUDIR)/tegra20/u-boot-spl.lds"
#define CONFIG_SYS_NAND_SELF_INIT +#define CONFIG_SYS_NAND_ONFI_DETECTION
#endif /* __TEGRA20_COMMON_H */

On 10/14/2012 10:32:52 AM, Lucas Stach wrote:
Scott, can I have your Acked-by for this, so Tom can take it through the Tegra tree?
Thanks, Lucas
Am Sonntag, den 07.10.2012, 23:29 +0200 schrieb Lucas Stach:
Add the missing bits to the Tegra NAND driver to make ONFI
detection work
properly.
Also add it to the Tegra default config, as it seems to be a
reasonable thing
to have it available on all boards that use any kind of NAND.
Signed-off-by: Lucas Stach dev@lynxeye.de
v2: use puts instead of printf
drivers/mtd/nand/tegra_nand.c | 36
++++++++++++++++++++++++++++++++++++
include/configs/tegra20-common.h | 1 + 2 Dateien geändert, 37 Zeilen hinzugefügt(+)
Acked-by: Scott Wood scottwood@freescale.com
-Scott

Lucas/Scott,
-----Original Message----- From: Scott Wood [mailto:scottwood@freescale.com] Sent: Monday, October 15, 2012 1:06 PM To: Lucas Stach Cc: u-boot@lists.denx.de; Tom Warren Subject: Re: [U-Boot] [PATCH v2] tegra: nand: make ONFI detection work
On 10/14/2012 10:32:52 AM, Lucas Stach wrote:
Scott, can I have your Acked-by for this, so Tom can take it through the Tegra tree?
Thanks, Lucas
Am Sonntag, den 07.10.2012, 23:29 +0200 schrieb Lucas Stach:
Add the missing bits to the Tegra NAND driver to make ONFI
detection work
properly.
Also add it to the Tegra default config, as it seems to be a
reasonable thing
to have it available on all boards that use any kind of NAND.
Signed-off-by: Lucas Stach dev@lynxeye.de
v2: use puts instead of printf
drivers/mtd/nand/tegra_nand.c | 36
++++++++++++++++++++++++++++++++++++
include/configs/tegra20-common.h | 1 + 2 Dateien geändert, 37 Zeilen hinzugefügt(+)
Acked-by: Scott Wood scottwood@freescale.com
Applied to u-boot-tegra/next and pushed to denx.de.
Tom
-Scott
participants (3)
-
Lucas Stach
-
Scott Wood
-
Tom Warren