#c #gcc #makefile #compilation #avr
#c #gcc #makefile #Сборник #avr
Вопрос:
У меня есть следующий 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
порядок, и раньше это, казалось, имело значение, но не в этот раз. Я чувствую, что это неправильно, потому что как бы кто-то со многими исходными файлами справился с этим, скажем, для большого проекта? Должен быть лучший способ.
Комментарии:
1. Что такое
One
и где это определено?2.
➭ grep -rn "One" bb_system.h:25:extern struct bb_system One; Sys_init.c:7://struct bb_system One; Sys_init.c:23: BB_Struct_init(amp;One); Binary file Sys_init.o matches BB_Timers.c:31: One.sys_count ; //this will provide the count for taks BB_Timers.c:37: return (One.sys_count ;); Binary file BB_Timers.o matches
3. Так
One
на самом деле нигде не определено , просто объявлено и используется? Кажется, что единственное определение закомментировано.4. Похоже, я определил это здесь, верно?
bb_system.h:25:extern struct bb_system One;
5. @BrocolliRob Нет, это декларация . В
Sys_init.c
строке 7 у вас есть закомментированное определение.
Ответ №1:
Итак, подводя итог комментариям по вопросу:
В bb_system.h
строке 25 вы помещаете это утверждение:
extern struct bb_system One;
Это означает, что где-то в ваших скомпилированных файлах должна быть переменная с именем One
типа struct bb_system
И в Sys_init.c
строке 7 вы прокомментировали утверждение
// struct bb_system One;
Что, вероятно, было бы решено.
Исправьте эту строку