TMS320F2800132: issue with HAL_clearDataRAM or LOAD_START in linker cmd file

Manu36

Expert 1350 points

Part Number: TMS320F2800132
Other Parts Discussed in Thread: LAUNCHXL-F2800137

Tool/software:

Hello,

I have an issue with my code.

The code based on universal_motor_control_lab.

At the begining of the code, in the main, some data RAM are cleared using for example:

    HAL_clearDataRAM((void *)loadStart_est_data, (uint16_t)loadSize_est_data);

loadStart_est_data and loadSize_est_data are defined by the linker with the following in cmd file:

    est_data             : >  RAMLS0F,
                              LOAD_START(loadStart_est_data),
                              LOAD_END(loadEnd_est_data),
                              LOAD_SIZE(loadSize_est_data)

And in hal.h, the following is declared.

extern uint32_t loadStart_est_data;
extern uint32_t loadEnd_est_data;
extern uint32_t loadSize_est_data;

I try to reproduce it for another RAM location but it doesn't work, the following clear a data section which is not the expected one (it clears from adress 0x00000000 so it clears some usefull information written by the boot rom code).

The issue is coming from :

HAL_clearDataRAM((void *)loadStart_sta_data, (uint16_t)loadSize_sta_data);

loadStart_sta_data and loadSize_sta_data are declared identically in hal.h.

And in the command file, it is a little bit different :

	.sectionRAM_DATA_safety : {sta_tests.obj(.data),
						stl_sp.obj(.data),
						ref_crc.obj(.data),
						sta_tests.obj(.bss),
						sta_user.obj(.bss),
						//sta_util.obj(.data,.bss),
						sta_motor_application.obj(.data,.bss)} > BASE_RAMLS0D_SAFETY_DATA,
                      LOAD_START(loadStart_sta_data),
                      LOAD_END(loadEnd_sta_data),
                      LOAD_SIZE(loadSize_sta_data)

Using the debugger, I can see that the value of loadStart_sta_data and loadSize_sta_data is correct (using a breakpoint, when the cursor is above the data name) but once the code execution is in HAL_clearDataRAM((void *)loadStart_sta_data, (uint16_t)loadSize_sta_data);

void HAL_clearDataRAM(void *pMemory, uint16_t lengthMemory)
{
    uint16_t *pMemoryStart;
    uint16_t loopCount, loopLength;

    pMemoryStart = pMemory;
    loopLength = lengthMemory;

    for(loopCount = 0; loopCount < loopLength; loopCount++)
    {
        *(pMemoryStart + loopCount) = 0x0000;
    }
}   //end of HAL_clearDataRAM() function

The start adress switch to zero which cause the issue.

I can not find the issue, can I have some help?

Regards.

10 months ago

0 Manu36 10 months ago

Expert 1350 points

Hi, I have made some tests with the existing HAL_clearDataRAM function call and for me it is the same issue, it did'nt work as expected.

I also go back to the original universal_motor_control_lab and LAUNCHXL-F2800137 and it seems it is the same.

I can get it runing correctly by using this:

    uint32_t Start;
    uint16_t Size;
    Start=_symval(&loadStart_sta_data);
    Size = _symval(&loadSize_sta_data);

    HAL_clearDataRAM((void *)Start, (uint16_t)Size);

Can you confirm there is an issue in the universal_motor_control_lab and what is the best way to fix it?

Regards.

0 Manu36 10 months ago in reply to Manu36

Expert 1350 points

Again some progress,

I finally change all the calls to HAL_clearDataRAM like this:

//HAL_clearDataRAM((void *)loadStart_est_data, (uint16_t)loadSize_est_data);    //original code
HAL_clearDataRAM((void *)&loadStart_est_data, _symval(&loadSize_est_data));     //modified

I read the following:

https://e2e.ti.com/support/microcontrollers/c2000-microcontrollers-group/c2000/f/c2000-microcontrollers-forum/1060635/tms320f28379d-what-is-the-data-type-of-the-load_start-parameter

To be honest, I'm not fully confident with the changes I made, that's why I would to have an approval from TI expert.

Regards.

0 Jiaxin Teng 10 months ago in reply to Manu36

TI__Expert 5590 points

Hi Manu,

Can you try to delete the contents defined in the curly bracket and just keep the definition below?

.section RAM_DATA_safety :> BASE_RAMLS0D_SAFETY_DATA,
LOAD_START(loadStart_sta_data),
LOAD_END(loadEnd_sta_data),
LOAD_SIZE(loadSize_sta_data)

Thanks,

Jiaxin

0 Manu36 10 months ago in reply to Jiaxin Teng

Expert 1350 points

Hi Jiaxin,

Thank you for the answer.

As I try to explain in the 2nd post, the issue is not related the code I added. The original code also have the same issue, for exemple:

HAL_clearDataRAM((void *)loadStart_est_data, (uint16_t)loadSize_est_data);

    est_data             : >  RAMLS0F,
                              LOAD_START(loadStart_est_data),
                              LOAD_END(loadEnd_est_data),
                              LOAD_SIZE(loadSize_est_data)

This is similar to your proposal and also failed.

From other function which are in the universal_motor_control_lab and also some documentation of the linker/compiler, It seems that "&" is mandatory, see for example:

    memcpy(&RamfuncsRunStart, &RamfuncsLoadStart, (size_t)&RamfuncsLoadSize);

C2000™︎ microcontrollers

C2000 microcontrollers forum

TMS320F2800132: issue with HAL_clearDataRAM or LOAD_START in linker cmd file