putchar funktion soll auf Progmem umgebaut werden

[Simon (Gast)]

Ich habe Folgende Function:

void uart_usb_send_buffer(U8 *buffer, U8 nb_data)

Diese wurde bisher mit:

int uart_usb_putchar(int data_to_send)
{
   uart_usb_send_buffer((U8*)&data_to_send, 1);
   return data_to_send;
}

verwendet.

Ich würde nun gerne PSTR("") damit verwenden.

Ich dachte da an soetwas hier: aber das funzt nicht :-)

void uart_usb_send_buffer_P(const char *data)
{
   while (pgm_read_byte(data) != 0x00)
     uart_usb_send_buffer(pgm_read_byte(data++));
}

[Stefan E. (sternst)]

Das hier

     uart_usb_send_buffer(pgm_read_byte(data++));

 passt ja auch überhaupt nicht zu

void uart_usb_send_buffer(U8 *buffer, U8 nb_data)

void uart_usb_send_buffer_P(const char *data)
{
   U8 byte;
   while ((byte = pgm_read_byte(data++)) != 0x00)
     uart_usb_send_buffer(&byte,1);
}

Und für die Zukunft:
"aber das funzt nicht" ist keine hinreichende Fehlerbeschreibung. Und 
wenn (wie offensichtlich in diesem Fall) der Compiler dir den Code um 
die Ohren haut, dann schreibe zumindest die Fehlermeldungen des Compiler 
mit dazu.

[Karl H. (kbuchegg)]

Simon schrieb:

> Ich dachte da an soetwas hier: aber das funzt nicht :-)
>
>

> void uart_usb_send_buffer_P(const char *data)
> {
>    while (pgm_read_byte(data) != 0x00)
>      uart_usb_send_buffer(pgm_read_byte(data++));
> }
>

logisch.
Das kann nicht gehen.

Die uart_usb_send_buffer will die Adresse einer Speicherfläche haben, 
von der sie sich die Daten holt. Du hast aber keine Speicherfläche, du 
versuchst die Daten selbst in die Funktion reinzustopfen.

void uart_usb_send_buffer_P(const char *data)
{
  char c = pgm_read_byte(data);

  while( c != 0x00) {
    uart_usb_send_buffer( &c, 1 );
    c = pgm_read_byte(++data);
  }
}

Aber: Hast du denn keine uart_send Funktion, die einfach nur einen U8 
übernimmt? Eine
void uart_usb_send_byte(U8 buffer);

Normalerweise benutzt eine Funktion wie uart_usb_send_buffer eine 
derartige Funktion als Basis um mehrere Zeichen auszugeben.

Edit:
Dein ganzer funktionaler Aufbau scheint nicht zu stimmen.
uart_usb_putchar ist die einfachere Funktion. Die sollte sich nicht auf 
uart_usb_send_buffer stützen, sondern genau umgekehrt: 
uart_usb_send_buffer benutzt uart_usb_putchar um seine Funktionalität zu 
erledigen. Und die int in uart_usb_putchar glaub ich dir auch nicht. 
Beides zusammen ergibt eine völlig unnötige Beschäftigungstherapie für 
den µC.

[Simon (Gast)]

Die sache ist das PSTR() es nicht im RAM ablegt, somit geht wenn ich
das richtig verstanden habe auch kein pointer auf den RAM.. und da liegt 
mein Problem, ich brauche irgendwie die Möglichkeit die Funktion auch 
mit PSTR verwenden zu können.




Es gibt ja noch in der unteren funktion ein
"Usb_write_byte(*buffer);"
aber ich wollte die funktionen da ansich nicht auseinander nehmen.
Gibts da keine einfacherer Möglichkeit?



Das ganze stammt aus der AVR usb lib (CDC Demo)

/** 
  * @brief This function fills the USB transmit buffer with the new data. This buffer
  * is sent if complete. To flush this buffer before waiting full, launch
  * the uart_usb_flush() function.
  * 
  * @param data_to_send 
  * 
  * @return 
  */
int uart_usb_putchar(int data_to_send)
{
   uart_usb_send_buffer((U8*)&data_to_send, 1);
   return data_to_send;
}


/** 
  * @brief This function transmits a ram buffer content to the USB.
  * This function is mode efficient in term of USB bandwith transfer.
  * 
  * @param U8 *buffer : the pointer to the RAM buffer to be sent 
  * @param data_to_send : the number of data to be sent
  */
void uart_usb_send_buffer(U8 *buffer, U8 nb_data)
{
   U8 zlp;
   
  if(!Is_device_enumerated())
     return;
   
   // Compute if zlp required
   if(nb_data%TX_EP_SIZE) 
   { zlp=FALSE;} 
   else { zlp=TRUE; }
   
   Usb_select_endpoint(TX_EP);
   while (nb_data)
   {
      while(Is_usb_write_enabled()==FALSE); // Wait Endpoint ready
      while(Is_usb_write_enabled() && nb_data)
      {
         Usb_write_byte(*buffer);
         buffer++;
         nb_data--;
   }
      Usb_ack_in_ready();
   }
   if(zlp)
   {
      while(Is_usb_write_enabled()==FALSE); // Wait Endpoint ready 
      Usb_ack_in_ready();
}
}

[Karl H. (kbuchegg)]

Simon schrieb:
> Die sache ist das PSTR() es nicht im RAM ablegt, somit geht wenn ich
> das richtig verstanden habe auch kein pointer auf den RAM.. und da liegt
> mein Problem, ich brauche irgendwie die Möglichkeit die Funktion auch
> mit PSTR verwenden zu können.

Na, ja. Hast du ja jetzt.
Stefans und mein Vorschlag sind gleichwertig.


> Es gibt ja noch in der unteren funktion ein
> "Usb_write_byte(*buffer);"
> aber ich wollte die funktionen da ansich nicht auseinander nehmen.
> Gibts da keine einfacherer Möglichkeit?

Könnte man wahrscheinlich machen. Aber da ist noch soviel drumherum zu 
machen, dass es sich wohl nicht lohnt.

Allerdings sollte man dann über die String Funktion noch mal nachdenken. 
In diesem Fall ist es dann nicht so schlau, wegen jedem Zeichen wieder 
durch die ganze Rundherum-Prozedur in uart_usb_send_buffer 
durchzulaufen.

Ich würde das zumindest soweit versuchen zu optimieren, dass aus dem 
Flash immer zb 4 Character geholt werden und diese 4 Character in einem 
aufwasch gesendet werden. Die 4 sind natürlich verhandelbar und könnten 
auch maximal zb 32 sein.

[Johann L. (gjlayde)]

Sowas?

void uart_usb_send_buffer_P (const char *pstr)
{
  size_t len = strlen_P (pstr);
  char str[len];

  strcpy_P (str, pstr);

  uart_usb_send_buffer (str, len);
}

[Karl H. (kbuchegg)]

Johann L. schrieb:

>   char str[len];

    char str[len+1];

[Simon (Gast)]

Funzt bestens!

Vielen Dank, morgen durchdenke ich das mal in Ruhe.

Wie kommts zu der Warnung:

../uart_usb_lib.c:163: warning: pointer targets in passing argument 1 of 
'uart_usb_send_buffer' differ in signedness

[Johann L. (gjlayde)]

Simon schrieb:

> Wie kommts zu der Warnung:
>
> ../uart_usb_lib.c:163: warning: pointer targets in passing argument 1 of
> 'uart_usb_send_buffer' differ in signedness

Du mixt signed und unsigned pointer wild durcheinander.