NAND page command refactoring.

Created a new function that handles sending a command and the address
information for pages to a NAND device.

[dbrownell@users.sourceforge.net: tweaked line lengths, name 'oob_only']

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
This commit is contained in:
Dean Glazeski 2009-11-20 00:19:39 -06:00 committed by David Brownell
parent 7e2dffbbff
commit 9a51b8b0e3
2 changed files with 35 additions and 59 deletions

View File

@ -863,20 +863,19 @@ static int nand_read_page(struct nand_device *nand, uint32_t page, uint8_t *data
return nand->controller->read_page(nand, page, data, data_size, oob, oob_size); return nand->controller->read_page(nand, page, data, data_size, oob, oob_size);
} }
int nand_read_page_raw(struct nand_device *nand, uint32_t page, uint8_t *data, uint32_t data_size, uint8_t *oob, uint32_t oob_size) int nand_page_command(struct nand_device *nand, uint32_t page,
uint8_t cmd, bool oob_only)
{ {
uint32_t i;
if (!nand->device) if (!nand->device)
return ERROR_NAND_DEVICE_NOT_PROBED; return ERROR_NAND_DEVICE_NOT_PROBED;
if (nand->page_size <= 512) if (oob_only && NAND_CMD_READ0 == cmd && nand->page_size <= 512)
{ cmd = NAND_CMD_READOOB;
nand->controller->command(nand, cmd);
if (nand->page_size <= 512) {
/* small page device */ /* small page device */
if (data)
nand->controller->command(nand, NAND_CMD_READ0);
else
nand->controller->command(nand, NAND_CMD_READOOB);
/* column (always 0, we start at the beginning of a page/OOB area) */ /* column (always 0, we start at the beginning of a page/OOB area) */
nand->controller->address(nand, 0x0); nand->controller->address(nand, 0x0);
@ -892,20 +891,17 @@ int nand_read_page_raw(struct nand_device *nand, uint32_t page, uint8_t *data, u
/* 5th cycle only on devices with more than 8 GiB */ /* 5th cycle only on devices with more than 8 GiB */
if (nand->address_cycles >= 5) if (nand->address_cycles >= 5)
nand->controller->address(nand, (page >> 24) & 0xff); nand->controller->address(nand, (page >> 24) & 0xff);
} } else {
else
{
/* large page device */ /* large page device */
nand->controller->command(nand, NAND_CMD_READ0);
/* column (0 when we start at the beginning of a page, /* column (0 when we start at the beginning of a page,
* or 2048 for the beginning of OOB area) * or 2048 for the beginning of OOB area)
*/ */
nand->controller->address(nand, 0x0); nand->controller->address(nand, 0x0);
if (data) if (oob_only)
nand->controller->address(nand, 0x0);
else
nand->controller->address(nand, 0x8); nand->controller->address(nand, 0x8);
else
nand->controller->address(nand, 0x0);
/* row */ /* row */
nand->controller->address(nand, page & 0xff); nand->controller->address(nand, page & 0xff);
@ -915,8 +911,9 @@ int nand_read_page_raw(struct nand_device *nand, uint32_t page, uint8_t *data, u
if (nand->address_cycles >= 5) if (nand->address_cycles >= 5)
nand->controller->address(nand, (page >> 16) & 0xff); nand->controller->address(nand, (page >> 16) & 0xff);
/* large page devices need a start command */ /* large page devices need a start command if reading */
nand->controller->command(nand, NAND_CMD_READSTART); if (NAND_CMD_READ0 == cmd)
nand->controller->command(nand, NAND_CMD_READSTART);
} }
if (nand->controller->nand_ready) { if (nand->controller->nand_ready) {
@ -926,6 +923,20 @@ int nand_read_page_raw(struct nand_device *nand, uint32_t page, uint8_t *data, u
alive_sleep(1); alive_sleep(1);
} }
return ERROR_OK;
}
int nand_read_page_raw(struct nand_device *nand, uint32_t page,
uint8_t *data, uint32_t data_size,
uint8_t *oob, uint32_t oob_size)
{
uint32_t i;
int retval;
retval = nand_page_command(nand, page, NAND_CMD_READ0, !data);
if (ERROR_OK != retval)
return retval;
if (data) if (data)
{ {
if (nand->controller->read_block_data != NULL) if (nand->controller->read_block_data != NULL)
@ -983,47 +994,9 @@ int nand_write_page_raw(struct nand_device *nand, uint32_t page, uint8_t *data,
int retval; int retval;
uint8_t status; uint8_t status;
if (!nand->device) retval = nand_page_command(nand, page, NAND_CMD_SEQIN, !data);
return ERROR_NAND_DEVICE_NOT_PROBED; if (ERROR_OK != retval)
return retval;
nand->controller->command(nand, NAND_CMD_SEQIN);
if (nand->page_size <= 512)
{
/* column (always 0, we start at the beginning of a page/OOB area) */
nand->controller->address(nand, 0x0);
/* row */
nand->controller->address(nand, page & 0xff);
nand->controller->address(nand, (page >> 8) & 0xff);
/* 4th cycle only on devices with more than 32 MiB */
if (nand->address_cycles >= 4)
nand->controller->address(nand, (page >> 16) & 0xff);
/* 5th cycle only on devices with more than 8 GiB */
if (nand->address_cycles >= 5)
nand->controller->address(nand, (page >> 24) & 0xff);
}
else
{
/* column (0 when we start at the beginning of a page,
* or 2048 for the beginning of OOB area)
*/
nand->controller->address(nand, 0x0);
if (data)
nand->controller->address(nand, 0x0);
else
nand->controller->address(nand, 0x8);
/* row */
nand->controller->address(nand, page & 0xff);
nand->controller->address(nand, (page >> 8) & 0xff);
/* 5th cycle only on devices with more than 128 MiB */
if (nand->address_cycles >= 5)
nand->controller->address(nand, (page >> 16) & 0xff);
}
if (data) if (data)
{ {

View File

@ -270,6 +270,9 @@ struct nand_device *get_nand_device_by_name(const char *name);
struct nand_device *get_nand_device_by_num(int num); struct nand_device *get_nand_device_by_num(int num);
int nand_page_command(struct nand_device *nand, uint32_t page,
uint8_t cmd, bool oob_only);
int nand_read_page_raw(struct nand_device *nand, uint32_t page, int nand_read_page_raw(struct nand_device *nand, uint32_t page,
uint8_t *data, uint32_t data_size, uint8_t *oob, uint32_t oob_size); uint8_t *data, uint32_t data_size, uint8_t *oob, uint32_t oob_size);
int nand_write_page_raw(struct nand_device *nand, uint32_t page, int nand_write_page_raw(struct nand_device *nand, uint32_t page,