Parameter Serielle Schnittstelle

[Thomas H. (datatom)]

Hallo zusammen,

ich möchte Daten vom PC zum Controller senden.

In meinem C++ Programm habe ich folgende Einstellungen vorgenommen:

hSerial = CreateFile("COM4",
    GENERIC_READ | GENERIC_WRITE,
    0,
    0,
    OPEN_EXISTING,
    FILE_ATTRIBUTE_NORMAL,
    0);

...

dcbSerialParams.BaudRate=CBR_19200; //CBR_57600;
dcbSerialParams.ByteSize=8;
dcbSerialParams.StopBits=ONE5STOPBITS;//ONESTOPBIT;
dcbSerialParams.Parity=NOPARITY;

Leider klappt das Senden nicht.

Mit den Einstellungen, die ich in Putty vorgenommen habe, klappt es. Die 
Einstellungen in Putty habe ich per jpg-Datei beigefügt.

Wo liegt der Fehler?

Vielen Dank im Voraus.

Grüße

datatom

[ingo (Gast)]

Versuche mal die HW-Flusskontrolle auszuschalten:
   dcbSerialParams.fOutxCtsFlow=0;
   dcbSerialParams.fOutxDsrFlow=0;

mfG ingo

[DirkB (Gast)]

Du hast auch 1.5 Stopbits eingestellt.

[Thomas (Gast)]

DirkB schrieb:
> Du hast auch 1.5 Stopbits eingestellt.

Die Stopbits stehen wie ich gerade noch einmal nachgeschaut habe, 
dierekt beim WriteFile auf 1.

[Thomas (Gast)]

ingo schrieb:
> Versuche mal die HW-Flusskontrolle auszuschalten:
>    dcbSerialParams.fOutxCtsFlow=0;
>    dcbSerialParams.fOutxDsrFlow=0;
>
> mfG ingo

Die beiden Parameter stehen, beim WriteFile auf 0.

[Thomas (Gast)]

Ich habe mal alle Parameter als jpg beigefügt.

Ich habe im Moment echt keine Ahnung woran es liegt:(

[Peter II (Gast)]

woher weisst du das nichts gesendet wird? eventuell sendest du ja das 
falsche?

[Thomas (Gast)]

Peter II schrieb:
> woher weisst du das nichts gesendet wird? eventuell sendest du ja das
> falsche?

Wenn etwas gesendet wird, blinkt das LED auf dem Controller. So passiert 
es auch bei Putty und beim Bascom-Terminal.

[Peter II (Gast)]

dann zeig doch mal die stellen vom senden, es ist eigentlich egal welche 
Paremter eingestellt sind, senden muss immer gehen. (ausser wenn die 
hardwareflusskontrolle eingeschaltet ist)

[Flusskontrolle (Gast)]

fDtrControl und fRtsControl stehen doch auf 1

[Thomas (Gast)]

Ich habe noch mal nachgeforscht. Der Fehler liegt bei den Stopbits. Egal 
ob ich eine 1 oder ONE5STOPBITS, ich bekomme bei SetCommState immer 
einen Fehler:

if(!SetCommState(hSerial, &dcbSerialParams))
  {
    //error setting serial port state
    printf("Fehler bei GetCommState beim Parameter belegen!\n");
  }
  else
  {
    printf("Kein Fehler bei GetCommState beim Parameter belegen!\n");
  }

Die Stopbits sind bei WinBase.h folgendermaßen deklariert:

#define ONESTOPBIT          0
#define ONE5STOPBITS        1
#define TWOSTOPBITS         2

Ich weiß nicht warum ich da einen Fehler bekomme.

[Thomas H. (datatom)]

Letzten Beitrag bitte komplett vergessen. Die Stopbits würden auf 1,5 
stehen, da gibts natürlich einen Fehler.

Allerdings klappt das Senden und Empfangen trotzdem noch nicht.

Irgendein Parameter scheint noch falsch gesetzt zu sein.

[Peter II (Gast)]

Thomas Holländer schrieb:
> Irgendein Parameter scheint noch falsch gesetzt zu sein.

glaube ich nicht! Auch mit falschen Parmeter sendet die Schnittstelle. 
Prüfe bitte erstmal deine Sende routine und schau mal nach wann die LED 
leuchten müssste. (also erst bei einem epmfangen byte oder hängt sie 
direkt an der sende leitung)

[Thomas H. (datatom)]

Hier mal der komplette Quellcode:

// SerielleSchnittstelle.cpp : Definiert den Einstiegspunkt für die Konsolenanwendung.
//

#include "stdafx.h"
#include "stdio.h"
#include "stdlib.h"
#include "string.h"
#include "windows.h"
#include "strsafe.h"


void ErrorExit(LPTSTR lpszFunction);

int i = 1;
HANDLE hSerial;
COMMTIMEOUTS timeouts;
DCB dcbSerialParams;
char szBuff;        
char szBuff1[8];
char szBuff2[8];
char szBuff3[8];
DWORD dwBytesRead;
DWORD dwBytesWrite;
int n = 8; //4096;

int _tmain(int argc, _TCHAR* argv[])
{
  hSerial = CreateFile("COM4",
    GENERIC_READ | GENERIC_WRITE,
    0,
    0,
    OPEN_EXISTING,
    FILE_ATTRIBUTE_NORMAL,
    0);
  if(hSerial==INVALID_HANDLE_VALUE)
  {
    if(GetLastError()==ERROR_FILE_NOT_FOUND)
    {
      // Generate an error
      // Fehlercode ausgeben
      if(!GetProcessId(NULL))
      {
        ErrorExit(TEXT("GetProcessId"));
      }
      //serial port does not exist. Inform user.
      printf("Fehler bei CreateFile. Serial Port existiert nicht!\n");
    }
    //some other error occurred. Inform user.
    printf("Fehler bei CreateFile. Sonstiger Fehler!\n");
  }
  else
  {
    printf("Kein Fehler bei CreateFile!\n");
  }
  //////////////////////////////
  if (!GetCommState(hSerial, &dcbSerialParams)) 
  {
    //error getting state
    printf("Fehler bei GetCommState!\n");
  }
  else
  {
    printf("Kein Fehler bei GetCommState!\n");
  }
  dcbSerialParams.BaudRate=CBR_19200; //CBR_57600;
  dcbSerialParams.ByteSize=8;
  dcbSerialParams.StopBits=ONESTOPBIT;
  dcbSerialParams.Parity=NOPARITY;
  dcbSerialParams.fOutX=1;
  dcbSerialParams.fInX=1;

  if(!SetCommState(hSerial, &dcbSerialParams))
  {
    //error setting serial port state
    printf("Fehler bei GetCommState beim Parameter belegen!\n");
  }
  else
  {
    printf("Kein Fehler bei GetCommState beim Parameter belegen!\n");
  }
  ///////////////////////////////
  timeouts.ReadIntervalTimeout=50;
  timeouts.ReadTotalTimeoutConstant=50;
  timeouts.ReadTotalTimeoutMultiplier=10;
  timeouts.WriteTotalTimeoutConstant=50;
  timeouts.WriteTotalTimeoutMultiplier=10;

  if(!SetCommTimeouts(hSerial, &timeouts))
  {
    //error occureed. Inform user
    printf("Fehler bei Timeout!\n");
  }
  else
  {
    printf("Kein Fehler bei Timeout!\n");
  }
  //////////////////////////////////
  while(i < 9999999)
  {   
    i = 0;
    if(!ReadFile(hSerial, &szBuff2, n, &dwBytesRead, NULL))

    {
      //error occurred. Report to user.
      printf("Fehler bei ReadFile!\n");
    }
    else
    {  i = atoi(szBuff2);
    _itoa_s(i,szBuff3,10);
    printf (szBuff3);
    printf ("\n");
    }
    ////////////////////////////////
    //szBuff[0] = 'r';
    i = i + 1000;
    _itoa_s(i,szBuff1,10);  
    if(!WriteFile(hSerial, &szBuff1, n, &dwBytesWrite, NULL))
    {
      //error occurred. Report to user.
      printf("Fehler bei WriteFile!\n"); 
    }
    else
    {
      printf (szBuff1);
      printf ("\n");
    }

  }
  ///////////////////////////////
  CloseHandle(hSerial);
  ////////////////////////////////
  system("pause");
  return 0;
}

// Fehlerfunktion
void ErrorExit(LPTSTR lpszFunction) 
{ 
  // Retrieve the system error message for the last-error code

  LPVOID lpMsgBuf;
  LPVOID lpDisplayBuf;
  DWORD dw = GetLastError(); 

  FormatMessage(
    FORMAT_MESSAGE_ALLOCATE_BUFFER | 
    FORMAT_MESSAGE_FROM_SYSTEM |
    FORMAT_MESSAGE_IGNORE_INSERTS,
    NULL,
    dw,
    MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
    (LPTSTR) &lpMsgBuf,
    0, NULL );

  // Display the error message and exit the process

  lpDisplayBuf = (LPVOID)LocalAlloc(LMEM_ZEROINIT, 
    (lstrlen((LPCTSTR)lpMsgBuf) + lstrlen((LPCTSTR)lpszFunction) + 40) * sizeof(TCHAR)); 
  StringCchPrintf((LPTSTR)lpDisplayBuf, 
    LocalSize(lpDisplayBuf) / sizeof(TCHAR),
    TEXT("%s failed with error %d: %s"), 
    lpszFunction, dw, lpMsgBuf); 
  MessageBox(NULL, (LPCTSTR)lpDisplayBuf, TEXT("Error"), MB_OK); 

  LocalFree(lpMsgBuf);
  LocalFree(lpDisplayBuf);
  ExitProcess(dw); 
}

[ingo (Gast)]

Hier mal eine DCB-Initialisierung, die bei mir zuverlässig funktioniert:

   // BuildCommDCB("baud=9600 parity=N data=8 stop=1",&MyDcb);
   // sscanf(Dialog1.baudrate,"%ld",&a);

   MyDcb.BaudRate=a; /* oder was Anders */
   MyDcb.DCBlength=sizeof(DCB); /* !!! */
   MyDcb.fBinary=1;
   MyDcb.fOutxCtsFlow=0;
   MyDcb.fOutxDsrFlow=0;
   MyDcb.fDtrControl=DTR_CONTROL_ENABLE;
   MyDcb.fDsrSensitivity=0;
   MyDcb.fRtsControl=RTS_CONTROL_ENABLE;
   SetCommState(ComHandle,&MyDcb);


Evtl fehlt dir die DCBlength!
mfG ingo

[Peter II (Gast)]

Thomas Holländer schrieb:
> i = i + 1000;
>     _itoa_s(i,szBuff1,10);
>     if(!WriteFile(hSerial, &szBuff1, n, &dwBytesWrite, NULL))

das sieht merkwürdig aus. du Sendest hier 8byte (n=8) aber in szBuff1 
sind nur 4byte + 0byte. Dann sendest du auch keine \n - kann sein das 
deine Hardware auf ein \n wartet?

[Thomas H. (datatom)]

ingo schrieb:
> Hier mal eine DCB-Initialisierung, die bei mir zuverlässig funktioniert:
>
>    // BuildCommDCB("baud=9600 parity=N data=8 stop=1",&MyDcb);
>    // sscanf(Dialog1.baudrate,"%ld",&a);
>
>    MyDcb.BaudRate=a; /* oder was Anders */
>    MyDcb.DCBlength=sizeof(DCB); /* !!! */
>    MyDcb.fBinary=1;
>    MyDcb.fOutxCtsFlow=0;
>    MyDcb.fOutxDsrFlow=0;
>    MyDcb.fDtrControl=DTR_CONTROL_ENABLE;
>    MyDcb.fDsrSensitivity=0;
>    MyDcb.fRtsControl=RTS_CONTROL_ENABLE;
>    SetCommState(ComHandle,&MyDcb);
>
>
> Evtl fehlt dir die DCBlength!
> mfG ingo

Hab ich übernommen klappt leider nicht:-(

Peter II schrieb:
> Thomas Holländer schrieb:
>> i = i + 1000;
>>     _itoa_s(i,szBuff1,10);
>>     if(!WriteFile(hSerial, &szBuff1, n, &dwBytesWrite, NULL))
>
> das sieht merkwürdig aus. du Sendest hier 8byte (n=8) aber in szBuff1
> sind nur 4byte + 0byte. Dann sendest du auch keine \n - kann sein das
> deine Hardware auf ein \n wartet?

Hab auch mal den Wert 12345678 mitgegeben, klappt trotzdem nicht.

[Peter II (Gast)]

Thomas Holländer schrieb:
> Hab auch mal den Wert 12345678 mitgegeben, klappt trotzdem nicht.

du senderst aber immer noch kein Zeilenende - woher soll deine Hardware 
wissen das nicht noch ein Zeilchen kommt?

[Thomas H. (datatom)]

Wie geht das denn mit dem Zeilenende?

[Peter II (Gast)]

Thomas Holländer schrieb:
> Wie geht das denn mit dem Zeilenende?

also noch deutlicher kann man es kaum sagen du must ein \r oder \n 
senden - keine ahnung was deine Hardware erwartet!

[Karl H. (kbuchegg)]

Thomas schrieb:
> Peter II schrieb:
>> woher weisst du das nichts gesendet wird? eventuell sendest du ja das
>> falsche?
>
> Wenn etwas gesendet wird, blinkt das LED auf dem Controller.

Wann genau blinkt diese LED?
Wenn der Controller ein gültiges Kommando erhalten hat oder bei jedem 
Zeichen, dass der Controller bekommt?

Wenn du noch eine LED + Vorwiderstand hast. Häng die mal direkt an die 
Rx Leitung vom Controller. Deren blinken zeigt dir zuverlässig an, ob da 
was über die Schnittstelle kommt oder nicht. Egal ob das was da 
daherkommt, Sinn macht oder ob es richtig ist. Sobald der PC an der 
Leitung wackelt (und das muss er, wenn er sendet), dann blinkt die LED 
auch.

[Karl H. (kbuchegg)]

hSerial = CreateFile("COM4",

häng da mal an die Gerätebezeichnung einen : hinten drann. "COM4:" wenn 
ich mich recht erinnere, dann ist der : der Auslöser für das 
Betriebssystem, dass es sich hierbei um ein Gerät handelt und nicht etwa 
um eine Datei namens "COM4".

[Peter II (Gast)]

Karl Heinz Buchegger schrieb:
> häng da mal an die Gerätebezeichnung einen : hinten drann. "COM4:" wenn
> ich mich recht erinnere, dann ist der : der Auslöser für das
> Betriebssystem, dass es sich hierbei um ein Gerät handelt und nicht etwa
> um eine Datei namens "COM4".

NEIN - com4 ist schon richtig.

[Udo S. (urschmitt)]

man kann auch auf PC Seite erst mal Send und Recieve verbinden um zu 
sehen ob etwas gesendet wird. Klar muss man das PC seitig dann auch 
lesen, aber die Lesefunktion braucht man in der Regel sowiso.
Auch klar, falsch gesetzte parameter erkenne ich so nicht aber es ging 
ja darum ob überhaupt gesendet wird.

[Thomas H. (datatom)]

Peter II schrieb:
> also noch deutlicher kann man es kaum sagen du must ein \r oder \n
> senden - keine ahnung was deine Hardware erwartet!

Klappt jetzt. Ich habe am Ende \r angefügt und nun werden Daten 
gesendet.

Danke an alle die mit Rat und Tat zur Seite waren.

Grüße

datatom

[Peter II (Gast)]

Thomas Holländer schrieb:
> Klappt jetzt. Ich habe am Ende \r angefügt und nun werden Daten
> gesendet.

sie wurden auch schon vorher gesendet ...

[Thomas H. (datatom)]

is ja gut:-)