the APP_ERROR in the simple message

[zhengrui]

I created a simple message mode based on the simple_on_off modle,when I debuged the codes in simple message client,I got an error(app_error),I never changed the codes for this.And there is no trouble when I debugged the simple_on_off modle.Could you tell me some efficacious methods for this problem?

<t: 0>, main.c, 487, ----- BLE Mesh Light Switch Client Demo -----
<t: 0>, nrf_mesh_sdk.c, 187, Initializing softdevice
<t: 0>, nrf_mesh_sdk.c, 122, Initializing SoftDevice...
<t: 2>, nrf_mesh_sdk.c, 131, Ram base: 0x200019C0
<t: 14>, nrf_mesh_sdk.c, 197, Initializing mesh stack
<t: 333>, nrf_mesh_sdk.c, 205, Enabling mesh stack
<t: 337>, main.c, 191, Setting up access layer and models
<t: 356>, nrf_mesh_sdk.c, 72, APP_ERROR: F:\nrf5_SDK_for_Mesh_v1.0.1_src\examples\light_switch\client\src\main.c:212 code 14

Thanks in advance.

[Hung Bui]

Hi Zhengrui, 

 

When you face an error and there is a line of code and code number provided in the log, please check the line of code in the file to see what throws the error and check the error code to see what it means. The error code number is listed in nrf_error.h

[zhengrui]

Thanks for your valuable reply.

I find the error by following your words.

#define NRF_ERROR_NULL                        (NRF_ERROR_BASE_NUM + 14) ///< Null Pointer

But I could not understand the means of the error and I don't know why.Maybe that's the point.I tried my best to solve this problem by myself,but I have to ask for your help after tried many methods.

Sorry to disturb you again.

[zhengrui]

Sorry to reply too late.I have checked it and it shows  the error happens because of set_cb == NULL.And the simple_message_set_cb_t is defined as: 

typedef bool (*simple_message_set_cb_t)(const simple_message_client_t * p_self,nrf_mesh_address_t src,nrf_mesh_address_t dst, uint8_t *data, uint8_t length);

/** Simple Message Client state structure. */
struct __simple_message_client
{

/** Status callback called after status received from server. */
simple_message_set_cb_t set_cb;

};

so I think it's a value of bool.In the function,I valued it as true:

for (uint32_t i = 0; i < CLIENT_COUNT; ++i)
{
m_clients[i].set_cb = true;
m_clients[i].status_cb = client_status_cb;
ERROR_CHECK(simple_message_client_init(&m_clients[i], i));
}

the error is eliminated.I don't know whether it is correct and  I still don't understand the principle of it works.

[Hung Bui]

It seems that you created your own "simple_message" model ? 

Why do you want to add set_cb into the structure ? what's the difference between set_cb and status_cb ? 

Do you know the use of those variant ? 

The status_cb is used to assign it to the address of the function you want to call when you receive a status reply from the server. There is no point assigning set_cb = true. 

You need to assign it value to the address of the function your want to call. If you don't have any use for it, remove it both in the declaration of the struct and in simple_message_client_init()

 

 

[zhengrui]

I created the model by following 

https://devzone.nordicsemi.com/f/nordic-q-a/29836/send-and-receive-a-string-via-access-layer

and the file is provided by an application engineer at Nordic as it said.I changed the simple_on_off_client.c to simple_message_client.c.so i can't find the value of set_cb and i'm confused about it.

[Hung Bui]

If you have a look at rx_set_cb() you can see that set_cb is called there. This is the way the model can forward the event to the application. 

The event here is the message with simple_message_OPCODE_SEND opcode. Check m_opcode_handlers[].

 

I strongly suggest you to investigate the simple_on_off model before you create your own model. You would need to understand how the application - model - access layer connect together. 

[zhengrui]

thanks for your advice,i'll investigate the model and it maybe a work for a long time.i'll ask for your help if there are other problems.