RFM70 schon wieder.

[Christoph B. (nuke)]

Hallo,

das Thema RFM70 ist ja hier schon öfter diskutiert worden, jetzt habe 
ich mich auch mal daran versucht, die Teile mittels AVR zum laufen zu 
bekommen.
Leider kommt auf der Gegenseite nur Mist an.

Ich habe auf einem Steckbrett mal einen Testaufbau gemacht, mit 2 
ATMega48 und einem PTX und einem PRX RFM70. Da ich die RFM70-Bibliothek 
von http://projects.weber-itam.de/?p=90 verwende, dachte ich eigentlich, 
es sollte keine Probleme geben. Die Kommunikation beider Module mit dem 
zugehörigen AVR funktioniert, ich habe mal testweise die Channels 
ausgelesen und über UART ausgegeben, klappt alles wunderbar.
Die RX-Seite reagiert auf ein ankommendes Paket und sendet das Ergebnis 
per UART ab, es kommen aber nur Linefeeds und Punkte (dez.: 10 und 46) 
an...

Hier mein Code:

//#define SEND

#include <inttypes.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <stdlib.h>

#include "uart.h"
#include "rfm70.h"
 
#define UART_BAUD_RATE 9600

#ifndef SEND
// ###################### RECEIVE CODE ######################

int main()
{
  uint8_t buffer[32];
  uint8_t val;

  sei();

  uart_init( UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
  uart_puts("System Ready\n");

  Begin();
  setMode(1);
  setChannel(25);

  while (1)
  {
    val = receivePayload(buffer);

    if (val) {

      uart_puts((char*) buffer);
      uart_puts("\n");
    }
  }
  
  return 0;
}

#else
// ###################### SEND CODE ######################

int main()
{
  uint8_t value1 = 0;

  sei();

  uart_init( UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
  uart_puts("System Ready\n");

  Begin();
  setMode(0);
  setChannel(25);

  while (1) {

    value1 = (uint8_t)uart_getc();

    if(value1) {

      PORTB |= (1<<PB0);

      sendPayload(value1,1,0);
      uart_putc(value1);

      PORTB &= ~(1<<PB0);
    }

  }
  
  return 0;
}

#endif

Ich hoffe, ihr könnt mir helfen!

Viele Grüße,
Christoph

[Fabian O. (xfr)]

Christoph Brennig schrieb:
> Ich hoffe, ihr könnt mir helfen!

Den Fehler kannst Du sehr einfach selber finden, indem Du die Warnung 
des Compilers beachtest. Diese Warnungen gibt es nicht umsonst. Sie 
bedeuten in aller Regel, dass das Programm nicht macht, was es soll.

Falls noch nicht geschehen, füge in den Projektoptionen -Wall ein, dann 
warnt der Compiler in noch mehr Situationen vor Fehlerquellen.

[Christoph B. (nuke)]

Hmm, die hatte ich übersehen...
Für alle anderen: Die Lösung war das fehlende &:

sendPayload(&value1,1,0);

Danke!

[Fabian O. (xfr)]

Du hast auch noch einen anderen, subtileren Fehler im Code. Die Funktion 
uart_puts() erwartet nämlich einen Null-terminierten String. 
receivePayload() liefert dagegen nur die Bytes, die Du per Funk 
übertragen hast, ohne abschließendes Null-Byte.

Der Code funktioniert nur zufällig, weil Dein Paket kürzer als 32 Byte 
ist und das Array buffer anscheinend zu Beginn des Programms mit Null 
gefüllt ist. Letzteres ist aber nicht selbstverändlich, da Du es nicht 
explizit initialisiert hast!

Du musst also entweder das Null-Byte mit übertragen oder die 
Ausgabefunktion im Empfänger ändern, damit es sauber funktioniert.