Updates example that uses interrupt

* Adds additional parameter to routeHardwareIntterupt so that the hardware pin can be selected, defaults to pin one
* Fixes the hardware interrupt selection values
* Adds power control to the various functions that can't be changed without the chip sleeping.

Author: edspark

examples/Ex2_Interrupts/Ex2_Interrupts.ino

@@ -4,13 +4,10 @@
 QwiicKX132 kxAccel;
 outputData myData; // This will hold the accelerometer's output. 
 int dataReadyPin = D1;
-int timeThen;
-int timeNow; 
 
 void setup() {
 
-  pinMode(dataReadyPin, OUTPUT);
-  digitalWrite(dataReadyPin, HIGH);
+  pinMode(dataReadyPin, INPUT);
 
   while(!Serial){
     delay(50);
@@ -30,40 +27,35 @@ void setup() {
 
 
   if( !kxAccel.initialize(INT_SETTINGS)){
-    Serial.println("Could not initialize the chip.");
+    Serial.println("Could not initialize the chip. Freezing.");
     while(1);
   }
   else
     Serial.println("Initialized...");
 
   // kxAccel.setRange(KX132_RANGE16G);
-  timeThen = millis();
 
 }
 
 void loop() {
 
-  timeNow = millis();
-
-  if(timeNow - timeThen > 1000){
-    timeThen = timeNow;
-    digitalWrite(dataReadyPin, LOW);
-    Serial.println("click.");
+  if( digitalRead(dataReadyPin) == HIGH ){ // Wait for new data to be ready.
+
+    myData = kxAccel.getAccelData();
+    digitalWrite(dataReadyPin, HIGH);
+    Serial.print("X: ");
+    Serial.print(myData.xData, 4);
+    Serial.print("g ");
+    Serial.print(" Y: ");
+    Serial.print(myData.zData, 4);
+    Serial.print("g ");
+    Serial.print(" Z: ");
+    Serial.print(myData.zData, 4);
+    Serial.println("g ");
+    
+     //kxAccel.clearInterrupt();// Because the data is being read in "burst"
+     //mode, meaning that all the acceleration data is being read at once, we don't
+     //need to clear the interrupt.
   }
-
-  myData = kxAccel.getAccelData();
-  //digitalWrite(dataReadyPin, HIGH);
-  //Serial.print("X: ");
-  //Serial.print(myData.xData, 4);
-  //Serial.print("g ");
-  //Serial.print(" Y: ");
-  //Serial.print(myData.zData, 4);
-  //Serial.print("g ");
-  //Serial.print(" Z: ");
-  //Serial.print(myData.zData, 4);
-  //Serial.println("g ");
-  
-  // kxAccel.clearInterrupt();
-
   delay(20); // Delay should be 1/ODR (Output Data Rate), default is 50Hz
 }

src/SparkFun_Qwiic_KX13X.cpp

@@ -75,7 +75,7 @@ bool QwiicKX13xCore::initialize(uint8_t settings)
   if( settings == DEFAULT_SETTINGS )
     returnError = writeRegister(KX13X_CNTL1, 0x00, DEFAULT_SETTINGS, 0);
   if( settings == INT_SETTINGS ){
-    setInterruptPin(true);
+    setInterruptPin(true, 1);
     routeHardwareInterrupt(HI_DATA_READY);
     returnError = writeRegister(KX13X_CNTL1, 0x00, INT_SETTINGS, 0);
   }
@@ -190,29 +190,51 @@ bool QwiicKX13xCore::setInterruptPin(bool enable, uint8_t polarity, uint8_t puls
   else if( latchControl < 0 | latchControl > 4 )
     return false;
 
+  uint8_t accelState = readAccelState(); // Put it back where we found it.
+  accelControl(false); // Can't adjust without putting to sleep
+
   uint8_t combinedArguments = ((pulseWidth << 6) | (enable << 5) | (polarity << 4) | (latchControl << 3));
   KX13X_STATUS_t returnError;
-  returnError = writeRegister(KX13X_INC1, 0x07, combinedArguments, 3);
-  if( returnError == KX13X_SUCCESS )
+  returnError = writeRegister(KX13X_INC1, 0x07, combinedArguments, 0);
+  if( returnError == KX13X_SUCCESS ){
+    accelControl(accelState); 
     return true;
+  }
   else
     return false;
 }
 
 
-// Address: 0x25, bit[4]: default value is 0: disabled
-// Enables the data ready bit to be reported on the hardware interrupt. 
-bool QwiicKX13xCore::routeHardwareInterrupt(uint8_t rdr){
+// Address: 0x25, bits[7:0]: default value is 0: disabled
+// Enables anyone of the various interrupt settings to be routed to hardware
+// interrupt pin one or pin two.
+bool QwiicKX13xCore::routeHardwareInterrupt(uint8_t rdr, uint8_t pin){
 
-  if( rdr < 0 | rdr > 6 )
+  if( rdr < 0 | rdr > 128 )
+    return false;
+  if( pin != 1 && pin != 2)
     return false;
 
+  uint8_t accelState = readAccelState(); // Put it back where we found it.
+  accelControl(false); // Can't adjust without putting to sleep
+
   KX13X_STATUS_t returnError;
-  returnError = writeRegister(KX13X_INC4, 0xEF, rdr, 4);
-  if( returnError == KX13X_SUCCESS )
-    return true;
+
+  if( pin == 1 ){
+    returnError = writeRegister(KX13X_INC4, 0x00, rdr, 0);
+    if( returnError == KX13X_SUCCESS ){
+      accelControl(accelState); 
+      return true;
+    }
   else
-    return false;
+    returnError = writeRegister(KX13X_INC6, 0x00, rdr, 0);
+    if( returnError == KX13X_SUCCESS ){
+      accelControl(accelState);
+      return true;
+    }
+  }
+
+  return false;
 
 }
 
@@ -513,6 +535,7 @@ KX13X_STATUS_t QwiicKX13xCore::writeRegister(uint8_t reg, uint8_t mask, uint8_t
     return KX13X_I2C_ERROR;
   tempRegVal &= mask;
   tempRegVal |= (data << bitPos); 
+  Serial.println(tempRegVal, HEX);
 
 	if( _i2cPort == NULL ) {
 

src/SparkFun_Qwiic_KX13X.h

@@ -171,13 +171,16 @@ typedef enum {
 } KX13X_STATUS_t;
 
 enum HARDWARE_INTERRUPT {
-  HI_FREEFALL = 0x00,
-  HI_WATERMARK, 
-  HI_DATA_READY,
-  HI_BACK_TO_SLEEP,
-  HI_TAP_DOUBLE_TAP,
-  HI_WAKE_UP, 
-  HI_TILT_POSITION
+
+  HI_TILT_POSITION = 0x01,
+  HI_WAKE_UP = 0x02, 
+  HI_TAP_DOUBLE_TAP = 0x04,
+  HI_BACK_TO_SLEEP = 0x08,
+  HI_DATA_READY = 0x10,
+  HI_WATERMARK = 0x20, 
+  HI_BUFFER_FULL = 0x40,
+  HI_FREEFALL = 0x80
+
 };
 
 
@@ -197,7 +200,7 @@ class QwiicKX13xCore
     bool setOutputDataRate(uint8_t);
     float readOutputDataRate();
     bool setInterruptPin(bool enable, uint8_t polarity = 0, uint8_t pulseWidth = 0, bool latchControl = false);
-    bool routeHardwareInterrupt(uint8_t);
+    bool routeHardwareInterrupt(uint8_t, uint8_t pin = 1);
     bool clearInterrupt();
     bool dataTrigger();
     bool setBufferThreshold(uint8_t);