TMS570LC4357: Diagnostic FLA12: Testing of Parity when DIAG_BUF_ SEL = 0x4 has no effect

Hi Max,

We are working on your issue, and we will give you an update very soon.

--

Thanks & regards,

Jagadish.

Hi Max,

Is it possible for you to share your complete project so that i can easily debug and see the issue at my end.

--

Thanks & regards,
Jagadish.

Hello Jagadish,

Unfortunatly I can not share our code, I can only share what we do:

Comamnd Partiy Check Port A:

1. DIAG_BUF_SEL = 0x0
2. PAR_OVR_SEL = 0x2
3. PAR_DIS_KEY = 0x5
4. PAR_OVR_KEY = 0x5 s
5. read from Adress 0x0
   
6. ADD_PAR_ERR set?
7. ESM 3.13 set?

This yields the expected result. When we try the following for Port B, it does not work:

1. DIAG_BUF_SEL = 0x4
2. PAR_OVR_SEL = 0x2
3. PAR_DIS_KEY = 0x5
4. PAR_OVR_KEY = 0x5 
5. read from Adress 0x0
   
6. ADD_PAR_ERR set?
7. ESM 3.13 set?

So is the Parity testing for Port B just not necessary/possible?

Thank you and best regards,
Max

Thanks Max for sharing the steps, i will try to replicate the issue at my end.

Hi Max,

Apologies for the delay, i am still trying to reproduce the issue on my end. I will get back to you on this soon.

--

Thanks & regards,
Jagadish.

Hi Max,

I am not even getting error for PORT A, here is my main function. Can you please guide me?

/* USER CODE BEGIN (2) */
#define DIAG_BUF_SEL (0x04 << 8)
/* USER CODE END */

int main(void)
{
/* USER CODE BEGIN (3) */
    uint32_t    readValue;
    //sl_flashWREG->FDIAGCTRL = DIAG_BUF_SEL;
    sl_flashWREG->FPAR_OVR = 0x25500;
    readValue = *(uint32_t *)0x00000000;

    while(1);
/* USER CODE END */
    return 0;
}

--

Thanks & regards,
Jagadish.

Hi Jagadish,

You need to do the following:

/* USER CODE BEGIN (2) */
#define DIAG_BUF_SEL_PORT_A (0x00 << 8) // Write this to select Port A
#define DIAG_BUF_SEL_PORT_B (0x04 << 8) // This should select Port B
/* USER CODE END */

int main(void)
{
/* USER CODE BEGIN (3) */
    uint32_t    readValue;
    
    sl_flashWREG->FDIAGCTRL = DIAG_BUF_SEL_PORT_A;
    
    sl_flashWREG->FPAR_OVR = 0x25000; // Configure Parity 
    
    sl_flashWREG->FPAR_OVR |= (0x05 << 9); //PAR_OVR_KEY is bit 9-11
    readValue = *(uint32_t *)0x00000000;
    
    // Here you should be able to observe the error
    
    while(1);
/* USER CODE END */
    return 0;
}

I have not compiled and tested this, but as far as I can tell you assumed the bit field for PAR_OVR_KEY was bit 8-11 in FPAR_OVR.
If you correct this you should be able to observe the expected error for Port A.

Thank you and best regards,
Max

Hi Max,

Thanks for sharing above code, that was helpful and i could see the issue you mentioned on my end. I will do further analysis and will provide an update soon.

--
Thanks & regards,
Jagadish.

Hi Jagadish,

Any updates on this?

Thank you and best regards,
Max

Hi Jagadish,

Any updates on this?

Thank you and best regards,
Max

Hi Max,

Sorry for the delay, i am on vacation for last week. I will provide an update soon.

--

Thanks & regards,
Jagadish.

Hi Max,

Still i didn't get feedback from internal team on this, i will again check with them.

--

Thanks & regards,
Jagadish.

Hi Max,

After discussing with internal team i got below feedback,

In normal running mode, the primary and duplicate address tag registers store the recently issued CPU addresses. The primary and duplicate address tag registers are continuously compared to each other If they are different, then an address tag register error event is generated. In diag mode, the FRAW_ADDR register is to supply the alternate address (as CPU address). The FRAW_ADDR value is compared with the primary and the duplicate address tag registers.

--

Thanks & regards,
Jagadish.

Hi Jagadish,

Does this mean that running the Diagnostic for Port A only, is enough to test this mechanism?

Thank you and best regards,
Max