Elektroda.pl
Elektroda.pl
X
Please add exception to AdBlock for elektroda.pl.
If you watch the ads, you support portal and users.

AVR Studio 4 i błąd kompilacji

Zlata_n 01 Dec 2011 20:35 3881 14
  • #1
    Zlata_n
    Level 14  
    Witam, mam problem z kompilacją programu.(pliki ogolnodostępne dla projektu LED CUBE 4x4x4),

    :arrow: http://www.instructables.com/id/LED-Cube-4x4x4/]

    Posiadam pliki main.c frames.c effect.c draw.c oraz plik Makefile.
    Odpalam AVR Studio, ustawiam procesor ATmega 16, ustawiam zegar 14745600 bo taki jest uzyty w programie, laduje wszystkie pliki (procz MakeFile bo nawet nie wiem gdzie go wczytac) i uruchamiam Build Project aby stworzyc nowego MakeFile oraz HEX.'a (zrobilem zmiany w kodzie i chce sprawdzic czy poprawnie działają)

    niestety wywala mi bład którego nie rozumiem :

    Code:
    Build started 1.12.2011 at 20:33:11
    
    MAKE Version 5.2  Copyright (c) 1987, 2000 Borland
    Error Makefile 80: Command syntax error
    *** 1 errors during make ***
    Build failed with 1 errors and 0 warnings...


    Makefile wygląda tak:
    Code:

    # Hey Emacs, this is a -*- makefile -*-
    #----------------------------------------------------------------------------
    # WinAVR Makefile Template written by Eric B. Weddington, J�rg Wunsch, et al.
    #
    # Released to the Public Domain
    #
    # Additional material for this makefile was written by:
    # Peter Fleury
    # Tim Henigan
    # Colin O'Flynn
    # Reiner Patommel
    # Markus Pfaff
    # Sander Pool
    # Frederik Rouleau
    #
    #----------------------------------------------------------------------------
    # On command line:
    #
    # make all = Make software.
    #
    # make clean = Clean out built project files.
    #
    # make coff = Convert ELF to AVR COFF.
    #
    # make extcoff = Convert ELF to AVR Extended COFF.
    #
    # make program = Download the hex file to the device, using avrdude.
    #                Please customize the avrdude settings below first!
    #
    # make debug = Start either simulavr or avarice as specified for debugging,
    #              with avr-gdb or avr-insight as the front end for debugging.
    #
    # make filename.s = Just compile filename.c into the assembler code only.
    #
    # make filename.i = Create a preprocessed source file for use in submitting
    #                   bug reports to the GCC project.
    #
    # To rebuild project do "make clean" then "make all".
    #----------------------------------------------------------------------------


    # MCU name
    MCU = atmega16


    # Processor frequency.
    #     This will define a symbol, F_CPU, in all source code files equal to the
    #     processor frequency. You can then use this symbol in your source code to
    #     calculate timings. Do NOT tack on a 'UL' at the end, this will be done
    #     automatically to create a 32-bit value in your source code.
    F_CPU = 14745600

    # Output format. (can be srec, ihex, binary)
    FORMAT = ihex


    # Target file name (without extension).
    TARGET = main


    # List C source files here. (C dependencies are automatically generated.)
    SRC = $(TARGET).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 =


    # Optimization level, can be [0, 1, 2, 3, s].
    #     0 = turn off optimization. s = optimize for size.
    #     (Note: 3 is not always the best optimization level. See avr-libc FAQ.)
    OPT = s


    # Debugging format.
    #     Native formats for AVR-GCC's -g are dwarf-2 [default] or stabs.
    #     AVR Studio 4.10 requires dwarf-2.
    #     AVR [Extended] COFF format requires stabs, plus an avr-objcopy run.
    DEBUG = dwarf-2


    # List any extra directories to look for include files here.
    #     Each directory must be seperated by a space.
    #     Use forward slashes for directory separators.
    #     For a directory that has spaces, enclose it in quotes.
    EXTRAINCDIRS =


    # Compiler flag to set the C Standard level.
    #     c89   = "ANSI" C
    #     gnu89 = c89 plus GCC extensions
    #     c99   = ISO C99 standard (not yet fully implemented)
    #     gnu99 = c99 plus GCC extensions
    CSTANDARD = -std=gnu99


    # Place -D or -U options here
    CDEFS = -DF_CPU=$(F_CPU)UL


    # Place -I options here
    CINCS =



    #---------------- Compiler Options ----------------
    #  -g*:          generate debugging information
    #  -O*:          optimization level
    #  -f...:        tuning, see GCC manual and avr-libc documentation
    #  -Wall...:     warning level
    #  -Wa,...:      tell GCC to pass this to the assembler.
    #    -adhlns...: create assembler listing
    CFLAGS = -g$(DEBUG)
    CFLAGS += $(CDEFS) $(CINCS)
    CFLAGS += -O$(OPT)
    CFLAGS += -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums
    CFLAGS += -Wall -Wstrict-prototypes
    CFLAGS += -Wa,-adhlns=$(<:.c=.lst)
    CFLAGS += $(patsubst %,-I%,$(EXTRAINCDIRS))
    CFLAGS += $(CSTANDARD)


    #---------------- Assembler Options ----------------
    #  -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


    #---------------- Library Options ----------------
    # Minimalistic printf version
    PRINTF_LIB_MIN = -Wl,-u,vfprintf -lprintf_min

    # Floating point printf version (requires MATH_LIB = -lm below)
    PRINTF_LIB_FLOAT = -Wl,-u,vfprintf -lprintf_flt

    # If this is left blank, then it will use the Standard printf version.
    PRINTF_LIB =
    #PRINTF_LIB = $(PRINTF_LIB_MIN)
    #PRINTF_LIB = $(PRINTF_LIB_FLOAT)


    # Minimalistic scanf version
    SCANF_LIB_MIN = -Wl,-u,vfscanf -lscanf_min

    # Floating point + %[ scanf version (requires MATH_LIB = -lm below)
    SCANF_LIB_FLOAT = -Wl,-u,vfscanf -lscanf_flt

    # If this is left blank, then it will use the Standard scanf version.
    SCANF_LIB =
    #SCANF_LIB = $(SCANF_LIB_MIN)
    #SCANF_LIB = $(SCANF_LIB_FLOAT)


    MATH_LIB = -lm



    #---------------- External Memory Options ----------------

    # 64 KB of external RAM, starting after internal RAM (ATmega128!),
    # used for variables (.data/.bss) and heap (malloc()).
    #EXTMEMOPTS = -Wl,-Tdata=0x801100,--defsym=__heap_end=0x80ffff

    # 64 KB of external RAM, starting after internal RAM (ATmega128!),
    # only used for heap (malloc()).
    #EXTMEMOPTS = -Wl,--defsym=__heap_start=0x801100,--defsym=__heap_end=0x80ffff

    EXTMEMOPTS =



    #---------------- Linker Options ----------------
    #  -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
    LDFLAGS += $(EXTMEMOPTS)
    LDFLAGS += $(PRINTF_LIB) $(SCANF_LIB) $(MATH_LIB)



    #---------------- Programming Options (avrdude) ----------------

    # 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 = usbtiny
    #AVRDUDE_PROGRAMMER = ponyser

    # com1 = serial port. Use lpt1 to connect to parallel port.
    #AVRDUDE_PORT = lpt1
    #AVRDUDE_PORT = COM1

    AVRDUDE_WRITE_FLASH = -U flash:w:$(TARGET).hex
    #AVRDUDE_WRITE_EEPROM = -U eeprom:w:$(TARGET).eep


    # 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_COUNTER = -y

    # Uncomment the following if you do /not/ wish a verification to be
    # performed after programming the device.
    #AVRDUDE_NO_VERIFY = -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_VERBOSE = -v -v

    AVRDUDE_FLAGS = -p $(MCU) -P $(AVRDUDE_PORT) -c $(AVRDUDE_PROGRAMMER)
    AVRDUDE_FLAGS += $(AVRDUDE_NO_VERIFY)
    AVRDUDE_FLAGS += $(AVRDUDE_VERBOSE)
    AVRDUDE_FLAGS += $(AVRDUDE_ERASE_COUNTER)



    #---------------- Debugging Options ----------------

    # For simulavr only - target MCU frequency.
    DEBUG_MFREQ = $(F_CPU)

    # Set the DEBUG_UI to either gdb or insight.
    # DEBUG_UI = gdb
    DEBUG_UI = insight



    # Set the debugging back-end to either avarice, simulavr.
    DEBUG_BACKEND = avarice
    #DEBUG_BACKEND = simulavr

    # GDB Init Filename.
    GDBINIT_FILE = __avr_gdbinit

    # When using avarice settings for the JTAG
    JTAG_DEV = /dev/com1

    # Debugging port used to communicate between GDB / avarice / simulavr.
    DEBUG_PORT = 4242

    # Debugging host used to communicate between GDB / avarice / simulavr, normally
    #     just set to localhost unless doing some sort of crazy debugging when
    #     avarice is running on a different computer.
    DEBUG_HOST = localhost



    #============================================================================


    # Define programs and commands.
    SHELL = sh
    CC = avr-gcc
    OBJCOPY = avr-objcopy
    OBJDUMP = avr-objdump
    SIZE = avr-size
    NM = avr-nm
    AVRDUDE = avrdude
    REMOVE = rm -f
    COPY = cp
    WINSHELL = cmd


    # 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:




    # Define all object files.
    OBJ = $(SRC:.c=.o) $(ASRC:.S=.o)

    # Define all listing files.
    LST = $(SRC:.c=.lst) $(ASRC:.S=.lst)


    # Compiler flags to generate dependency files.
    GENDEPFLAGS = -MD -MP -MF .dep/$(@F).d


    # Combine all necessary flags and optional flags.
    # Add target processor to flags.
    ALL_CFLAGS = -mmcu=$(MCU) -I. $(CFLAGS) $(GENDEPFLAGS)
    ALL_ASFLAGS = -mmcu=$(MCU) -I. -x assembler-with-cpp $(ASFLAGS)





    # Default target.
    all: begin gccversion sizebefore build sizeafter end

    build: elf hex eep lss sym

    elf: $(TARGET).elf
    hex: $(TARGET).hex
    eep: $(TARGET).eep
    lss: $(TARGET).lss
    sym: $(TARGET).sym



    # 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)

    end:
       @echo $(MSG_END)
       @echo


    # Display size of file.
    HEXSIZE = $(SIZE) --target=$(FORMAT) $(TARGET).hex
    ELFSIZE = $(SIZE) -A $(TARGET).elf
    AVRMEM = avr-mem.sh $(TARGET).elf $(MCU)

    sizebefore:
       @if test -f $(TARGET).elf; then echo; echo $(MSG_SIZE_BEFORE); $(ELFSIZE); \
       $(AVRMEM) 2>/dev/null; echo; fi

    sizeafter:
       @if test -f $(TARGET).elf; then echo; echo $(MSG_SIZE_AFTER); $(ELFSIZE); \
       $(AVRMEM) 2>/dev/null; echo; fi



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



    # Program the device. 
    program: $(TARGET).hex $(TARGET).eep
       $(AVRDUDE) $(AVRDUDE_FLAGS) $(AVRDUDE_WRITE_FLASH) $(AVRDUDE_WRITE_EEPROM)


    # Generate avr-gdb config/init file which does the following:
    #     define the reset signal, load the target file, connect to target, and set
    #     a breakpoint at main().
    gdb-config:
       @$(REMOVE) $(GDBINIT_FILE)
       @echo define reset >> $(GDBINIT_FILE)
       @echo SIGNAL SIGHUP >> $(GDBINIT_FILE)
       @echo end >> $(GDBINIT_FILE)
       @echo file $(TARGET).elf >> $(GDBINIT_FILE)
       @echo target remote $(DEBUG_HOST):$(DEBUG_PORT)  >> $(GDBINIT_FILE)
    ifeq ($(DEBUG_BACKEND),simulavr)
       @echo load  >> $(GDBINIT_FILE)
    endif   
       @echo break main >> $(GDBINIT_FILE)
       
    debug: gdb-config $(TARGET).elf
    ifeq ($(DEBUG_BACKEND), avarice)
       @echo Starting AVaRICE - Press enter when "waiting to connect" message displays.
       @$(WINSHELL) /c start avarice --jtag $(JTAG_DEV) --erase --program --file \
       $(TARGET).elf $(DEBUG_HOST):$(DEBUG_PORT)
       @$(WINSHELL) /c pause
       
    else
       @$(WINSHELL) /c start simulavr --gdbserver --device $(MCU) --clock-freq \
       $(DEBUG_MFREQ) --port $(DEBUG_PORT)
    endif
       @$(WINSHELL) /c start avr-$(DEBUG_UI) --command=$(GDBINIT_FILE)
       



    # 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



    # 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) $@
       $(NM) -n $< > $@



    # Link: create ELF output file from object files.
    .SECONDARY : $(TARGET).elf
    .PRECIOUS : $(OBJ)
    %.elf: $(OBJ)
       @echo
       @echo $(MSG_LINKING) $@
       $(CC) $(ALL_CFLAGS) $^ --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 $@

    # Create preprocessed source for use in sending a bug report.
    %.i : %.c
       $(CC) -E -mmcu=$(MCU) -I. $(CFLAGS) $< -o $@


    # Target: clean project.
    clean: begin clean_list end

    clean_list :
       @echo
       @echo $(MSG_CLEANING)
       $(REMOVE) $(TARGET).hex
       $(REMOVE) $(TARGET).eep
       $(REMOVE) $(TARGET).cof
       $(REMOVE) $(TARGET).elf
       $(REMOVE) $(TARGET).map
       $(REMOVE) $(TARGET).sym
       $(REMOVE) $(TARGET).lss
       $(REMOVE) $(OBJ)
       $(REMOVE) $(LST)
       $(REMOVE) $(SRC:.c=.s)
       $(REMOVE) $(SRC:.c=.d)
       $(REMOVE) .dep/*



    # Include the dependency files.
    -include $(shell mkdir .dep 2>/dev/null) $(wildcard .dep/*)


    # Listing of phony targets.
    .PHONY : all begin finish end sizebefore sizeafter gccversion \
    build elf hex eep lss sym coff extcoff \
    clean clean_list program debug gdb-config
  • #2
    tmf
    Moderator of Microcontroller designs
    Używasz programu make nie z pakietu WinAVR tylko jakiego pakietu Borlanda - tak to nie zadziała. Sprawdź ścieżki do make, albo zmień jego nazwę np. na avr-make i tak to wywołuj.
  • #3
    Zlata_n
    Level 14  
    nie wiedzialem jak to zmienić, więc odinstalowalem starą wersję WinAVR i zainstalowalem najnowszą 20100110 i teraz mam inny błąd:

    Code:
    Build started 1.12.2011 at 23:31:27
    
    mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99   -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT main.o -MF dep/main.o.d  -c  ../main.c
    /usr/bin/sh: -Wall: command not found
    make: [main.o] Error 127 (ignored)
    mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99   -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT frames.o -MF dep/frames.o.d  -c  ../frames.c
    /usr/bin/sh: -Wall: command not found
    make: [frames.o] Error 127 (ignored)
    mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99   -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT effect.o -MF dep/effect.o.d  -c  ../effect.c
    /usr/bin/sh: -Wall: command not found
    make: [effect.o] Error 127 (ignored)
    mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99   -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT draw.o -MF dep/draw.o.d  -c  ../draw.c
    /usr/bin/sh: -Wall: command not found
    make: [draw.o] Error 127 (ignored)
    mmcu=atmega16 -Wl,-Map=main.map main.o frames.o effect.o draw.o     -o main.elf
    /usr/bin/sh: -Wl,-Map=main.map: command not found
    make: [main.elf] Error 127 (ignored)
    avr-objcopy -O ihex -R .eeprom -R .fuse -R .lock -R .signature  main.elf main.hex
    avr-objcopy: 'main.elf': No such file
    make: *** [main.hex] Error 1
    Build failed with 1 errors and 0 warnings...
  • Helpful post
    #4
    tmf
    Moderator of Microcontroller designs
    Ustaw w opcjach projektu w AVR Studio ścieżkę dostępu do make i avr-gcc znajdujących się w katalogu instalacyjnym WinAVR.
  • #5
    Zlata_n
    Level 14  
    Tak też zrobiłem i oczywiście inny błąd:
    Code:

    avr-gcc  -mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99     -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT frames.o -MF dep/frames.o.d  -c  ../frames.c
    ../frames.c:12: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'PROGMEM'
    make: *** [frames.o] Error 1
    Build failed with 1 errors and 26 warnings...


    kod tego pliku:

    Code:

    const unsigned char spinning_line[6][2] PROGMEM =
    {
       { 0x84, 0x21 },
       { 0x0c, 0x30 },
       { 0x03, 0xc0 },
       { 0x12, 0x48 },
       { 0x22, 0x44 },
       { 0x44, 0x22 },
    };


    zaczynam dopiero zabawę z C++ na qC więc jeśli mogłby ktoś to wytłumaczyc, będę bardzo wdzięczny.
  • #6
    tmf
    Moderator of Microcontroller designs
    To nie jest C++ tylko zwykłe C. Teraz błąd wynika nie ze złego programu make, tylko ze złego kodu. Konkretnie w podanym przez ciebie kodzie nie powinno być ostatniego przecinka przed zamykającym nawiasem.
  • #7
    Zlata_n
    Level 14  
    dziękuję, pokazał się następny błąd, jednak naprawdę go nie widzę w funkcji, średniki są, nawiasy pozamykane a mimo to wywala:

    Code:
    In file included from ../main.c:80:
    
    ../effect.c: At top level:
    ../effect.c:12: error: expected ',' or ';' before 'void'
    ../effect.c: In function 'effect_rain':

    gdy klikne na error to podswietla mi pierwszą funkcję:

    kod funkcji:
    Code:
    ->>> void effect_spiral(int direction, int iterations, int delay);
    
    void effect_spinning_plane(int direction, int iterations, int delay);
    void effect_rain (int iterations, int delay, int hold, int speed);
    void sendvoxel_z (unsigned char x, unsigned char y, unsigned char z, int delay);
    void sendplane_rand_z (unsigned char z, int delay, int wait);
    void blinky(void);
    void flyplane (unsigned char plane,unsigned char direction, int delay);
    void loadbar(int delay);
    void random_1 (int iterations, int voxels, int delay);
    void random_2 (void);
    void random_filler (int iterations, int pixels, int delay, int state);
    void random_filler2 (int delay, int state);
    void planeflip (unsigned char axis1, int pos1, unsigned char axis2, int pos2, int delay);
    void flip_playback (int delay, unsigned char flip_x, unsigned char flip_y, unsigned char flip_z);
    void boingboing (uint16_t iterations, int delay, unsigned char mode, unsigned char drawmode);

    // Shows an animation of a spinning spiral
    void effect_spiral(int direction, int iterations, int delay)
    {
       int i;
       int z;      // cube level

       for (i=0;i<iterations;i++)
       {
          // Loop cube levels
          for (z=0;z<4;z++)
          {
             // Read the animation from program memory and put it in the cube buffer.
             // y=0 and y=2 is stored in the upper nibble of the byte, and must be bitshifted into place.
             cube[z][0] = (pgm_read_byte(&spinning_line[(i+z)%6][0]) >> 4);
             cube[z][1] = pgm_read_byte(&spinning_line[(i+z)%6][0]);
             cube[z][2] = (pgm_read_byte(&spinning_line[(i+z)%6][1]) >> 4);
             cube[z][3] = pgm_read_byte(&spinning_line[(i+z)%6][1]);
             // the +z makes the animation iterate one step for each plane
             // making it into a spiral
          }
          delay_ms(delay);
       }
    }
  • #8
    Brutus_gsm
    Level 25  
    Błąd leży gdzieś wcześniej. Brakuje średnika przed tą funkcją.
  • #9
    tmf
    Moderator of Microcontroller designs
    Błędy kompilatora często leżą wcześniej niż je sygnalizuje. Stąd musisz przeanalizować wcześniejszy kod. Druga sprawa - często błędy sygnalizowane prze kompilator są konsekwencją innego błędu, więc wcale nie musi chodzić o ten średnik, tylko np. o jakiś niezamknięty nawiast itd.
    Swoją droga jeśli w linii nr 80 masz dyrektywę include to jak sądzę cały kod jest nieźle pokopany i wymaga poważnego czyszczenia i poprawy.
    Weź też pod uwagę, że oryginalny makefile załącza biblioteki printf i scanf związane z obsługa typów float, czego domyślny makefile z AVR Studio nie robi. Czy jest to problemem dodatkowym to trudno ocenić nie znając programu.
    Ale może czas zacząć od podstaw, tak, żebyś zrozumiał co się z czego bierze a nie działał na ślepo?
  • #10
    Zlata_n
    Level 14  
    Sprawa wygląda tak że termin goni, chciałem po prostu dodać swoje funkcje wyświetlające lub zmienić już będące ale bazując na tych co są już napisane, czyli wprowadzam lekkie zmiany <-> kompiluje i patrze jak działa i tak krok po kroku....
    Nie znam się aż tak żeby pisać wszystko od nowa...Chociaż mam w planach zacząć naukę programowania mikroprocesorów ponieważ strasznie mnie wciągnęło, przeanalizować cały kod i napisać go od podstaw.

    Zdziwiło mnie to bo autor podał pliki kodu i gotowy plik HEX. sądzilem ze jak skompiluje program to stworze podobny plik HEX, jednak ku mojemu zdziwieniu jest dużo błędów...i tu zaczęły się schody....Mam podstawy języka C++ lecz nie na bazie mikroprocesorów.
  • #11
    Zlata_n
    Level 14  
    Sprawa się rozwiązała, ściągnąłem gotowy plik Makefile dołączony od autora projektu. I w ustawieniach projektu zaznaczyłem żeby "używał zewnetrznego Makefile", program się kompiluje i generuje plik HEX.

    Zastanawiam mnie to, dlaczego teraz nie widzi tego błędu z linii 80. o którym napisałem post wyżej oraz błądu który był spowodowany (średnikiem)

    Code:
     const unsigned char spinning_line[6][2] PROGMEM =
    
    {
       { 0x84, 0x21 },
       { 0x0c, 0x30 },
       { 0x03, 0xc0 },
       { 0x12, 0x48 },
       { 0x22, 0x44 },
       { 0x44, 0x22 },
    };


    też musiałem go dopisać żeby program się poprawnie skompilował.
    Pytanie w jakim języku autor pisał ten kod ? bo uczyli mnie że
    Code:
    ;
    nie daje się po funkcji chyba ze po klasie. Czy to możliwe ze autor sam sobie zdefiniował takie obejście w pliku Makefile ?
  • #12
    Brutus_gsm
    Level 25  
    Przecież to nie jest funkcja, tylko tablica w pamięci programu.
  • #13
    Zlata_n
    Level 14  
    Brutus_gsm wrote:
    Przecież to nie jest funkcja, tylko tablica w pamięci programu.


    Czyli zawsze jak uzywam dyrektywy Progmem to mam pisać
    Code:
    {};
  • Helpful post
    #14
    Brutus_gsm
    Level 25  
    PROGMEM to nic innego jak atrybut, który umieszcza zmienną/tablicę zmiennych w pamięci programu. Przy inicjowaniu tablic wielowymiarowych zawsze powinno się stosować zagłębianie klamer, czyli "każdy wiersz we własnych klamrach".

    Code: c
    Log in, to see the code


    lub
    Code: c
    Log in, to see the code


    To tylko kwestia czytelności.
  • #15
    Zlata_n
    Level 14  
    Ok, dzięki za pomoc, temat zamknięty ;)