HAL_UART_Transmit Problem

OP #5404310
Lesenswert?

Guten Morgen Leute,

die Verbindung funktioniert und ich empfange die Daten mit Putty auf 
meinem Rechner. Ich wollte noch die Sekunden übertragen aber irgendwie 
bekomme ich es nicht hin. Ich habe auch in Erfahrung gebracht das man 
den printf Befehl bearbeiten kann um direkt ans UART auszugeben. Habe 
ich aber auch nicht hinbekommen.

Das hier ist die Anleitung mit dem printf (wäre nur praktisch aber nicht 
nötig wenn das funktioniert)
https://www.waveshare.com/wiki/STM32CubeMX_Tutorial_Series:_USART

Und hier mein Code
  while (1)
  {

  int i=1;
  char *msg = "Hello World! \n\r";
  char *msg1 = "Programm runs ";
  char *msg2;
  HAL_UART_Transmit(&huart2, (uint8_t*)msg, strlen(msg), 0xFFFF);

  while(1){

  sprintf(msg2,"%d",i);
  HAL_Delay(1000);
  HAL_UART_Transmit(&huart2, (uint8_t*)msg1, strlen(msg), 0xFFFF);
  HAL_UART_Transmit(&huart2, (uint8_t*)msg2, strlen(msg), 0xFFFF);
  i = i + 1;

  }

Vielen dank im Voraus
(Firma: Torrox.de) #5405745
Lesenswert?

Walt N. schrieb:
> Und hier mein Code

>   char *msg2;
>   sprintf(msg2,"%d",i);

msg2 ist eine Zeiger auf ein char, der nicht initialiert ist und damit 
irgendwo hin zeigt. In C ist char* häufig ein Synonym für einen String, 
in dem der Zeiger auf dass erste Zeichen eines Array von char zeigt. 
sprintf() erwartet einen Zeiger auf das erste Zeichen, dass ausreichend 
groß ist, um den Erzeugten Text dort hin zu kopieren:
1
    char msg2[ 100 ];
2
    sprintf( msg2, "%d", i );

HTH,
Torsten
Gast #5407868
Lesenswert?

Zu deinem Problem: wie Torsten schrieb, klappt das mit der 
Pointerinitialisierung nicht so ganz, da ein Pointer nur einen 
Platzverbrauch vone einer Speicheradresse hat. Besser wäre da ein Array:
1
 char msg[] = "Hello, World!\r\n";
denn dann wird gleich noch Speicher für deinen sTring belegt und 
befüllt.

Ansosnten hier meine Lösung. Ich vermute mal, du arbeitest mit nem 
STM32Fxxx.

So hab ich den UART mit printf zum Laufen gebracht:
1: USART aktivieren:
1
MX_USART1_UART_Init();

2: die putchar- und getchar-Funktionen hinzufügen:
1
int __io_putchar(int ch) {
2
  HAL_UART_Transmit(&huart1, (uint8_t *) &ch, 1, 0xFFFF);
3
  return ch;
4
}
5

6
int __io_getchar(void) {
7
  int retval = -1;
8
  HAL_UART_Receive(&huart1, (uint8_t*) &retval, 1, 0xF);
9
  return retval;
10
}

3: In deinem Verzeichnis sollte irgendwo ne syscalls.c rumliegen, die 
ins Source-verzeichnis kopieren (CubeMX erzeugt die für mich mit).

4: optional noch ein
1
setvbuf(stdout, NULL, _IONBF, 0);
damit printf sofort Daten schickt und nicht auf ein Newline wartet.


Alternativ geht das auch als Makro:
1
#define PRINTF(...) do{char text[255]; \
2
  int l = snprintf(text,255, __VA_ARGS__);\
3
  HAL_UART_Transmit(&huart1,(uint8_t*)text,l,0xFF);}while(0)

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren