Coexistence driver limits the pins for BLE_EIP from P00 up to P07

ID: LPCBARESDK-707

Status: Open

First reported: 6.0.14.1114

Fixed in: TBD

Description

The DA14531 and the DA14585 are able to expose the diagnostic pins (the BLE_EIP in the case of the coexistence driver) at pins:

  • For the DA14531 Diagnostic pins are available from P00 up to P011.

  • For the DA14585 Diagnostic pins are available from P00 up to P07 and from P10 up to P13.

The existing coex driver limits the BLE_EIP signal in both devices from P00 to P07. If the user tries to output the BLE_EIP signal on any other Diagnostic capable pin the SDK will assert an error. An update is required in the wlan_coex.c file for the following functions:

  • wlan_coex_gpio_cfg() for modifying the Assertion

  • wlan_coex_BLE_set() for properly configuring the diagnostic at the other available pins.

In the workaround section the updated functions are given supporting both DA14531 and DA14585/586 devices.

Workaround

void wlan_coex_gpio_cfg(void)
{
    // WLAN_COEX_BLE_EVENT can be applied on any pin for DA14531 and only on pins P0[0-7] and P1[1-3] for DA14585.
#if!(__DA14531__)
        ASSERT_WARNING((wlan_coex_cfg.ble_eip_pin <= GPIO_PIN_7) && (wlan_coex_cfg.ble_eip_port == GPIO_PORT_0) ||
                        (wlan_coex_cfg.ble_eip_pin <= GPIO_PIN_3) && (wlan_coex_cfg.ble_eip_port == GPIO_PORT_1));
#endif

    // Drive to inactive state the pin used for the BLE event in progress signal
    wlan_coex_set_ble_eip_pin_inactive();

    // Configure selected GPIO as output - BLE priority
    GPIO_ConfigurePin(wlan_coex_cfg.ble_prio_port, wlan_coex_cfg.ble_prio_pin, OUTPUT, PID_GPIO, false);

    // Configure selected GPIO as input with pull-down - 2.4GHz external device event in progress
    GPIO_ConfigurePin(wlan_coex_cfg.ext_24g_eip_port, wlan_coex_cfg.ext_24g_eip_pin, INPUT_PULLDOWN, PID_GPIO, false);

    // Configure 2.4GHz external device event in progress signal to act as a GPIO interrupt
    GPIO_RegisterCallback((IRQn_Type)(GPIO0_IRQn + wlan_coex_cfg.irq), ext_24g_eip_handler);
    GPIO_EnableIRQ(wlan_coex_cfg.ext_24g_eip_port, wlan_coex_cfg.ext_24g_eip_pin, (IRQn_Type)(GPIO0_IRQn + wlan_coex_cfg.irq),
                    /*low_input*/ false, /*release_wait*/ false, /*debounce_ms*/ 0);

#if defined (CFG_WLAN_COEX_DEBUG)
    GPIO_ConfigurePin(wlan_coex_cfg.debug_a_port, wlan_coex_cfg.debug_a_pin, OUTPUT, PID_GPIO, false);
    GPIO_ConfigurePin(wlan_coex_cfg.debug_b_port, wlan_coex_cfg.debug_b_pin, OUTPUT, PID_GPIO, false);
#endif
}
void wlan_coex_BLE_set(void)
{
    uint8_t shift;
    GPIO_PIN diag_en;
#if (__DA14531__)
    if ((wlan_coex_cfg.ble_eip_pin < GPIO_PIN_4) || (wlan_coex_cfg.ble_eip_pin > GPIO_PIN_7))
#else
    if (wlan_coex_cfg.ble_eip_pin < GPIO_PIN_4)
#endif
    {
#if (__DA14531__)
        if(wlan_coex_cfg.ble_eip_pin > GPIO_PIN_7)
        {
            diag_en = (GPIO_PIN)(wlan_coex_cfg.ble_eip_pin - GPIO_PIN_8);
            shift = diag_en * 8;
        }
        else
#endif
        {
            shift = wlan_coex_cfg.ble_eip_pin * 8;
            diag_en = wlan_coex_cfg.ble_eip_pin;
        }
        // Enable BLE event in progress
        SetBits32(BLE_DIAGCNTL_REG, DIAG0 << shift, 0x1F);
        SetBits32(BLE_DIAGCNTL_REG, DIAG0_EN << shift, 1);
    }
    else
    {
        shift = (wlan_coex_cfg.ble_eip_pin - 4) * 8;
        diag_en = wlan_coex_cfg.ble_eip_pin;
        // Enable BLE event in progress
        SetBits32(BLE_DIAGCNTL2_REG, DIAG4 << shift, 0x1F);
        SetBits32(BLE_DIAGCNTL2_REG, DIAG4_EN << shift, 1);
    }

    SetWord16(GPIO_BASE + 0x6 + (wlan_coex_cfg.ble_eip_port << 5) + (wlan_coex_cfg.ble_eip_pin << 1), 0x312);

#if defined (CFG_WLAN_COEX_BLE_EVENT_INV)
    SetBits32(BLE_DIAGCNTL3_REG, (DIAG0_INV << (diag_en * 4)), 1);
#endif
    SetBits32(BLE_DIAGCNTL3_REG, (DIAG0_BIT << (diag_en * 4)), 7);
}