RE: mesh_config_entry_get()/mesh_config_entry_set() are not working as expected

choehyunho — Tue, 03 Mar 2020 04:32:51 GMT

I found what is the problem.

my model_config_load() call is too early in model_init_cb(), so it failed.

And also I figured out that mesh_config_load() automatically includes loading each of my custom entries.

So, because mesh_config_load(0 will call getter/setter for my entries, I added memcpy() inside getter/setter, even it seems to be redundant.

After then, it works fine.

RE: mesh_config_entry_get()/mesh_config_entry_set() are not working as expected

choehyunho — Mon, 02 Mar 2020 16:56:03 GMT

I followed the emocean example.

Below is my code for declaring & getting entries.

#define LOCATION_FILE_ID                (0x0010)
#define GLOBAL_LOCATION_RECORD          (0x0001)
#define LOCAL_LOCATION_RECORD           (0x0002)
#define GLOBAL_LOCATION_ENTRY_ID        MESH_CONFIG_ENTRY_ID(LOCATION_FILE_ID, GLOBAL_LOCATION_RECORD)
#define LOCAL_LOCATION_ENTRY_ID         MESH_CONFIG_ENTRY_ID(LOCATION_FILE_ID, LOCAL_LOCATION_RECORD)

MESH_CONFIG_FILE(m_location_file, LOCATION_FILE_ID, MESH_CONFIG_STRATEGY_CONTINUOUS);

static uint32_t global_location_setter(mesh_config_entry_id_t id, const void * p_entry);
static void global_location_getter(mesh_config_entry_id_t id, void * p_entry);
static void global_location_deleter(mesh_config_entry_id_t id);
static uint32_t local_location_setter(mesh_config_entry_id_t id, const void * p_entry);
static void local_location_getter(mesh_config_entry_id_t id, void * p_entry);
static void local_location_deleter(mesh_config_entry_id_t id);

MESH_CONFIG_ENTRY(global_location,
                  GLOBAL_LOCATION_ENTRY_ID,
                  1,
                  sizeof(generic_location_global_status_params_t),
                  global_location_setter,
                  global_location_getter,
                  global_location_deleter,
                  false);

MESH_CONFIG_ENTRY(local_location,
                  LOCAL_LOCATION_ENTRY_ID,
                  1,
                  sizeof(generic_location_local_status_params_t),
                  local_location_setter,
                  local_location_getter,
                  local_location_deleter,
                  false);

static generic_location_global_status_params_t m_location_global =
{
    .global_latitude = 0x80000000,
    .global_longitude = 0x80000000,
    .global_altitude = 0x7FFF,
};

static generic_location_local_status_params_t m_location_local =
{
    .local_north = 0x8000,
    .local_east = 0x8000,
    .local_altitude = 0x7FFF,
    .floor_number = 0xFF,
    .uncertainty = 0x0000,
};

static uint32_t global_location_setter(mesh_config_entry_id_t id, const void * p_entry)
{
    return NRF_SUCCESS;
}

static void global_location_getter(mesh_config_entry_id_t id, void * p_entry)
{
}

static void global_location_deleter(mesh_config_entry_id_t id)
{
    m_location_global.global_latitude = 0x80000000;
    m_location_global.global_longitude = 0x80000000;
    m_location_global.global_altitude = 0x7FFF;
}

static uint32_t local_location_setter(mesh_config_entry_id_t id, const void * p_entry)
{
    return NRF_SUCCESS;
}

static void local_location_getter(mesh_config_entry_id_t id, void * p_entry)
{
}

static void local_location_deleter(mesh_config_entry_id_t id)
{
    m_location_local.local_north = 0x8000;
    m_location_local.local_east = 0x8000;
    m_location_local.local_altitude = 0x7FFF;
    m_location_local.floor_number = 0xFF;
    m_location_local.uncertainty = 0x0000;
}

static void model_config_load(void)
{
    mesh_config_entry_id_t idx = GLOBAL_LOCATION_ENTRY_ID;
    uint32_t status = mesh_config_entry_get(idx, &m_location_global);
    if (status == NRF_SUCCESS)
    {
        __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Load global location: 0x%08X,0x%08X,0x%04X\n",
              m_location_global.global_latitude, m_location_global.global_longitude, m_location_global.global_altitude);
    }
    idx = LOCAL_LOCATION_ENTRY_ID;
    status = mesh_config_entry_get(idx, &m_location_local);
    if (status == NRF_SUCCESS)
    {
        __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Load local location: 0x%04X,0x%04X,0x%04X,0x%02X,0x%04X\n",
              m_location_local.local_north, m_location_local.local_east, m_location_local.local_altitude,
              m_location_local.floor_number, m_location_local.uncertainty);
    }
}

And used the following code to set entries.

    // Save to configuration entry
    {
        mesh_config_entry_id_t entry_id = GLOBAL_LOCATION_ENTRY_ID;
        uint32_t status = mesh_config_entry_set(entry_id, &m_location_global);
        if (status == NRF_SUCCESS)
        {
            __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Save global location: 0x%08X,0x%08X,0x%04X\n",
                  m_location_global.global_latitude, m_location_global.global_longitude, m_location_global.global_altitude);
        }
    }
...
    // Save to configuration entry
    {
        mesh_config_entry_id_t entry_id = LOCAL_LOCATION_ENTRY_ID;
        uint32_t status = mesh_config_entry_set(entry_id, &m_location_local);
        if (status == NRF_SUCCESS)
        {
            __LOG(LOG_SRC_APP, LOG_LEVEL_INFO, "Save local location: 0x%04X,0x%04X,0x%04X,0x%02X,0x%04X\n",
                  m_location_local.local_north, m_location_local.local_east, m_location_local.local_altitude,
                  m_location_local.floor_number, m_location_local.uncertainty);
        }
    }

I am a little bit suspicious of using getter/setter code.

Because I just already have static module variable and try to share it between functions, no code is used inside getter/setter function.

Can it be problem?

RE: mesh_config_entry_get()/mesh_config_entry_set() are not working as expected

Hung Bui — Mon, 02 Mar 2020 15:15:18 GMT

Hi Choe,

Could you let me know how you define your config ID , have you followed what we did, for example in mesh_opt_provisioner.h in the light switch provisioner example ?

You need to define config file using MESH_CONFIG_FILE().

Please note that you can store your data using the flash library directly by creating your own flash area outside of the mesh config area. You can call this: mesh_stack_persistence_flash_usage() to know the mesh stack flash usage and allocate your own flash area outside of the mesh stack.