Forums

Hi Russell,

I don't use CCS.  But it seems to me that the ".cinit : " line should be above the ".text : " line so that that linker places it first.

To your specific question, it looks like the entire 256KB is configured correctly in the linker script and that the code will span appropriately.  It's just that ".cinit" apparently must reside in the lower 64K.  Again, I don't use CCS, so I don't know if the linker honors the order of sections listed, but I would move all sections that require "FLASH" __above__ all of the sections that can be "FLASH" or "FLASH2".

Jeff

PS.  Why does this command file apparently put ISRs in FLASH2 (@ 0x10000 and above)?  That doesn't seem right.

Try replacing the following part of the linker script:

.bss : {} > RAM | RAM2 //Global and static variables
.sysmem : {} > RAM | RAM2 //Dynamic memory alloc
.stack : {} > RAM2 (HIGH) //Stack
.text : {}>> FLASH //Code
 .text:_isr : {} > FLASH2 //Interrupt Service Routines

.bss : {} > RAM | RAM2 //Global and static variables
.sysmem : {} > RAM | RAM2 //Dynamic memory alloc
.data : {} > RAM | RAM2
.stack : {} > RAM2 (HIGH) //Stack
.text : {}>> FLASH2 | FLASH //Code
 .text:_isr : {} > FLASH //Interrupt Service Routines

1) With CCS 5.2 and MSP430 Compiler v4.1.1 the linker was reporting the warning:

warning #10247-D: creating output section ".data" without a SECTIONS specification

2) Setting the .text to be placed in FLASH2 in preference to FLASH allows the code to fit without getting the "can not fit code into memory". I don't understand why, but had a similar problem with a MSP430F5438 project.

3) As Jeff already pointed out the .text:_isr needs to be placed in the first 64Kbytes of memory (as per the default linker script when a new MSP430F5438A project is created).
 

Chester,

This is a logical idea, and certainly fixes the issue of data being stored in the FLASH section, instead of RAM. However, when I make this correction, it blows the security fuse.

Russell

Jeff,

Those definitions come from TI's CCS linker file via the "User Experience" application for this chip. I'm not sure why that's the case, but with very small code models, it seems to work.

Seems like everyone using micros at TI use IAR Workbench. If so, why does TI even create this tool at all?

Russell

Hi Russel,

Russell Lemburg

Seems like everyone using micros at TI use IAR Workbench. If so, why does TI even create this tool at all?

As far as i know, IAR is much more expensive than CCS.

Russell Lemburg

This is a logical idea, and certainly fixes the issue of data being stored in the FLASH section, instead of RAM. However, when I make this correction, it blows the security fuse.

a) The current lnk_msp430f5438a.cmd linker command file.

b) The CCS console build output for a rebuild of the project, to see if there are any #10247-D: creating output section "<section>" without a SECTIONS warnings. See Should linker warning #10247-D really be an error for why these warnings are a sign that the linked program may not run.

(it the linker ends up placing sections intended for ram in the peripheral address space the start-up code can get stuck continuously generating watchdog resets which means CCS may report that the security fuse is blown)

c) The LEDGoggles_MSP430_Sw_Rev_B_F5438A.map file produced by the linker, to be able to check where sections have been placed in memory.

Chester,

Thanks for the reply.

Yes, I've read over the Should linker warning #10247-D really be an error article. This was sort of what we thought was happening, but CCS no longer gives that warning.

In zip file:

  lnk_msp430f5438a.cmd

  LEDGoggles_MSP430_Sw_Rev_B_F5438A.map

CCS Build output:

Description    Resource    Path    Location    Type
#303-D typedef name has already been declared (with same type)    main.c    /LEDGoggles    line 32    C/C++ Problem
#70-D integer conversion resulted in truncation    Factory.h    /LEDGoggles    line 204    C/C++ Problem
#70-D integer conversion resulted in truncation    Factory.h    /LEDGoggles    line 214    C/C++ Problem
#70-D integer conversion resulted in truncation    Factory.h    /LEDGoggles    line 224    C/C++ Problem

Thank you

Looking at the linker command file and map file I can't see any problem with the sections. i.e. appears that the flash and RAM sections are correct.

The project uses flash above 64Kbyte, has the CCS project selected the "large" memory model?

Chester Gillon

The project uses flash above 64Kbyte, has the CCS project selected the "large" memory model?

Having the option as "blank" causes the compiler to select the code memory model based upon the type of CPU - small for a CPU module (16-bit register / address bus) and large for a CPUX module (20-bit registers / address bus).

To confirm the actual settings in use the CCS Project Properties -> CCS Build -> MSP430 Compiler -> Advanced Options -> Assembler Options -> Generate listing file (--asm_listing, -al) can be selected. The resulting .lst file reports the memory model used by the compiler.

Given that the attached LEDGoggles_MSP430_Sw_Rev_B_F5438A.zip doesn't change the default memory models, it should be using a large code memory model and a small data model which is correct (the map file shows all constants and data are in the lower 32Kbytes).

The attached CCS project in the attached LEDGoggles_MSP430_Sw_Rev_B_F5438A.zip contains a system_pre_init.c to "Disable Watchdog timer to prevent reset during long variable initialization sequences".

Using CCS 5.2.1.00018 and MSP430 compiler v4.1.0 to compile the project in the attached LEDGoggles_MSP430_Sw_Rev_B_F5438A.zip with the lnk2_msp430f5438a.cmd from the attached LEDGoggles_map_and_cmd.zip, the resulting linker map shows that the _system_pre_init function from the project system_pre_init.c gets linked into the executable:

0001a2b4 0000000a system_pre_init.obj (.text:_system_pre_init)

00027af6 00000004 rts430x_lc_sd_eabi.lib : pre_init.obj (.text:_system_pre_init)

Has anything in the project, apart from lnk2_msp430f5438a.cmd changed since LEDGoggles_MSP430_Sw_Rev_B_F5438A.zip was attached?

Chester,

This is a good suggestion. I've made sure that we are using the large memory model and small data model (this is correct based on the .map file). I thought I had these compiler arguments enabled.

CCS has seemed to stop working with Spy-by-Wire as a result of this (I get a "device not recognized" if the FET430UIF is connected in a Spy-by-Wire configuration). However, I managed to get the code to store into memory using JTAG. If I break execution a second or so after running, I get the following runtime message:

No source available for "_TI_zero_init(unsigned long, unsigned long) at 0x27810"

Seems like the processor is resetting itself.

I removed system_pre_init.c and disable the watchdog at the beginning of main().

Including rts430x_lc_sd_eabi.lib in the runtime support library and adding system_pre_init.c seemed to work. I'm now able to compile and run the code (~90K), but ONLY WITH JTAG.

CCS is giving me an "unknown device" if I connect with Spy-by-Wire.

I could not find any documentation on compiler options for forcing either JTAG or Spy-by-Wire modes in CCS. I know in IAR you can specify either one explicitly.

Russ

Russell,

from your last post I wasn't sure whether you were able to completely resolve the issue. Could you confirm either scenario?

As far as the .text: {} >> FLASH2 | FLASH is concerned, the splitting instruction essentially prioritizes splitting and placing .text objects into FLASH2 first and only switches to FLASH when it runs out of FLASH2 space. This is indicated in your map file with most of the code is located in FLASH2 and minimal space is consumed in FLASH.

I gave your project a quick try, with the latest linker command file that you attached, as well as with some other combinations of placing .text to >/>> FLASH2 | FLASH, and the project seems to build. However, the interrupt vectors, especially the reset vector always shows up as 0xFFFF. This might be due to my hacked-up test setup, but I want to make sure whether you can see the same thing or it was something related to something specific in the code.

Regarding the system_pre_init.c and its function int _system_pre_init(void), do you plan on adding any code in there other than the watchdog holding line? If not, you can safely remove the file and let CCS use the system default file. You'll need to add the watchdog line to the top of your main() function however.

Regards,

Dung

Dung,

The linker file is working now. most of the code should reside in FLASH2. That's now working.

system_pre_init.c needs to be there. If I disable the watchdog in main, the processor stays in reset.

The problem I am now having seems to be a result of fixing these other problems. That is, I can no longer debug with Spy-by-Wire. JTAG seems to be the only thing that works.

Russell

Dung Dang

I gave your project a quick try, with the latest linker command file that you attached, as well as with some other combinations of placing .text to >/>> FLASH2 | FLASH, and the project seems to build. However, the interrupt vectors, especially the reset vector always shows up as 0xFFFF. This might be due to my hacked-up test setup, but I want to make sure whether you can see the same thing or it was something related to something specific in the code.

There are two warnings 'ignoring .CLINK directive' warnings which are probably related to the vectors being zero:

**** Build of configuration Debug for project LEDGoggles_MSP430_Sw_Rev_B_F5438A ****

C:\ti\ccsv5\utils\bin\gmake -k all
'Building file: ../FFT_430.asm'
'Invoking: MSP430 Compiler'
"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/bin/cl430" -vmspx --abi=eabi -O4 --opt_for_speed=0 -g --include_path="C:/ti/ccsv5/ccs_base/msp430/include" --include_path="C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/include" --define=__MSP430F5438A__ --diag_warning=225 --display_error_number --silicon_errata=CPU21 --silicon_errata=CPU22 --silicon_errata=CPU23 --silicon_errata=CPU40 --printf_support=minimal --preproc_with_compile --preproc_dependency="FFT_430.pp" "../FFT_430.asm"
'Finished building: ../FFT_430.asm'
' '
'Building file: ../main.c'
'Invoking: MSP430 Compiler'
"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/bin/cl430" -vmspx --abi=eabi -O4 --opt_for_speed=0 -g --include_path="C:/ti/ccsv5/ccs_base/msp430/include" --include_path="C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/include" --define=__MSP430F5438A__ --diag_warning=225 --display_error_number --silicon_errata=CPU21 --silicon_errata=CPU22 --silicon_errata=CPU23 --silicon_errata=CPU40 --printf_support=minimal --preproc_with_compile --preproc_dependency="main.pp" "../main.c"
"..\Factory.h", line 204: warning #70-D: integer conversion resulted in truncation
"..\Factory.h", line 214: warning #70-D: integer conversion resulted in truncation
"..\Factory.h", line 224: warning #70-D: integer conversion resulted in truncation
"../main.c", line 23: warning #303-D: typedef name has already been declared (with same type)
"C:\DOCUME~1\MR_HAL~1\LOCALS~1\Temp\0328013", WARNING! at line 34570:
[W0000]
Section .text:_isr:TIMER1_A0_ISR already has .RETAIN specified -
ignoring .CLINK directive

.clink

No Assembly Errors, 1 Assembly Warning
'Finished building: ../main.c'
' '
'Building file: ../system_pre_init.c'
'Invoking: MSP430 Compiler'
"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/bin/cl430" -vmspx --abi=eabi -O4 --opt_for_speed=0 -g --include_path="C:/ti/ccsv5/ccs_base/msp430/include" --include_path="C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/include" --define=__MSP430F5438A__ --diag_warning=225 --display_error_number --silicon_errata=CPU21 --silicon_errata=CPU22 --silicon_errata=CPU23 --silicon_errata=CPU40 --printf_support=minimal --preproc_with_compile --preproc_dependency="system_pre_init.pp" "../system_pre_init.c"
'Finished building: ../system_pre_init.c'
' '
'Building target: LEDGoggles_MSP430_Sw_Rev_B_F5438A.out'
'Invoking: MSP430 Linker'
"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/bin/cl430" -vmspx --abi=eabi -O4 --opt_for_speed=0 -g --define=__MSP430F5438A__ --diag_warning=225 --display_error_number --silicon_errata=CPU21 --silicon_errata=CPU22 --silicon_errata=CPU23 --silicon_errata=CPU40 --printf_support=minimal -z --stack_size=256 -m"LEDGoggles_MSP430_Sw_Rev_B_F5438A.map" --heap_size=256 --use_hw_mpy=F5 -i"C:/ti/ccsv5/ccs_base/msp430/include" -i"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/lib" -i"C:/ti_ccs5_2/ccsv5/tools/compiler/msp430_4.1.0/include" --reread_libs --warn_sections --display_error_number --rom --rom_model -o "LEDGoggles_MSP430_Sw_Rev_B_F5438A.out" "./system_pre_init.obj" "./main.obj" "./FFT_430.obj" -l"libc.a" "../lnk2_msp430f5438a.cmd"
<Linking>
"C:\DOCUME~1\MR_HAL~1\LOCALS~1\Temp\0354412", WARNING! at line 52802:
[W0000]
Section [1].text:_isr:TIMER1_A0_ISR already has .RETAIN specified -
ignoring .CLINK directive

.clink

No Assembly Errors, 1 Assembly Warning
'Finished building target: LEDGoggles_MSP430_Sw_Rev_B_F5438A.out'
' '

**** Build Finished ****