X-Git-Url: https://git.zerfleddert.de/cgi-bin/gitweb.cgi/fhem-stuff/blobdiff_plain/b2720844ad3ca5ae4d2b92d6d76650756f41ace0..HEAD:/culfw/culfw-asksin-fix.diff diff --git a/culfw/culfw-asksin-fix.diff b/culfw/culfw-asksin-fix.diff index e63b87f..65dc3bb 100644 --- a/culfw/culfw-asksin-fix.diff +++ b/culfw/culfw-asksin-fix.diff @@ -1,160 +1,156 @@ Index: clib/rf_asksin.c =================================================================== ---- clib/rf_asksin.c (revision 373) +--- clib/rf_asksin.c (revision 378) +++ clib/rf_asksin.c (working copy) -@@ -11,7 +11,7 @@ +@@ -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, 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,7 +29,7 @@ 0x11, 0x93, 0x12, 0x03, 0x15, 0x34, -- 0x17, 0x30, // always go into IDLE -+ 0x17, 0x3F, // always go into RX, 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 -+ 0xff, 0xff +@@ -39,6 +42,7 @@ + 0x3e, 0xc3 }; - void -@@ -56,7 +58,7 @@ - my_delay_us(100); ++static inline uint8_t read_cc1100_rxbytes(void); + static void rf_asksin_reset_rx(void); - // load configuration -- for (uint8_t i = 0; i<50; i += 2) { -+ for (uint8_t i = 0; i < sizeof(ASKSIN_CFG); i += 2) { - - if (pgm_read_byte( &ASKSIN_CFG[i] )>0x40) - break; -@@ -67,7 +69,9 @@ - - ccStrobe( CC1100_SCAL ); + void +@@ -74,12 +78,29 @@ + } while (cc1100_readReg(CC1100_MARCSTATE) != MARCSTATE_RX); + } -- my_delay_ms(1); -+ my_delay_ms(4); ++// Workaround for CC1101 Errata 3 ++static inline uint8_t ++read_cc1100_rxbytes(void) ++{ ++ uint8_t rxbytes, rxbytes2; ++ ++ rxbytes = cc1100_readReg(CC1100_RXBYTES); ++ while((rxbytes2 = cc1100_readReg(CC1100_RXBYTES)) != rxbytes) ++ rxbytes = rxbytes2; ++ ++ return rxbytes; ++} + -+ ccRX(); + static void + rf_asksin_reset_rx(void) + { + ccStrobe( CC1100_SFRX ); + ccStrobe( CC1100_SIDLE ); + ccStrobe( CC1100_SNOP ); ++ ++ while (read_cc1100_rxbytes() & 0x7f) ++ cc1100_readReg(CC1100_RXFIFO); ++ + ccStrobe( CC1100_SRX ); } - void -@@ -97,14 +101,10 @@ - } - - rssi = cc1100_sendbyte( 0 ); -+ /* LQI = */ cc1100_sendbyte( 0 ); - - CC1100_DEASSERT; - -- ccStrobe( CC1100_SFRX ); -- ccStrobe( CC1100_SIDLE ); -- ccStrobe( CC1100_SNOP ); -- ccStrobe( CC1100_SRX ); -- - dec[0] = enc[0]; - dec[1] = (~enc[1]) ^ 0x89; - -@@ -113,7 +113,6 @@ - - dec[l] = enc[l] ^ dec[2]; - -- - if (tx_report & REP_BINTIME) { - - DC('a'); -@@ -133,24 +132,14 @@ - } +@@ -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; -- - } -- -- -- 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; -- -- } -+ 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 @@ - 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; +- // 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--; -@@ -195,6 +171,13 @@ - - enc[l] = dec[l] ^ dec[2]; + if (enc[0] >= MAX_ASKSIN_MSG) { + // Something went horribly wrong, out of sync? +@@ -104,6 +141,27 @@ + return; + } -+ 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 -+ } ++ 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; ++ } + - // send - CC1100_ASSERT; - cc1100_sendbyte(CC1100_WRITE_BURST | CC1100_TXFIFO); -@@ -205,12 +188,10 @@ ++ if (!timeout) { ++ // Not enough data received, out of sync? ++ rf_asksin_reset_rx(); ++ return; ++ } ++ } ++ + CC1100_ASSERT; + cc1100_sendbyte( CC1100_READ_BURST | CC1100_RXFIFO ); + +@@ -112,14 +170,19 @@ + } + + rssi = cc1100_sendbyte( 0 ); +- /* LQI = */ cc1100_sendbyte( 0 ); + + CC1100_DEASSERT; - CC1100_DEASSERT; +- 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); ++ } -- ccStrobe( CC1100_SFRX ); -- ccStrobe( CC1100_STX ); -+ // wait for TX to finish -+ while(cc1100_readReg( CC1100_MARCSTATE ) == MARCSTATE_TX) -+ ; - -- while( cc1100_readReg( CC1100_MARCSTATE ) != 1 ) -- my_delay_ms(5); -- - if(asksin_on) { - ccRX(); - } else { ++ l = cc1100_readReg(CC1100_RXFIFO); ++ ++ if (!(l & 0x80)) // CRC not ok ++ continue; ++ + dec[0] = enc[0]; + dec[1] = (~enc[1]) ^ 0x89; +