Возврат из функции в gcc., не возвращается=( Опции

[osminog]

Компилю проект в avr-gcc . Так вот проблема такая: не происходит возврата из функций - процессор зависает на следующей строчке программы после функции, при этом сама функция нормально вызывается и исполняется. Может кто-то с этим сталкивался, подскажите плиз.

[_Bill]

Компилю проект в avr-gcc . Так вот проблема такая: не происходит возврата из функций - процессор зависает на следующей строчке программы после функции, при этом сама функция нормально вызывается и исполняется. Может кто-то с этим сталкивался, подскажите плиз.

Код в студию!

Сообщение отредактировал _Bill - Jun 27 2006, 12:40

[osminog]

Код

#include <include.h>
volatile BASIC_RF_SETTINGS rfSettings;

//  void basicRfInit(BASIC_RF_RX_INFO *pRRI, UINT8 channel, WORD panId, WORD myAddr)

void basicRfInit(BASIC_RF_RX_INFO *pRRI, UINT8 channel, WORD panId, WORD myAddr) {
    UINT8 n;

    // Make sure that the voltage regulator is on, and that the reset pin is inactive
    SET_VREG_ACTIVE();
    halWait(1000);
    SET_RESET_ACTIVE();
    halWait(1);
    SET_RESET_INACTIVE();
    halWait(5);

    // Initialize the FIFOP external interrupt
    FIFOP_INT_INIT();
    ENABLE_FIFOP_INT();

    // Turn off all interrupts while we're accessing the CC2420 registers
    DISABLE_GLOBAL_INT();
    // Register modifications
    FASTSPI_STROBE(CC2420_SXOSCON);
    FASTSPI_SETREG(CC2420_MDMCTRL0, 0x0AF2); // Turn on automatic packet acknowledgment
    FASTSPI_SETREG(CC2420_MDMCTRL1, 0x0500); // Set the correlation threshold = 20
    FASTSPI_SETREG(CC2420_IOCFG0, 0x007F);   // Set the FIFOP threshold to maximum
    FASTSPI_SETREG(CC2420_SECCTRL0, 0x01C4); // Turn off "Security enable"
    // Set the RF channel
    halRfSetChannel(channel);

    // Turn interrupts back on
    ENABLE_GLOBAL_INT();

    // Set the protocol configuration
    rfSettings.pRxInfo = pRRI;
    rfSettings.panId = panId;
    rfSettings.myAddr = myAddr;
    rfSettings.txSeqNumber = 0;
    rfSettings.receiveOn = FALSE;

    // Wait for the crystal oscillator to become stable
    halRfWaitForCrystalOscillator();

    // Write the short address and the PAN ID to the CC2420 RAM (requires that the XOSC is on and stable)
       DISABLE_GLOBAL_INT();
    FASTSPI_WRITE_RAM_LE(&myAddr, CC2420RAM_SHORTADDR, 2, n);
    FASTSPI_WRITE_RAM_LE(&panId, CC2420RAM_PANID, 2, n);
      ENABLE_GLOBAL_INT();

} // basicRfInit

Соответсвенно косяк происходит с функциями halWait(), halRfSetChannel(), ну и наверное со всеми функциями в проекте, я просто больше пока не проверил эти функции находятся в отдельных файлах, вот к примеру halWait():

Код

#include <include.h>


//-------------------------------------------------------------------------------------------------------
//    void halWait(UINT16 timeout)

void halWait(UINT16 timeout) {

    // This sequence uses exactly 8 clock cycle for each round
    do {
        NOP();
        NOP();
        NOP();
        NOP();
    } while (--timeout);

} // halWait

Кстати такой же косяк происходит со структурой rfSettings, виснет на операции присваивания, просто может я чето в Makefile не так пишу. Если надо Makefile приложу=). Со стеком по идее проблем быть не должно, функций не так много, если только в gcc это как то регулируется.

Сообщение отредактировал osminog - Jun 27 2006, 14:14

[beer_warrior]

Код классный, пожалуй проблема не в нем.
Действительно похоже на стек.
Возможно мэйкфайл(выкладывайте) .
Кстати, в gcc есть опция назначения глубины стека.
А вообще произвродит впечетление, что не для того кристалла собирается или RAMTOP стоит неправильный, или что-то накручено не то в стартапе.
Вы в отладчике прогоняли? И еще стоит взглянуть на листинг.
Извините, что сумбурно - мыслей сразу много

[osminog]

В отладчике пока не смотрел, понял в чем проблема исключительно в полевых условиях. Вот собственно Makefile:

Код

# MCU name
MCU = atmega128
# Output format. (can be srec, ihex, binary)
FORMAT = ihex
# Target file name (without extension).
TARGET = chipcon
# Optimization level, can be [0, 1, 2, 3, s]. 0 turns off optimization.
# (Note: 3 is not always the best optimization level. See avr-libc FAQ.)
OPT = s
# List C source files here. (C dependencies are automatically generated.)
SRC = $(TARGET).c
SRC += C:\Chipcon\lib\hal\atmega128\hal_wait.c
SRC += C:\Chipcon\lib\hal\hal_rf_set_channel.c
SRC += C:\Chipcon\lib\hal\hal_rf_wait_for_crystal_oscillator.c
SRC += C:\Chipcon\lib\basic_rf\basic_rf_init.c
SRC += C:\Chipcon\lib\basic_rf\basic_rf_send_packet.c
SRC += C:\Chipcon\lib\basic_rf\basic_rf_receive.c
# List Assembler source files here.
# Make them always end in a capital .S.  Files ending in a lowercase .s
# will not be considered source files but generated files (assembler
# output from the compiler), and will be deleted upon "make clean"!
# Even though the DOS/Win* filesystem matches both .s and .S the same,
# it will preserve the spelling of the filenames, and gcc itself does
# care about how the name is spelled on its command-line.
ASRC =
# List any extra directories to look for include files here.
#     Each directory must be seperated by a space.
EXTRAINCDIRS = C:\Chipcon\include
# Optional compiler flags.
#  -g:        generate debugging information (for GDB, or for COFF conversion)
#  -O*:       optimization level
#  -f...:     tuning, see gcc manual and avr-libc documentation
#  -Wall...:  warning level
#  -Wa,...:   tell GCC to pass this to the assembler.
#    -ahlms:  create assembler listing
CFLAGS = -g -O$(OPT) \
-funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums \
-Wall \
-Wa,-adhlns=$(<:.c=.lst) \
$(patsubst %,-I%,$(EXTRAINCDIRS))
# Set a "language standard" compiler flag.
#   Unremark just one line below to set the language standard to use.
#   gnu99 = C99 + GNU extensions. See GCC manual for more information.
#CFLAGS += -std=c89
#CFLAGS += -std=gnu89
#CFLAGS += -std=c99
CFLAGS += -std=gnu99
# Optional assembler flags.
#  -Wa,...:   tell GCC to pass this to the assembler.
#  -ahlms:    create listing
#  -gstabs:   have the assembler create line number information; note that
#             for use in COFF files, additional information about filenames
#             and function names needs to be present in the assembler source
#             files -- see avr-libc docs [FIXME: not yet described there]
ASFLAGS = -Wa,-adhlns=$(<:.S=.lst),-gstabs
# Optional linker flags.
#  -Wl,...:   tell GCC to pass this to linker.
#  -Map:      create map file
#  --cref:    add cross reference to  map file
LDFLAGS = -Wl,-Map=$(TARGET).map,--cref
# Additional libraries
# Minimalistic printf version
#LDFLAGS += -Wl,-u,vfprintf -lprintf_min
# Floating point printf version (requires -lm below)
#LDFLAGS += -Wl,-u,vfprintf -lprintf_flt
# -lm = math library
LDFLAGS += -lm
# Programming support using avrdude. Settings and variables.
# Programming hardware: alf avr910 avrisp bascom bsd
# dt006 pavr picoweb pony-stk200 sp12 stk200 stk500
#
# Type: avrdude -c ?
# to get a full listing.
#
AVRDUDE_PROGRAMMER = stk500
AVRDUDE_PORT = com1       # programmer connected to serial device
#AVRDUDE_PORT = lpt1    # programmer connected to parallel port
AVRDUDE_WRITE_FLASH = -U flash:w:$(TARGET).hex
#AVRDUDE_WRITE_EEPROM = -U eeprom:w:$(TARGET).eep
AVRDUDE_FLAGS = -p $(MCU) -P $(AVRDUDE_PORT) -c $(AVRDUDE_PROGRAMMER)
# Uncomment the following if you want avrdude's erase cycle counter.
# Note that this counter needs to be initialized first using -Yn,
# see avrdude manual.
#AVRDUDE_ERASE += -y
# Uncomment the following if you do /not/ wish a verification to be
# performed after programming the device.
#AVRDUDE_FLAGS += -V
# Increase verbosity level.  Please use this when submitting bug
# reports about avrdude. See <http://savannah.nongnu.org/projects/avrdude>
# to submit bug reports.
#AVRDUDE_FLAGS += -v -v
# Define directories, if needed.
DIRAVR = c:/winavr
DIRAVRBIN = $(DIRAVR)/bin
DIRAVRUTILS = $(DIRAVR)/utils/bin
DIRINC = .
DIRLIB = $(DIRAVR)/avr/lib
# Define programs and commands.
SHELL = sh
CC = avr-gcc
OBJCOPY = avr-objcopy
OBJDUMP = avr-objdump
SIZE = avr-size
# Programming support using avrdude.
AVRDUDE = avrdude
REMOVE = rm -f
COPY = cp
HEXSIZE = $(SIZE) --target=$(FORMAT) $(TARGET).hex
ELFSIZE = $(SIZE) -A $(TARGET).elf
# Define Messages
# English
MSG_ERRORS_NONE = Errors: none
MSG_BEGIN = -------- begin --------
MSG_END = --------  end  --------
MSG_SIZE_BEFORE = Size before:
MSG_SIZE_AFTER = Size after:
MSG_COFF = Converting to AVR COFF:
MSG_EXTENDED_COFF = Converting to AVR Extended COFF:
MSG_FLASH = Creating load file for Flash:
MSG_EEPROM = Creating load file for EEPROM:
MSG_EXTENDED_LISTING = Creating Extended Listing:
MSG_SYMBOL_TABLE = Creating Symbol Table:
MSG_LINKING = Linking:
MSG_COMPILING = Compiling:
MSG_ASSEMBLING = Assembling:
MSG_CLEANING = Cleaning project:
TARGET_PLATFORM  = -DCC2420DB
# Define all object files.
OBJ = $(SRC:.c=.o)
# Define all listing files.
LST = $(ASRC:.S=.lst) $(SRC:.c=.lst)
# Combine all necessary flags and optional flags.
# Add target processor to flags.
ALL_CFLAGS = -mmcu=$(MCU) -I. $(CFLAGS) $(TARGET_PLATFORM)
ALL_ASFLAGS = -mmcu=$(MCU) -I. -x assembler-with-cpp $(ASFLAGS)
# Default target.
all: begin gccversion sizebefore $(TARGET).elf $(TARGET).hex $(TARGET).eep \
    $(TARGET).lss $(TARGET).sym sizeafter finished end
# Eye candy.
# AVR Studio 3.x does not check make's exit code but relies on
# the following magic strings to be generated by the compile job.
begin:
    @echo
    @echo $(MSG_BEGIN)

finished:
    @echo $(MSG_ERRORS_NONE)

end:
    @echo $(MSG_END)
    @echo

# Display size of file.
sizebefore:
    @if [ -f $(TARGET).elf ]; then echo; echo $(MSG_SIZE_BEFORE); $(ELFSIZE); echo; fi

sizeafter:
    @if [ -f $(TARGET).elf ]; then echo; echo $(MSG_SIZE_AFTER); $(ELFSIZE); echo; fi

# Display compiler version information.
gccversion :
    @$(CC) --version


# Convert ELF to COFF for use in debugging / simulating in
# AVR Studio or VMLAB.
COFFCONVERT=$(OBJCOPY) --debugging \
    --change-section-address .data-0x800000 \
    --change-section-address .bss-0x800000 \
    --change-section-address .noinit-0x800000 \
    --change-section-address .eeprom-0x810000
coff: $(TARGET).elf
    @echo
    @echo $(MSG_COFF) $(TARGET).cof
    $(COFFCONVERT) -O coff-avr $< $(TARGET).cof
extcoff: $(TARGET).elf
    @echo
    @echo $(MSG_EXTENDED_COFF) $(TARGET).cof
    $(COFFCONVERT) -O coff-ext-avr $< $(TARGET).cof
# Program the device.  
program: $(TARGET).hex $(TARGET).eep
    $(AVRDUDE) $(AVRDUDE_FLAGS) $(AVRDUDE_WRITE_FLASH) $(AVRDUDE_WRITE_EEPROM)
# Create final output files (.hex, .eep) from ELF output file.
%.hex: %.elf
    @echo
    @echo $(MSG_FLASH) $@
    $(OBJCOPY) -O $(FORMAT) -R .eeprom $< $@

%.eep: %.elf
    @echo
    @echo $(MSG_EEPROM) $@
    -$(OBJCOPY) -j .eeprom --set-section-flags=.eeprom="alloc,load" \
    --change-section-lma .eeprom=0 -O $(FORMAT) $< $@

# Create extended listing file from ELF output file.
%.lss: %.elf
    @echo
    @echo $(MSG_EXTENDED_LISTING) $@
    $(OBJDUMP) -h -S $< > $@

# Create a symbol table from ELF output file.
%.sym: %.elf
    @echo
    @echo $(MSG_SYMBOL_TABLE) $@
    avr-nm -n $< > $@
# Link: create ELF output file from object files.
.SECONDARY : $(TARGET).elf
.PRECIOUS : $(OBJ)
%.elf: $(OBJ)
    @echo
    @echo $(MSG_LINKING) $@
    $(CC) $(ALL_CFLAGS) $(OBJ) --output $@ $(LDFLAGS)
# Compile: create object files from C source files.
%.o : %.c
    @echo
    @echo $(MSG_COMPILING) $<
    $(CC) -c $(ALL_CFLAGS) $< -o $@
# Compile: create assembler files from C source files.
%.s : %.c
    $(CC) -S $(ALL_CFLAGS) $< -o $@
# Assemble: create object files from assembler source files.
%.o : %.S
    @echo
    @echo $(MSG_ASSEMBLING) $<
    $(CC) -c $(ALL_ASFLAGS) $< -o $@
# Target: clean project.
clean: begin clean_list finished end

clean_list :
    @echo
    @echo $(MSG_CLEANING)
    $(REMOVE) $(TARGET).hex
    $(REMOVE) $(TARGET).eep
    $(REMOVE) $(TARGET).obj
    $(REMOVE) $(TARGET).cof
    $(REMOVE) $(TARGET).elf
    $(REMOVE) $(TARGET).map
    $(REMOVE) $(TARGET).obj
    $(REMOVE) $(TARGET).a90
    $(REMOVE) $(TARGET).sym
    $(REMOVE) $(TARGET).lnk
    $(REMOVE) $(TARGET).lss
    $(REMOVE) $(OBJ)
    $(REMOVE) $(LST)
    $(REMOVE) $(SRC:.c=.s)
    $(REMOVE) $(SRC:.c=.d)
# Automatically generate C source code dependencies.
# (Code originally taken from the GNU make user manual and modified
# (See README.txt Credits).)
#
# Note that this will work with sh (bash) and sed that is shipped with WinAVR
# (see the SHELL variable defined above).
# This may not work with other shells or other seds.
#
%.d: %.c
    set -e; $(CC) -MM $(ALL_CFLAGS) $< \
    | sed 's,\(.*\)\.o[ :]*,\1.o \1.d : ,g' > $@; \
    [ -s $@ ] || rm -f $@


# Remove the '-' if you want to see the dependency files generated.
-include $(SRC:.c=.d)


# Listing of phony targets.
.PHONY : all begin finish end sizebefore sizeafter gccversion coff extcoff \
    clean clean_list program

[beer_warrior]

На первый взгляд все в порядке. В отладчик его

[aesok]

> MCU = atmega128

Какое значение у бита M103C в Extended Fuse Byte?

[osminog]

> MCU = atmega128

Какое значение у бита M103C в Extended Fuse Byte?

Правильное вроде бы-0.

[Harbour]

Если стек в ROM попадает, то оно так и будет работать - нужно оbjdump'ом глянуть карту elf файла - похоже что с расположением секций намудрили.

[osminog]

Если стек в ROM попадает, то оно так и будет работать - нужно оbjdump'ом глянуть карту elf файла - похоже что с расположением секций намудрили.

Подскажите пожалуйста с каким ключом смотреть эту карту, и что вобще она из себя представляет, и как понять, что в ней что-то не так?

[3.14]

> MCU = atmega128

Какое значение у бита M103C в Extended Fuse Byte?

Правильное вроде бы-0.

Дык 0 этож как раз он и будет прикидываться M103, хотя насколько я понимаю, в Вашем случае это не должно иметь значения.

[aesok]

> MCU = atmega128

Какое значение у бита M103C в Extended Fuse Byte?

Правильное вроде бы-0.

Запишите туда "1". Отключите режим совместимости с ATMega103. У 128/103 RAM по paзным адресам.

[Harbour]

Если стек в ROM попадает, то оно так и будет работать - нужно оbjdump'ом глянуть карту elf файла - похоже что с расположением секций намудрили.

Подскажите пожалуйста с каким ключом смотреть эту карту, и что вобще она из себя представляет, и как понять, что в ней что-то не так?

Если не будем читать доки - все будет плохо в этой жизни. objdump -h <elf>. Так как у Вас прога во внещнем озу - неплохо бы тестик для начала написать и результат как-то индицировать, а вдруг микросхему не припаяли ? Вобщем произведите ряд лабораторных работ - для освоения оно весьма полезно будет.

[osminog]

Если не будем читать доки - все будет плохо в этой жизни. objdump -h <elf>. Так как у Вас прога во внещнем озу - неплохо бы тестик для начала написать и результат как-то индицировать, а вдруг микросхему не припаяли ? Вобщем произведите ряд лабораторных работ - для освоения оно весьма полезно будет.

А у меня программа во внутреннем ОЗУ. В Makefile что-то указывает на внешнее ОЗУ? Я просто первый раз с gcc работаю, много чего не понимаю.

[vet]

AVR в принципе не умеет выполнять код из ОЗУ.
osminog, снятие fuse-бита совместимости с мегой103, скорее всего, поможет.

[osminog]

Fuse bit поставил равным единице. Не заработало=(.

Код

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .data         00000000  00800100  000008ea  0000097e  2**0
                  CONTENTS, ALLOC, LOAD, DATA
  1 .text         000008ea  00000000  00000000  00000094  2**0
                  CONTENTS, ALLOC, LOAD, READONLY, CODE
  2 .bss          00000106  00800100  00800100  0000097e  2**0
                  ALLOC
  3 .noinit       00000000  00800206  00800206  0000097e  2**0
                  CONTENTS
  4 .eeprom       00000000  00810000  00810000  0000097e  2**0
                  CONTENTS
  5 .stab         00000408  00000000  00000000  00000980  2**2
                  CONTENTS, READONLY, DEBUGGING
  6 .stabstr      00000084  00000000  00000000  00000d88  2**0
                  CONTENTS, READONLY, DEBUGGING
  7 .debug_aranges 0000008c  00000000  00000000  00000e0c  2**0
                  CONTENTS, READONLY, DEBUGGING
  8 .debug_pubnames 0000019f  00000000  00000000  00000e98  2**0
                  CONTENTS, READONLY, DEBUGGING
  9 .debug_info   00000ec0  00000000  00000000  00001037  2**0
                  CONTENTS, READONLY, DEBUGGING
10 .debug_abbrev 000004b9  00000000  00000000  00001ef7  2**0
                  CONTENTS, READONLY, DEBUGGING
11 .debug_line   00000c78  00000000  00000000  000023b0  2**0
                  CONTENTS, READONLY, DEBUGGING
12 .debug_str    000003cd  00000000  00000000  00003028  2**0
                  CONTENTS, READONLY, DEBUGGING

вот что выводит objdump.

Сообщение отредактировал osminog - Jun 28 2006, 07:46

[Igor_U]

void basicRfInit(BASIC_RF_RX_INFO *pRRI, UINT8 channel, WORD panId, WORD myAddr)

А gcc умеет передавать 4 параметра? В ИАРе насколько я помню максимально можно было передать 3 параметра.

[_Bill]

void basicRfInit(BASIC_RF_RX_INFO *pRRI, UINT8 channel, WORD panId, WORD myAddr)

А gcc умеет передавать 4 параметра? В ИАРе насколько я помню максимально можно было передать 3 параметра.

С чего Вы это взяли? Только через регистры можно передать до 8 параметров, а про стек я уже не говорю.

[Harbour]

AVR в принципе не умеет выполнять код из ОЗУ.
osminog, снятие fuse-бита совместимости с мегой103, скорее всего, поможет.

А какая ему разница, program memory выбирается по адресам, которые для данного случая не перекрываются, просто нужно сделать alloc нужной text section во внешнее озу, загрузчик при ресете его туды перенесет. В большинстве случаев это не имеет смысла - этот же код будет хранится и во флеше, но для оверлеев, которые хранятся где-то на SD/MMC можно применить.

void basicRfInit(BASIC_RF_RX_INFO *pRRI, UINT8 channel, WORD panId, WORD myAddr)

А gcc умеет передавать 4 параметра? В ИАРе насколько я помню максимально можно было передать 3 параметра.

Да хоть 404.

[Harbour]

Fuse bit поставил равным единице. Не заработало=(.

Код

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .data         00000000  00800100  000008ea  0000097e  2**0
                  CONTENTS, ALLOC, LOAD, DATA
  1 .text         000008ea  00000000  00000000  00000094  2**0
                  CONTENTS, ALLOC, LOAD, READONLY, CODE
  2 .bss          00000106  00800100  00800100  0000097e  2**0
                  ALLOC
  3 .noinit       00000000  00800206  00800206  0000097e  2**0
                  CONTENTS
  4 .eeprom       00000000  00810000  00810000  0000097e  2**0
                  CONTENTS
  5 .stab         00000408  00000000  00000000  00000980  2**2
                  CONTENTS, READONLY, DEBUGGING
  6 .stabstr      00000084  00000000  00000000  00000d88  2**0
                  CONTENTS, READONLY, DEBUGGING
  7 .debug_aranges 0000008c  00000000  00000000  00000e0c  2**0
                  CONTENTS, READONLY, DEBUGGING
  8 .debug_pubnames 0000019f  00000000  00000000  00000e98  2**0
                  CONTENTS, READONLY, DEBUGGING
  9 .debug_info   00000ec0  00000000  00000000  00001037  2**0
                  CONTENTS, READONLY, DEBUGGING
10 .debug_abbrev 000004b9  00000000  00000000  00001ef7  2**0
                  CONTENTS, READONLY, DEBUGGING
11 .debug_line   00000c78  00000000  00000000  000023b0  2**0
                  CONTENTS, READONLY, DEBUGGING
12 .debug_str    000003cd  00000000  00000000  00003028  2**0
                  CONTENTS, READONLY, DEBUGGING

вот что выводит objdump.

Побробуйте добавить к опциям линкера что-то типа -Wl,-Tdata=0x801100,--defsym=__heap_end=0x80ffff