diff options
author | Kinsey Moore <kinsey.moore@oarcorp.com> | 2023-10-19 14:20:25 -0500 |
---|---|---|
committer | Joel Sherrill <joel@rtems.org> | 2023-10-27 11:33:44 -0500 |
commit | bb6ed3bed7927e9a56f2a6569f3e24c8669afba2 (patch) | |
tree | a0ea886579729aa89b52ef9a8fd173b8a069fdd9 /bsps/shared/dev/nand/xnandpsu.c | |
parent | bsps/xnandpsu: Avoid loop counter reset (diff) | |
download | rtems-bb6ed3bed7927e9a56f2a6569f3e24c8669afba2.tar.bz2 |
bsps/xnandpsu: Always wrap page to device size
The xnandpsu driver conditionally tries to wrap page index to NAND chip
size causing an off-by-one error where the first page of the second chip
is not wrapped correctly. This removes the conditional so that page
index is always wrapped.
Diffstat (limited to '')
-rw-r--r-- | bsps/shared/dev/nand/xnandpsu.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/bsps/shared/dev/nand/xnandpsu.c b/bsps/shared/dev/nand/xnandpsu.c index 5aeee90084..db4e625e5a 100644 --- a/bsps/shared/dev/nand/xnandpsu.c +++ b/bsps/shared/dev/nand/xnandpsu.c @@ -1482,7 +1482,11 @@ s32 XNandPsu_Write(XNandPsu *InstancePtr, u64 Offset, u64 Length, u8 *SrcBuf) } Target = (u32) (OffsetVar/InstancePtr->Geometry.TargetSize); +#ifdef __rtems__ + { +#else if (Page > InstancePtr->Geometry.NumTargetPages) { +#endif Page %= InstancePtr->Geometry.NumTargetPages; } @@ -1597,7 +1601,11 @@ s32 XNandPsu_Read(XNandPsu *InstancePtr, u64 Offset, u64 Length, u8 *DestBuf) } Target = (u32) (OffsetVar/InstancePtr->Geometry.TargetSize); +#ifdef __rtems__ + { +#else if (Page > InstancePtr->Geometry.NumTargetPages) { +#endif Page %= InstancePtr->Geometry.NumTargetPages; } /* Check if partial read */ |