C# Serial Port funktioniert nicht richtig

[Dominic A. (neo123)]

Hallo zusammen,

Ich habe mir eine Schaltung zur Steuerung von RGB-Leds entwickelt. Diese 
funktioniert auch so wie sie sollte. Auf der Schaltung befindet sich ein 
PIC16F1509 mit einem MAX232 zur Komunikation mit einem Computer. Diese 
Komunikation funktioniert auch ohne Probleme über Putty oder anderen 
Terminal- Programmen mit 250k Baud.
Nun zum Problem: Ich habe in C# ein kleines Programm geschrieben das die 
eingestellten Farbwerte über einen COM Port an den Mikrocontroller 
sendet. Dies funktioniert jedoch nicht richtig. Als erstes übergebe ich 
ein Zeichen welches definiert welche Farbe gesetzt werden muss. 
Anschliessend die Helligkeit dieser Farbe.
Der Code sieht so aus:

serialPort.Write("g");  //Farbe Grün
serialPort.Write("63"); // Helligkeitsstufe 63

Jedoch funktioniert dieser Code so nicht.

Wenn ich es jedoch so mache:

serialPort.Write("g"); //Farbe Grün
serialPort.Write("6"); // Helligkeitsstufe 63
serialPort.Write("3");

Funktioniert es.
Auch wenn ich in Putty g63 tippe funktioniert es.

Ausgewertet werden die Empfangenen Zeichen mit folgendem Programm auf 
dem Mikrocontroller:

unsigned int rot = 0, grun = 0, blau = 0;
void main(void)
{
    init();
    initPWM();
    rs232_init();
    PEIE = 1; //Peripherie-Interrupts akivieren
    GIE = 1; //Globale Interrupts akiviere
    RCIE = 1; //Receive Interru
    while (1)
    {
        PWMsetlin(rot, grun, blau);
        delay_ms(1);
    }
}
void interrupt isr_rs232(void)
{
    if (RCIE && RCIF)
    {
        RCIE = 0;
        char c;
        c = rs232_getc();
        if (c == 'r')
        {
            rot = rs232_getint();
        }
        else if (c == 'g')
        {
            grun = rs232_getint();
        }
        else if (c == 'b')
        {
            blau = rs232_getint();
        }
        RCIE = 1;
    }
}

Weiss jemand aus welchem Grund es nicht richtig funktioniert?

Grüsse

[Frank M. (aktenasche)]

ich vermute einfach mal, dass du diese funktion verwendest:

uint16_t RS232_getint(void)
{
    uint8_t high;
    uint8_t low;
   
    low = RS232_getc();
    high = RS232_getc();
   
    return (high<<8) + low;
}

die streng genommen nicht funktionieren dürfte, da der char "6" und der 
int "6" nicht die gleichen werte in der asciitabelle haben.

1. alle relevanten funktionen posten.
2. "funktioniert so nicht" ist als fehlerbeschreibung nicht ausreichend
3. kopple das Tx signal des rechners zurück in den Rx port und schau dir 
an was du wirklich rausschickst.
4. stell die baudrate runter, es kann sein dass die zweite zahl (in dem 
fall 3) ankommt während RCIE=0 ist.

[Peter II (Gast)]

welche codierung hast du auf dem com port eingestellt? C# Verwendet 
standardmäßig Unicode zum senden von Strings.

Senden lieber bytes, dann ist sicher was raus geht.

http://msdn.microsoft.com/de-de/library/vstudio/ms143551.aspx

[Dominic A. (neo123)]

Frank Meier schrieb:
> ich vermute einfach mal, dass du diese funktion verwendest:
Nein, ich verwende eine andere. Eigentlich ist der Name rs232_getint 
falsch:

signed int rs232_getint(void)
{
    char string[2];
    char c;
    unsigned int laenge = 0, i = 0, a = 0;
    do
    {
        a++;
        c = rs232_getc(); //Einzelnes Zeichen empfangen
        string[laenge] = c; //Empfangenes Zeichen in String ablegen
        laenge++;
    }
    while (a < 2); //Bis 2 Zeichen empfangen wurden

    i = atoi(string);
    return i;
}

> 2. "funktioniert so nicht" ist als fehlerbeschreibung nicht ausreichend
Es passiert schlicht gar nichts.

> 3. kopple das Tx signal des rechners zurück in den Rx port und schau dir
> an was du wirklich rausschickst.
Ich habe den Logic Analyzer angeschlossen. Es kommen genau diese 3 
Zeichen/Zahlen.

> 4. stell die baudrate runter, es kann sein dass die zweite zahl (in dem
> fall 3) ankommt während RCIE=0 ist.

Genau das war das Problem. Ich habe die Baudrate nun auf 2400 gestellt 
und nun funktioniert es. AUf welche Art würde es dann mit einer höheren 
Baudrate funktionieren?

[Frank M. (aktenasche)]

ah ok na die funktion macht schon mehr sinn. die frage ist, wieso mehr 
baudrate ;)

ehrlichgesagt hab ich nach kurzem blick ins datenblatt ein paar fälle im 
kopf durchgespielt woran es liegen könnte. mir ist nix eingefallen, du 
könntest aber mal OERR in RCSTA checken (siehe 22.1.2.5)

ich persönlich hab immer versucht, die isrs schlank zu halten. für 
diesen fall könnte man einen puffer verwenden, in den die empfangenen 
bytes abgespeichert werden. hierzu wird in der isr nur das zeichen im 
puffer gespeichert. das stellt sicher, dass auch wirklich alle zeichen 
verfügbar sind, wenn die umwandlung erfolgt (die uU lange dauern kann, 
kenne die performance von pics nicht)

in etwa so:

char _buffer[3];
INT8U _currentPosition;

void main()
{
...
  _currentPosition = 0;

  while(1)
  {
    if(_currentPosition > 2)
    {
      _currentPosition = 0;
      SetNewPWMValue();
    }    
  }
}

void interrupt isr_rs232(void)
{
  GIE = 0; //global interrupt disable
  if(RCIF) 
  {
    buffer[_currentPosition] = RCREG;
    _currentPosition++;
  }
  GIE = 1;
}

void SetNewPWMValue()
{
  //lets hope there will be no interrupt here. set GIE=0 if shit happens
  INT8U newVal;
  //no fast way to work with atoi here. code performance shoudl be comparable
  newVal = (_buffer[1] - '0')*10 + (_buffer[2] - '0');  
  switch(_buffer[0])
  {
    case 'r': rot = newVal; break;
    case 'g': gruen = newVal; break;
    case 'b': blau = newVal; break;
    default : rot = 0xFF; gruen = 0; blau = 0; break;//error indicator
  }
  PWMsetlin(rot, grun, blau);
}

der puffer ließe sich beliebig erweitern, d.h. du könntest auch mehr 
befehle als nur diesen senden. ein passendes kommando zum setzen der PWM 
wäre zB PWMg63. wenn du nun PWM in deinem puffer erkennst, weisst du die 
nächsten 3 zeichen sind dafür da, die farbe neu zu setzen.

ich weiss es ist unschön den fehler im eigenen code nicht zu finden, 
aber manchmal muss man einfach redesignen ;)

[Dominic A. (neo123)]

Frank Meier schrieb:

> du könntest aber mal OERR in RCSTA checken (siehe 22.1.2.5)

Vielen Dank für den Tipp. In der Funktion zum Empfangen eines Zeichen 
lösche ich das Errorbit CREN jedes mal.

unsigned char rs232_getc(void)
{
    CREN = 0;
    CREN = 1; //clear OERR-Overflow-Error-Bit
    while (!RCIF); //Warten auf Empfangs-Interrupt-Flag
    return RCREG; //Empfangens Byte zurückgeben
}

> in etwa so:
>
> der puffer ließe sich beliebig erweitern, d.h. du könntest auch mehr
> befehle als nur diesen senden. ein passendes kommando zum setzen der PWM
> wäre zB PWMg63. wenn du nun PWM in deinem puffer erkennst, weisst du die
> nächsten 3 zeichen sind dafür da, die farbe neu zu setzen.

Das ist genial. Funktioniert wunderbar. Auch das mit den weiteren 
Befehlen kommt mir sehr entgegen da ich noch verschiedene Übergänge usw. 
geplant habe.

Nun läuft alles mit 19.2k Baud.
Vielen Dank für deine Unterstützung

[Juergen R. (stumpjumper)]

Hallo Dominic,
ich würde die Zeichenkette string 3 Zeichen lang machen und mit Nullen 
vorinitialisieren. Normalerweise wird ein Ende der Zeichenkette durch 
ein Nullbyte erkannt, wenn die Kette aber nur 2 Zeichen hat ist es 
meiner Ansicht nach nur Zufall dass es funktioniert weil der Speicher 
hinter der Zeichenkette zufällig auf null steht.

>char string[2];