move all LOGI to more severe log levels

This commit is contained in:
Morgan 'ARR\!' Allen 2023-01-16 22:58:40 -08:00
parent 41065d791e
commit 9d1bd57a2e
1 changed files with 33 additions and 31 deletions

View File

@ -74,7 +74,7 @@ double lora32_calc_datarate(lora32_cfg_t *lora) {
double cr = (4.0 / (long)lora->codingRate); double cr = (4.0 / (long)lora->codingRate);
double sf = pow(2, lora->spreadingFactor); double sf = pow(2, lora->spreadingFactor);
double c2 = sf / bandwidths[lora->bandwidth]; double c2 = sf / bandwidths[lora->bandwidth];
ESP_LOGI(TAG, "codingRate: %d cr: %f sf: %f c2: %f", lora->codingRate, cr, sf, c2); ESP_LOGD(TAG, "codingRate: %d cr: %f sf: %f c2: %f", lora->codingRate, cr, sf, c2);
return lora->spreadingFactor * cr / c2 * 1000; return lora->spreadingFactor * cr / c2 * 1000;
} }
@ -151,12 +151,12 @@ void lora32_send(lora32_cfg_t *lora, uint8_t *data, uint8_t len) {
} }
void lora32_set_frequency(lora32_cfg_t *lora, long frequency) { void lora32_set_frequency(lora32_cfg_t *lora, long frequency) {
ESP_LOGI(TAG, "lora32_set_frequency: %lu", frequency); ESP_LOGD(TAG, "lora32_set_frequency: %lu", frequency);
uint64_t frf = ((uint64_t)frequency << 19) / 32000000; uint64_t frf = ((uint64_t)frequency << 19) / 32000000;
ESP_LOGI(TAG, "REG_FRF_MSB: 0x%2X", (uint8_t)(frf >> 16)); ESP_LOGD(TAG, "REG_FRF_MSB: 0x%2X", (uint8_t)(frf >> 16));
ESP_LOGI(TAG, "REG_FRF_MID: 0x%2X", (uint8_t)(frf >> 8)); ESP_LOGD(TAG, "REG_FRF_MID: 0x%2X", (uint8_t)(frf >> 8));
ESP_LOGI(TAG, "REG_FRF_LSB: 0x%2X", (uint8_t)(frf >> 0)); ESP_LOGD(TAG, "REG_FRF_LSB: 0x%2X", (uint8_t)(frf >> 0));
lora32_write_reg(lora, REG_FRF_MSB, (uint8_t)(frf >> 16)); lora32_write_reg(lora, REG_FRF_MSB, (uint8_t)(frf >> 16));
lora32_write_reg(lora, REG_FRF_MID, (uint8_t)(frf >> 8)); lora32_write_reg(lora, REG_FRF_MID, (uint8_t)(frf >> 8));
@ -194,7 +194,7 @@ void lora32_set_tx_power(lora32_cfg_t *lora, uint8_t level, uint8_t output) {
lora32_write_reg(lora, REG_PA_CONFIG, PA_BOOST | (level - 2)); lora32_write_reg(lora, REG_PA_CONFIG, PA_BOOST | (level - 2));
} }
ESP_LOGI(TAG, "set_tx_power(%d, %d)", level, output); ESP_LOGD(TAG, "set_tx_power(%d, %d)", level, output);
ESP_LOGD(TAG, "REG_PA_CONFIG: 0x%2X", lora32_read_reg(lora, REG_PA_CONFIG)); ESP_LOGD(TAG, "REG_PA_CONFIG: 0x%2X", lora32_read_reg(lora, REG_PA_CONFIG));
} }
@ -231,7 +231,7 @@ void lora32_dump_regs(lora32_cfg_t *lora) {
void lora32_toggle_reset(lora32_cfg_t *config) { void lora32_toggle_reset(lora32_cfg_t *config) {
// toggle reset (L/H) // toggle reset (L/H)
ESP_LOGI(TAG, "Toggling reset pin %d", config->reset); ESP_LOGD(TAG, "Toggling reset pin %d", config->reset);
gpio_set_level(config->reset, 0); gpio_set_level(config->reset, 0);
vTaskDelay(100 / portTICK_PERIOD_MS); // requires 100us vTaskDelay(100 / portTICK_PERIOD_MS); // requires 100us
@ -257,7 +257,7 @@ void lora32_set_spreadfactor(lora32_cfg_t *lora, uint8_t factor) {
lora32_write_reg(lora, REG_DETECTION_THRESHOLD, DETECT_THRES_OTHER); lora32_write_reg(lora, REG_DETECTION_THRESHOLD, DETECT_THRES_OTHER);
} }
ESP_LOGI(TAG, "lora32_set_spreadfactor: %d", factor); ESP_LOGV(TAG, "lora32_set_spreadfactor: %d", factor);
lora32_write_reg(lora, REG_MODEM_CONFIG_2, (lora32_read_reg(lora, REG_MODEM_CONFIG_2) & 0x0F) | ((factor << 4) & 0xF0)); lora32_write_reg(lora, REG_MODEM_CONFIG_2, (lora32_read_reg(lora, REG_MODEM_CONFIG_2) & 0x0F) | ((factor << 4) & 0xF0));
@ -312,11 +312,11 @@ int16_t lora32_get_packet_rssi(lora32_cfg_t *lora) {
long lora32_get_bandwidth(lora32_cfg_t *lora) { long lora32_get_bandwidth(lora32_cfg_t *lora) {
uint8_t bw = (lora32_read_reg(lora, REG_MODEM_CONFIG_1) >> 4); uint8_t bw = (lora32_read_reg(lora, REG_MODEM_CONFIG_1) >> 4);
ESP_LOGI(TAG, "lora32_get_bandwidth: %d", bw); ESP_LOGV(TAG, "lora32_get_bandwidth: %d", bw);
if(bw > 9) return -1; if(bw > 9) return -1;
ESP_LOGI(TAG, "lora32_get_bandwidth: %ld", bandwidths[bw]); ESP_LOGD(TAG, "lora32_get_bandwidth: %ld", bandwidths[bw]);
return bandwidths[bw]; return bandwidths[bw];
} }
@ -324,20 +324,20 @@ long lora32_get_bandwidth(lora32_cfg_t *lora) {
void lora32_set_ldo_flag(lora32_cfg_t *lora) { void lora32_set_ldo_flag(lora32_cfg_t *lora) {
long symbolDuration = 1000 / (lora32_get_bandwidth(lora) / (1L << lora32_get_spreadfactor(lora))); long symbolDuration = 1000 / (lora32_get_bandwidth(lora) / (1L << lora32_get_spreadfactor(lora)));
ESP_LOGI(TAG, "symbolDuration: %ld", symbolDuration); ESP_LOGD(TAG, "symbolDuration: %ld", symbolDuration);
bool ldoOn = symbolDuration > 16; bool ldoOn = symbolDuration > 16;
ESP_LOGI(TAG, "ldoOn: %d", ldoOn); ESP_LOGD(TAG, "ldoOn: %d", ldoOn);
uint8_t modem_config_3 = lora32_read_reg(lora, REG_MODEM_CONFIG_3); uint8_t modem_config_3 = lora32_read_reg(lora, REG_MODEM_CONFIG_3);
ESP_LOGI(TAG, "modem_config_3: %d", modem_config_3); ESP_LOGD(TAG, "modem_config_3: %d", modem_config_3);
modem_config_3 |= ldoOn << 3; modem_config_3 |= ldoOn << 3;
ESP_LOGI(TAG, "modem_config_3: %d", modem_config_3); ESP_LOGD(TAG, "modem_config_3: %d", modem_config_3);
lora32_write_reg(lora, REG_MODEM_CONFIG_3, modem_config_3); lora32_write_reg(lora, REG_MODEM_CONFIG_3, modem_config_3);
} }
void lora32_set_bandwidth(lora32_cfg_t *lora, uint8_t bw) { void lora32_set_bandwidth(lora32_cfg_t *lora, uint8_t bw) {
ESP_LOGI(TAG, "lora32_set_bandwidth: %d", bw); ESP_LOGD(TAG, "lora32_set_bandwidth: %d", bw);
uint8_t modem_config_1 = lora32_read_reg(lora, REG_MODEM_CONFIG_1); uint8_t modem_config_1 = lora32_read_reg(lora, REG_MODEM_CONFIG_1);
lora32_write_reg(lora, REG_MODEM_CONFIG_1, (modem_config_1 & 0x0F) | (bw << 4)); lora32_write_reg(lora, REG_MODEM_CONFIG_1, (modem_config_1 & 0x0F) | (bw << 4));
@ -351,7 +351,7 @@ void lora32_set_coding_rate(lora32_cfg_t *lora, uint8_t cr) {
cr = cr - 4; cr = cr - 4;
ESP_LOGI(TAG, "lora32_set_coding_rate: %d", cr + 4); ESP_LOGD(TAG, "lora32_set_coding_rate: %d", cr + 4);
lora32_write_reg(lora, REG_MODEM_CONFIG_1, (lora32_read_reg(lora, REG_MODEM_CONFIG_1) & 0xF1) | (cr << 1)); lora32_write_reg(lora, REG_MODEM_CONFIG_1, (lora32_read_reg(lora, REG_MODEM_CONFIG_1) & 0xF1) | (cr << 1));
} }
@ -365,7 +365,7 @@ void lora32_set_lna(lora32_cfg_t *lora, uint8_t gain) {
uint8_t lna = lora32_read_reg(lora, REG_LNA); uint8_t lna = lora32_read_reg(lora, REG_LNA);
lora32_write_reg(lora, REG_LNA, lna | gain); lora32_write_reg(lora, REG_LNA, lna | gain);
ESP_LOGI(TAG, "set lna: 0x%2X", lna | gain); ESP_LOGD(TAG, "set lna: 0x%2X", lna | gain);
} }
static void lora32_handle_receive(lora32_cfg_t *lora) { static void lora32_handle_receive(lora32_cfg_t *lora) {
@ -384,7 +384,7 @@ static void lora32_handle_receive(lora32_cfg_t *lora) {
static void IRAM_ATTR lora32_dio_task(void *arg) { static void IRAM_ATTR lora32_dio_task(void *arg) {
// allocate lora32_cfg_t to receive config from Queu // allocate lora32_cfg_t to receive config from Queu
lora32_cfg_t *lora = malloc(sizeof(lora32_cfg_t)); lora32_cfg_t *lora = malloc(sizeof(lora32_cfg_t));
ESP_LOGI(TAG, "starting DIO handler task"); ESP_LOGD(TAG, "starting DIO handler task");
while(1) { while(1) {
// wait for event over Queue // wait for event over Queue
@ -431,12 +431,12 @@ static void IRAM_ATTR lora32_dio_task(void *arg) {
} }
if((irqs & IRQ_FHSS_CHANGE) == IRQ_FHSS_CHANGE) { if((irqs & IRQ_FHSS_CHANGE) == IRQ_FHSS_CHANGE) {
ESP_LOGI(TAG, "switching channel %d: %d", lora->channel, lora->channels[lora->channel]); ESP_LOGD(TAG, "switching channel %d: %d", lora->channel, lora->channels[lora->channel]);
if(lora->channel == 0 && lora->channels[lora->channel] == 0) continue; if(lora->channel == 0 && lora->channels[lora->channel] == 0) continue;
if(lora->channels[lora->channel] == 0) { if(lora->channels[lora->channel] == 0) {
ESP_LOGI(TAG, "reseting to channel 0"); ESP_LOGD(TAG, "reseting to channel 0");
lora->channel = 0; lora->channel = 0;
} }
@ -451,7 +451,7 @@ void lora32_read_data(lora32_cfg_t *lora, uint8_t *data) {
uint8_t i = 0; uint8_t i = 0;
uint8_t len = lora32_read_reg(lora, (lora->implicitHeader ? REG_PAYLOAD_LENGTH : REG_RX_NB_BYTES)); uint8_t len = lora32_read_reg(lora, (lora->implicitHeader ? REG_PAYLOAD_LENGTH : REG_RX_NB_BYTES));
ESP_LOGI(TAG, "Reading %d bytes", len); ESP_LOGD(TAG, "Reading %d bytes", len);
for(i = 0; i < len; i++) { for(i = 0; i < len; i++) {
data[i] = lora32_read_reg(lora, REG_FIFO); data[i] = lora32_read_reg(lora, REG_FIFO);
@ -463,7 +463,7 @@ static void IRAM_ATTR lora32_on_dio(void *arg) {
} }
uint8_t lora32_spi_init(lora32_cfg_t *lora) { uint8_t lora32_spi_init(lora32_cfg_t *lora) {
ESP_LOGI(TAG, "Initializing SPI bus"); ESP_LOGD(TAG, "Initializing SPI bus");
esp_err_t err = ESP_OK; esp_err_t err = ESP_OK;
@ -499,7 +499,7 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
lora32_toggle_reset(lora); lora32_toggle_reset(lora);
// set NSS high // set NSS high
ESP_LOGI(TAG, "Bringing NSS high: %d", lora->nss); ESP_LOGD(TAG, "Bringing NSS high: %d", lora->nss);
gpio_set_level(lora->nss, 1); gpio_set_level(lora->nss, 1);
vTaskDelay(10 / portTICK_PERIOD_MS); vTaskDelay(10 / portTICK_PERIOD_MS);
@ -521,16 +521,18 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
uint8_t version = lora32_read_reg(lora, REG_VERSION); uint8_t version = lora32_read_reg(lora, REG_VERSION);
ESP_LOGD(TAG, "lora32_get_id() == 0x%2X", version); ESP_LOGD(TAG, "lora32_get_id() == 0x%2X", version);
lora->version = version;
// if ID does not match, something is likely wrong on the SPI bus // if ID does not match, something is likely wrong on the SPI bus
if(version != 0x12) { if(version != 0x12) {
ESP_LOGD(TAG, "REG_VERSION returned incorrectly. Expected 0x12 got 0x%02X", version); ESP_LOGE(TAG, "REG_VERSION returned incorrectly. Expected 0x12 got 0x%02X", version);
return ERR_LOR_VERSION_MISMATCH; return ERR_LOR_VERSION_MISMATCH;
} }
// TODO: confirm this is happening. Before/after power measurements? // TODO: confirm this is happening. Before/after power measurements?
lora32_sleep(lora); lora32_sleep(lora);
ESP_LOGI(TAG, "lora32_sleep"); ESP_LOGV(TAG, "lora32_sleep");
// TODO: VERIFY // TODO: VERIFY
lora32_set_frequency(lora, lora->frequency); lora32_set_frequency(lora, lora->frequency);
@ -540,7 +542,7 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
lora32_write_reg(lora, REG_FIFO_TX_BASE_ADDR, 0x00); lora32_write_reg(lora, REG_FIFO_TX_BASE_ADDR, 0x00);
lora32_write_reg(lora, REG_FIFO_RX_BASE_ADDR, 0x00); lora32_write_reg(lora, REG_FIFO_RX_BASE_ADDR, 0x00);
ESP_LOGI(TAG, "clear rx/tx fifos"); ESP_LOGV(TAG, "clear rx/tx fifos");
lora32_set_lna(lora, 0x03); lora32_set_lna(lora, 0x03);
@ -549,10 +551,10 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
// TODO make based on config // TODO make based on config
lora32_set_tx_power(lora, 17, PA_OUTPUT_PA_BOOST_PIN); lora32_set_tx_power(lora, 17, PA_OUTPUT_PA_BOOST_PIN);
ESP_LOGI(TAG, "lora32_set_tx_power"); ESP_LOGV(TAG, "lora32_set_tx_power");
lora32_standby(lora); lora32_standby(lora);
ESP_LOGI(TAG, "lora32_standby"); ESP_LOGV(TAG, "lora32_standby");
if(lora->implicitHeader) if(lora->implicitHeader)
lora23_set_implicit_header(lora); lora23_set_implicit_header(lora);
@ -561,7 +563,7 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
// TODO setup shouldn't be based on just receive callback // TODO setup shouldn't be based on just receive callback
if(lora->receive != NULL) { if(lora->receive != NULL) {
ESP_LOGI(TAG, "Setting GPIO Interrupt"); ESP_LOGV(TAG, "Setting GPIO Interrupt");
// TODO check at least one DIOx pin is not NULL // TODO check at least one DIOx pin is not NULL
@ -578,7 +580,7 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
// the DIO interrupt handling for every device is done from one task // the DIO interrupt handling for every device is done from one task
if(dio_task_handle == NULL) { if(dio_task_handle == NULL) {
ESP_LOGI(TAG, "Setting callback handler and ISR service"); ESP_LOGV(TAG, "Setting callback handler and ISR service");
// enable global ISR service // enable global ISR service
gpio_install_isr_service(0); gpio_install_isr_service(0);
@ -590,7 +592,7 @@ uint8_t lora32_init(lora32_cfg_t *lora) {
} }
// add ISR handler to the global service started (once) above // add ISR handler to the global service started (once) above
ESP_LOGI(TAG, "Installing ISR handler for GPIO%d", lora->dio0); ESP_LOGV(TAG, "Installing ISR handler for GPIO%d", lora->dio0);
if(lora->dio0 > -1) if(lora->dio0 > -1)
gpio_isr_handler_add(lora->dio0, lora32_on_dio, (void*)lora); gpio_isr_handler_add(lora->dio0, lora32_on_dio, (void*)lora);