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.

[brett_anderson]

So, there is a second block, in ble_advertising.c, called phy_is_valid(), that also blocks unless the S140 is specified. 

Turns out, if I comment out both of these blocks, the S340 does indeed advertise using the coded phy.

So my question remains, is the coded phy fully supported or not in S340? If it is, then I will simply leave my edits in place. I assume that Nordic will want to add a further check to include the S340 for future SDK releases. If it is not fully supported, when do we expect it to be?

[Simonr]

Hi

According to thisisant.com (which is the guys distributing the ANT SoftDevice for us) S340 does support long-range BLE (Coded PHY). Have you included all the necessary header files for S340 in the /components/softdevice/S340/headers path? 

You can ask the ANT guys for confirmation, but as far as I can see, the S340 should fully support Coded PHY.

Best regards,

Simon

[brett_anderson]

Hi Simon,

ble_advertising.c is not a part of the soft device header files, so the Garmin guys don't control that. SDK 16.0 seems to block the S340 from using the coded phy. I was able to modify the SDK to allow it, and it is working.

I guess maybe just report the issue so it can be resolved in the SDK?

[Simonr]

Hi Brett

I agree. I have reported this internally now. This will likely be fixed in an upcoming release of the SDK, but until then, it should be fine adding S340 to the phy_is_valid() check.

Best regards,

Simon