Question

我有以下Makefile：

# Name: Makefile
# Author: <insert your name here>
# Copyright: <insert your copyright message here>
# License: <insert your license reference here>

# DEVICE ....... The AVR device you compile for
# CLOCK ........ Target AVR clock rate in Hertz
# OBJECTS ...... The object files created from your source files. This list is
#                usually the same as the list of source files with suffix ".o".
# PROGRAMMER ... Options to avrdude which define the hardware you use for
#                uploading to the AVR and the interface where this hardware
#                is connected.
# FUSES ........ Parameters for avrdude to flash the fuses appropriately.

CSRC       = main.c Sys_init.c BB_Timers.c SPI.c Radio_interface.c  
DEVICE     = atmega328p 
MCU        = m328p 
CLOCK      = 800000
PROGRAMMER = -c dragon_isp 
PORT       = -P usb 
#PROGRAMMER = -c /dev/buspirate 
OBJECTS    = main.o 
FUSES      = -U efuse:w:0xff:m -U lfuse:w:0xe2:m -U hfuse:w:0xde:m
OBJECTS    = $(CSRC:.c=.o) $(ASRC:.s=.o)


######################################################################
######################################################################

# Tune the lines below only if you know what you are doing:

AVRDUDE = avrdude $(PROGRAMMER) $(PORT) -p $(DEVICE) -vv
COMPILE = avr-gcc -Wall -Os -DF_CPU=$(CLOCK) -mmcu=$(DEVICE) # symbolic targets:
all:    main.hex

.c.o:
    $(COMPILE) -c $< -o $@

.S.o:
    $(COMPILE) -x assembler-with-cpp -c $< -o $@
# "-x assembler-with-cpp" should not be necessary since this is the default
# file type for the .S (with capital S) extension. However, upper case
# characters are not always preserved on Windows. To ensure WinAVR
# compatibility define the file type manually.

.c.s:
    $(COMPILE) -S $< -o $@

flash:  all
    $(AVRDUDE) -U flash:w:main.hex:i

fuse:
    $(AVRDUDE) $(FUSES)

install: flash fuse

# if you use a bootloader, change the command below appropriately:
load: all
    bootloadHID main.hex

clean:
    rm -f main.hex main.elf $(OBJECTS)

# file targets:
main.elf: $(OBJECTS)
    $(COMPILE) -o main.elf $(OBJECTS)

main.hex: main.elf
    rm -f main.hex
    avr-objcopy -j .text -j .data -O ihex main.elf main.hex
# If you have an EEPROM section, you must also create a hex file for the
# EEPROM and add it to the "flash" target.

# Targets for code debugging and analysis:
disasm: main.elf
    avr-objdump -d main.elf

cpp:
    $(COMPILE) -E main.c

当我使用make进行编译时，出现以下错误：

>> make    
avr-gcc -Wall -Os -DF_CPU=800000 -mmcu=atmega328p   -o main.elf main.o Sys_init.o BB_Timers.o SPI.o Radio_interface.o 
    Sys_init.o: In function `System_init':
    Sys_init.c:(.text+0x0): undefined reference to `One'
    Sys_init.c:(.text+0x2): undefined reference to `One'
    BB_Timers.o: In function `__vector_14':
    BB_Timers.c:(.text+0x2e): undefined reference to `One'
    BB_Timers.c:(.text+0x30): undefined reference to `One'
    collect2: error: ld returned 1 exit status
    make: *** [Makefile:66: main.elf] Error 1

我试过改变CSRC左右的顺序，过去似乎有所作为，但这次不是。我觉得这是错误的，因为有很多源文件的人如何处理这个问题，比如一个大项目？必须有更好的方法。

Answer 1

总结对这个问题的评论：

在第25行的bb_system.h中，您提出了以下声明：

extern struct bb_system One;

这意味着在编译文件中的某处必须有一个名为One struct bb_system的变量

在第7行的Sys_init.c中你注释掉了陈述

// struct bb_system One;

它可能已经解决了。

修正该行

AVR-GCC Makefile链接问题

1 个答案: