Hello, which SX127* module are you using for dev this library ? i guess hoperf rfm95 or any board with an esp32 and lora module like Liligo.

Why i ask ? Because im looking for support of EbYTES sx1276/78 modules like E19-868M30S Those modules are designed with some added pin like TXEN/RXEN that must be respetively HIGH when sending or receiving. But i never see ESP-IDF library which implement this.

I've ported part of the IRQ handling code from https://github.com/bkgoodman/esp-idf-lora, to your implementation. However, I'm seeing a conflict with both the IRQ handling and the SPI bus initialization:

    esp_err_t err = ESP_OK;

#ifdef LED_PIN
    gpio_set_direction(LED_PIN, GPIO_MODE_OUTPUT);
#endif

#if CONFIG_I2C_INTERFACE
    int i2c_sda = CONFIG_SDA_GPIO;
    int i2c_sclk = CONFIG_SCL_GPIO;
    int reset_pin = CONFIG_RESET_GPIO;

	ESP_LOGI(TAG, "INTERFACE is i2c");
	ESP_LOGI(TAG, "CONFIG_SDA_GPIO=%d",CONFIG_SDA_GPIO);
	ESP_LOGI(TAG, "CONFIG_SCL_GPIO=%d",CONFIG_SCL_GPIO);
	ESP_LOGI(TAG, "CONFIG_RESET_GPIO=%d",CONFIG_RESET_GPIO);

    i2c_config_t i2c_config = {
		.mode = I2C_MODE_MASTER,
		.sda_io_num = i2c_sda,
		.scl_io_num = i2c_sclk,
        .sda_pullup_en = GPIO_PULLUP_ENABLE,
        .scl_pullup_en = GPIO_PULLUP_ENABLE,
        .master.clk_speed = I2C_MASTER_FREQ_HZ
	};

	ESP_ERROR_CHECK(i2c_param_config(I2C_NUM, &i2c_config));
	ESP_ERROR_CHECK(i2c_driver_install(I2C_NUM, I2C_MODE_MASTER, 0, 0, 0));

    if (reset_pin >= 0) {
		//gpio_pad_select_gpio(reset);
		gpio_reset_pin(reset_pin);
		gpio_set_direction(reset_pin, GPIO_MODE_OUTPUT);
		gpio_set_level(reset_pin, 0);
		vTaskDelay(50 / portTICK_PERIOD_MS);
		gpio_set_level(reset_pin, 1);
	}

#ifdef HAS_OLED_DISPLAY
    ssd1306_init(&ssd1306_dev, 128, 64);
    ssd1306_clear_screen(&ssd1306_dev, false);
	ssd1306_contrast(&ssd1306_dev, 0xff);
#endif
#endif // CONFIG_I2C_INTERFACE

#ifdef USE_SDCARD
    spi_bus_config_t bus_cfg = {
        .mosi_io_num = SDCARD_MOSI,
        .miso_io_num = SDCARD_MISO,
        .sclk_io_num = SDCARD_SCK,
        .quadwp_io_num = -1,
        .quadhd_io_num = -1,
        .max_transfer_sz = 4000,
    };
    ret = spi_bus_initialize(host.slot, &bus_cfg, SDSPI_DEFAULT_HOST);
    if (ret != ESP_OK) {
        ESP_LOGE(TAG, "Failed to initialize bus.");
        return ret;
    }
#endif

    err = lora_setup(cfg);
    if (err != ESP_OK)
        ESP_LOGW(TAG, "RF communications not functional. Missing component?");

My SD card code is identical to:

https://raw.githubusercontent.com/espressif/esp-idf/a82e6e63d98bb051d4c59cb3d440c537ab9f74b0/examples/storage/sd_card/sdspi/main/sd_card_example_main.c

My TTGO LORA32 board configuration:

#define OLED_RST    -1
#define OLED_SDA    21
#define OLED_SCL    22

#define CONFIG_SDA_GPIO OLED_SDA
#define CONFIG_SCL_GPIO OLED_SCL
#define CONFIG_RESET_GPIO OLED_RST

#define SDCARD_SCK      14
#define SDCARD_MOSI     15
#define SDCARD_MISO     2
#define SDCARD_CS       13

#define LORA_MOSI       27
#define LORA_MISO       19
#define LORA_SCLK       5
#define LORA_CS         18
#define LORA_RST        23
#define LORA_DIO0       26
#define LORA_ISR        LORA_DIO0

#define CONFIG_LORA_CS_GPIO  LORA_CS
#define CONFIG_LORA_RST_GPIO LORA_RST
#define CONFIG_LORA_MISO_GPIO LORA_MISO
#define CONFIG_LORA_MOSI_GPIO LORA_MOSI
#define CONFIG_LORA_SCK_GPIO LORA_SCLK
#define CONFIG_LORA_IRQ_GPIO LORA_ISR

LORA setup:

esp_err_t lora_init(void)
{
   esp_err_t ret;

   /*
    * Configure CPU hardware to communicate with the radio chip
    */
   gpio_pad_select_gpio(CONFIG_LORA_RST_GPIO);
   gpio_set_direction(CONFIG_LORA_RST_GPIO, GPIO_MODE_OUTPUT);

   gpio_reset_pin(CONFIG_LORA_CS_GPIO);
   gpio_pad_select_gpio(CONFIG_LORA_CS_GPIO);
   gpio_set_direction(CONFIG_LORA_CS_GPIO, GPIO_MODE_OUTPUT);
   gpio_set_level(CONFIG_LORA_CS_GPIO, 1);


   spi_bus_config_t bus = {
      .miso_io_num = CONFIG_LORA_MISO_GPIO,
      .mosi_io_num = CONFIG_LORA_MOSI_GPIO,
      .sclk_io_num = CONFIG_LORA_SCK_GPIO,
      .quadwp_io_num = -1,
      .quadhd_io_num = -1,
      .max_transfer_sz = 0
   };
   
   ret = spi_bus_initialize(SPI_HOST_ID, &bus, SPI_DMA_CH_AUTO);
   if (ret != ESP_OK)
      return ret;

   spi_device_interface_config_t dev = {
      .clock_speed_hz = 9000000,
      .mode = 0,
      .spics_io_num = CONFIG_LORA_CS_GPIO,
      .queue_size = 7,
      .flags = 0,
      .pre_cb = NULL
   };

   ret = spi_bus_add_device(SPI_HOST_ID, &dev, &__spi);
   if (ret != ESP_OK)
      return ret;

   /*
    * Perform hardware reset.
    */
   lora_reset();

   /*
    * Check version.
    */
   uint8_t version;
   uint8_t i = 0;
   while(i++ < TIMEOUT_RESET) {
      version = lora_read_reg(REG_VERSION);
      if(version == 0x12) break;
      vTaskDelay(2);
   }
   assert(i <= TIMEOUT_RESET + 1); // at the end of the loop above, the max value i can reach is TIMEOUT_RESET + 1

   /*
    * Default configuration.
    */
   lora_sleep();
   lora_write_reg(REG_FIFO_RX_BASE_ADDR, 0);
   lora_write_reg(REG_FIFO_TX_BASE_ADDR, 0);
   lora_write_reg(REG_LNA, lora_read_reg(REG_LNA) | 0x03);
   lora_write_reg(REG_MODEM_CONFIG_3, 0x04);
   lora_set_tx_power(17);

   lora_idle();
   return ESP_OK;
}

I've been trying to follow the steps described here: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/peripherals/sdspi_share.html

Guru Meditation Error: Core  0 panic'ed (InstrFetchProhibited). Exception was unhandled.

Core  0 register dump:
PC      : 0x00000000  PS      : 0x00060030  A0      : 0x00000000  A1      : 0x3ffbc510  
A2      : 0x3ffb9750  A3      : 0x00000000  A4      : 0x3ffb4610  A5      : 0x3ffb4620  
A6      : 0x00000001  A7      : 0x00000060  A8      : 0x8008cec4  A9      : 0x3ffbc4f0  
A10     : 0x3ffb9750  A11     : 0x00000092  A12     : 0x00000091  A13     : 0x00000060  
A14     : 0x00000000  A15     : 0x00000001  SAR     : 0x00000000  EXCCAUSE: 0x00000014  
EXCVADDR: 0x00000000  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0x00000000  

I'm happy to send you the entire lora.c file. Thank you for your excellent work again, your repositories are an absolute time saver and great reference.

creating task with 1024*2 stack size is too small and therefore lead to carsh loop with stack overflow when lauching either task_rx or task_tx in basic exemple.

#if CONFIG_SENDER                                                                                                                                                                                          
         xTaskCreate(&task_tx, "task_tx", 1024*4, NULL, 5, NULL);                                                                                                                                           
#endif                                                                                                                                                                                                     
#if CONFIG_RECEIVER                                                                                                                                                                                                  
         xTaskCreate(&task_rx, "task_rx", 1024*4, NULL, 5, NULL);                

Solution is to increase stack size at task creation.

I'd like to make a few enhancements to your LoRa library. Would you be open to me making the following enhancements?

1. Add support for FSK operation e.g.: typedef enum { sx_mode_lora, sx_mode_fsk } sx_radio_mode; void lora_set_mode(sx_radio_mode new_mode); // default init mode is LoRa void lora_set_fsk_bitrate(unsigned bps); // auto-sets other FSK parameters void lora_get_fsk_fei(void); // read frequency error

2. Add support for making the reset pin optional if set to -1 and/or adding an installer for an optional external reset function (Because the ESP32 is so short on pins, I use an external GPIO expander wherever possible)

3. Add optional support for interrupt-driven operation using at least DIO0 rather than polling using vTaskDelay()

Thanks for providing the library!

I'm trying to move over to the idf from Arduino because the esp32c3 usb support is absolutely terrible there. I was using the radiolib arduino library for the sx1276 and it works great. But it doesn't work in the idf obviously. It does have support for dio0 dio1 (and rxen txen pins that a lot of modules have). Are these pins going to be added in the future?