Go Down

Topic: Needing to Re-Start my Waveshield after 35ish button presses (Read 1 time) previous topic - next topic

robinschwartzman

Hello-

I'm working with a Waveshield 1.1, an Arduino Uno, and a 1GB SD card. I have written this code so that every time button 0 is pressed, it plays one of 52 random .wav files and everytime button 1 is pressed, it plays one of two random .wav files. It works great- until about the 35th time (varies a bit each test) the button 0 is pressed. Then the audio stops working and I have to hit the reset button. Once I reset it, it works fine again, for about the same amount of time. I need it to continue to work without re-setting. Here is my code:
Code: [Select]
#include <FatReader.h>
#include <SdReader.h>
#include <avr/pgmspace.h>
#include "WaveUtil.h"
#include "WaveHC.h"


#include "WProgram.h"
int freeRam(void);
void sdErrorCheck(void);
void setup();
void check_switches();
void loop();
void playcomplete(char *name);
void playfile(char *name);
SdReader card;    // This object holds the information for the card
FatVolume vol;    // This holds the information for the partition on the card
FatReader root;   // This holds the information for the filesystem on the card
FatReader f;      // This holds the information for the file we're play

WaveHC wave;      // This is the only wave (audio) object, since we will only play one at a time

#define DEBOUNCE 5  // button debouncer
#define NUMSOUNDS 52  // how many clover sounds i want to play off the sdcard.
#define CHERSOUNDS 2 //how many cheer sounds i want to play off the sdcard.

// here is where we define the buttons that we'll use. button "1" is the first, button "6" is the 6th, etc
byte buttons[] = {14, 15, 16, 17, 18, 19};


char* cloverSounds[]={"C1.wav", "C2.wav", "C3.wav", "C4.wav", "C5.wav", "C6.wav", "C7.wav", "C8.wav", "C9.wav", "C10.wav", "C11.wav", "C12.wav", "C13.wav", "C14.wav", "C15.wav", "16.wav","C17.wav", "C18.wav", "C19.wav", "C20.wav", "C21.wav", "C22.wav", "C23.wav", "C24.wav", "C25.wav", "C26.wav", "C27.wav", "C28.wav", "C29.wav", "C30.wav", "C31.wav","C32.wav", "C33.wav", "C34.wav", "C35.wav", "C36.wav", "C37.wav", "C38.wav", "C39.wav","C57.wav", "C58.wav", "C59.wav", "C60.wav", "C61.wav", "C62.wav", "C63.wav", "C64.wav", "C65.wav", "66.wav","C67.wav", "C68.wav", "C69.wav", "C70.wav"};
char* cheerSounds[]={"H1.wav", "H4.wav"};

// This handy macro lets us determine how big the array up above is, by checking the size
#define NUMBUTTONS sizeof(buttons)
// we will track if a button is just pressed, just released, or 'pressed' (the current state)
volatile byte pressed[NUMBUTTONS], justpressed[NUMBUTTONS], justreleased[NUMBUTTONS];

// this handy function will return the number of bytes currently free in RAM, great for debugging!   
int freeRam(void)
{
  extern int  __bss_end;
  extern int  *__brkval;
  int free_memory;
  if((int)__brkval == 0) {
    free_memory = ((int)&free_memory) - ((int)&__bss_end);
  }
  else {
    free_memory = ((int)&free_memory) - ((int)__brkval);
  }
  return free_memory;
}

void sdErrorCheck(void)
{
  if (!card.errorCode()) return;
  putstring("\n\rSD I/O error: ");
  Serial.print(card.errorCode(), HEX);
  putstring(", ");
  Serial.println(card.errorData(), HEX);
  while(1);
}

void setup() {
  byte i;
 
  // set up serial port
  Serial.begin(9600);
  putstring_nl("WaveHC with ");
  Serial.print(NUMBUTTONS, DEC);
  putstring_nl("buttons");
 
  putstring("Free RAM: ");       // This can help with debugging, running out of RAM is bad
  Serial.println(freeRam());      // if this is under 150 bytes it may spell trouble!
 
  // Set the output pins for the DAC control. This pins are defined in the library
  pinMode(2, OUTPUT);
  pinMode(3, OUTPUT);
  pinMode(4, OUTPUT);
  pinMode(5, OUTPUT);

  // pin13 LED
  pinMode(13, OUTPUT);

  // Make input & enable pull-up resistors on switch pins
  for (i=0; i< NUMBUTTONS; i++) {
    pinMode(buttons[i], INPUT);
    digitalWrite(buttons[i], HIGH);
  }
 
  //  if (!card.init(true)) { //play with 4 MHz spi if 8MHz isn't working for you
  if (!card.init()) {         //play with 8 MHz spi (default faster!) 
    putstring_nl("Card init. failed!");  // Something went wrong, lets print out why
    sdErrorCheck();
    while(1);                            // then 'halt' - do nothing!
  }
 
  // enable optimize read - some cards may timeout. Disable if you're having problems
  card.partialBlockRead(true);

// Now we will look for a FAT partition!
  uint8_t part;
  for (part = 0; part < 5; part++) {     // we have up to 5 slots to look in
    if (vol.init(card, part))
      break;                             // we found one, lets bail
  }
  if (part == 5) {                       // if we ended up not finding one  :(
    putstring_nl("No valid FAT partition!");
    sdErrorCheck();      // Something went wrong, lets print out why
    while(1);                            // then 'halt' - do nothing!
  }
 
  // Lets tell the user about what we found
  putstring("Using partition ");
  Serial.print(part, DEC);
  putstring(", type is FAT");
  Serial.println(vol.fatType(),DEC);     // FAT16 or FAT32?
 
  // Try to open the root directory
  if (!root.openRoot(vol)) {
    putstring_nl("Can't open root dir!"); // Something went wrong,
    while(1);                             // then 'halt' - do nothing!
  }
 
  // Whew! We got past the tough parts.
  putstring_nl("Ready!");
 
  TCCR2A = 0;
  TCCR2B = 1<<CS22 | 1<<CS21 | 1<<CS20;

  //Timer2 Overflow Interrupt Enable
  TIMSK2 |= 1<<TOIE2;


}

SIGNAL(TIMER2_OVF_vect) {
  check_switches();
}

void check_switches()
{
  static byte previousstate[NUMBUTTONS];
  static byte currentstate[NUMBUTTONS];
  byte index;

  for (index = 0; index < NUMBUTTONS; index++) {
    currentstate[index] = digitalRead(buttons[index]);   // read the button
   
    /*     
    Serial.print(index, DEC);
    Serial.print(": cstate=");
    Serial.print(currentstate[index], DEC);
    Serial.print(", pstate=");
    Serial.print(previousstate[index], DEC);
    Serial.print(", press=");
    */
   
    if (currentstate[index] == previousstate[index]) {
      if ((pressed[index] == LOW) && (currentstate[index] == LOW)) {
          // just pressed
          justpressed[index] = 1;
      }
      else if ((pressed[index] == HIGH) && (currentstate[index] == HIGH)) {
          // just released
          justreleased[index] = 1;
      }
      pressed[index] = !currentstate[index];  // remember, digital HIGH means NOT pressed
    }
    //Serial.println(pressed[index], DEC);
    previousstate[index] = currentstate[index];   // keep a running tally of the buttons
  }
}


void loop() {
  byte i;
 
    if (justpressed[0]) {
    justpressed[0] = 0;
       
       int n = random(NUMSOUNDS);
       playfile(cloverSounds[n]);

  }
  else if (justpressed[1]) {
    justpressed[1] = 0;
   
       int x = random(CHERSOUNDS);
       playfile(cheerSounds[x]);
  }

}



// Plays a full file from beginning to end with no pause.
void playcomplete(char *name) {
  // call our helper to find and play this name
  playfile(name);
  while (wave.isplaying) {
  // do nothing while its playing
  }
  // now its done playing
}

void playfile(char *name) {
  // see if the wave object is currently doing something
  if (wave.isplaying) {// already playing something, so stop it!
    wave.stop(); // stop it
  }
  // look in the root directory and open the file
  if (!f.open(root, name)) {
    putstring("Couldn't open file "); Serial.print(name); return;
  }
  // OK read the file and turn it into a wave object
  if (!wave.create(f)) {
    putstring_nl("Not a valid WAV"); return;
  }
 
  // ok time to play! start playback
  wave.play();
}

int main(void)
{
init();

setup();
   
for (;;)
loop();
       
return 0;
}



Is there something in the code I need to write to help clear out the memory as it uses each file? Any help/suggestions would be much appreciated!!

johnwasser

Code: [Select]

  // OK read the file and turn it into a wave object
  if (!wave.create(f)) {
    putstring_nl("Not a valid WAV"); return;
  }


Is the wave object something you are supposed to delete when you are done with it?
Send Bitcoin tips to: 1G2qoGwMRXx8az71DVP1E81jShxtbSh5Hp

robinschwartzman

I guess in this case, deleting the object when I'm done with it would be helpful to keep things clean. Do you know of how to go about this?

MarkT

You need to call close() on the FatReader don't you?
[ I will NOT respond to personal messages, I WILL delete them, use the forum please ]

robinschwartzman

Probably so- I'm a newbie coder, so I'm not sure where this command would go in the code?

robinschwartzman

Good news- problem solved!
I just had to void out this line of code (as recommended right in the code). Silly mistake!
Code: [Select]
 // enable optimize read - some cards may timeout. Disable if you're having problems
 //card.partialBlockRead(true);


Go Up