MCU-PLUS-SDK-AM243X: Calculate CRC of sections with MCRC and compare with linker results

changed assignment to Sitara MPU

Hello Andrea Politano,

I am looking at the your queries and you may expect reply in one or two days .

Regards,

Anil.

Hello Andrea Politano,

My suggestion is that instead of calculating CRC for more data like,text size, use 10 bytes of data, calculate the CRC from the linker and calculate the CRC from the MCRC module and check both the CRC values.

This can give a clue to us.

I assume that the method is calculated by TMS570_CRC64_ISO is swapped the data bytes and preforms the CRC based on the link below.

https://software-dl.ti.com/codegen/docs/tiarmclang/compiler_tools_user_guide/compiler_manual/linker_description/09_linker_generated_crc_tables/a-note-on-the-tms570-crc64-iso-algorithm-stdz0756196.html

And, we are not swapping this data and passing it to the MCRC module.

Finally, we will be checking if both are the same or not.

So, please do CRC validation for 10 bytes or less and share all the CRC values this gives clue to us.

From the next time onwards please share your project. This will be helpful to resolve the issue quicker.

I don't need your entire project and need only the CRC module Application.

Regards,

Anil.

Hi Anil,

thank you for your response.

We are still working on it, we will have answers to your suggestions next week.

Thank you.

Best regards,
Andrea

Hello Andrea,

Thanks for the update.

Once you have done the testing please share the test results.

Regards,

Anil.

Hi,

I did this test by passing only a 64-bit value to the MCRC module and checking the result, i used the same code I wrote in this post. The data I write is 0x24421A80240003C0, the result is signVal.regL = 0xB4033CED and signVal.regH = 0x0AF753AD.

I also did a test by simply not writing any data, and as expected here the result is signVal.regL = 0x0 and signVal.regH = 0x0.

For now I have temporarily solved this problem by implementing the CRC by hand following this link that shows how CRC should be calculated by the linker, but it would be very useful then to be able to take advantage of the MCRC module and calculate the CRC64 algorithm instead of CRC8-16-32 used in the link.

I tried calculating the CRC of that 64-bit value with this tool online using the CRC 64 ISO algorithm but, even trying changing the endianess of the value, I can't figure out how to get that result.

Thank you,

Best regards,

Andrea

Hello Andrea,

can you please share the crc value of the above 64bit value from the Linker ?

Regards,

Anil

Hi,

sure, the linker instead calculates the CRC as 0x0C00266D0AF75383.

Thank you,

Best regards,

Andrea

Hello Andrea,

The CRC ISO 64 bit does not match the MCRC module that is available on SOC.

With CRC ISO, initially, it will take the initial Hex data and will perform the CRC and this CRC ISO results does not matches with the MCRC module.

You can use this tool below and keep the settings as below.

https://www.lddgo.net/en/encrypt/crc

For the "0x24421A80240003C0", I am getting different CRC values from the tool.

Can you please confirm if you gave the above data is correct or not?

Based on the above data, I am getting this 0AF753830C00266D CRC result.

Regards,

Anil.

Andrea Politano said:

sure, the linker instead calculates the CRC as 0x0C00266D0AF75383.

Andreas,

The above linker result exactly matches with the Tool and the results in the linker is given swapped 32 bits.

Please look at the tool result and in this case the MCRC module is giving the wrong CRC calculation.

Regards,

Anil.

Hello Andreas,

The MCRC module is connected to the 32bit VBUSP, so the 64bit atomic write is not possible.

So, the MCRC module that is available on the AM64X does not support the 64bit CRC and supports only 8, 16 and 32 bit.

In the above code, you need to type cast with 32bit and perform the crc calculation for the 32 source buffer.

And, similarly, you need to calculate the CRC from the linker also with the 32-bit.

Then only both SOC CRC and Linker CRC should be matched.

    uint32_t* buffer = (uint32_t*) crc_rec.addr;

    /* compute the CRC by writing the data buffer on which CRC computation is needed */
    for (loopCnt = 0; loopCnt < patternCnt ; loopCnt++)
    {
        *(volatile uint32_t *) ((uintptr_t) psaSignRegAddr.regL) = buffer[loopCnt];
    }

Regards,

Anil.

Hi Anil,

thank you for your response.

We will test your suggestions next week.

Thank you.

Best regards,
Andrea