RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

bsps/zynqmp/jffs2: Use BBT information directly

Browse Source 
The XNandPsu_IsBlockBad() function is insufficient for JFFS2 to
determine whether a block is usable since the function does not report
reserved blocks. JFFS2 is also unable to use the last 4 blocks of each
target attached to the NAND controller since they are reserved for the
Bad Block Table (BBT), but not necessarily marked as such.
This commit is contained in:
Kinsey Moore
2023-10-25 16:22:07 -05:00
committed by Joel Sherrill
parent 0883f7ec58
commit a6aa6c6385
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
bsps/aarch64/xilinx-zynqmp/jffs2_xnandpsu.c
View File

@@ -130,6 +130,10 @@ static int flash_block_is_bad(
{
XNandPsu *nandpsu = get_flash_control(super)->nandpsu;
uint32_t BlockIndex;
uint8_t BlockData;
uint8_t BlockShift;
uint8_t BlockType;
uint32_t BlockOffset;
assert(bad);
@@ -137,10 +141,28 @@ static int flash_block_is_bad(
return -EIO;
}
*bad = true;
BlockIndex = offset / nandpsu->Geometry.BlockSize;
rtems_mutex_lock(&(get_flash_control(super)->access_lock));
*bad = (XNandPsu_IsBlockBad(nandpsu, BlockIndex) == XST_SUCCESS);
/* XNandPsu_IsBlockBad() is insufficient for this use case */
BlockOffset = BlockIndex >> XNANDPSU_BBT_BLOCK_SHIFT;
BlockShift = XNandPsu_BbtBlockShift(BlockIndex);
BlockData = nandpsu->Bbt[BlockOffset];
BlockType = (BlockData >> BlockShift) & XNANDPSU_BLOCK_TYPE_MASK;
if (BlockType == XNANDPSU_BLOCK_GOOD) {
*bad = false;
}
int TargetBlockIndex = BlockIndex % nandpsu->Geometry.NumTargetBlocks;
/* The last 4 blocks of every device target are reserved for the BBT */
if (nandpsu->Geometry.NumTargetBlocks - TargetBlockIndex <= 4) {
*bad = true;
}
rtems_mutex_unlock(&(get_flash_control(super)->access_lock));
return 0;
}