Adding a second model to the example: Publish adress handle not set.

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Tue, 22 May 2018 12:16:43 GMT

no in main.c. also please open a new issue.

RE: Adding a second model to the example: Publish adress handle not set.

Igor — Tue, 22 May 2018 11:55:24 GMT

config? You mean the do_config_step() method?

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Mon, 21 May 2018 19:29:44 GMT

also; if you open a second ticket with more information Nordic staff will help you out (they're quire responsive) and if you linkit here i'll also take a look.

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Mon, 21 May 2018 19:28:53 GMT

at the time i was using the mesh sdk v1.0.1 and my answer fixed it - the config wasn't right. are you using the sdk v2.0.1? anyways - make sure that you configure the model correctly

RE: Adding a second model to the example: Publish adress handle not set.

Igor — Mon, 21 May 2018 15:37:23 GMT

@sommmen I have the same problem. Code is written as in your example above. How did fix this?

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Mon, 12 Mar 2018 11:26:15 GMT

Figured it out;

1. set the publish address in void provisioner_config_successful_cb

2. add it to local storage in void provisioner_prov_complete_cb

messed something up there, it is fixed now.

Could still do with a proper tutorial on the provisioning/configurating part though.

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Mon, 12 Mar 2018 09:08:55 GMT

Hi, thanks for your response!

I did, however nowhere is described how to properly use the provisioner/configuration in the example.

The example itself is still working fine.

The on/off example uses a mechanism to add several servers to the clients and im now trying to figure out how to add my own model properly.

Basically it comes down to the fact that i don't know how to do these parts;

void provisioner_config_successful_cb(void)
{
    __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Configuration of device %u successful\n", m_configured_devices);

    /* Set publish address for the client to the corresponding server. */
    ERROR_CHECK(access_model_publish_address_set(m_clients[m_configured_devices].model_handle,
                                                 m_server_handles[m_configured_devices]));
    
    //@TODO add own model here

    access_flash_config_store();

    hal_led_pin_set(BSP_LED_0 + m_configured_devices, false);
    m_configured_devices++;

    if (m_configured_devices < SERVER_COUNT)
    {
        provisioner_wait_for_unprov(UNPROV_START_ADDRESS + m_provisioned_devices);
        hal_led_pin_set(BSP_LED_0 + m_configured_devices, true);
    }
    else
    {
        __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "All servers provisioned\n");
        hal_led_blink_ms(LEDS_MASK, 100, 4);
    }
}

void provisioner_prov_complete_cb(const nrf_mesh_prov_evt_complete_t * p_prov_data)
{
    /* We should not get here if all servers are provisioned. */
    NRF_MESH_ASSERT(m_configured_devices < SERVER_COUNT);

    __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Provisioning complete. Adding address 0x%04x.\n", p_prov_data->address);

    /* Add to local storage. */
    ERROR_CHECK(dsm_address_publish_add(p_prov_data->address, &m_server_handles[m_provisioned_devices]));
    ERROR_CHECK(dsm_devkey_add(p_prov_data->address, m_netkey_handle, p_prov_data->p_devkey, &m_devkey_handles[m_provisioned_devices]));

    //@TODO add own model to local storage and bind client/config.

    /* Bind the device key to the configuration server and set the new node as the active server. */
    ERROR_CHECK(config_client_server_bind(m_devkey_handles[m_provisioned_devices]));
    ERROR_CHECK(config_client_server_set(m_devkey_handles[m_provisioned_devices],
                                         m_server_handles[m_provisioned_devices]));

    m_provisioned_devices++;

    /* Move on to the configuration step. */
    provisioner_configure(UNPROV_START_ADDRESS + m_configured_devices);
}

I bound my client/server to element 0.

I can publish server -> client fine, but a client -> server publish gives the invalid param error.

RE: Adding a second model to the example: Publish adress handle not set.

Håkon Alseth — Mon, 12 Mar 2018 08:54:15 GMT

Hi,

Did you follow this howto in the mesh docs when creating your own model?

http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.meshsdk.v1.0.1/md_doc_getting_started_how_to_models.html?cp=4_1_0_3_4

When you updated your code with the configuration from on/off server, did you get different behavior?

Best regards,

Håkon

RE: Adding a second model to the example: Publish adress handle not set.

sommmen — Thu, 08 Mar 2018 15:45:39 GMT

Update:

I copied the configuration code from the on/off server:

        case PROV_STATE_CONFIG_PUBLICATION_XXX:
        {
            /* also configure XXX @TODO move to own state */
            config_publication_state_t pubstate2 = {0};
            pubstate2.element_address = m_target_address;
            pubstate2.publish_address.type = NRF_MESH_ADDRESS_TYPE_UNICAST;
            pubstate2.publish_address.value = PROVISIONER_ADDRESS;//PROVISIONER_ADDRESS + m_target_address - UNPROV_START_ADDRESS; //@TODO what should this be?
            pubstate2.appkey_index = 0;
            pubstate2.frendship_credential_flag = false;
            pubstate2.publish_ttl = SERVER_COUNT;
            pubstate2.publish_period.step_num = 0;
            pubstate2.publish_period.step_res = ACCESS_PUBLISH_RESOLUTION_100MS;
            pubstate2.retransmit_count = 1;
            pubstate2.retransmit_interval = 0;
            pubstate2.model_id.company_id = ACCESS_COMPANY_ID_NORDIC;
            pubstate2.model_id.model_id = XXX_SERVER_MODEL_ID;
            __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Setting publication address for the XXX server to 0x%04x\n", pubstate2.publish_address.value);

            ERROR_CHECK(config_client_model_publication_set(&pubstate2));          

            break;
        }