UART auslesen funktioniert nicht

[Dominik (Gast)]

Hallo zusammen,
ich habe ein Zybo Z7 Board und möchte Daten, die ich per UART an den PC 
verschicke auf der Konsole (in einem ersten Schritt) und irgendwann 
später in eine Datei speichern, auslesen.
Leider funktioniert mein Code nicht.
Hat hier jemand Erfahrungen warum nicht?

Dieses open allein funktioniert schon nicht, wenn ich aber fd als FILE 
definiere und dann ein fopen verwende funktioniert es.

 #include <stdio.h>
 #include <fcntl.h>   /* File Control Definitions           */
 #include <termios.h> /* POSIX Terminal Control Definitions */
 #include <unistd.h>  /* UNIX Standard Definitions      */ 
 #include <errno.h>   /* ERROR Number Definitions           */
  
 void main(void)
 {
     int fd;/*File Descriptor*/
     fd = open ("/dev/ttyUSB1", 'r'); //fd = open("/dev/ttyUSB1",O_RDWR | O_NOCTTY);  
     if(fd == -1)            /* Error Checking */
       printf("\n  Error! in Opening ttyUSB1  ");
     else
       printf("\n  ttyUSB1 Opened Successfully ");

     struct termios SerialPortSettings;  /* Create the structure                          */
     tcgetattr(fd, &SerialPortSettings);  /* Get the current attributes of the Serial port */

     /* Setting the Baud rate */
     cfsetispeed(&SerialPortSettings,B115200); /* Set Read  Speed as 115200                       */
     cfsetospeed(&SerialPortSettings,B115200); /* Set Write Speed as 115200                       */

    /* 8N1 Mode */
    SerialPortSettings.c_cflag &= ~PARENB;   /* Disables the Parity Enable bit(PARENB),So No Parity   */
    SerialPortSettings.c_cflag &= ~CSTOPB;   /* CSTOPB = 2 Stop bits,here it is cleared so 1 Stop bit */
    SerialPortSettings.c_cflag &= ~CSIZE;   /* Clears the mask for setting the data size             */
    SerialPortSettings.c_cflag |=  CS8;      /* Set the data bits = 8                                 */
    
    SerialPortSettings.c_cflag &= ~CRTSCTS;       /* No Hardware flow Control                         */
    SerialPortSettings.c_cflag |= CREAD | CLOCAL; /* Enable receiver,Ignore Modem Control lines       */ 
    
    
    SerialPortSettings.c_iflag &= ~(IXON | IXOFF | IXANY);          /* Disable XON/XOFF flow control both i/p and o/p */
    SerialPortSettings.c_iflag &= ~(ICANON | ECHO | ECHOE | ISIG);  /* Non Cannonical mode                            */

    SerialPortSettings.c_oflag &= ~OPOST;/*No Output Processing*/
    
    /* Setting Time outs */
    SerialPortSettings.c_cc[VMIN] = 10; /* Read at least 10 characters */
    SerialPortSettings.c_cc[VTIME] = 0; /* Wait indefinetly   */


    if((tcsetattr(fd,TCSANOW,&SerialPortSettings)) != 0) /* Set the attributes to the termios structure*/
        printf("\n  ERROR ! in Setting attributes");
    else
                    printf("\n  BaudRate = 115200 \n  StopBits = 1 \n  Parity   = none");
      
          /*------------------------------- Read data from serial port -----------------------------*/

    tcflush(fd, TCIFLUSH);   /* Discards old data in the rx buffer            */

    char read_buffer[32];   /* Buffer to store the data received              */
    int  bytes_read = 0;    /* Number of bytes read by the read() system call */
     int i = 0;

    bytes_read = read(fd,&read_buffer,32); /* Read the data                   */
      
    printf("\n\n  Bytes Rxed -%d", bytes_read); /* Print the number of bytes read */
    printf("\n\n  ");

    for(i=0;i<bytes_read;i++)   /*printing only the received characters*/
        printf("%c",read_buffer[i]);
  
    printf("\n +----------------------------------+\n\n\n");

    close(fd); /* Close the serial port */

}

[Gaensefueschen (Gast)]

fd = open ("/dev/ttyUSB1", 'r'); //fd = open("/dev/ttyUSB1",O_RDWR | 
O_NOCTTY);

'r'? Mach da mal "r" drauss.

[Dominik (Gast)]

Hab ich auch schon ausprobiert. Funktioniert hat es nicht.

[Ingo W. (uebrig)]

Dominik schrieb:
> fd = open ("/dev/ttyUSB1", 'r'); //fd = open("/dev/ttyUSB1",O_RDWR |
> O_NOCTTY);

Was da nach dem "//" (auskommentiert) steht, sieht für mein Gefühl 
besser aus, als das davor.
Der Filemodus "rw" ist für "fopen", also eine ganz andere Baustelle.

[Dominik (Gast)]

Hi,
ich hab beides ausprobiert. Funktioniert aber beides nicht

[devzero (Gast)]

Was heisst den ueberhaupt funktioniert nicht? Das steht nirgendwo.

Kommt open() mit -1 zurueck?
Schau auch mal, ob dmesg irgendwas sagt.

[Ingo W. (uebrig)]

Wie ist denn das Verhalten?
Keine Fehlermeldungen, nur Warten und keine Ausgaben?

Dominik schrieb:
> /* Setting Time outs */
>     SerialPortSettings.c_cc[VMIN] = 10; /* Read at least 10 characters
> */
>     SerialPortSettings.c_cc[VTIME] = 0; /* Wait indefinetly   */

bedeutet, das mindestens 10 Zeichen reinkommen müssen, bevor "read" 
zurückkommt.

[DPA (Gast)]

Dominik schrieb:
> if(fd == -1)            /* Error Checking */
>        printf("\n  Error! in Opening ttyUSB1  ");

Wieso ist das newline am Anfang? Ausserdem ist die Fehlermeldung 
normalerweise noch aufschlussreich. perror oder strerror sind dazu gut 
geeignet.

Empfangen bestehende Tools, wie z.B. minicom denn etwas?

[Jack (Gast)]

Dominik schrieb:
>

>      tcgetattr(fd, &SerialPortSettings);  /* Get the current attributes 
> of the Serial port */
>

Warum? Du willst doch eine spezielle Konfiguration, nicht irgendwas, was 
zufällig gesetzt ist. Daher:

// Set flags to a defined starting point
memset(&SerialPortSettings, 0, sizeof(SerialPortSettings));

Übrigens, der Variablenname SerialPortSettings nervt. Das ist eine 
temporäre Variable, die man kurzzeitig braucht, also warum der Roman?

>      /* Setting the Baud rate */
>      cfsetispeed(&SerialPortSettings,B115200); /* Set Read  Speed as
> 115200                       */
>      cfsetospeed(&SerialPortSettings,B115200); /* Set Write Speed as
> 115200                       */
>
>     /* 8N1 Mode */
>     SerialPortSettings.c_cflag &= ~PARENB;   /* Disables the Parity
> Enable bit(PARENB),So No Parity   */
>     SerialPortSettings.c_cflag &= ~CSTOPB;   /* CSTOPB = 2 Stop
> bits,here it is cleared so 1 Stop bit */
>     SerialPortSettings.c_cflag &= ~CSIZE;   /* Clears the mask for
> setting the data size             */
>     SerialPortSettings.c_cflag |=  CS8;      /* Set the data bits = 8
> */
>
>     SerialPortSettings.c_cflag &= ~CRTSCTS;       /* No Hardware flow
> Control                         */
>     SerialPortSettings.c_cflag |= CREAD | CLOCAL; /* Enable
> receiver,Ignore Modem Control lines       */
>
>
>     SerialPortSettings.c_iflag &= ~(IXON | IXOFF | IXANY);          /*
> Disable XON/XOFF flow control both i/p and o/p */
>     SerialPortSettings.c_iflag &= ~(ICANON | ECHO | ECHOE | ISIG);  /*
> Non Cannonical mode                            */
>
>     SerialPortSettings.c_oflag &= ~OPOST;/*No Output Processing*/
>
>     /* Setting Time outs */
>     SerialPortSettings.c_cc[VMIN] = 10; /* Read at least 10 characters
> */
>     SerialPortSettings.c_cc[VTIME] = 0; /* Wait indefinetly   */

Alles viel zu kompliziert, da du mit tcgetattr arbeitest. Wenn du die 
Struktur ausgenullt hast, dann:

// 8N1
SerialPortSettings.c_cflag = CS8 | CREAD | CLOCAL;
SerialPortSettings.c_iflag = 0;
SerialPortSettings.c_oflag = 0;
SerialPortSettings.c_lflag = 0;

// Wait for a few min chars, and return once a "burst" of chars
// dies down.
SerialPortSettings.c_cc[VMIN] = 8;
SerialPortSettings.c_cc[VTIME] = 1;

// Configure speed
cfsetispeed(&SerialPortSettings,B115200);
cfsetospeed(&SerialPortSettings,B115200);

So, jetzt wird es ernst:

fd = open("/dev/ttyUSB1", O_RDWR | O_NONBLOCK);
if(fd < 0) {
    perror("open /dev/ttyUSB1");
    return;
}
if(tcsetattr(fd, TCSANOW, &SerialPortSettings)) {
    perror("tcsetattr");
    close(fd);
    return;
}

[Ingo W. (uebrig)]

Jack schrieb:
> Übrigens, der Variablenname SerialPortSettings nervt. Das ist eine
> temporäre Variable, die man kurzzeitig braucht, also warum der Roman?

Es ist dein Variablenname aus deinem Programm. Wenn er dir nicht mehr 
gefällt, dann einfach suchen/ersetzen.

[PittyJ (Gast)]

Man kann nach einenm Fehlermit errno den Grund abfragen.
Und mit strerror den Code noch als String ausgeben.

Wäre doch mal praktisch nachzuschauen, warum es nicht geht.

[Rolf M. (rmagnus)]

Jack schrieb:
> So, jetzt wird es ernst:fd = open("/dev/ttyUSB1", O_RDWR | O_NONBLOCK);

Warum O_NONBLOCK? Das führt dazu, dass read() sofort zurückkehrt, auch 
wenn es gar nix zu lesen gibt.