try even harder to not drop any packets

[fhem-stuff] / culfw / culfw-asksin-fix.diff

diff --git a/culfw/culfw-asksin-fix.diff b/culfw/culfw-asksin-fix.diff
index e63b87faecb9d1d985a44b78b8937115d7318d3e..07a587f31afb0887c98906c901a196c1c0908822 100644 (file)
--- a/culfw/culfw-asksin-fix.diff
+++ b/culfw/culfw-asksin-fix.diff
@@ -2,21 +2,29 @@ Index: clib/rf_asksin.c
 ===================================================================
 --- clib/rf_asksin.c   (revision 373)
 +++ clib/rf_asksin.c   (working copy)
-@@ -11,7 +11,7 @@
+@@ -11,13 +11,13 @@
  
  uint8_t asksin_on = 0;
  
 -const uint8_t PROGMEM ASKSIN_CFG[50] = {
+-     0x00, 0x07,
 +const uint8_t PROGMEM ASKSIN_CFG[] = {
-      0x00, 0x07,
++     0x00, 0x01,
       0x02, 0x2e,
       0x03, 0x0d,
-@@ -26,16 +26,18 @@
+      0x04, 0xE9,
+      0x05, 0xCA,
+-     0x07, 0x0C,
++     0x07, 0x04,
+      0x0B, 0x06,
+      0x0D, 0x21,
+      0x0E, 0x65,
+@@ -26,18 +26,21 @@
       0x11, 0x93,
       0x12, 0x03,
       0x15, 0x34,
 -     0x17, 0x30, // always go into IDLE
-+     0x17, 0x3F, // always go into RX, CCA, ELV uses 0x03
++     0x17, 0x3F, // always go into RX after TX, CCA, ELV uses 0x03
       0x18, 0x18,
       0x19, 0x16,
       0x1B, 0x43,
@@ -26,22 +34,29 @@ Index: clib/rf_asksin.c
 +     0x29, 0x59,
 +     0x2c, 0x81,
       0x2D, 0x35,
-      0x3e, 0xc3,
+-     0x3e, 0xc3,
 -     0xff
-+     0xff, 0xff
++     0x3e, 0xc3
  };
  
++static inline uint8_t read_cc1100_rxbytes();
++
  void
-@@ -56,7 +58,7 @@
+ rf_asksin_init(void)
+ {
+@@ -56,20 +59,30 @@
    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) {
-        
-     if (pgm_read_byte( &ASKSIN_CFG[i] )>0x40)
-       break;
-@@ -67,7 +69,9 @@
+     cc1100_writeReg( pgm_read_byte(&ASKSIN_CFG[i]),
+                      pgm_read_byte(&ASKSIN_CFG[i+1]) );
+   }
    
    ccStrobe( CC1100_SCAL );
  
@@ -51,24 +66,70 @@ Index: clib/rf_asksin.c
 +  ccRX();
  }
  
++// Workaround for CC1101 Errata 3
++static inline uint8_t read_cc1100_rxbytes()
++{
++  uint8_t rxbytes, rxbytes2;
++  
++  rxbytes = cc1100_readReg(CC1100_RXBYTES);
++  while((rxbytes2 = cc1100_readReg(CC1100_RXBYTES)) != rxbytes)
++    rxbytes = rxbytes2;
++
++  return rxbytes;
++}
++
  void
-@@ -97,14 +101,10 @@
+ rf_asksin_task(void)
+ {
+@@ -77,15 +90,27 @@
+   uint8_t dec[MAX_ASKSIN_MSG];
+   uint8_t rssi;
+   uint8_t l;
++  uint8_t rxfifo_cnt;
+ 
+   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;
++
+     enc[0] = cc1100_readReg( CC1100_RXFIFO ) & 0x7f; // read len
+ 
++    while ((enc[0] + 2) < (rxfifo_cnt - 1)) { // Wait for more data
++      my_delay_ms(1);
++      rxfifo_cnt = (read_cc1100_rxbytes() & 0x7f) + 1;
++    }
++
+     if (enc[0]>=MAX_ASKSIN_MSG)
+          enc[0] = MAX_ASKSIN_MSG-1;
+     
+@@ -97,13 +122,12 @@
      }
      
      rssi = cc1100_sendbyte( 0 );
-+    /* LQI = */ cc1100_sendbyte( 0 );
-     
+-    
++    l = cc1100_sendbyte( 0 );
++
      CC1100_DEASSERT;
  
 -    ccStrobe( CC1100_SFRX  );
 -    ccStrobe( CC1100_SIDLE );
 -    ccStrobe( CC1100_SNOP  );
 -    ccStrobe( CC1100_SRX   );
--
++    if (!(l & 0x80)) // CRC not ok
++      continue;
+ 
      dec[0] = enc[0];
      dec[1] = (~enc[1]) ^ 0x89;
-     
-@@ -113,7 +113,6 @@
+@@ -113,7 +137,6 @@
      
      dec[l] = enc[l] ^ dec[2];
      
@@ -76,11 +137,22 @@ Index: clib/rf_asksin.c
      if (tx_report & REP_BINTIME) {
        
        DC('a');
-@@ -133,24 +132,14 @@
+@@ -131,26 +154,17 @@
+       
+       DNL();
      }
++  }
  
-     return;
+-    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;
    }
 -       
 -       
@@ -97,17 +169,11 @@ Index: clib/rf_asksin.c
 -    break;
 -       
 -  }
- 
-+  if (cc1100_readReg( CC1100_MARCSTATE ) == MARCSTATE_RXFIFO_OVERFLOW) {
-+      ccStrobe( CC1100_SFRX  );
-+      ccStrobe( CC1100_SIDLE );
-+      ccStrobe( CC1100_SNOP  );
-+      ccStrobe( CC1100_SRX   );
-+  }
+-
  }
  
  void
-@@ -173,20 +162,7 @@
+@@ -173,20 +187,7 @@
      my_delay_ms(3);             // 3ms: Found by trial and error
    }
  
@@ -128,7 +194,7 @@ Index: clib/rf_asksin.c
    enc[0] = dec[0];
    enc[1] = (~dec[1]) ^ 0x89;
  
-@@ -195,6 +171,13 @@
+@@ -195,6 +196,15 @@
    
    enc[l] = dec[l] ^ dec[2];
  
@@ -137,12 +203,14 @@ Index: clib/rf_asksin.c
 +    // 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);
 +  }
 +
    // send
    CC1100_ASSERT;
    cc1100_sendbyte(CC1100_WRITE_BURST | CC1100_TXFIFO);
-@@ -205,12 +188,10 @@
+@@ -205,12 +215,17 @@
  
    CC1100_DEASSERT;
  
@@ -151,6 +219,13 @@ Index: clib/rf_asksin.c
 +  // wait for TX to finish
 +  while(cc1100_readReg( CC1100_MARCSTATE ) == MARCSTATE_TX)
 +    ;
++
++  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);