S340 v6.1.1 advertise on coded phy?

[brett_anderson]

I'm trying to advertise on the coded phy, but SDK 16 seems to have a block in place against doing this when you use S340. I can obviously edit this, but I don't normally edit the SDK without checking first. Does the S340 fully support the coded phy?

static bool config_is_valid(ble_adv_modes_config_t const * const p_config)
{
    if ((p_config->ble_adv_directed_high_duty_enabled == true) &&
        (p_config->ble_adv_extended_enabled == true))
    {
        return false;
    }
#if !defined (S140)
    else if ( p_config->ble_adv_primary_phy == BLE_GAP_PHY_CODED ||
              p_config->ble_adv_secondary_phy == BLE_GAP_PHY_CODED)
    {
        return false;
    }
#endif // !defined (S140)
    else
    {
        return true;
    }
}

As you can see, in ble_advertising.c if you are not specified with S140, it will not init the advertising module with the coded phy.

[brett_anderson]

I just did a quick check and swapped to the S140. I can advertise successfully on the S140 using the coded phy, but I cannot on the S340. If I set the phy to be coded on the S340 (and comment out the block), it instead advertises on the 1M phy. Same code on the S140, and it uses the coded phy.

Any thoughts on this? I need to be able to advertise using the coded phy, but I also need the ANT stack.