Algmeine Funktion

[Markus (Gast)]

Hallo,
ich versuche gerade mit einer Funktion mein Programm zu verschönern.
Das Gesamte Programm liest eine I2C RTC(PCF8583P) aus, deshalb verwende 
ich die Bibliothek von Peter Fleury.

Die zwei ausgelesnen Bytes werden in diesen Variabeln gespeichert:
buff_sekunde
buff_minute

Nun hab ich mir gedach ich schreib mir eine Funktion

uint8_t getsm (uint8_t  ) {
  uint8_t zehn = 0 ;
  
  
  
  ergebnis = buff & 0b00001111; // maskieren der 1er Stelle 
  zehn = buff & 0b01110000; // hier die 10er Stelle
  zehn = zehn >> 4 ; //um 4 Stellen zurückschieben für richtige Wertichkeit 
  ergebnis = (ergebnis >> 4) * 10 + (ergebnis & 0x0F); // umwandeln von BDC in Binär
  ergebnis = ergebnis + zehn *10 ; //Gesamt Ergebnis
  
  
  
  
  
  
  return (); // In der Klammer sollte die Rückgabevariable stehen
  
};

Im "main" frage ich einefach das Ergbniss ab

sekunde= getsm(buff_sekunde) ;
minute= getsm(buff_minute);

Zu erwähnen ist noch, dass ich die Funktion in eine *.c Datei 
ausgelagert habe. Aber ich weiß nicht wie die Variablen Übergabe 
aussehen soll? Was soll ich in return() schreiben?

Vielen Danke wenn jemand so nett wäre und mich einwenig unterstützen 
kann.
Ich lerne grade µC programmieren in C nur durch Tutorials. I2C und LCD 
laufen schon mal die Uhr läuft auch schon =) Die komplette Uhr ist so zu 
sagen mein Lehrnprojekt.

[Georg G. (df2au)]

Vorschlag: Nachlesen, wie man eine Funktion deklariert.

Markus schrieb:
> uint8_t getsm (uint8_t  ) {
Da fehlt doch was? Wie wäre es mit
uint8_t getsam(uint8_t buff) {

>   return (); // In der Klammer sollte die Rückgabevariable stehen
Warum nicht einfach
return(ergebniss);

[Easylife (Gast)]

ergebnis = buff & 0b00001111; // maskieren der 1er Stelle 
ergebnis = (ergebnis >> 4) * 10 + (ergebnis & 0x0F); // umwandeln von BDC

die 2. Zeile ist vollkommen überflüssig.
(ergebnis >> 4) * 10
ist in deinem Fall immer null


im Grunde läuft es auf das hier hinaus:

uint8_t getsm (uint8_t value_bcd) {
  uint8_t value_dec;
  value_dec = 10 * (value_bcd >> 4) + (value_bcd & 0x0f);
  return (value_dec);
};