uart printf vs fprintf atmega328p

[Simon W. (wysi)]

Hallo zusammen

Mit dem atmega328p möchte ich Text über UART ausgeben. Mit printf() 
funktioniert dies. Mit fprintf() wird immer nur das erste Zeichen des 
Strings über UART ausgegeben.

Versucht habe ich auch FILE str_uart als "static" zu definieren. Dies 
brachte auch keine Verbesserung.

Folgend das minimal Beispiel:

//minimal example for UART printf vs fprintf test
//compiled with AtmelStudio 6.2
//controller: ATMEGA328P (same mikrocontroller as on Arduino UNO)

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

//F = 16MHz
//add under Project/ Toolchain/ AVR/GNU C Compiler -> Symbols -> add F_CPU=16000000UL

void init_UART(void)
{
  //Set baud rate 115200
  UBRR0H = 0;
  UBRR0L = 16;
  //Enable receiver and transmitter
  UCSR0B = (1<<RXEN0)|(1<<TXEN0);
  //Double Clock Speed
  UCSR0A = (1<<U2X0);
  // Set frame format: 8data, 1stop bit
  UCSR0C = (3<<UCSZ00); //(1<<USBS0)|
}

void uart_putchar(char c, FILE *stream){
  //wait until buffer empty
  while ( !( UCSR0A & (1<<UDRE0)) );
  //Put data into buffer
  UDR0 = c;
}

int main(void)
{
  init_UART();
  FILE str_uart = FDEV_SETUP_STREAM(uart_putchar, NULL, _FDEV_SETUP_WRITE);
  stdout = &str_uart;
  
  //works
  printf( "Test UART 1 \n");
  
  //doesn't work, prints only first char ('T') to UART
  fprintf(&str_uart, "Test UART 2 \n");
  
  
  while(1);
}

Was ist an obenstehendem Code falsch, dass immer nur das erste Zeichen 
des Strings in fprintf() ausgegeben wird?

Vilen Dank für eure Hilfe.

[Jörg W. (dl8dtl) (Moderator)]

Dein printf() wird vermutlich durch ein puts() ersetzt, während
das andere tatsächlich intern in ein vfprintf() verzweigt (dort
enden alle Aufrufe irgendwelcher *printf*-Funktionen).

uart_putchar() soll eine 0 zurückgeben, wenn sie das Zeichen
erfolgreich geschrieben hat.  Möglicherweise ist das ja dein Problem.

[Simon W. (wysi)]

Vielen Dank für die schnelle und zielführende Hilfe. Nach dem Hinzufügen 
von return 0 in uart_putchar() funktioniert fprintf() auch korrekt.

Hier die komplette, funktionierende Lösung:

//minimal example for UART printf vs fprintf test
//compiled with AtmelStudio 6.2
//controller: ATMEGA328P (same mikrocontroller as on Arduino UNO)

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

//F = 16MHz
//add under Project/ Toolchain/ AVR/GNU C Compiler -> Symbols -> add F_CPU=16000000UL

void init_UART(void)
{
  //Set baud rate 115200
  UBRR0H = 0;
  UBRR0L = 16;
  //Enable receiver and transmitter
  UCSR0B = (1<<RXEN0)|(1<<TXEN0);
  //Double Clock Speed
  UCSR0A = (1<<U2X0);
  // Set frame format: 8data, 1stop bit
  UCSR0C = (3<<UCSZ00); //(1<<USBS0)|
}

int uart_putchar(char c, FILE *stream){
  //wait until buffer empty
  while ( !( UCSR0A & (1<<UDRE0)) );
  //Put data into buffer
  UDR0 = c;
  
  return 0;
}

int main(void)
{
  init_UART();
  FILE str_uart = FDEV_SETUP_STREAM(uart_putchar, NULL, _FDEV_SETUP_WRITE);
  stdout = &str_uart;
  
  //works
  printf( "Test UART 1 \n");
  
  //works (after adding "return 0" to "uart_putchar()")
  fprintf(&str_uart, "Test UART 2 \n");
  
  
  while(1);
}

[Jörg W. (dl8dtl) (Moderator)]

Simon W. schrieb:
> // Set frame format: 8data, 1stop bit
>   UCSR0C = (3<<UCSZ00); //(1<<USBS0)|

8N1 ist übrigens die Voreinstellung der UART.