MSP432P4111: Header File Changes in MSP432 SDK

Jejomar Ildefonso

Part Number: MSP432P4111

Hi Team,

Posting for our customer. They would like to ask some information on changes made from v3230 to 3231 of the msp432p4111.h header file in the MSP432 SDK v3.40.01.02.

They are specifically interested in why some base addresses in the header files are now signed integers. (i.e. the "U" has been removed). They following MISRA rules and this actually gives warnings when writing low level drivers for an MSP432P4111.

Thanks in advance!

Kind Regards,

Jejomar

over 5 years ago

0 Dennis Lehman over 5 years ago

TI__Guru 71216 points

Hi Jejomar,

Let me look into this.

0 Dennis Lehman over 5 years ago in reply to Dennis Lehman

TI__Guru 71216 points

Hi Jejomar,

I'm looking at SDK SDK v3.40.01.02, msp430p4111.h.

#define __MSP432_HEADER_VERSION__ 3231

I'm not finding any register that is defined as an (int16_t) vs (uint16_t) or (int32_t) vs (uint32_t).

Were you able to locate an instance? Can the customer be specific where in the header file?

0 Jejomar Ildefonso over 5 years ago in reply to Dennis Lehman

TI__Genius 9700 points

Hello Dennis,

Many thanks for looking into this. Our customer is looking specifically at the UCA3 UART module and in the msp432p4111.h file version 3.40.01.02, the base address is defined as:

#define EUSCI_A3_BASE (PERIPH_BASE +0x00001C00)

where
#define PERIPH_BASE ((uint32_t)0x40000000)

It seems that even though the peripheral address uses uint32_t, the lack of a "U" on the UART base address causes a MISRA-C 2004 warning.

The following prevents a warning:
#define EUSCI_A3_BASE (PERIPH_BASE +0x00001C00U)

0 Dennis Lehman over 5 years ago in reply to Jejomar Ildefonso

TI__Guru 71216 points

Hi Jejomar,

Got it. Let me reach out to SW team.

0 Dennis Lehman over 5 years ago in reply to Dennis Lehman

TI__Guru 71216 points

Hello Jejomar,

According to our SW team reports the following:

Is it possible the previous file mentioned by the customer was modified manually? I found the alleged version 3230 and I even went to the older version available in the git SDK, and none have the register bases declared with ‘U’.

You’re right, the MSP432 SDK did not run MISRA-C. And I agree, looking at previous versions of PERIPH_BASE and EUSCI_A3_BASE in msp432p4111.h, it doesn’t look like those register bases were ever declared with ‘U’, so perhaps the customer was using their own copy of that file.