ATmega168 GPIO mit Zeigern

[Andreas L. (feuerlink)]

Hallo zusammen,

ich sitze zur Zeit dabei, meinen Rechner so einzurichten, dass ich uCs 
mit einem Arduino Uno als ISP flashen kann. Dazu nutze ich zum Testen 
einen Atmega168. avr-gcc und avrdude läuft. Noch nutze ich notepad++ und 
cmd (Win10) zum proggen, bei Zeiten werde ich eine IDE einrichten.

Typischer erster Versuch: LED blinken lassen, bzw. erst einmal 
einschalten. Mithilfe der avr/io.h funktioniert's auch prima, mit 
direkten Registerzugriffen über selbstdefinierte Zeiger aber nicht. Ich 
möchte aber gerne, dass das auch klappt.


Mein Code sieht wie folgt aus:

//#define __AVR_ATmega168A__

#include <stdint.h>

volatile uint8_t *DDRB_p = (uint8_t*)0x24;  //Pointer to DDRB:   Direction
volatile uint8_t *PORTB_p = (uint8_t*)0x25; //Pointer to PORRTB; DATA

int main(){

  *DDRB_p = 0xFF; //set output
  
  while(1) {                
    *PORTB_p |= (1<<1);    //set Bit
  }
  return 0;
}

Mein erstmal minimalistisches Makefile:

#MAKEFILE

all: test.o
#avr-gcc -o -mmcu=atmega168a test test.o | DOESNT WORK
  avr-gcc -o test test.o
  avr-objcopy -O ihex test.o test.hex
  avrdude -c avrisp -p atmega168 -P COM4 -b 19200 -U flash:w:test.hex
  
  
%.o: %.c
  avr-gcc -c $<

clean:
  rm -f test.o test.hex test

Kann mir jemand sagen, was ich falsch mache? Die Registeradressen stehen 
so im Datenblatt. Zusätzlich habe ich die avr/io.h und die 
avr/sfr_defs.h nachverfolgt und komme auf die gleichen Adressen.


Gruß

[jemand (Gast)]

Zeiger im Adressraum sind 16 bit nicht 8 bit

[Johann L. (gjlayde)]

Andreas L. schrieb:
> #avr-gcc -o -mmcu=atmega168a test test.o | DOESNT WORK

weil du "-mmcu=atmega168a" als Ausgabedatei angibst.

>   avr-gcc -o test test.o
>   avr-objcopy -O ihex test.o test.hex

Die Ausgabe nach dem Linken heißt "test", aber du wandelst "test.o" nach 
IHEX um.  test.o ist vermutlich ein nicht-lokatiertes Object-File.

> volatile uint8_t *DDRB_p = (uint8_t*)0x24;  //Pointer to DDRB:
> Direction

Besser:
 

#include <avr/io.h>
volatile uint8_t *DDRB_p = &DDRB;

vermeidet magische Zahlen.

[spess53 (Gast)]

Hi

> Die Registeradressen stehen so im Datenblatt.

Kommt darauf an wie auf die Adressen zugegriffen wird. Probiere doch mal 
0x04/0x05.

MfG spess

[Andreas L. (feuerlink)]

Danke für eure Antworten!

jemand schrieb:
> Zeiger im Adressraum sind 16 bit nicht 8 bit

Denke ich nicht und selbst wenn, macht das hier keinen Unterschied.

Johann L. schrieb:
> weil du "-mmcu=atmega168a" als Ausgabedatei angibst.

Nein, die Zeile ist auskommentiert.

> Die Ausgabe nach dem Linken heißt "test", aber du wandelst "test.o" nach
> IHEX um.  test.o ist vermutlich ein nicht-lokatiertes Object-File.

geändert:

#MAKEFILE

all: test.o
  avr-gcc -o test test.o
  avr-objcopy -O ihex test test.hex
  avrdude -c avrisp -p atmega168 -P COM4 -b 19200 -U flash:w:test.hex
  
  
%.o: %.c
  avr-gcc -c $<

clean:
  rm -f test.o test.hex test

> vermeidet magische Zahlen.

Naja, es geht mir aber ja gerade genau darum, dass es ohne die AVR 
Bibliotheken auch funktionieren muss.

spess53 schrieb:
> Hi
>
>> Die Registeradressen stehen so im Datenblatt.
>
> Kommt darauf an wie auf die Adressen zugegriffen wird. Probiere doch mal
> 0x04/0x05.
>
> MfG spess

Habe ich auch ausprobiert. Also ohne den Offset. Hilft leider auch 
nicht.

[Andreas L. (feuerlink)]

Johann L. schrieb:
> weil du "-mmcu=atmega168a" als Ausgabedatei angibst.

Oh, sorry, hab verstanden wie du's meinst.

Jetzt läufts. Makefile sieht jetzt wie folgt aus:

#MAKEFILE

all: test.o
  avr-gcc -mmcu=atmega168a -o test test.o
  avr-objcopy -O ihex test test.hex
  avrdude -c avrisp -p atmega168 -P COM4 -b 19200 -U flash:w:test.hex
  
  
%.o: %.c
  avr-gcc -c $<

clean:
  rm -f test.o test.hex test

Also war es ein Fehler vom compilieren und nicht vom Code? Muss der 
Compiler zwingend wissen, für welchen Controller er compilieren muss?


EDIT: Ups, sorry für den Doppelpost

[Johann L. (gjlayde)]

Andreas L. schrieb:
> Johann L. schrieb:
>> weil du "-mmcu=atmega168a" als Ausgabedatei angibst.
>
> Nein, die Zeile ist auskommentiert.

Genau.  Weil sie falsch war.

>   avr-gcc -o test test.o

Hier fehlt ein -mmcu=...

>   avr-gcc -c $<

Ditto.

>> vermeidet magische Zahlen.
>
> Naja, es geht mir aber ja gerade genau darum, dass es ohne die AVR
> Bibliotheken auch funktionieren muss.

Die Bibliotheken brauchst du nicht, die Header genügen.

Und wenn du auf (Standard)-Header und Bibliotheken verzichten willst, 
dann willst du mindestens mit

-nostartfiles -nostdlib -nodefaultlibs -nostdinc

übersetzen / linken.

> spess53 schrieb:
>> Kommt darauf an wie auf die Adressen zugegriffen wird. Probiere doch mal
>> 0x04/0x05.
>
> Habe ich auch ausprobiert. Also ohne den Offset. Hilft leider auch
> nicht.

avr-gcc verwendet RAM-Adressen.