Fixed

Author: developer328

NRF24.c

@@ -100,7 +100,7 @@ void nrf24_data_rate(uint8_t bps){
 
 	data &= ~(1 << RF_DR_LOW) & ~(1 << RF_DR_HIGH);
 
-	if(bps == _2Mbps){
+	if(bps == _2mbps){
 		data |= (1 << RF_DR_HIGH);
 	}else if(bps == _250kbps){
 		data |= (1 << RF_DR_LOW);
@@ -258,17 +258,26 @@ void nrf24_clear_max_rt(void){
 }
 
 uint8_t nrf24_read_bit(uint8_t reg, uint8_t bit){
+
+	if(nrf24_r_reg(reg, 1) & (1 << bit)){
+		return 1;
+	}
+
+	return 0;
+}
+
+void nrf24_set_bit(uint8_t reg, uint8_t bit, uint8_t val){
 	uint8_t data = 0;
 
 	data = nrf24_r_reg(reg, 1);
 
-	data &= (1 << bit);
-
-	if(data){
-		return 1;
+	if(val){
+		data |= (1 << bit);
+	}else{
+		data &= ~(1 << bit);
 	}
 
-	return 0;
+    nrf24_w_reg(reg, &data, 1);
 }
 
 uint8_t nrf24_r_pld_wid(void){
@@ -290,6 +299,8 @@ void nrf24_listen(void){
 	data |= (1 << PRIM_RX);
 
 	nrf24_w_reg(CONFIG, &data, 1);
+
+	ce_high();
 }
 
 void nrf24_stop_listen(void){
@@ -414,6 +425,7 @@ uint8_t nrf24_transmit(uint8_t *data, uint8_t size){
 
 	nrf24_clear_tx_ds();
 
+
 	csn_low();
 	nrf24_w_spec_cmd(W_TX_PAYLOAD);
 	nrf24_w_spec_reg(data, size);
@@ -423,19 +435,9 @@ uint8_t nrf24_transmit(uint8_t *data, uint8_t size){
 	HAL_Delay(1);
 	ce_low();
 
-	uint32_t ms = HAL_GetTick();
 
-	while((HAL_GetTick()-ms) < 150){
-		uint8_t stat = nrf24_r_reg(STATUS, 1);
-		if(stat & (1 << TX_DS)){
-			nrf24_clear_tx_ds();
-			return 1;
-		}
-
-		if(stat & (1 << MAX_RT)){
-			nrf24_clear_max_rt();
-			return 2;
-		}
+	if(nrf24_read_bit(STATUS, MAX_RT) == 1){
+		return 1;
 	}
 
 	return 0;
@@ -475,6 +477,10 @@ void nrf24_transmit_rx_ack_pld(uint8_t pipe, uint8_t *data, uint8_t size){
 
 }
 
+uint8_t nrf24_carrier_detect(void){
+	return nrf24_r_reg(RPD, 1);
+}
+
 uint8_t nrf24_data_available(void){
 
  	uint8_t reg_dt = nrf24_r_reg(FIFO_STATUS, 1);
@@ -486,22 +492,15 @@ uint8_t nrf24_data_available(void){
 	return 0;
 }
 
-void nrf24_flush_on_full_rx(void){
-	uint8_t reg_dt = nrf24_r_reg(FIFO_STATUS, 1);
-
-	if((reg_dt & (1 << RX_FULL))){
-		nrf24_flush_rx();
-	}
-}
-
 void nrf24_receive(uint8_t *data, uint8_t size){
-	ce_high();
 
 	csn_low();
 	nrf24_w_spec_cmd(R_RX_PAYLOAD);
 	nrf24_r_spec_reg(data, size);
 	csn_high();
 
+	nrf24_clear_rx_dr();
+
 	nrf24_flush_rx();
 }
 
@@ -527,7 +526,7 @@ void nrf24_defaults(void){
 	nrf24_auto_retr_limit(3);
 
 
-	for(uint8_t i = 0; i <= 5; i++){
+	for(uint8_t i = 0; i < 5; i++){
 		nrf24_pipe_pld_size(i, 0);
 		nrf24_cls_rx_pipe(i);
 		nrf24_set_rx_dpl(i, disable);
@@ -537,4 +536,16 @@ void nrf24_defaults(void){
 	ce_high();
 }
 
+void nrf24_init(void){
+
+	nrf24_pwr_up();
+
+	nrf24_flush_tx();
+	nrf24_flush_rx();
+
+	nrf24_clear_rx_dr();
+	nrf24_clear_tx_ds();
+	nrf24_clear_max_rt();
+}
+
 

NRF24.h

@@ -3,7 +3,7 @@
 
 enum data_rate {
 	_1mbps   = 0,
-	_2Mbps   = 1,
+	_2mbps   = 1,
 	_250kbps = 2
 };
 
@@ -69,13 +69,15 @@ void nrf24_clear_tx_ds(void);
 void nrf24_clear_max_rt(void);
 
 uint8_t nrf24_read_bit(uint8_t reg, uint8_t bit);
+void nrf24_set_bit(uint8_t reg, uint8_t bit, uint8_t val);
 uint8_t nrf24_r_pld_wid(void);
 
 void nrf24_listen(void);
 void nrf24_stop_listen(void);
 
 void nrf24_dpl(uint8_t en);
 void nrf24_set_rx_dpl(uint8_t pipe, uint8_t en);
+
 void nrf24_auto_ack(uint8_t pipe, uint8_t ack);
 void nrf24_auto_ack_all(uint8_t ack);
 void nrf24_en_ack_pld(uint8_t en);
@@ -87,11 +89,12 @@ uint8_t nrf24_transmit(uint8_t *data, uint8_t size);
 void nrf24_transmit_no_ack(uint8_t *data, uint8_t size);
 void nrf24_transmit_rx_ack_pld(uint8_t pipe, uint8_t *data, uint8_t size);
 
+uint8_t nrf24_carrier_detect(void);
 uint8_t nrf24_data_available(void);
-void nrf24_flush_on_full_rx(void);
 void nrf24_receive(uint8_t *data, uint8_t size);
 
 void nrf24_defaults(void);
+void nrf24_init(void);
 
 #endif
 

NRF24_conf.h

@@ -2,7 +2,7 @@
 #define _NRF_24_CONF_H_
 
 #define hspiX hspi1
-#define spi_w_timeout 1000
+#define spi_w_timeout 100
 #define spi_r_timeout 1000
 
 #define csn_gpio_port GPIOA
@@ -15,3 +15,4 @@
 #define irq_gpio_pin GPIO_PIN_1
 
 #endif
+