undefined reference to 'sreg'

[S. R. (svenska)]

Hi,

ich bastle gerade an einem größeren AVR-Projekt. Ursprünglich komplett 
in C geschrieben (avr-gcc unter Linux) ist die Performance des 
Hauptmoduls eher mäßig, also schreibe ich das Modul gerade auf Assembler 
um. Mit AVR-Assembler (und speziell avr-as) habe ich bisher nie 
gearbeitet, alles Neuland.

Minimalbeispiel (x.S):

#include <avr/io.h>
#define avrf r21 /* copy of SREG */

    in avrf, sreg ;1 /* store H, C */

Ich möchte also einfach nur SREG in ein (benamstes) Register speichern, 
wie es auch überall gemacht wird. Aber irgendwas funktioniert da nicht:

$ avr-gcc -mmcu=atmega8515 -std=gnu11 -Wall -Wextra -Os -c -o x.o x.S
$ avr-nm x.o|grep sreg
       U sreg

$ avr-gcc -mmcu=atmega8515 -std=gnu11 -Wall -Wextra -Os -o test.elf main.o x.o
x.o:(.text+0x0): undefined reference to `sreg'

Was zur Hölle mache ich falsch?

Gruß,
svenska

[Christian S. (roehrenvorheizer)]

Hallo,

der Name sreg ist eben nicht vereinbart und deshalb unbekannt. Entweder 
Du gibst die absolute Adresse an oder sicherst das Statusregister 
mittels push auf den Stack und holst es mit pop zurück.

MfG

[S. R. (svenska)]

Warum benutzt die halbe Welt dann "in r24, sreg" und es funktioniert?
In welchem Header wäre "sreg" denn definiert? (Mit SREG geht es auch 
nicht.)

[Thomas E. (thomase)]

S. R. schrieb:
> Warum benutzt die halbe Welt dann "in r24, sreg" und es funktioniert?

Weil sie "common.h" einbindet.

[Christian S. (roehrenvorheizer)]

Steht das in deiner 8515.inc nicht drin?
Hast Du sie eingebunden?

;***** Specify Device
.device AT90S8515

;***** I/O Register Definitions
.equ  SREG  =$3f
.equ  SPH  =$3e
.equ  SPL  =$3d
.equ  GIMSK  =$3b
.equ  GIFR  =$3a
.equ  TIMSK  =$39
.equ  TIFR  =$38
.equ  MCUCR  =$35

[Leo C. (rapid)]

#include <avr/io.h>
#define avrf r21 /* copy of SREG */
    sreg = 0x3f

    in avrf,sreg ;1 /* store H, C */

$ avr-gcc -mmcu=atmega8515 -std=gnu11 -Wall -Wextra -Os -c -o x.o x.S
$ avr-objdump -d x.o

x.o:     file format elf32-avr


Disassembly of section .text:

00000000 <.text>:
   0:  5f b7         in  r21, 0x3f  ; 63
$

avr-gcc definiert bei Bedarf "__SREG__"

$ avr-gcc -mmcu=atmega1281 -S ser.c -o -

  .file  "ser.c"
__SP_H__ = 0x3e
__SP_L__ = 0x3d
__SREG__ = 0x3f
__RAMPZ__ = 0x3b
__tmp_reg__ = 0
__zero_reg__ = 1
  .text
.global  __vector_25
  .type  __vector_25, @function
__vector_25:
  push r1
  push r0
  in r0,__SREG__
  push r0
  clr __zero_reg__
  push r24
/* prologue: Signal */
/* frame size = 0 */
...

[S. R. (svenska)]

Danke, schaue ich mir im Detail an.

Thomas E. schrieb:
> Weil sie "common.h" einbindet.
Wenn ich <avr/common.h> einbinde, ändert sich nichts.

Christian S. schrieb:
> Steht das in deiner 8515.inc nicht drin?
> Hast Du sie eingebunden?
Nein, das Beispiel ist komplett.
Ich dachte, dass <avr/io.h> ausreicht.
Aber gebracht hat das auch nichts.

Leo C. schrieb:
> #include <avr/io.h>
> #define avrf r21 /* copy of SREG */
>     sreg = 0x3f
>     in avrf,sreg ;1 /* store H, C */

Naja, da definierst du das Symbol selbst, dann ist es natürlich 
vorhanden... ich dachte nur, dass es eigentlich hätte definiert sein 
müssen.

_SREG_ scheint nur bei C-Quellen zu existieren.

[Leo C. (rapid)]

S. R. schrieb:
> "_SREG_" scheint nur bei C-Quellen zu existieren.

Das existiert bei mir gar nicht.
Der Compiler definiert SREG (ohne die Unterstriche) so, daß der 
Assembler nichts damit anfangen kann:

$ avr-gcc -mmcu=atmega8515 -dM -E x.S |grep -i SREG
#define SREG_C (0)
#define SREG_H (5)
#define SREG_I (7)
#define SREG_N (2)
#define SREG_S (4)
#define SREG_T (6)
#define SREG_V (3)
#define SREG_Z (1)
#define AVR_STATUS_REG SREG
#define SREG _SFR_IO8(0x3F)
#define AVR_STATUS_ADDR _SFR_IO_ADDR(SREG)

[Patrick J. (ho-bit-hun-ter)]

Hi

Leo C. schrieb:
> #define SREG_C (0)
> #define SREG_H (5)
> #define SREG_I (7)
> #define SREG_N (2)
> #define SREG_S (4)
> #define SREG_T (6)
> #define SREG_V (3)
> #define SREG_Z (1)
> #define SREG _SFR_IO8(0x3F)

Da sehe ich aber durchaus mir auch vom Assembler bekannt vorkommende 
Werte.
Die Bitpositionen der Test-Flags, wie auch die Position des 
sReg-Register.

MfG

[Jacko (Gast)]

Bei allen avr-asm-Projekten habe ich bisher nur

das  xxxxdef.inc - File für den jeweiligen avr-µC eingebunden.
Habe bisher noch keine Register-Namen etc. aus dem zugehörigen
µC-Datenblatt vermisst.

Es fehlt also nur die Zeile:

.include "m8515def.inc"

Im Studio 7 wird es nur nervig, weil da schon nach der
Projekt-Eröffnung das *def.inc-File des angegebenen µCs
automatisch eingebunden wird. Nimmt man eine alte Studio-5
asm-Datei und macht kein Semikolon cor der Zeile
.include "xxxxdef.inc"
kommen beim Compile-Lauf endlose Warnings/Errors wegen
doppelter Definitionen...

Andersrum gesehen:
Bei Studio 7 dürfte dieser Fehler garnicht auftreten.

[S. R. (svenska)]

Christian S. schrieb:
> Steht das in deiner 8515.inc nicht drin?

Falsche Datei, du meinst m8515def.inc. ;-)
Spielt aber auch keine Rolle, denn avr-as kann damit nichts anfangen.

Jacko schrieb:
> Es fehlt also nur die Zeile:
> .include "m8515def.inc"

Falscher Assembler. Ich benutze nicht AVR Studio.

Ich habe den Code jetzt auf AVRA umgeschrieben. Der nutzt die 
INC-Dateien und es baut. Damit ist das Projekt leider ein reines 
Assemblerprojekt geworden, was ich ursprünglich vermeiden wollte. Naja, 
dafür kann ich jetzt einigermaßen AVR-Assembler...

Im Prinzip ist das ein Rewrite des i8080-Emulators aus dem AVR-CP/M, der 
aber externes SRAM benutzt und ein gutes Stück schneller sein sollte. 
Die (unfertige) C-Version war doch arg langsam. Naja, dafür kann ich 
jetzt einigermaßen i8080-Assembler...

Jetzt muss ich nur noch UART-Treiber, HEX-Loader und Mini-BDOS-Emulator 
portieren, dann kann ich den Exerciser wieder starten...