Arduino Forum

Using Arduino => Audio => Topic started by: robinschwartzman on Mar 15, 2011, 06:38 pm

Title: Needing to Re-Start my Waveshield after 35ish button presses
Post by: robinschwartzman on Mar 15, 2011, 06:38 pm
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!!
Title: Re: Needing to Re-Start my Waveshield after 35ish button presses
Post by: johnwasser on Mar 15, 2011, 08:27 pm
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?
Title: Re: Needing to Re-Start my Waveshield after 35ish button presses
Post by: robinschwartzman on Mar 15, 2011, 09:25 pm
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?
Title: Re: Needing to Re-Start my Waveshield after 35ish button presses
Post by: MarkT on Mar 16, 2011, 12:38 am
You need to call close() on the FatReader don't you?
Title: Re: Needing to Re-Start my Waveshield after 35ish button presses
Post by: robinschwartzman on Mar 16, 2011, 03:52 am
Probably so- I'm a newbie coder, so I'm not sure where this command would go in the code?
Title: Re: Needing to Re-Start my Waveshield after 35ish button presses
Post by: robinschwartzman on Mar 16, 2011, 08:31 pm
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);