Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2025-12-06 09:01:15 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

arch/mpfs/mpfs_ddr.c: Improve CA training validity check heuristics

Browse Source 
Update the CA training result check to match the HSS reference code.

The original check which required phase delays to be 5 taps separated was wrong.
This has also been removed from the HSS.

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
This commit is contained in:
Jukka Laitinen
2025-11-05 16:10:10 +02:00
committed by Xiang Xiao
parent 3560a28eef
commit 5ae35b619a
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
arch/risc-v/src/mpfs/mpfs_ddr.c
View File

@@ -3449,6 +3449,7 @@ static int mpfs_training_verify(void)
if ((LIBERO_SETTING_TRAINING_SKIP_SETTING & ADDCMD_BIT) != ADDCMD_BIT)
{
unsigned low_ca_dly_count = 0;
unsigned decrease_count = 0;
uint8_t ca_status[8] =
{
((addcmd_status0) & 0xff),
@@ -3467,21 +3468,28 @@ static int mpfs_training_verify(void)
* Expected result is increasing numbers, starting at index n and
* wrapping around. For example:
* [0x35, 0x3b, 0x4, 0x14, 0x1b, 0x21, 0x28, 0x2f].
*
* Also they need to be separated by at least 5
*/
for (i = 0; i < 8; i++)
{
if (ca_status[i] < last + 5)
if (ca_status[i] < 5)
{
low_ca_dly_count++;
}
if (ca_status[i] <= last)
{
decrease_count++;
}
last = ca_status[i];
}
if (low_ca_dly_count > 1)
/* Check against thresholds. Allow one low and one extra
* backwards jump, in addition to the wrap-around point
*/
if (low_ca_dly_count > 1 || decrease_count > 2)
{
/* Retrain via reset */