SD_Karte Directory erstellen

[Marco G. (grmg2010)]

Moin,

ich versuche mich gerade am Schreiben auf SD-Karte. Dazu benutze ich die 
hier verlinkte Bibliothek von Daniel R.

Das Schreiben auf Datei funktioniert auch ganz gut, nun wollte ich ein 
neues Verzeichnis anlegen und dann darin die Datei erstellen. Ich 
bekomme jetzt von Atmelstudio die Meldung, "undefined reference to 
`ffmkdir' ", obwohl diese auch in der file.h zu finden sind, wie zum 
Beispiel "ffwrites()" auch.

Hier mal mein Code:

#define F_CPU 8000000UL

#include <stdlib.h>
#include <avr/io.h>
#include <util/delay.h>

#include "config.h"  
#include "file.h"
#include "fat.h"
#include "mmc.h"    
#include "uart.h"    


char buffer[];
int counter = 0;
unsigned char file_name[]="Output.txt";
unsigned char dir[]="test";

int main(void){

  DDRC |=(1<<DDC0);
  PORTC |= (1<<PC0);

  
  UART_Init();

  uart_puts("Boot");

  while (FALSE == mmc_init()){
    nop();
  }

  uart_puts("...");

  if(TRUE == fat_loadFatData()){

    uart_puts("Ok");

  }
  
  while(1){
    
      counter++;
      
      itoa(counter,buffer,10);
      PORTC ^=(1<<PC0); 
      
      ffmkdir(dir);
      ffcd(dir);
      
      // Datei existiert nicht, also anlegen !
      if(MMC_FILE_NEW == ffopen(file_name)){
      
        
       ffwrites(buffer);

       // Neue Zeile in der Datei
       ffwrite(0x0D);
       ffwrite(0x0A);

       // Schließt Datei
       ffclose();
       }


       // Datei existiert, also anhaengen !
      if(MMC_FILE_EXISTS == ffopen(file_name)){

    
       ffseek(file.length);   

       ffwrites("Countervalue: ");
       ffwrites(buffer);

       ffwrite(0x0D);
       ffwrite(0x0A);
       ffclose();
       }
     
    }

}

[Peter II (Gast)]

Marco G. schrieb:
> char buffer[];
> itoa(counter,buffer,10);

halte ich für sehr bedenklich. Die schreibst damit wild im Speicher rum

if(TRUE == fat_loadFatData()){

    uart_puts("Ok");

  }

und was ist wenn es schief gelaufen ist, dann machst du einfach weiter?

zu dem eigentlichen Problem:

> "undefined reference to `ffmkdir' ",
ist eine Fehlermeldung von Linker, es scheint also eine lib oder ein 
objekt-File zu fehler.

> obwohl diese auch in der file.h zu finden sind, wie zum
> Beispiel "ffwrites()" auch.
das hat nichts zu sagen.

[Marco G. (grmg2010)]

Peter II schrieb:
>> char buffer[];
>> itoa(counter,buffer,10);
>
> halte ich für sehr bedenklich. Die schreibst damit wild im Speicher rum

Wie würdest du es denn besser lösen?

Das es eine Fehlermeldung von Linker ist hatte ich mir bereits gedacht; 
wie könnte ich den Fehler weiter eingrenzen, um zu erfahren was genau 
fehlt?

[Peter II (Gast)]

Marco G. schrieb:
> Wie würdest du es denn besser lösen?

eine größe für die Variabel vorgeben.

> Das es eine Fehlermeldung von Linker ist hatte ich mir bereits gedacht;
> wie könnte ich den Fehler weiter eingrenzen, um zu erfahren was genau
> fehlt?
schau doch mal nach, in welcher *.c Datei die Funktion ist, gehört die 
zu deinen Projekt?

[Marco G. (grmg2010)]

Ok, bis auf die Fehlende Größe ist die Zeile also in Ordnung gewesen. 
Gut, dann vergebe ich eine Größe, um den Bereich einzugrenzen.

Die Funktion findet sich, genau wie ffwrites() in der Datei file.c. Dies 
File, wie auch die file.h gehören in mein Projekt. Es gibt auch die 
Objekt-Datei, file.o

[Peter II (Gast)]

Marco G. schrieb:
> Ok, bis auf die Fehlende Größe ist die Zeile also in Ordnung gewesen.
> Gut, dann vergebe ich eine Größe, um den Bereich einzugrenzen.

richtig, bis auf den Fehler ist alles richtig

> Die Funktion findet sich, genau wie ffwrites() in der Datei file.c. Dies
> File, wie auch die file.h gehören in mein Projekt. Es gibt auch die
> Objekt-Datei, file.o
merkwürdig, gibt es Warnungen?

[Marco G. (grmg2010)]

Ja, aber nur folgende:   Line
pointer targets in passing argument 1 of 'ffwrites' differ in signedness

Also nichts zu den anderen Funktionen, nur zu ffwrites

[Peter II (Gast)]

Marco G. schrieb:
> Ja, aber nur folgende:   Line
> pointer targets in passing argument 1 of 'ffwrites' differ in signedness
>
> Also nichts zu den anderen Funktionen, nur zu ffwrites

dann behebe das doch erst mal.

[Marco G. (grmg2010)]

Behoben. Der Fehler ist aber leider weiterhin vorhanden. Nur alle 
Warnungen sind zurückgesetzt.

[Peter II (Gast)]

suche mal ein der objekt Datei nach "ffmkdir".

[Marco G. (grmg2010)]

in der File.o? Wenn ich die mit Notepad++ öffne, kommt nur Datenwirrwar 
heraus.

[Peter II (Gast)]

Marco G. schrieb:
> in der File.o? Wenn ich die mit Notepad++ öffne, kommt nur Datenwirrwar
> heraus.

egal, suche einfach den dem string.

[Marco G. (grmg2010)]

Gibt es nicht, ich finde nur ffwrites und ffwrite

[Peter II (Gast)]

Marco G. schrieb:
> Gibt es nicht, ich finde nur ffwrites und ffwrite

lade mal die file.c hoch.

[Marco G. (grmg2010)]

Ok, wie ich in meinem Eingangspost bereits erwähnt hatte habe ich sie 
aber nicht selbst geschrieben.

[Peter II (Gast)]

Marco G. schrieb:
> Ok, wie ich in meinem Eingangspost bereits erwähnt hatte habe ich sie
> aber nicht selbst geschrieben.

merkwürde Datei...


#if (MMC_WRITE==TRUE && MMC_MKDIR==TRUE)

hast du diese defines auch gesetzt?

[Marco G. (grmg2010)]

Das fand ich auch.

Nein, müsste ich das noch zusätzlich?

[Marco G. (grmg2010)]

Deine Frage hat mich zur Lösung geführt. Ich muss in der config.h die 
Funktion zum Erstellen erst einschalten. Standardmäßig ist sie auf false 
gesetzt.

Danke für deine Hilfe :)

[Peter II (Gast)]

Marco G. schrieb:
> Deine Frage hat mich zur Lösung geführt. Ich muss in der config.h die
> Funktion zum Erstellen erst einschalten. Standardmäßig ist sie auf false
> gesetzt.
>
> Danke für deine Hilfe :)

normalerweise müssten die gleiche abfragen auch in der Header Datei drin 
sein, dann hätte man direkt einen Compiler-fehler erhalten.

Auch das "unsigned char" für String halte ich verbesserungswürdig.

[Marco G. (grmg2010)]

Welchen Daten-Typ würdest du als String vorschlagen?

[Peter II (Gast)]

Marco G. schrieb:
> Welchen Daten-Typ würdest du als String vorschlagen?

char*

[Marco G. (grmg2010)]

Ich habe noch ein wenig weiter "experimentiert" und den Code erweitert.
Ich übergebe einem MCP4921 WErte und lese sie wieder über den internen 
ADC ein. Die eingelesenen Werte schreibe ich auf die SD-Karte. 
Merkwürdigerweise ignoriert er das vorhandene File und legt alle ca 
6-7Werte ein neues File mit selben Namen an, was ja eigentlich schon 
nicht geht.
Ist das ein Problem der Programmierung, oder ein Problem unter Windows 
(wenn ich die Daten mir ansehen möchte).

Hier noch mal der Code dazu:

#define F_CPU 8000000UL

#define VTESC uart_putc(0x1b)            //Definieren des Befehls 'VTESC'
#define VTAUTO uart_puts(">")            //Definieren des Befehls 'VTAuto'
#define VTPosZero uart_puts("[f")          //Definieren des Befehls 'VTPosZero'
#define VTRESET uart_puts("c")            //Definieren des Befehls 'VTReset'
#define VTCLSLine uart_puts("[K")          //Definieren des Befehls 'VTCLSLine'
#define VTCLSbelow uart_puts("[2J")          //Definieren des Befehls 'VTCLSbelow'
#define VTCURSERoff uart_puts("[?25l")        //Definieren des Befehls 'VTCURSERoff'
#define VTdown uart_puts("#3")            //Definieren des Befehls 'VTdown'
#define VTup uart_puts("#4")            //Definieren des Befehls 'VTup'
#define VTUnderline uart_puts("[4m")        //Definieren des Befehls 'VTUnderline'
#define VTnormal uart_puts("[0")          //Definieren des Befehls 'VTnormal'
#define LF uart_putc(0x0A)              //Definieren des Befehls 'LF'
#define CR uart_putc(0x0D)              //Definieren des Befehls 'CR'

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <stdlib.h>
#include "avr.h"
#include "mcp4921.h"
#include "UART.h"
#include "SPI.h"
#include "ADC.h"
#include "VT100.h"

#include "config.h"  // Hier werden alle nötigen Konfigurationen vorgenommen, umbedingt anschauen !
#include "file.h"
#include "fat.h"
#include "mmc.h"

char buffer[15], buffer1[15], buffer2[15], buffer3[15], buffer_adc[20];
int counter = 256;
int counter_high = 0;
volatile int test = 0;
uint16_t adc_result;
int next = 0, input, counter_sd = 0;
char number;
double ADC_value = 0.0;
unsigned char file_name[]="test.txt";
int new_file = 0;


ISR(ADC_vect)
{
  //PORTD ^=(1<<PD7);
  adc_result = (double) (ADCL + (ADCH<<8));
  next = 1;
}

ISR(USART0_RX_vect)
{
  number = UDR0;
  
  switch (number)
  {
    case 'a':
      //select_adc_ch0();
      PORTD |= (1<<PD7);
    break;
    
    case 'b':
      //select_adc_ch1();
      PORTD &= ~(1<<PD7);
    break;
  }

}



int main(void)
{  
  
  avr_init();
  UART_Init();
  _delay_ms(500);
  
  uart_puts("Boot");

  // Versuch Karte zu Initialisieren, bis es klappt. Umbedingt so, weil die Initialiesierung nicht immer auf anhieb klappt.
  while (FALSE == mmc_init()){
    nop();
  }

  uart_puts("...");


  // Fat initialisieren. Nur wenn das klappt sind weitere Aktionen sinnvoll, sonst aendet das Programm !
  if(TRUE == fat_loadFatData()){

    uart_puts("Ok");    // Wenn auf dem terminal "Boot... OK" zu lesen ist, ist Init OK. Jetzt kann man schreiben/anhaengen/lesen

  }
  
  //SPI_MasterInit();
  
  VTESC;                      //Senden eines ESC-Zeichens
  VTRESET;                    //Reseten der Terminalausgabe
  
  VTESC;
  uart_puts("[02;2H");
  Send_UART_Char("Initialisiere Device: ");
  VTESC;
  uart_puts("[03;5H");
  uart_puts("- Initialisiere Register");
  VTESC;
  uart_puts("[03;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  
  VTESC;
  uart_puts("[04;5H");
  uart_puts("- Initialisiere ADC");
  VTESC;
  uart_puts("[04;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  
  VTESC;
  uart_puts("[05;5H");
  uart_puts("- Initialisiere SPI");
  VTESC;
  uart_puts("[05;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  _delay_ms(2500);
  
  
  
  VTESC;                      //Senden eines ESC-Zeichens
  VTRESET;                    //Reseten der Terminalausgabe
  VTESC;                      //Senden eines ESC-Zeichens
  VTAUTO;                      //Schalten des Terminals auf VT100
  VTESC;                      //Senden eines ESC-Zeichens
  VTPosZero;                    //Setzen des Cursors auf die Position 0/0
  VTESC;                      //Senden eines ESC-Zeichens
  VTCLSbelow;
  
  sei();
    
  /*ADCSRA |= (1<<ADSC);*/

    while(1)
    {
    vt100();
    //select_adc(number);
  
    counter++;
    counter_high = (counter >> 8);
    test = counter_high | 0b00110000;
    
    if (counter == 4090)
    {
      counter = 256;
    }
    
        MCP4921_setdata(test, counter);
    //MCP4921_latchdata();
    _delay_ms(50);
    
    //PORTC ^=(1<<PC0);
    

    if (next == 1)
    {

      ADC_value = calculate_adc(adc_result);
      
      dtostrf(ADC_value, 5, 3, buffer_adc);
      VTESC;
      uart_puts("[07;2H");
      uart_puts("Read Voltage: ");
      uart_puts(buffer_adc);
      Send_UART_Char("V ");
      next = 0;  
    }
    
    
    
    
    MCP4921_showvoltage(counter);
    
    if (counter_sd <= 100)
    {
    // Datei existiert nicht, also anlegen !
    if(MMC_FILE_NEW == ffopen(file_name)){
      
      
            ffwrites((unsigned char*)"Read Voltage: ");
      ffwrites((unsigned char*)buffer_adc);

      // Neue Zeile in der Datei
      ffwrite(0x0D);
      ffwrite(0x0A);

      // Schließt Datei
      ffclose();
      new_file = 1;
    }
    
    
    //if (ffileExsists(file_name) == TRUE)
    //{
    //  ffopen(file_name);
    //}
    
    // Datei existiert, also anhaengen !
    if(MMC_FILE_EXISTS == ffopen(file_name)){

      
      ffseek(file.length);  

      ffwrites((unsigned char*)"Read Voltage: ");
      ffwrites((unsigned char*)buffer_adc);

      ffwrite(0x0D);
      ffwrite(0x0A);
      ffclose();
      
      PORTC ^=(1<<PC0);
    }
    
    }  
    
    counter_sd++;  
    _delay_ms(100);
    
    start_converting();
    
    }
}

[Karl H. (kbuchegg)]

unsigned char ffopen( unsigned char name[]){

  unsigned char file_flag = fat_loadFileDataFromDir(name);  //pruefung ob datei vorhanden. wenn vorhanden, laden der daten auf das file struct.

  if( file_flag==TRUE ){                    /** datei existiert, anlegen nicht noetig! **/
    fat_getFatChainClustersInRow( file.firstCluster );    // verkettete cluster aus der fat-chain suchen.
    file.name = name;

    #if (MMC_MULTI_BLOCK==TRUE && MMC_OVER_WRITE==FALSE)
...

wie hast du denn MMC_MULTI_BLOCK bzw. MMC_OVER_WRITE gesetzt?

[Marco G. (grmg2010)]

Multi_Block ist False und Overwrite ist true

[Marco G. (grmg2010)]

Ok, ich habe jetzt noch mal MMC_OVERWRITE auf false gesetzt. Nun bekomme 
ich wieder nur eine Datei, wie gewünscht.

[Marco G. (grmg2010)]

Merkwürdig, jetzt bleibt der Code kurz nach dem Eintritt in die Main 
stehen, nichts geht mehr.

Hier mal der Code:

int main(void)
{  
  avr_init();
  UART_Init();
  _delay_ms(50);
  
  uart_puts("Boot");
  PORTC ^=(1<<PC1);
  
  // Versuch Karte zu Initialisieren, bis es klappt. Umbedingt so, weil die Initialiesierung nicht immer auf anhieb klappt.
  while (FALSE == mmc_init()){
    nop();
  }

  uart_puts("...");


  // Fat initialisieren. Nur wenn das klappt sind weitere Aktionen sinnvoll, sonst aendet das Programm !
  if(TRUE == fat_loadFatData()){

    uart_puts("Ok");    // Wenn auf dem terminal "Boot... OK" zu lesen ist, ist Init OK. Jetzt kann man schreiben/anhaengen/lesen
    //PORTD ^=(1<<PD7);

  }
  
  
  VTESC;                      //Senden eines ESC-Zeichens
  VTRESET;                    //Reseten der Terminalausgabe
  
  VTESC;
  uart_puts("[02;2H");
  Send_UART_Char("Initialisiere Device: ");
  VTESC;
  uart_puts("[03;5H");
  uart_puts("- Initialisiere Register");
  VTESC;
  uart_puts("[03;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  
  VTESC;
  uart_puts("[04;5H");
  uart_puts("- Initialisiere ADC");
  VTESC;
  uart_puts("[04;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  
  VTESC;
  uart_puts("[05;5H");
  uart_puts("- Initialisiere SPI");
  VTESC;
  uart_puts("[05;30H");
  _delay_ms(500);
  uart_puts("=> Done!");
  _delay_ms(2500);
  
  
  
  VTESC;                      //Senden eines ESC-Zeichens
  VTRESET;                    //Reseten der Terminalausgabe
  VTESC;                      //Senden eines ESC-Zeichens
  VTAUTO;                      //Schalten des Terminals auf VT100
  VTESC;                      //Senden eines ESC-Zeichens
  VTPosZero;                    //Setzen des Cursors auf die Position 0/0
  VTESC;                      //Senden eines ESC-Zeichens
  VTCLSbelow;
  
  sei();
  PORTC &=~(1<<PC1);  
  //ffmkdir(file_dir);
  //ffcd(file_dir);
    
  /*ADCSRA |= (1<<ADSC);*/

    while(1)
    {
    itoa(counter, buffer1, 10);
    VTESC;
    uart_puts("[09;2H");    
    uart_puts(buffer1);
    Send_UART_Char("V ");
    
    
    vt100();
    //select_adc(number);
  
    counter++;
    counter_high = (counter >> 8);
    test = counter_high | 0b00110000;
    
    if (counter == 4090)
    {
      counter = 256;
    }
    
        MCP4921_setdata(test, counter);
    //MCP4921_latchdata();
    _delay_ms(50);
    
    //PORTC ^=(1<<PC0);
    

    if (next == 1)
    {

      ADC_value = calculate_adc(adc_result);
      
      dtostrf(ADC_value, 5, 3, buffer_adc);
      VTESC;
      uart_puts("[07;2H");
      uart_puts("Read Voltage: ");
      uart_puts(buffer_adc);
      Send_UART_Char("V ");
      next = 0;  
    }
    
    
    
    
    MCP4921_showvoltage(counter);
    
    if (counter_sd <= 100)
    {
      //ffcd(file_dir);
      counter_anzahl++;
      itoa(counter_anzahl, buffer3, 10);
    // Datei existiert nicht, also anlegen !
    if(MMC_FILE_NEW == ffopen(file_name)){
      
      
      // Schreibt String auf Karte ! Nur richtige Strings koennen mit ffwrites geschrieben werden !
      ffwrites((unsigned char*)"Read Voltage: ");
      ffwrites((unsigned char*)buffer_adc);

      // Neue Zeile in der Datei
      ffwrite(0x0D);
      ffwrite(0x0A);

      // Schließt Datei
      ffclose();
      new_file = 1;
    }
    
    
    // Datei existiert, also anhaengen !
    if(MMC_FILE_EXISTS == ffopen(file_name)){

      
      ffseek(file.length);   // Spult bis zum Dateiende vor um anzuhaengen, geht auch ohne Option MMC_OVER_WRITE !
      ffwrites((unsigned char*) "Wert Nr: ");
      ffwrites((unsigned char*)buffer3);
      ffwrites((unsigned char*)" Read Voltage: ");
      ffwrites((unsigned char*)buffer_adc);

      ffwrite(0x0D);
      ffwrite(0x0A);
      ffclose();
      
      PORTC ^=(1<<PC0);
      }
    
    }  
    
    counter_sd++;  
    _delay_ms(100);
    
    start_converting();

    
    }
}

[Marco G. (grmg2010)]

Bei dem Makro MMC_OVERWRITE hätte ich gedacht, dass es das File 
Überschreibt, wenn es vorhanden ist, nicht immer neue mit dem selben 
Namen erzeugt.

Das der µC natürlich geringere Ressourcen hat als ein PC ist mir klar. 
Auch das dadurch einige Operationen nicht funktionieren.

Bei meinem Code war es allerdings so, dass er erst funktioniert hatte. 
Ich hatte dann die Anzahl der Durchläufe zum schreiben erhöht. Danach 
ging nichts mehr. Ich hatte dann zum Testen alle Dateioperationen 
auskommentiert, aber dennoch hat es nicht wieder funktioniert.
Ich denke, ich werde morgen mal den Code komplett neu schreiben (Stück 
für Stück) um zu testen, wo es beginnt zu haken.

Da ich nur eine Datei beschreibe, werde ich das ffclose() nur noch alle 
x Operationen nutzen. Anstelle der Funktion werde ich in der Schleife 
mal die Funktion ffflushData ausprobieren.

[Falk B. (falk)]

@ Marco G. (grmg2010)

>
>Bei meinem Code war es allerdings so, dass er erst funktioniert hatte.
>Ich hatte dann die Anzahl der Durchläufe zum schreiben erhöht. Danach
>ging nichts mehr.

Was ein klares Anzeichen für einen Fehler ist!

>Da ich nur eine Datei beschreibe, werde ich das ffclose() nur noch alle
>x Operationen nutzen.

FALSCH!

> Anstelle der Funktion werde ich in der Schleife
>mal die Funktion ffflushData ausprobieren.

Auch falsch!

Bei normalen Dateioperationen mit einer Lib, die den Namen FAT-Lib 
verdient, muss man mit flush gar nichts machen, darum kümmert sich die 
LIB! Man schreibt nur mit .._write() schön seine Daten in die Datei und 
am ENDE macht man EINMALIG ein close(). Damit ist der Dateizugriff 
beendet. Alles andere ist Murks!

Ein flush() ist nur in SEHR speziellen Fällen nötig.