AVR Unterprogramm

[M. M. (pockined)]

Guten Abend,

Bin noch recht neu in AVR und bin gerade dabei den PCF 8574P über das 
TWI und einem ATmega 32 zu programmieren. Die Ansteuerung über das TWI 
ist mir bereits  gelungen, jedoch funktioniert das ganze mit 
Unterprogrammen nicht.
Folgendes Problem:

Schreibe ich den Code so funktioniert alles perfekt:

 
#include <avr/io.h>
#include <util/delay.h>
#include <util/twi.h>

int main(void)
{

  while(1)
  {
      TWBR = 0x01;
      TWSR = (0<<TWPS1) | (0<<TWPS0);
      TWCR = (1<<TWINT)|(1<<TWSTA)|(1<<TWEN);
      while (!(TWCR & (1<<TWINT)));
      TWDR = 0x40;
      TWCR = (1<<TWINT)|(1<<TWEN);
      while (!(TWCR & (1<<TWINT)));
      TWDR = 0x00;
      TWCR = (1<<TWINT)|(1<<TWEN);
      while (!(TWCR & (1<<TWINT)));
      TWCR = (1<<TWINT)|(1<<TWEN)|(1<<TWSTO);
  }
        return 0;
}

jetzt wollte ich das ganze in mehrere Unterprogramme packen:

#include <avr/io.h>
#include <util/delay.h>
#include <util/twi.h>

void TWI_begin(void)
{
  TWBR = 0x01;
  TWSR = (0<<TWPS1) | (0<<TWPS0);
}

void TWI_send_start(void)
{
  TWCR = (1<<TWINT) | (1<<TWSTA) | (1<<TWEN);
  while(!(TWCR & (1 << TWINT)));
}

void TWI_send_address(unsigned char add)
{
  TWDR = add;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while(!(TWCR & (1<<TWINT)));
}

void TWI_send_data(unsigned char dat)
{
  TWDR = dat;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while(!(TWCR & (1<<TWINT)));
}

void TWI_send_stop(void)
{
  TWCR = (1<<TWINT) | (1<<TWSTO) | (1<<TWEN);
}


int main(void)
{
  while(1)
  {
    TWI_begin();
    TWI_send_start();
    TWI_send_address(0x40);
    TWI_send_data(0x00);
    TWI_send_stop();
  }
  return 0;
}

Allerdings funktioniert dieses nicht und ich komme nicht auf den Fehler 
:(
Ich bitte um eure Hilfe

[Hmm (Gast)]

Analytisch ist es nicht besonders wertvoll nur festzustellen, das es 
nicht funktioniert. Wesentlich wertvoller wäre es zu beschreiben, was 
nicht funktioniert, an welcher_ Stelle im Code und _woran Du das genau 
bemerkst.

[M. M. (pockined)]

Soweit ich es feststellen konnte funktioniert das Programm bis zum 
Unterprogramm "TWI_send_data(0x00);" anscheinend bleibit er dort bei der 
Abfrage while(!(TWCR & (1<<TWINT))); hängen.

[Spess53 (Gast)]

Hi

Von einer Fehlerbehandlung über TWSR scheinst du auch nicht viel zu 
halten.

MfG Spess

[Hmm (Gast)]

>anscheinend...

Bitte noch präziser. Welches Ereigniss bzw. das Ausbleiben welchen 
Ereignisses konkret vermittelt Dir den Anschein, das es dort hängen 
bleibt ?
Woran merkst Du das? Glaskugel, Pendel, Geomantie, Wahrsagerei? :-)

[M. M. (pockined)]

tut mir leid bin noch neu in AVR :b

Da das Programm an dieser Stelle nicht mehr weiterläuft und in dieser 
Schleife bleibt.

[Hmm (Gast)]

Braucht Dir nicht leid tun.

>Da das Programm an dieser Stelle nicht mehr weiterläuft und in dieser
>Schleife bleibt.

Aha. Und wie, auf welche Weise, mit welchen Mitteln stellst Du das fest?
Kartenlegen, Orakel, Innereien-Schau?

[M. M. (pockined)]

Mit der Noobmethode, dass ich einfach ein LED einmal vor der Schleife 
und einmal nach der Schleife am Port D eingeschaltet habe und dabei 
bemerkt habe, dass der Befehl vor der Schleife ausgeführt wird, danach 
allerdings nicht mehr und auch im folgenden Programmteil wird nichts 
mehr ausgeführt.

[holger (Gast)]

>Mit der Noobmethode, dass ich einfach ein LED einmal vor der Schleife
>und einmal nach der Schleife am Port D eingeschaltet habe

Warum zeigst du dann nicht den Code mit den LEDs?

[Hmm (Gast)]

>...dass ich einfach ein LED einmal vor der Schleife
>...und einmal nach der Schleife am Port D eingeschaltet habe

Gut. Das hätte schon mal ins erste Posting gehört, dann braucht man 
nicht erst nachzufragen.
Wir können nicht ohne Zauberei einzusetzen, einschätzen ob eine 
Behauptung auf konkreten Informationen beruht oder nur auf Annahmen.

Was hast Du an Werkzeugen? JTAG-Debugger? (z.B. JTAG-ICE MKII)

Spess hat ja schon das TWSR-Register erwähnt. Kannst Du feststellen, was 
darin steht? Etwa über den Debugger oder Ausgabe auf einen Port?
Hast Du die Beschreibung schon gelesen?

[Hmm (Gast)]

Es hängt auch davon ab, wie die Person einzuschätzen ist. Wenn Spess uns 
schreibt, das er in der while-Schleife hängen bleibt, dann glauben wir 
ihm das unbesehen (bis es ernste Anzeichen gibt, das es nicht so ist). 
Aber uns selbst und Unbekannten, glauben wir es erst wenn wir sicher 
sind und materielle Beweise dafür vorliegen.
Nimm das Nachbohren also nicht so schwer. Ist nichts persönliches. Rein 
geschäftlich. :-)

[M. M. (pockined)]

Ok hätte mir klar sein sollen :)

Programmiert wird über den Prog-s von Atmel auf dem Atmel 
Evaluationboard.
Ich werden es versuchen jedoch dachte ich das es sich um einen 
Anfängerfehler handelt, da das Programm ohne Unterprogramme einwandfrei 
funktioniert hat und danach, wo ich es doch nur aufgeiteil habe nicht 
mehr.

[Spess53 (Gast)]

Hi

>Wenn Spess uns schreibt, das er in der while-Schleife hängen bleibt, dann
>glauben wir ihm das unbesehen ....

Da ich notorischer Assemblerprogrammierer bin, wäre das eher ein 
Anzeichen geistiger Umnachtung.

MfG Spess

[Hmm (Gast)]

>... dachte ich das es sich um einen Anfängerfehler handelt ...

Nein, tut es wohl nicht. Sei beruhigt. Falls ich nichts übersehen habe, 
sollte der Code eigentlich genauso funktionieren. Du hast ja lediglich 
die Zeilen aus der While-Schleife in Funktionen gepackt.

Dieser "Prog S" ist mitnichten von Atmel sondern von DIAMEX. (Immer 
schön genau hingucken, gelle :-) )
Damit kannst Du leider nicht debuggen.

Es bleibt also nur, das TWSR-Register irgendwie auszugeben. Serielle 
Schnittstelle, Port, Display oder sonstwie.

Was hast Du an Messmitteln? Multimeter? Oszilloskop? Aber zur Not gehts 
auch mit ner LED.

[Hmm (Gast)]

>Da ich notorischer Assemblerprogrammierer bin...

OK. Ich korrigiere mich: Wenn Spess sagt, das er in einer LD, CPI, 
BRNE-Schleife hängenbleibt, glauben wir ihm das unbesehen. Grins.

[M. M. (pockined)]

Ok werde versuchen mit einem Display etwas verwertbares aus dem Register 
zu bekommen :)

[M. M. (pockined)]

So habe jetzt das TWSR zu meinem Erstaunen erfolgreich ausgelesen und zu 
meiner Überraschung konnte ich keine Fehler entdecken.
Hab das Register mit 5 LED's ausgelesen.
Ich poste hier noch schnell den Code mit Ergebnissen:

#include <avr/io.h>
#include <util/delay.h>
#include <util/twi.h>

void TWI_begin(void)
{
  TWBR = 0x01;
  TWSR = (0<<TWPS1) | (0<<TWPS0);
}

void TWI_send_start(void)
{
  int x = 4;
  TWCR = (1<<TWINT) | (1<<TWSTA) | (1<<TWEN);
  while(!(TWCR & (1 << TWINT)));
  for(int i=7; i>=3; i--)
  {
    PORTD |= ((TWSR>>i) & 0x01) << x;    //Ergebnis: 0b0000 1xxx --> 0x08 --> A START condition has been transmitted 
    x--;
  }
  _delay_ms(1000);
  PORTD = 0x00;
  _delay_ms(1000);
}

void TWI_send_address(unsigned char add)
{
  int x = 4;
  TWDR = add;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while(!(TWCR & (1<<TWINT)));
  for(int i=7; i>=3; i--)
  {
    PORTD |= ((TWSR>>i) & 0x01) << x;    //Ergebnis: 0b0010 0xxx --> 0x20 --> SLA+W has been transmitted NOT ACK has been received
    x--;
  }
  _delay_ms(1000);
  PORTD = 0x00;
  _delay_ms(1000);
}

void TWI_send_data(unsigned char dat)
{
  int x = 4;
  TWDR = dat;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while(!(TWCR & (1<<TWINT)));
  for(int i=7; i>=3; i--)
  {
    PORTD |= ((TWSR>>i) & 0x01) << x;    //Ergebnis: 0b0011 0xxx  --> 0x30 --> Data byte has been transmitted NOT ACK has been received
    x--;
  }
  _delay_ms(1000);
  PORTD = 0x00;
  _delay_ms(1000);
}

void TWI_send_stop(void)
{
  TWCR = (1<<TWINT) | (1<<TWSTO) | (1<<TWEN);
}


int main(void)
{
  DDRD = 0xFF;
  while(1)
  {
    TWI_begin();
    TWI_send_start();
    TWI_send_address(0x40);
    TWI_send_data(0x00);
    TWI_send_stop();
  }
  return 0;
}

Ich konnte auch kein hängen an der Schleife entdecken.
Das Programm läuft ohne Probleme durch nur Funktioniert es nicht :(
Defekt am Portexpander ist auszuschließen, das es mit dem anderen 
Programm ohne Probleme funktioniert.

[Hmm (Gast)]

Nun ja. Du hast gerade mehrere Sachen auf einmal geändert, was Die 
Ursache dafür sein kann, das sich das Verhalten geändert hat.

Nimm zunächst mal die delays raus. Auch den Port brauchst Du nicht auf 
Null zu setzen. (Denke ich).

Auch diese Schleife ist unnötig kompliziert.

void TWI_send_data(unsigned char dat)
{
  // int x = 4;
  TWDR = dat;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while(!(TWCR & (1<<TWINT)));
  // for(int i=7; i>=3; i--)
  //{
    PORTD TWSR & 0xF8;
  //  x--;
  //}
  //_delay_ms(1000);
  //PORTD = 0x00;
  //_delay_ms(1000);
}

Ändere das auch in den anderen Funktionen.

Vielleicht ist das mit den Ausgaben allein nicht hinreichend. Schau Dir 
mal die Codebeispiele im Datenblatt an. Dort wird bei den entsprechenden 
Bedingungen zu Fehlern verzweigt. Vielleicht ist es besser dort 
entsprechende Portbits zu setzen und in einen Endlos-Schleife zu gehen.

Fehlendes ACK ist natürlich ungünstig.

[Hmm (Gast)]

Ooops.

PORTD TWSR & 0xF8;

muss natürlich heissen:

PORTD = TWSR & 0xF8;

[M. M. (pockined)]

Problem gelöst!
Bei den Fusebits waren die CKOPT Oscillator Options aktiviert.
Hab mich wohl beim setzen verlesen :)
Trotzdem danke für euere Hilfe!