Function und sprintf problem / frage

[Christian B. (christian_buchner)]

Hallo,

ich habe eine Funktion mit der ich über "SendDisplayData(Buffer)" den 
Buffer eines sprintf an mein Display sende. Wenn ich nun einen Text an 
das Display senden möchte muss ich immer folgendes machen:

SendDisplayData("#ZL,100,100,Hier der Text\r");

wenn ich z.b. den Wert eines ADCs ausgeben möchte dann muss ich sprintf 
vorher nutzen, das sieht dann so aus:

char Buffer[50];
sprintf(Buffer, "#ZL,100,100,ADC-Wert: %d\r", adc_variable);
SendDisplayData(Buffer);

Ich hätte das ganze aber gerne so:

write_display(100,100, "Hier der Text");

oder

write_display(100,100, adc_variable);

und die function soll dann so aussehen:

void write_display(int x_variable, int y_variable, char ausgabewert)
{
    sprintf(Buffer, "#ZL {hier die x_variable}, {hier die y_variable}, {hier den ausgabetext} \r", x_variable, y_variable, ausgabewert);
    SendDisplayData(Buffer);
}

Leider habe ich mit sprintf nur sehr wenige erfahrungen. Der compiler 
mekkert immer und sagt was von %c oder %s und wie ich mehr als nur einen 
Wert mit sprintf in den Buffer "übergebe" weiss ich leider auch nicht.

Wenn der Compiler dann mal nicht mekkert dann geht es entweder nicht, 
also ich bekomme auf die Display keine Ausgabe oder die Ausgabe erfolgt 
schon ab der, in diesem falle, 100, was nicht sein dürfte.

Ich danke euch schonmal.

[Peter II (Gast)]

void write_display(int x_variable, int y_variable, char* ausgabewert) {
   sprintf(Buffer, "#ZL %d, %d, %s \r", x_variable, y_variable, ausgabewert);
    SendDisplayData(Buffer);
};

sollte gehen.

[Christian B. (christian_buchner)]

Hallo, danke. Klappt sogar war nur ein semikolon zu viel. Aber was mich 
wundern, warum mekkert Win-Avr trotzdem?

display.c: In function 'write_display':
display.c:158: warning: implicit declaration of function 'sprintf'
display.c:158: warning: incompatible implicit declaration of built-in function 'sprintf'

[Peter II (Gast)]

Christian B. schrieb:
> warum mekkert Win-Avr trotzdem?

vermutlich hast du das #include <stdio.h> vergessen.

[Christian B. (christian_buchner)]

Jau. Danke. Das brauchte ich für das Display ja nicht. Jetzt kam ja eine 
funktion dazu die die stdio.h benötigt g

Manchmal sind es die kleinen Dinge :)

Super.

Danke Danke

Wenn wir schon dabei sind. Warum mekkert der Compiler z.b. "function is 
an prototype? Habe mir für den ds1307 ein paar funktionen erstellt. die 
gehen zwar aber er mekkert immer prototyp bla

[Christian B. (christian_buchner)]

Hier mal die Dateien:

ds1307.h

#include <avr/io.h>
#include <stdio.h>

void ds1307_second_write(unsigned char second);
void ds1307_minute_write(unsigned char minute);
void ds1307_hour_write(unsigned char hour,unsigned char hour_format, unsigned char am_pm);
void ds1307_day_write(unsigned char day);
void ds1307_date_write(unsigned char date);
void ds1307_month_write(unsigned char month);
void ds1307_year_write(unsigned char year);

unsigned char ds1307_read_second();
unsigned char ds1307_read_minute();
unsigned char ds1307_read_hour();
unsigned char ds1307_read_day();
unsigned char ds1307_read_date();
unsigned char ds1307_read_month();
unsigned char ds1307_read_year();
unsigned char convert_decimal_to_bcd(unsigned char decimal_number);
unsigned char convert_bcd_to_decimal(unsigned char bcd_number);

void read_uhr();

und die ds1307.c

#include "i2c.h"
#include "ds1307.h"
#include "bcdata.h"
#include "display.h"

unsigned char convert_decimal_to_bcd(unsigned char decimal_number)
{
decimal_number=((decimal_number/10)*16)+(decimal_number%10);
return decimal_number;
}

unsigned char convert_bcd_to_decimal(unsigned char bcd_number)
{
bcd_number=((bcd_number>>4)*10)+(bcd_number & 0x0f);
return bcd_number;
} 

// Schreibe funktionen
void ds1307_second_write(unsigned char second)
{
  i2c_start(0xd0);
  i2c_write(0x00);
  second=convert_decimal_to_bcd(second);
  i2c_write(second);
  i2c_stop();
}

void ds1307_minute_write(unsigned char minute)
{
  i2c_start(0xd0);
  i2c_write(0x01);
  minute=convert_decimal_to_bcd(minute);
  i2c_write(minute);
  i2c_stop();
}

void ds1307_hour_write(unsigned char hour,unsigned char hour_format, unsigned char am_pm)
{
  i2c_start(0xd0);
  i2c_write(0x02);
  if(hour_format==1)
  {
    hour=convert_decimal_to_bcd(hour);
    hour=hour | (1<<6) | (am_pm<<5);
    i2c_write(hour);
  }
  else
  {
    hour=convert_decimal_to_bcd(hour);
    i2c_write(hour);
  }
  i2c_stop();
}

void ds1307_day_write(unsigned char day)
{
  i2c_start(0xd0);
  i2c_write(0x03);
  day=convert_decimal_to_bcd(day);
  i2c_write(day);
  i2c_stop();
}

void ds1307_date_write(unsigned char date)
{
  i2c_start(0xd0);
  i2c_write(0x04);
  date=convert_decimal_to_bcd(date);
  i2c_write(date);
  i2c_stop();
}

void ds1307_month_write(unsigned char month)
{
  i2c_start(0xd0);
  i2c_write(0x05);
  month=convert_decimal_to_bcd(month);
  i2c_write(month);
  i2c_stop();
}

void ds1307_year_write(unsigned char year)
{
  i2c_start(0xd0);
  i2c_write(0x06);
  year=convert_decimal_to_bcd(year);
  i2c_write(year);
  i2c_stop();
}


// Lese funktionen
unsigned char ds1307_read_second()
{
  unsigned char sec;
  i2c_start(0xd0);
  i2c_write(0x00);
  i2c_start(0xd1);
  sec=i2c_readNak();
  i2c_stop();
  return sec;
}

unsigned char ds1307_read_minute()
{
  unsigned char min;
  i2c_start(0xd0);
  i2c_write(0x01);
  i2c_start(0xd1);
  min=i2c_readNak();
  i2c_stop();
  return min;
}

unsigned char ds1307_read_hour()
{
  unsigned char hour;
  i2c_start(0xd0);
  i2c_write(0x02);
  i2c_start(0xd1);
  hour=i2c_readNak();
  i2c_stop();
  return hour;
}

unsigned char ds1307_read_day()
{
  unsigned char day;
  i2c_start(0xd0);
  i2c_write(0x03);
  i2c_start(0xd1);
  day=i2c_readNak();
  i2c_stop();
  return day;
}

unsigned char ds1307_read_date()
{
  unsigned char date;
  i2c_start(0xd0);
  i2c_write(0x04);
  i2c_start(0xd1);
  date=i2c_readNak();
  i2c_stop();
  return date;
}

unsigned char ds1307_read_month()
{
  unsigned char month;
  i2c_start(0xd0);
  i2c_write(0x05);
  i2c_start(0xd1);
  month=i2c_readNak();
  i2c_stop();
  return month;
}

unsigned char ds1307_read_year()
{
  unsigned char year;
  i2c_start(0xd0);
  i2c_write(0x06);
  i2c_start(0xd1);
  year=i2c_readNak();
  i2c_stop();
  return year;
}

void read_uhr()
{
  bcdata[UHR_SEKUNDE] = convert_bcd_to_decimal(ds1307_read_second());
  bcdata[UHR_MINUTE] = convert_bcd_to_decimal(ds1307_read_minute());
  bcdata[UHR_STUNDE] = convert_bcd_to_decimal(ds1307_read_hour());
  sprintf(Buffer, "#ZB730,20,800,20,4,%d:%d\r", bcdata[UHR_STUNDE], bcdata[UHR_MINUTE]);
  SendDisplayData(Buffer);
}

und dann der Compiler

In file included from main.c:16:
ds1307.h:12: warning: function declaration isn't a prototype
ds1307.h:13: warning: function declaration isn't a prototype
ds1307.h:14: warning: function declaration isn't a prototype
ds1307.h:15: warning: function declaration isn't a prototype
ds1307.h:16: warning: function declaration isn't a prototype
ds1307.h:17: warning: function declaration isn't a prototype
ds1307.h:18: warning: function declaration isn't a prototype
ds1307.h:22: warning: function declaration isn't a prototype

Compiling C: ds1307.c
avr-gcc -c -mmcu=atmega128 -I. -gdwarf-2 -DF_CPU=16000000UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Wall -Wstrict-prototypes -Wa,-adhlns=./ds1307.lst  -std=gnu99 -MMD -MP -MF .dep/ds1307.o.d ds1307.c -o ds1307.o 
In file included from ds1307.c:2:
ds1307.h:12: warning: function declaration isn't a prototype
ds1307.h:13: warning: function declaration isn't a prototype
ds1307.h:14: warning: function declaration isn't a prototype
ds1307.h:15: warning: function declaration isn't a prototype
ds1307.h:16: warning: function declaration isn't a prototype
ds1307.h:17: warning: function declaration isn't a prototype
ds1307.h:18: warning: function declaration isn't a prototype
ds1307.h:22: warning: function declaration isn't a prototype
ds1307.c:94: warning: function declaration isn't a prototype
ds1307.c:105: warning: function declaration isn't a prototype
ds1307.c:116: warning: function declaration isn't a prototype
ds1307.c:127: warning: function declaration isn't a prototype
ds1307.c:138: warning: function declaration isn't a prototype
ds1307.c:149: warning: function declaration isn't a prototype
ds1307.c:160: warning: function declaration isn't a prototype
ds1307.c:171: warning: function declaration isn't a prototype

[Rolf M. (rmagnus)]

Das macht er, da du vernünftigerweise -Wstrict-prototypes angegeben 
hast.

Mit dieser Zeile

unsigned char ds1307_read_second();

sagst du dem Compiler, dass es eine Funktion ds1307_read_second gibt, 
die einen unsigned char zurückgibt und irgendeine unbekannte Liste von 
Parametern nimmt. Soll die Funktion keine Parameter haben, musst du das 
explizit sagen, indem du schreibst:

unsigned char ds1307_read_second(void);

[Christian B. (christian_buchner)]

Hallo,

ja, das habe ich versucht. Wenn ich das jedoch mache, dann mekkert der 
compiler zwar nicht, aber eine ausgabe bekomme ich dann auch nicht mehr. 
Lasse ich das als "prototyp" laufen dann bekomme ich eine ausgabe.

Das ist das was mich verwundert

[Christian B. (christian_buchner)]

Und da haben wir wieder gleich ein Problem mit dem write_display() von 
vorhin

main.c: In function 'main':
main.c:76: warning: passing argument 3 of 'write_display' makes pointer from integer without a cast

und dort steht

// Display I2C Buffer
        DisplayBufferStatus = ReadDisplayBufferStatus();
        if (DisplayBufferStatus == 1)
          {
            readdata = ReadDisplayData();
          }
      // Display I2C Buffer ende
      
      write_display(100,150,readdata);

[Peter II (Gast)]

Christian B. schrieb:
> Und da haben wir wieder gleich ein Problem mit dem write_display() von
> vorhin

du zeigst und viel zu wenig.

readdata was ist das für eine Variable?

[Christian B. (christian_buchner)]

int DisplayBufferStatus;
char readdata;

int main (void)
{

while (1)
  {
      
      // Display I2C Buffer
        DisplayBufferStatus = ReadDisplayBufferStatus();
        if (DisplayBufferStatus == 1)
          {
            readdata = ReadDisplayData();
          }
      // Display I2C Buffer ende
      
      write_display(100,150,readdata);

      read_fotowiderstand();
      display_helligkeit();
      write_display(100,100,"das ist ein test");
      
      
      
      // Display Buffer löschen
      readdata = 0;
      // Display Buffer löschen ende
  }

}

Das write_display mit dem "das ist ein test" funktioniert.

[Peter II (Gast)]

char readdata;

da wird wohl kein String reinpassen