Hello,
I have two codes:
From sample reference code
Edited code as per datasheet
As per algorithm, need to read twice again after DQ5 status and nee...
Show More
Hello,
I have two codes:
- From sample reference code
- Edited code as per datasheet
- As per algorithm, need to read twice again after DQ5 status and need to issue reset command if timeout occur.
- Or do I need to handle Timeout condition outside of function with issuing reset command and check it again until its NOT_BUSY ?
I want to confirm the edited code (2nd) is correct or not. And if it is not correct then why?
For reference please check the algorithm below,
1. Below, sample reference code.
/******************************************************************************
*
* lld_StatusGet - Determines Flash operation status
*
*
* RETURNS: DEVSTATUS
*
*/
DEVSTATUS lld_StatusGet
(
FLASHDATA * base_addr, /* device base address in system */
ADDRESS offset /* address offset from base address */
)
{
FLASHDATA dq2_toggles;
FLASHDATA dq6_toggles;
FLASHDATA status_read_1;
FLASHDATA status_read_2;
FLASHDATA status_read_3;
status_read_1 = FLASH_RD(base_addr, offset);
status_read_2 = FLASH_RD(base_addr, offset);
status_read_3 = FLASH_RD(base_addr, offset);
/* Any DQ6 toggles */
dq6_toggles = ((status_read_1 ^ status_read_2) & /* Toggles between read1 and read2 */
(status_read_2 ^ status_read_3) & /* Toggles between read2 and read3 */
DQ6_MASK ); /* Check for DQ6 only */
/* Any DQ2 toggles */
dq2_toggles = ((status_read_1 ^ status_read_2) & /* Toggles between read1 and read2 */
(status_read_2 ^ status_read_3) & /* Toggles between read2 and read3 */
DQ2_MASK ); /* Check for DQ6 only */
if (dq6_toggles)
{
/* Checking WriteBuffer Abort condition:
Check for all devices that have DQ6 toggling also have Write Buffer Abort DQ1 set */
if ((!dq2_toggles) &&
((DQ6_TGL_DQ1_MASK & status_read_1) == DQ6_TGL_DQ1_MASK)
)
return DEV_WRITE_BUFFER_ABORT;
/* Checking Timeout condition:
Check for all devices that have DQ6 toggling also have Time Out DQ5 set. */
if ((DQ6_TGL_DQ5_MASK & status_read_1) == DQ6_TGL_DQ5_MASK )
return DEV_EXCEEDED_TIME_LIMITS;
/* No timeout, no WB error */
return DEV_BUSY;
}
else /* no DQ6 toggles on all devices */
{
/* Checking Erase Suspend condition */
status_read_1 = FLASH_RD(base_addr, offset);
status_read_2 = FLASH_RD(base_addr, offset);
/* Checking Erase Suspend condition */
if ((((status_read_1 ^ status_read_2) & DQ2_MASK) == 0)
)
return DEV_NOT_BUSY; /* All devices DQ2 not toggling */
else // at least one device is suspended
{
return DEV_SUSPEND; /* At least some devices toggle DQ2 */
}
}
}
2. Below, edited Sample reference code as per datasheet
DEVSTATUS lld_StatusGet
(
FLASHDATA * base_addr, /* device base address in system */
ADDRESS offset /* address offset from base address */
)
{
FLASHDATA dq2_toggles;
FLASHDATA dq6_toggles;
FLASHDATA status_read_1;
FLASHDATA status_read_2;
status_read_1 = FLASH_RD(base_addr, offset);
status_read_2 = FLASH_RD(base_addr, offset);
/* Any DQ6 toggles */
dq6_toggles = ((status_read_1 ^ status_read_2) & /* Toggles between read1 and read2 */
DQ6_MASK ); /* Check for DQ6 only */
/* Any DQ2 toggles */
dq2_toggles = ((status_read_1 ^ status_read_2) & /* Toggles between read1 and read2 */
DQ2_MASK ); /* Check for DQ6 only */
if (dq6_toggles)
{
/* Checking WriteBuffer Abort condition:
Check for all devices that have DQ6 toggling also have Write Buffer Abort DQ1 set */
if ((!dq2_toggles) &&
((DQ6_TGL_DQ1_MASK & status_read_1) == DQ6_TGL_DQ1_MASK)
)
return DEV_WRITE_BUFFER_ABORT;
/* Checking Timeout condition:
Check for all devices that have DQ6 toggling also have Time Out DQ5 set. */
if ((DQ6_TGL_DQ5_MASK & status_read_1) == DQ6_TGL_DQ5_MASK )
{
status_read_1 = FLASH_RD(base_addr, offset);
status_read_2 = FLASH_RD(base_addr, offset);
dq6_toggles = ((status_read_1 ^ status_read_2) & /* Toggles between read1 and read2 */
DQ6_MASK );
if (dq6_toggles)
{
//Issue reset command here as per algorithm in the datasheet
return DEV_BUSY; // To continue checking of status until dq6 not toggle (or DEV_NOT_BUSY).
}
else
{
return DEV_BUSY;// Here also need to return BUSY because, need to check DQ2 pin for SUSPEND or DEV_NOT_BUSY status in next iteration of loop.
}
}
/* No timeout, no WB error */
return DEV_BUSY;
}
else /* no DQ6 toggles on all devices */
{
/* Checking Erase Suspend condition */
status_read_1 = FLASH_RD(base_addr, offset);
status_read_2 = FLASH_RD(base_addr, offset);
/* Checking Erase Suspend condition */
if ((((status_read_1 ^ status_read_2) & DQ2_MASK) == 0)
)
return DEV_NOT_BUSY; /* All devices DQ2 not toggling */
else // at least one device is suspended
{
return DEV_SUSPEND; /* At least some devices toggle DQ2 */
}
}
}
Show Less
Oct 12, 2022
Solved
