]> git.zerfleddert.de Git - fhem-stuff/blobdiff - culfw/culfw-asksin-fix.diff
rebase against r378
[fhem-stuff] / culfw / culfw-asksin-fix.diff
index 168ad477b42dfdda427084da0dcdcbcf1be4ce5f..65dc3bbe15bbbc89422460e1627751822a0c8319 100644 (file)
 Index: clib/rf_asksin.c
 ===================================================================
---- clib/rf_asksin.c   (revision 373)
+--- clib/rf_asksin.c   (revision 378)
 +++ clib/rf_asksin.c   (working copy)
-@@ -11,8 +11,8 @@
+@@ -9,15 +9,18 @@
  
+ #include "rf_asksin.h"
++//we receive a new byte approximately every 8 ms...
++#define RX_TIMEOUT_MS 10
++
  uint8_t asksin_on = 0;
  
--const uint8_t PROGMEM ASKSIN_CFG[50] = {
+ const uint8_t PROGMEM ASKSIN_CFG[] = {
 -     0x00, 0x07,
-+const uint8_t PROGMEM ASKSIN_CFG[] = {
 +     0x00, 0x01,
       0x02, 0x2e,
       0x03, 0x0d,
       0x04, 0xE9,
-@@ -26,18 +26,21 @@
+      0x05, 0xCA,
+-     0x07, 0x0C,
++     0x07, 0x04,
+      0x0B, 0x06,
+      0x0D, 0x21,
+      0x0E, 0x65,
+@@ -26,7 +29,7 @@
       0x11, 0x93,
       0x12, 0x03,
       0x15, 0x34,
--     0x17, 0x30, // always go into IDLE
-+     0x17, 0x33, // go into RX after TX, CCA; ELV uses 0x03
+-     0x17, 0x33, // go into RX after TX, CCA; EQ3 uses 0x03
++     0x17, 0x3F, // always go into RX, CCA; EQ3 uses 0x03
       0x18, 0x18,
       0x19, 0x16,
       0x1B, 0x43,
-      0x21, 0x56,
-      0x25, 0x00,
-      0x26, 0x11,
-+     0x29, 0x59,
-+     0x2c, 0x81,
-      0x2D, 0x35,
--     0x3e, 0xc3,
--     0xff
-+     0x3e, 0xc3
+@@ -39,6 +42,7 @@
+      0x3e, 0xc3
  };
  
-+static void rf_asksin_reset_rx(void);
-+
- void
- rf_asksin_init(void)
- {
-@@ -56,20 +59,27 @@
-   my_delay_us(100);
-   // load configuration
--  for (uint8_t i = 0; i<50; i += 2) {
--       
--    if (pgm_read_byte( &ASKSIN_CFG[i] )>0x40)
--      break;
--
-+  for (uint8_t i = 0; i < sizeof(ASKSIN_CFG); i += 2) {
-     cc1100_writeReg( pgm_read_byte(&ASKSIN_CFG[i]),
-                      pgm_read_byte(&ASKSIN_CFG[i+1]) );
-   }
-   
-   ccStrobe( CC1100_SCAL );
++static inline uint8_t read_cc1100_rxbytes(void);
+ static void rf_asksin_reset_rx(void);
  
--  my_delay_ms(1);
-+  my_delay_ms(4);
-+
-+  ccRX();
+ void
+@@ -74,12 +78,29 @@
+   } while (cc1100_readReg(CC1100_MARCSTATE) != MARCSTATE_RX);
  }
  
-+static void
-+rf_asksin_reset_rx(void)
++// Workaround for CC1101 Errata 3
++static inline uint8_t
++read_cc1100_rxbytes(void)
 +{
-+  ccStrobe( CC1100_SFRX  );
-+  ccStrobe( CC1100_SIDLE );
-+  ccStrobe( CC1100_SNOP  );
-+  ccStrobe( CC1100_SRX   );
++  uint8_t rxbytes, rxbytes2;
++  
++  rxbytes = cc1100_readReg(CC1100_RXBYTES);
++  while((rxbytes2 = cc1100_readReg(CC1100_RXBYTES)) != rxbytes)
++    rxbytes = rxbytes2;
++
++  return rxbytes;
 +}
 +
- void
- rf_asksin_task(void)
static void
+ rf_asksin_reset_rx(void)
  {
-@@ -83,12 +93,14 @@
+   ccStrobe( CC1100_SFRX  );
+   ccStrobe( CC1100_SIDLE );
+   ccStrobe( CC1100_SNOP  );
++
++  while (read_cc1100_rxbytes() & 0x7f)
++    cc1100_readReg(CC1100_RXFIFO);
++
+   ccStrobe( CC1100_SRX   );
+ }
  
-   // see if a CRC OK pkt has been arrived
-   if (bit_is_set( CC1100_IN_PORT, CC1100_IN_PIN )) {
--
+@@ -90,13 +111,29 @@
+   uint8_t dec[MAX_ASKSIN_MSG];
+   uint8_t rssi;
+   uint8_t l;
++  uint8_t rxfifo_cnt;
++  uint16_t timeout;
+   if(!asksin_on)
+     return;
+-  // see if a CRC OK pkt has been arrived
+-  if (bit_is_set( CC1100_IN_PORT, CC1100_IN_PIN )) {
++  // see if there is data to be read
++  while (bit_is_set( CC1100_IN_PORT, CC1100_IN_PIN )) {
++    rxfifo_cnt = read_cc1100_rxbytes();
++
++    if (rxfifo_cnt & 0x80) // Overflow
++      break;
++
++    rxfifo_cnt &= 0x7f;
++
++    // We must not read the last byte from the RX fifo while RX is in progress (Errata 1)
++    while ((rxfifo_cnt < 2) && (cc1100_readReg(CC1100_PKTSTATUS) & (1 << 3))) {
++      my_delay_ms(1);
++      rxfifo_cnt = read_cc1100_rxbytes() & 0x7f;
++    }
++
      enc[0] = cc1100_readReg( CC1100_RXFIFO ) & 0x7f; // read len
++    rxfifo_cnt--;
+     if (enc[0] >= MAX_ASKSIN_MSG) {
+       // Something went horribly wrong, out of sync?
+@@ -104,6 +141,27 @@
+       return;
+     }
  
--    if (enc[0]>=MAX_ASKSIN_MSG)
--         enc[0] = MAX_ASKSIN_MSG-1;
--    
-+    if (enc[0] >= MAX_ASKSIN_MSG) {
-+      // Something went horribly wrong, out of sync?
-+      rf_asksin_reset_rx();
-+      return;
++    if ((enc[0] + 2) > rxfifo_cnt) {
++      timeout = RX_TIMEOUT_MS * ((enc[0] + 2) - rxfifo_cnt);
++      while (timeout-- && ((enc[0] + 2) > rxfifo_cnt)) { // Wait for more data
++        my_delay_ms(1);
++        rxfifo_cnt = read_cc1100_rxbytes();
++
++        if (rxfifo_cnt & 0x80) { // Overflow
++          rf_asksin_reset_rx();
++          return;
++      }
++ 
++        rxfifo_cnt &= 0x7f;
++      }
++
++      if (!timeout) {
++        // Not enough data received, out of sync?
++        rf_asksin_reset_rx();
++        return;
++      }
 +    }
 +
      CC1100_ASSERT;
      cc1100_sendbyte( CC1100_READ_BURST | CC1100_RXFIFO );
      
-@@ -97,12 +109,10 @@
+@@ -112,14 +170,19 @@
      }
      
      rssi = cc1100_sendbyte( 0 );
--    
-+    l = cc1100_sendbyte( 0 );
-+
-     CC1100_DEASSERT;
--    ccStrobe( CC1100_SFRX  );
--    ccStrobe( CC1100_SIDLE );
--    ccStrobe( CC1100_SNOP  );
-     ccStrobe( CC1100_SRX   );
+-    /* LQI = */ cc1100_sendbyte( 0 );
  
-     dec[0] = enc[0];
-@@ -113,7 +123,6 @@
-     
-     dec[l] = enc[l] ^ dec[2];
-     
--    
-     if (tx_report & REP_BINTIME) {
-       
-       DC('a');
-@@ -131,26 +140,17 @@
-       
-       DNL();
-     }
-+  }
--    return;
--       
-+  switch(cc1100_readReg( CC1100_MARCSTATE )) {
-+    case MARCSTATE_RXFIFO_OVERFLOW:
-+      ccStrobe( CC1100_SFRX  );
-+    case MARCSTATE_IDLE:
-+      ccStrobe( CC1100_SIDLE );
-+      ccStrobe( CC1100_SNOP  );
-+      ccStrobe( CC1100_SRX   );
-+      break;
-   }
--       
--       
--  switch (cc1100_readReg( CC1100_MARCSTATE )) {
--            
--       // RX_OVERFLOW
--  case 17:
--       // IDLE
--  case 1:
--    ccStrobe( CC1100_SFRX  );
--    ccStrobe( CC1100_SIDLE );
--    ccStrobe( CC1100_SNOP  );
--    ccStrobe( CC1100_SRX   );
--    break;
--       
--  }
--
- }
- void
-@@ -173,20 +173,7 @@
-     my_delay_ms(3);             // 3ms: Found by trial and error
-   }
--  ccStrobe(CC1100_SIDLE);
--  ccStrobe(CC1100_SFRX );
--  ccStrobe(CC1100_SFTX );
--
--  if (dec[2] & (1 << 4)) { //BURST-bit set?
--    ccStrobe(CC1100_STX  ); //We need to send a burst
--
--    //According to ELV, devices get activated every 300ms, so send burst for 360ms
--    for(l = 0; l < 3; l++)
--      my_delay_ms(120); //arg is uint_8, so loop
--  }
--
-   // "crypt"
--
-   enc[0] = dec[0];
-   enc[1] = (~dec[1]) ^ 0x89;
+     CC1100_DEASSERT;
  
-@@ -195,6 +182,15 @@
-   
-   enc[l] = dec[l] ^ dec[2];
+-    do {
+-      ccStrobe(CC1100_SRX);
+-    } while (cc1100_readReg(CC1100_MARCSTATE) != MARCSTATE_RX);
++    // We must not read the last byte from the RX fifo while RX is in progress (Errata 1)
++    while (((read_cc1100_rxbytes() & 0x7f) < 2) && (cc1100_readReg(CC1100_PKTSTATUS) & (1 << 3))) {
++      my_delay_ms(1);
++    }
  
-+  ccTX();
-+  if (dec[2] & (1 << 4)) { // BURST-bit set?
-+    // According to ELV, devices get activated every 300ms, so send burst for 360ms
-+    for(l = 0; l < 3; l++)
-+      my_delay_ms(120); // arg is uint_8, so loop
-+  } else {
-+      my_delay_ms(10);
-+  }
++    l = cc1100_readReg(CC1100_RXFIFO);
 +
-   // send
-   CC1100_ASSERT;
-   cc1100_sendbyte(CC1100_WRITE_BURST | CC1100_TXFIFO);
-@@ -205,12 +201,17 @@
-   CC1100_DEASSERT;
--  ccStrobe( CC1100_SFRX  );
--  ccStrobe( CC1100_STX   );
-+  // wait for TX to finish
-+  while(cc1100_readReg( CC1100_MARCSTATE ) == MARCSTATE_TX)
-+    ;
++    if (!(l & 0x80)) // CRC not ok
++      continue;
 +
-+  if (cc1100_readReg( CC1100_MARCSTATE ) == MARCSTATE_TXFIFO_UNDERFLOW) {
-+      ccStrobe( CC1100_SFTX  );
-+      ccStrobe( CC1100_SIDLE );
-+      ccStrobe( CC1100_SNOP  );
-+      ccStrobe( CC1100_SRX   );
-+  }
-   
--  while( cc1100_readReg( CC1100_MARCSTATE ) != 1 )
--    my_delay_ms(5);
--  
-   if(asksin_on) {
-     ccRX();
-   } else {
+     dec[0] = enc[0];
+     dec[1] = (~enc[1]) ^ 0x89;
+     
Impressum, Datenschutz