CTL setup server subscription sharing

[DeveloperZ]

Hi,

Mesh SDK 4.2.0.

In light_ctl_setup_server.c,

if (NRF_SUCCESS == status)
    {
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->model_handle));
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->generic_ponoff_setup_srv.generic_ponoff_srv.model_handle));
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->generic_ponoff_setup_srv.generic_ponoff_srv.generic_onoff_srv.model_handle));
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->generic_ponoff_setup_srv.generic_dtt_srv.model_handle));
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->light_lightness_srv.model_handle));
        NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->light_lightness_srv.generic_level_srv.model_handle));

        m_total_ctl_ss_instances++;
    }

This subscription list sharing doesn't share it with Generic Power OnOff Setup server but only does it with Generic Power OnOff server. 

I added the below code and that fixed it.

NRF_MESH_ERROR_CHECK(access_model_subscription_lists_share(p_s_server->model_handle,
                                                                   p_ll_s_server->generic_ponoff_setup_srv.model_handle));

Thank you.

[tesc]

Hi,

Thank you for notifying us about this, and also providing a proposed fix. It is highly appreciated!

I have registered the issue in our internal tracker, for the mesh team to look into and fix.

Regards,
Terje

[tesc]

Hi again,

I am happy to report that the mesh team has now added the lines, so it will be part of next nRF5 SDK for Mesh release.

Until then, you need to patch this manually in light_ctl_setup_server.c, and add the two lines inside that if block.

Regards,
Terje