Interruption Vector

[flylowgofast]

Hello all,

few days ago I sent a post to know if I can adapt my hardware to MySensors. SomeOne explained me that is possible, because there isn't constraint with pin to use. That's right, I already have two nodes which work with temperature and door switch.

But now I have an issue with a node which runs two triacs. I need AC power synchronization to drive my triacs and I use ISR (INT0_vect) { }, but when I compile the sketche I have : multiple definition of `__vector_1'

I think it occurs because MySensors use this interruption too ?

Could you help me ?

regards

[sundberg84]

@flylowgofast Post your sketch so we can have a look.

[flylowgofast]

ok, but it's a little bit long, this is the part : ISR(INT0_vect) {

#include <MySensor.h>  
#include <SPI.h>
#include <DallasTemperature.h>
#include <OneWire.h>
#include "config.h"           //Définition des constantes pour le projet DomoFab

#define COMPARE_TEMP 1 // Send temperature only if changed? 1 = Yes 0 = No

#define MAX_ATTACHED_DS18B20 1
unsigned long SLEEP_TIME = 30000; // Sleep time between reads (in milliseconds)
OneWire oneWire(ONE_WIRE_BUS); // Setup a oneWire instance to communicate with any OneWire devices (not just Maxim/Dallas temperature ICs)
DallasTemperature sensors(&oneWire); // Pass the oneWire reference to Dallas Temperature.
MyTransportNRF24 transport(CE, CSN, RF24_PA_LEVEL_GW);
MySensor gw(transport);// signerRADIO_RX_LED_PIN, RADIO_TX_LED_PIN, RADIO_ERROR_LED_PIN);
float lastTemperature[MAX_ATTACHED_DS18B20];
int numSensors=0;
boolean receivedConfig = false;
boolean metric = true; 
// Initialize temperature message
MyMessage msgTemp(0,V_TEMP);
MyMessage msgDoor(1,V_STATUS);

uint32_t triacTempo;
volatile uint8_t triac1 = false;
volatile uint8_t triac2 = false;
uint16_t analogUp;
uint16_t analogDown;
uint8_t doorOpenState;
uint8_t doorOpenChange = 99; //pour passer la première fois

unsigned long myTime = 15000; // Pour faire une lecture des valeurs tout de suite

// ???????????????????????????  Who am I ?????????????????????????????????????????????????????????????????????????????????????????????????????????????
 byte role = API2; // première carte avec pro mini capteur de température et capteur de porte
// ??????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????????

// the setup routine runs once when you press reset:
void setup() {

  printf("DomoFab rRF24 API n°: %u start\n\n\r", role);

  // initialize the digital pin as an output.
  pinMode(HALL, INPUT);
  pinMode(TRIAC1, OUTPUT); // output pin for OC2B
  pinMode(TRIAC2, OUTPUT); // output pin for OC1B
  pinMode(ZEROCROSS, INPUT); //passage par zéro du réseau INT0
  digitalWrite(UP, HIGH);    //Set pull up resistor
  digitalWrite(DOWN, HIGH);  //Set pull up resistor

  EIMSK = (1 << INT0);       //INT 0 enable
  EICRA = (1 << ISC00);      //autoriser n'importe quel changement sur INT0 (D2) à générer une interruption;

  printf ("\nConfiguration : API2\n\r");

  // Startup up the OneWire library
  sensors.begin();
  // requestTemperatures() will not block current thread
  sensors.setWaitForConversion(false);

  // Startup and initialize MySensors library. Set callback for incoming messages. 
  gw.begin();

  // Send the sketch version information to the gateway and Controller
  gw.sendSketchInfo("Temperature Sensor", "1.1");

  // Fetch the number of attached temperature sensors  
  numSensors = sensors.getDeviceCount();
  Serial.print("nombre de capteur : ");
  Serial.println(numSensors);

  // Present all sensors to controller
  gw.present(0, S_TEMP);
  gw.present(1, S_DOOR);
}

/*ISR(TIMER2_COMPB_vect)        // interrupt service routine
{
}

ISR(TIMER1_COMPB_vect)        // interrupt service routine
{
}
*/
ISR(INT0_vect) {
  if (triac1) {
    digitalWrite(TRIAC1, HIGH);
    digitalWrite(TRIAC1, LOW);
    delayMicroseconds(100);
    digitalWrite(TRIAC1, HIGH);
    digitalWrite(TRIAC1, LOW);
    digitalWrite(TRIAC1, HIGH);
    digitalWrite(TRIAC1, LOW);
    digitalWrite(TRIAC1, HIGH);
    digitalWrite(TRIAC1, LOW);
  } else {
    if (triac2) {
      digitalWrite(TRIAC2, HIGH);
      digitalWrite(TRIAC2, LOW);
      delayMicroseconds(100);
      digitalWrite(TRIAC2, HIGH);
      digitalWrite(TRIAC2, LOW);
      digitalWrite(TRIAC2, HIGH);
      digitalWrite(TRIAC2, LOW);
      digitalWrite(TRIAC2, HIGH);
      digitalWrite(TRIAC2, LOW);
    }
  }
}

// the loop routine runs over and over again forever:
void loop() {
  // Process incoming messages (like config from server)
  gw.process(); 

  //---------------Lecture des valeurs------------------
  if ((millis() - myTime) > 10000) { // toutes les dix secondes
    myTime = millis();
    // Attention à la fréquence 16 ou 8 Mhz pro mini Chinois ou pas !!!
    // Sinon ca ne marche pas et je cherche pendant 3 heures !!!
    // Fetch temperatures from Dallas sensors
    sensors.requestTemperatures();
  }
  // Fetch and round temperature to one decimal
  float temperature = static_cast<float>(static_cast<int>((gw.getConfig().isMetric?sensors.getTempCByIndex(0):sensors.getTempFByIndex(0)) * 10.)) / 10.;
 
  // Only send data if temperature has changed and no error
  #if COMPARE_TEMP == 1
  if (lastTemperature[0] != temperature && temperature != -127.00 && temperature != 85.00) {
  #else
  if (temperature != -127.00 && temperature != 85.00) {
  #endif
    // Send in the new temperature
    gw.send(msgTemp.setSensor(0).set(temperature,1));
    // Save new temperatures for next compare
    lastTemperature[0]=temperature;
  }
  doorOpenState = digitalRead(HALL); // lecture à chaque cycle (urgence)
  if (doorOpenState != doorOpenChange){
     
    gw.send(msgDoor.setSensor(1).set(doorOpenState));
    doorOpenChange = doorOpenState;
  }

[sundberg84]

Hi.
Cant se any definition of the vector_1 in your code - maybe in any of the libraries?
There are some info in the arduino forum: http://forum.arduino.cc/index.php?topic=47992.0 have a look and see if that helps you out.

[flylowgofast]

Yes you're right,
before MySensors my code was ok, but now MySensors use a librairy in ~/arduino-1.6.7/harware/arduino/avr/cores/arduino/WInterrupts.c
So this use of interrupt is already declare in this library !
I don't now how to avoid the issue ?

[martinhjelmare]

@flylowgofast

Hi!

Use attachInterrupt instead.
http://gammon.com.au/interrupts

And don't use delay inside the ISR. Variables should be volatile.

[flylowgofast]

@martinhjelmare

Hi, thank you for your reply, the link is very usefull.
You said variables should be Volatile, this is the case for my variable triac1 & 2, but I use two other "const" :

static const uint8_t TRIAC1= 3;    //Triac 1 associé au timer 2 OC2B pour la sortie D3
static const uint8_t TRIAC2= 10;   //Triac 2 associé au timer 1 OC1B pour la sortie D10 

Have I to add volatile in this case too ?

Thanks

[mfalkvidd]

"C's volatile keyword is a qualifier that is applied to a variable when it is declared. It tells the compiler that the value of the variable may change at any time--without any action being taken by the code the compiler finds nearby" (source: http://www.barrgroup.com/Embedded-Systems/How-To/C-Volatile-Keyword )

So volatile only affects changes to variables. Your const variables will not change, so there is no need to declare them volatile.

[flylowgofast]

thanks to all your contributions, it becomes clear for me.