Help! Problems with PIR Sensor

Hi there

Been reading a lot on the forum, adafruit and searching on google. My eyes are bleeding. Please help me :slight_smile:

So I'm building a setup with an Arduino Uno, Adafruit waveshield and a PIR sensor. My goal is to trigger random wavefiles when motion is detected by the PIR sensor.

My problem is:
My serial monitor doesn't show any data?

And it seems like the sensor is triggered continuously and playing the wavefiles in a loop?

I'm also getting this warning message: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings] playcomplete("WAVESH~6.WAV");

Here is a pic of the setup:

And here is my code:

#include <FatReader.h>
#include <SdReader.h>
#include <avr/pgmspace.h>
#include "WaveUtil.h"
#include "WaveHC.h"


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

uint8_t dirLevel; // indent level for file/dir names    (for prettyprinting)
dir_t dirBuf;     // buffer for directory reads

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

// for PIR
  int inputPin = 1;               // choose the input pin (for PIR sensor)
  int pirState = LOW;             // we start, assuming no motion detected
  int val = 0;                    // variable for reading the pin statu

#define DEBOUNCE 100  // button debouncer

// 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() {
  // set up serial port
  Serial.begin(9600);
  putstring_nl("WaveHC with 6 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);

  //  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!");
}

void loop() {
  //putstring(".");            // uncomment this to see if the loop isnt running
  switch (check_switches()) {
    case 1:
      playcomplete("WAVESH~1.WAV");
      break;
    case 2:
      playcomplete("WAVESH~2.WAV");
      break;
    case 3:
      playcomplete("WAVESH~3.WAV");
      break;
    case 4:
      playcomplete("WAVESH~4.WAV");
      break;
    case 5:
      playcomplete("WAVESH~5.WAV");
      break;
    case 6:
      playcomplete("WAVESH~6.WAV");
  }
}

byte check_switches()
{
  if (digitalRead(1) == HIGH)  // check for high signal on pin 1 (analog 0)
    {
       return random(1,6);
    }
  return 0;
}


// 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();
}

Thank you soo much!

  if (digitalRead(1) == HIGH)  // check for high signal on pin 1 (analog 0)

There seems to be some confusion between digital and analogue pins and pin numbering. The statement above will treat pin 1 (not A1) as a digital pin. Which pin is the PIR connected to ?

If it is pin 1 (not A1) then it will interfere with Serial comms on a Uno because it is used by the Serial interface

Start with a simpler program that just loops

• reading the sensor and

• printing the value and/or illuminating an LED on another pin, or use the build-in LED on your board.

If the sensor, um, makes no sense, it is pointless to worry about why the rest of the code doesn't work.

Divide and conquer:

Replace the sensor with a pushbutton for simulating the sensor. This will allow you to test the logic that doe what the sensor is gonna end up doing.

With both parts working, combine your successes.

HTH

a7

Of course, that did the trick. Switched input to pin 12 and now it works like a charm. Thank you so much!

This topic was automatically closed 120 days after the last reply. New replies are no longer allowed.