CCS/MSP430FR5994: CCS/MSP430FR5994

ok I get it these values are in the linker file as per;

4.3.4 Initialization of the FRAM Memory Protection Unit The linker supports initialization of the FRAM memory protection unit (MPU). The linker uses a boot routine that performs MPU initialization based on the definition of certain symbols. The TI-provided linker command files that are used by default for different devices define the necessary symbols so MPU initialization happens automatically. Code and data sections are automatically given the correct access permissions. If you want to manually adjust how the MPU is initialized you can modify the __mpuseg and __mpusam definitions in the linker command file. The MPU-specific boot routine is used when these two symbols are defined and it sets the value of the MPUSEG and MPUSAM registers based on these values. If you do not want the MPU initialized you can remove these definitions from the linker command file.

Craig,

The linker decides to include the __mpu_init() routine in the link if it sees either:

   __mpu_enable

Or

    __mpusam && __mpuseg

__mpusam/__mpuseg are from a much older initial implementation of mpu init support before it was updated to better support more devices.

You should only need to either update CCS MPU dialog in Project_properties->General->MPU tab and check  the box for "Enable Memory Protection Unit (MPU)", or add predefined symbol with -D_MPU_ENABLE (which is what checking the box does anyways). That should work with most new linker cmd files. 

If you are still having linker errors then please provide more information about your CCS project, msp device, compiler options, etc.

Thanks
Greg

Greg,

Thanks so much for the reply.  I have selected Enable Memory Protection Unit (MPU). I started to get these errors when I enabled MPU. I'm using 

Code Composer Studio

Version: 10.0.0.00010

(c) Copyright Texas Instruments, 2016. All rights reserved.

Attached in my linker command file

/******************************************************************************
*
* Copyright (C) 2012 - 2020 Texas Instruments Incorporated - http://www.ti.com/
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
* are met:
*
* Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
*
* Redistributions in binary form must reproduce the above copyright
* notice, this list of conditions and the following disclaimer in the
* documentation and/or other materials provided with the
* distribution.
*
* Neither the name of Texas Instruments Incorporated nor the names of
* its contributors may be used to endorse or promote products derived
* from this software without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*
* Default linker command file for Texas Instruments MSP430FR5994
*
*****************************************************************************/

/******************************************************************************/
/* */
/* Usage: lnk430 <obj files...> -o <out file> -m <map file> lnk.cmd */
/* cl430 <src files...> -z -o <out file> -m <map file> lnk.cmd */
/* */
/*----------------------------------------------------------------------------*/
/* These linker options are for command line linking only. For IDE linking, */
/* you should set your linker options in Project Properties */
/* -c LINK USING C CONVENTIONS */
/* -stack 0x0100 SOFTWARE STACK SIZE */
/* -heap 0x0100 HEAP AREA SIZE */
/* */
/*----------------------------------------------------------------------------*/
/* 1.209 */
/*----------------------------------------------------------------------------*/

/****************************************************************************/
/* SPECIFY THE SYSTEM MEMORY MAP */
/****************************************************************************/

MEMORY
{
TINYRAM : origin = 0xA, length = 0x16
BSL : origin = 0x1000, length = 0x800
INFOD : origin = 0x1800, length = 0x80
INFOC : origin = 0x1880, length = 0x80
INFOB : origin = 0x1900, length = 0x80
INFOA : origin = 0x1980, length = 0x80
RAM : origin = 0x1C00, length = 0x1000
FRAM : origin = 0x4000, length = 0xBF80
FRAM2 : origin = 0x10000,length = 0x34000
JTAGSIGNATURE : origin = 0xFF80, length = 0x0004, fill = 0xFFFF
BSLSIGNATURE : origin = 0xFF84, length = 0x0004, fill = 0xFFFF
IPESIGNATURE : origin = 0xFF88, length = 0x0008, fill = 0xFFFF
INT00 : origin = 0xFF90, length = 0x0002
INT01 : origin = 0xFF92, length = 0x0002
INT02 : origin = 0xFF94, length = 0x0002
INT03 : origin = 0xFF96, length = 0x0002
INT04 : origin = 0xFF98, length = 0x0002
INT05 : origin = 0xFF9A, length = 0x0002
INT06 : origin = 0xFF9C, length = 0x0002
INT07 : origin = 0xFF9E, length = 0x0002
INT08 : origin = 0xFFA0, length = 0x0002
INT09 : origin = 0xFFA2, length = 0x0002
INT10 : origin = 0xFFA4, length = 0x0002
INT11 : origin = 0xFFA6, length = 0x0002
INT12 : origin = 0xFFA8, length = 0x0002
INT13 : origin = 0xFFAA, length = 0x0002
INT14 : origin = 0xFFAC, length = 0x0002
INT15 : origin = 0xFFAE, length = 0x0002
INT16 : origin = 0xFFB0, length = 0x0002
INT17 : origin = 0xFFB2, length = 0x0002
INT18 : origin = 0xFFB4, length = 0x0002
INT19 : origin = 0xFFB6, length = 0x0002
INT20 : origin = 0xFFB8, length = 0x0002
INT21 : origin = 0xFFBA, length = 0x0002
INT22 : origin = 0xFFBC, length = 0x0002
INT23 : origin = 0xFFBE, length = 0x0002
INT24 : origin = 0xFFC0, length = 0x0002
INT25 : origin = 0xFFC2, length = 0x0002
INT26 : origin = 0xFFC4, length = 0x0002
INT27 : origin = 0xFFC6, length = 0x0002
INT28 : origin = 0xFFC8, length = 0x0002
INT29 : origin = 0xFFCA, length = 0x0002
INT30 : origin = 0xFFCC, length = 0x0002
INT31 : origin = 0xFFCE, length = 0x0002
INT32 : origin = 0xFFD0, length = 0x0002
INT33 : origin = 0xFFD2, length = 0x0002
INT34 : origin = 0xFFD4, length = 0x0002
INT35 : origin = 0xFFD6, length = 0x0002
INT36 : origin = 0xFFD8, length = 0x0002
INT37 : origin = 0xFFDA, length = 0x0002
INT38 : origin = 0xFFDC, length = 0x0002
INT39 : origin = 0xFFDE, length = 0x0002
INT40 : origin = 0xFFE0, length = 0x0002
INT41 : origin = 0xFFE2, length = 0x0002
INT42 : origin = 0xFFE4, length = 0x0002
INT43 : origin = 0xFFE6, length = 0x0002
INT44 : origin = 0xFFE8, length = 0x0002
INT45 : origin = 0xFFEA, length = 0x0002
INT46 : origin = 0xFFEC, length = 0x0002
INT47 : origin = 0xFFEE, length = 0x0002
INT48 : origin = 0xFFF0, length = 0x0002
INT49 : origin = 0xFFF2, length = 0x0002
INT50 : origin = 0xFFF4, length = 0x0002
INT51 : origin = 0xFFF6, length = 0x0002
INT52 : origin = 0xFFF8, length = 0x0002
INT53 : origin = 0xFFFA, length = 0x0002
INT54 : origin = 0xFFFC, length = 0x0002
RESET : origin = 0xFFFE, length = 0x0002
}

/****************************************************************************/
/* Specify the LEA memory map */
/****************************************************************************/

#define LEASTACK_SIZE 0x138

MEMORY
{
LEARAM : origin = 0x2C00, length = 0x1000 - LEASTACK_SIZE
LEASTACK : origin = 0x3C00 - LEASTACK_SIZE, length = LEASTACK_SIZE
}

/****************************************************************************/
/* SPECIFY THE SECTIONS ALLOCATION INTO MEMORY */
/****************************************************************************/

SECTIONS
{
.text:EraseMe > 0x43F00 /*This is the Erase function at the top of Memory */

GROUP(RW_IPE)
{

GROUP(READ_WRITE_MEMORY)
{

.TI.persistent : {} /* For #pragma persistent */
.cio : {} /* C I/O Buffer */
.sysmem : {} /* Dynamic memory allocation area */
} PALIGN(0x0400), RUN_START(fram_rw_start)

GROUP(IPENCAPSULATED_MEMORY)
{

.ipestruct : {} /* IPE Data structure */
.ipe : {} /* IPE */
.ipe_const : {} /* IPE Protected constants */
.ipe:_isr : {} /* IPE ISRs */
} PALIGN(0x0400), RUN_START(fram_ipe_start) RUN_END(fram_ipe_end) RUN_END(fram_rx_start)

} > 0x4000

.cinit : {} > FRAM /* Initialization tables */
.binit : {} > FRAM /* Boot-time Initialization tables */
.pinit : {} > FRAM /* C++ Constructor tables */
.init_array : {} > FRAM /* C++ Constructor tables */
.mspabi.exidx : {} > FRAM /* C++ Constructor tables */
.mspabi.extab : {} > FRAM /* C++ Constructor tables */
.text:_isr : {} > FRAM /* Code ISRs */

#ifndef __LARGE_DATA_MODEL__
.const : {} > FRAM /* Constant data */
#else
.const : {} >> FRAM | FRAM2 /* Constant data */
#endif

#ifndef __LARGE_CODE_MODEL__
.text : {} > FRAM /* Code */
#else
.text : {} >> FRAM2 | FRAM /* Code */
#endif

#ifdef __TI_COMPILER_VERSION__
#if __TI_COMPILER_VERSION__ >= 15009000
#ifndef __LARGE_CODE_MODEL__
.TI.ramfunc : {} load=FRAM, run=RAM, table(BINIT)
#else
.TI.ramfunc : {} load=FRAM | FRAM2, run=RAM, table(BINIT)
#endif
#endif
#endif

.jtagsignature : {} > JTAGSIGNATURE
.bslsignature : {} > BSLSIGNATURE

GROUP(SIGNATURE_SHAREDMEMORY)
{
.ipesignature : {} /* IPE Signature */
.jtagpassword : {} /* JTAG Password */
} > IPESIGNATURE

.bss : {} > RAM /* Global & static vars */
.data : {} > RAM /* Global & static vars */
.TI.noinit : {} > RAM /* For #pragma noinit */
.stack : {} > RAM (HIGH) /* Software system stack */

.tinyram : {} > TINYRAM /* Tiny RAM */

/* MSP430 INFO memory segments */
.infoA : type = NOINIT{} > INFOA
.infoB : type = NOINIT{} > INFOB
.infoC : type = NOINIT{} > INFOC
.infoD : type = NOINIT{} > INFOD

.leaRAM : {} > LEARAM /* LEA RAM */
.leaStack : {} > LEASTACK (HIGH) /* LEA STACK */

/* MSP430 interrupt vectors */

.int00 : {} > INT00
.int01 : {} > INT01
.int02 : {} > INT02
.int03 : {} > INT03
.int04 : {} > INT04
.int05 : {} > INT05
.int06 : {} > INT06
.int07 : {} > INT07
.int08 : {} > INT08
.int09 : {} > INT09
.int10 : {} > INT10
.int11 : {} > INT11
.int12 : {} > INT12
.int13 : {} > INT13
.int14 : {} > INT14
.int15 : {} > INT15
.int16 : {} > INT16
.int17 : {} > INT17
LEA : { * ( .int18 ) } > INT18 type = VECT_INIT
PORT8 : { * ( .int19 ) } > INT19 type = VECT_INIT
PORT7 : { * ( .int20 ) } > INT20 type = VECT_INIT
EUSCI_B3 : { * ( .int21 ) } > INT21 type = VECT_INIT
EUSCI_B2 : { * ( .int22 ) } > INT22 type = VECT_INIT
EUSCI_B1 : { * ( .int23 ) } > INT23 type = VECT_INIT
EUSCI_A3 : { * ( .int24 ) } > INT24 type = VECT_INIT
EUSCI_A2 : { * ( .int25 ) } > INT25 type = VECT_INIT
PORT6 : { * ( .int26 ) } > INT26 type = VECT_INIT
PORT5 : { * ( .int27 ) } > INT27 type = VECT_INIT
TIMER4_A1 : { * ( .int28 ) } > INT28 type = VECT_INIT
TIMER4_A0 : { * ( .int29 ) } > INT29 type = VECT_INIT
AES256 : { * ( .int30 ) } > INT30 type = VECT_INIT
RTC_C : { * ( .int31 ) } > INT31 type = VECT_INIT
PORT4 : { * ( .int32 ) } > INT32 type = VECT_INIT
PORT3 : { * ( .int33 ) } > INT33 type = VECT_INIT
TIMER3_A1 : { * ( .int34 ) } > INT34 type = VECT_INIT
TIMER3_A0 : { * ( .int35 ) } > INT35 type = VECT_INIT
PORT2 : { * ( .int36 ) } > INT36 type = VECT_INIT
TIMER2_A1 : { * ( .int37 ) } > INT37 type = VECT_INIT
TIMER2_A0 : { * ( .int38 ) } > INT38 type = VECT_INIT
PORT1 : { * ( .int39 ) } > INT39 type = VECT_INIT
TIMER1_A1 : { * ( .int40 ) } > INT40 type = VECT_INIT
TIMER1_A0 : { * ( .int41 ) } > INT41 type = VECT_INIT
DMA : { * ( .int42 ) } > INT42 type = VECT_INIT
EUSCI_A1 : { * ( .int43 ) } > INT43 type = VECT_INIT
TIMER0_A1 : { * ( .int44 ) } > INT44 type = VECT_INIT
TIMER0_A0 : { * ( .int45 ) } > INT45 type = VECT_INIT
ADC12_B : { * ( .int46 ) } > INT46 type = VECT_INIT
EUSCI_B0 : { * ( .int47 ) } > INT47 type = VECT_INIT
EUSCI_A0 : { * ( .int48 ) } > INT48 type = VECT_INIT
WDT : { * ( .int49 ) } > INT49 type = VECT_INIT
TIMER0_B1 : { * ( .int50 ) } > INT50 type = VECT_INIT
TIMER0_B0 : { * ( .int51 ) } > INT51 type = VECT_INIT
COMP_E : { * ( .int52 ) } > INT52 type = VECT_INIT
UNMI : { * ( .int53 ) } > INT53 type = VECT_INIT
SYSNMI : { * ( .int54 ) } > INT54 type = VECT_INIT
.reset : {} > RESET /* MSP430 reset vector */

}
/****************************************************************************/
/* MPU/IPE SPECIFIC MEMORY SEGMENT DEFINITONS */
/****************************************************************************/

#ifdef _IPE_ENABLE
#define IPE_MPUIPLOCK 0x0080
#define IPE_MPUIPENA 0x0040
#define IPE_MPUIPPUC 0x0020

// Evaluate settings for the control setting of IP Encapsulation
#if defined(_IPE_ASSERTPUC1)
#if defined(_IPE_LOCK ) && (_IPE_ASSERTPUC1 == 0x08))
fram_ipe_enable_value = (IPE_MPUIPENA | IPE_MPUIPPUC |IPE_MPUIPLOCK);
#elif defined(_IPE_LOCK )
fram_ipe_enable_value = (IPE_MPUIPENA | IPE_MPUIPLOCK);
#elif (_IPE_ASSERTPUC1 == 0x08)
fram_ipe_enable_value = (IPE_MPUIPENA | IPE_MPUIPPUC);
#else
fram_ipe_enable_value = (IPE_MPUIPENA);
#endif
#else
#if defined(_IPE_LOCK )
fram_ipe_enable_value = (IPE_MPUIPENA | IPE_MPUIPLOCK);
#else
fram_ipe_enable_value = (IPE_MPUIPENA);
#endif
#endif

// Segment definitions
#ifdef _IPE_MANUAL // For custom sizes selected in the GUI
fram_ipe_border1 = (_IPE_SEGB1>>4);
fram_ipe_border2 = (_IPE_SEGB2>>4);
#else // Automated sizes generated by the Linker
fram_ipe_border2 = fram_ipe_end >> 4;
fram_ipe_border1 = fram_ipe_start >> 4;
#endif

fram_ipe_settings_struct_address = Ipe_settingsStruct >> 4;
fram_ipe_checksum = ~((fram_ipe_enable_value & fram_ipe_border2 & fram_ipe_border1) | (fram_ipe_enable_value & ~fram_ipe_border2 & ~fram_ipe_border1) | (~fram_ipe_enable_value & fram_ipe_border2 & ~fram_ipe_border1) | (~fram_ipe_enable_value & ~fram_ipe_border2 & fram_ipe_border1));
#endif

#ifdef _MPU_ENABLE
#define MPUPW (0xA500) /* MPU Access Password */
#define MPUENA (0x0001) /* MPU Enable */
#define MPULOCK (0x0002) /* MPU Lock */
#define MPUSEGIE (0x0010) /* MPU Enable NMI on Segment violation */

__mpu_enable = 1;
// Segment definitions
#ifdef _MPU_MANUAL // For custom sizes selected in the GUI
mpu_segment_border1 = _MPU_SEGB1 >> 4;
mpu_segment_border2 = _MPU_SEGB2 >> 4;
mpu_sam_value = (_MPU_SAM0 << 12) | (_MPU_SAM3 << 8) | (_MPU_SAM2 << 4) | _MPU_SAM1;
#else // Automated sizes generated by Linker
#ifdef _IPE_ENABLE //if IPE is used in project too
//seg1 = any read + write persistent variables
//seg2 = ipe = read + write + execute access
//seg3 = code, read + execute only
mpu_segment_border1 = fram_ipe_start >> 4;
mpu_segment_border2 = fram_rx_start >> 4;
mpu_sam_value = 0x1573; // Info R, Seg3 RX, Seg2 RWX, Seg1 RW
#else
mpu_segment_border1 = fram_rx_start >> 4;
mpu_segment_border2 = fram_rx_start >> 4;
mpu_sam_value = 0x1513; // Info R, Seg3 RX, Seg2 R, Seg1 RW
#endif
#endif
#ifdef _MPU_LOCK
#ifdef _MPU_ENABLE_NMI
mpu_ctl0_value = MPUPW | MPUENA | MPULOCK | MPUSEGIE;
#else
mpu_ctl0_value = MPUPW | MPUENA | MPULOCK;
#endif
#else
#ifdef _MPU_ENABLE_NMI
mpu_ctl0_value = MPUPW | MPUENA | MPUSEGIE;
#else
mpu_ctl0_value = MPUPW | MPUENA;
#endif
#endif
#endif

/****************************************************************************/
/* INCLUDE PERIPHERALS MEMORY MAP */
/****************************************************************************/

-l msp430fr5994.cmd

Greg,

So check this out.

First I retried with Project->Properties->General->MPU->Enable Memory Protection Unit (MPU) and thta resulted in the symbol errors.

MPUSEG /home/ckeller/ti/ccs1000/ccs/tools/compiler/ti-cgt-msp430_20.2.0.LTS/lib/rts430x_lc_ld_eabi.lib<mpu_init.c.obj>
__mpusam /home/ckeller/ti/ccs1000/ccs/tools/compiler/ti-cgt-msp430_20.2.0.LTS/lib/rts430x_lc_ld_eabi.lib<mpu_init.c.obj>
__mpuseg /home/ckeller/ti/ccs1000/ccs/tools/compiler/ti-cgt-msp430_20.2.0.LTS/lib/rts430x_lc_ld_eabi.lib<mpu_init.c.obj>

However, if I dselect Enable Memory Protection Unit (MPU) and put the symbol D_MPU_ENABLE in Projects->Properties->Build->MSP430 Compiler->Predefined Symbols then build links.

What is the difference?

Craig

Craig, Can you send a few more details on what you did to originally create the project?

When I create blink LED for FR5994 with CCS10.0 with 20.2.0.LTS compiler, it builds clean with no link errors and includes mpu_init (ie that's set by default).

If I then uncheck MPU init checkbox, then it builds clean without any link errors.  And no mpu_init.

If I then check the MPU init checkbox, it once again builds clean with mpu_init enabled.

So I can't reproduce your issue.

I'm not sure if the CCS MPU enable checkbox does anything else beyond adding _MPU_ENABLE as a predefined symbol but have asked another colleague who has worked with the feature.

Thanks
Greg

Hi Greg,

The project was originally created using ccs 8.3.1, I recently upgraded to ccs10.0.0.  I didn't have a previous need under 8.3.1 to enable the MPU, only after migrating to 10.0.0. Let's see, oh,  When I installed ccs10.0.0 I only included resources for MSP430, so maybe something has yet to be installed or configured.

General:

Family=MSP430

Compiler version= TI v20.2.0.LTS

Output Format=elf

linker command= lnk_msp430fr5994.cmd (modified to include '     .text:EraseMe > 0x43F00                  /*This is the Erase function at the top of Memory */" under SECTIONS

Runtime support library=automatic

General->build->MSP430 Compiler->Processor Options

Silicon Version=mspx

code memory model=large

data model=large

Indicates what data must be near=none

General->build->MSP430 Compiler->Advice Options

FRAM Configuration Setting is checked

1,1: FRAM LockIO  (checked)

1.2 FRAM wait states (checked)

Note: Last night I added the symbol _MPU_ENABLE to the Predefined Symbols, and this morning it is not persistent

General->build->MSP430 Compiler->Advamced Options->Language Options

C Dialect=C11

Floating point precision....=all

chars=unsigned

level of printf/scanf...=minimal

General->build->MSP430 Linker->Basic Options

Heap=4096

Set C system stack size=128

Hold WDT=on

General->build->MSP430 Linker->File Search Path

${COM_TI_XDC_LIBRARY_PATH}

${CCS_BASE_ROOT}/msp430/include

/home/ckeller/ti/ccs1000/ccs/ccs_base/msp430/include

/home/ckeller/ti/ccs1000/ccs/ccs_base/msp430/lib/5xx_6xx_FRxx

/home/ckeller/ti/ccs1000/ccs/ccs_base/msp430/lib/FR59xx

Craig

Greg,

I looked at ./tools/compiler/ti-cgt-msp430_20.2.0.LTS/lib/src/mpu_init.c and if I'm pulling in this all of the missing symbols are there? So how can I be using the wrong mpu_init.c?

void __mpu_init(void)
{
MPUCTL0 = 0xA500; /* Unlock MPU */
MPUSEG = (unsigned int)_symval(&__mpuseg); /* Set segment boundaries */
MPUSAM = (unsigned int)_symval(&__mpusam); /* Set RWX permissions */
MPUCTL0 = 0xA501; /* Enable MPU */
MPUCTL0_H = 0x0; /* Lock MPU */
}

Craig,

You should not be linking in the RTS lib/src/mpu_init.c as that is the older legacy implementation only for FR57.

If libc.a is being linked prior to libmpu_init.a then you could be pulling in the wrong older implementation

Please confirm the order in General-build->MSP430 Linker->File Search Path to be:
   libmath.a
   libmpu_init.a
   libc.a

Also, console output for link step should have libmpu_init.a linked ahead of libc.a
cl430 ... -o "msp_mpu_fr5994.out" "./blink.obj" "../lnk_msp430fr5994.cmd" -llibmath.a -llibmpu_init.a -llibc.a

Thanks
Greg