nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

gwayne — Fri, 10 Apr 2015 15:02:37 GMT

I just spent an entire day updating my project from SDK 7.1 components to 8.0. I ran into this and a number of other issues I still haven't resolved. Nordic should really publish a thorough migration app note for major releases like this.

I found nAN-36 to be one of the most informative and useful documents Nordic publishes on developing applications for the nRF51. It should be updated along with the Github example or replaced with something just as comprehensive encompassing SDK 8.0 and implementation via IAR (not just Keil).

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

Stian Røed Hafskjold — Fri, 13 Mar 2015 09:29:46 GMT

You are correct that if you got the most recent SDK (which is version 8.0) APP_BUTTON_INIT() is not defined. Thank you for pointing this out. The Github version of the lbs example is based on SDK 7.1 (which is stated in the readme file). The nAN-36 app note is really old now, and in my opinion should have been removed from the site, so I understand you frustration, sorry about that. We are working on an alternative to the app note. We sure also have to update the Github example to the recent SDK 8 version which was released a few days ago.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

bulbse — Thu, 12 Mar 2015 18:56:25 GMT

out of the box it fails, I got the most recent sdk and keil. I grabbed the git hub example because there were a lot of inconsistencies with the app note and the current release. since the git hub project did not compile i took the source files from that directory and added it to a new template project to make this thing work. which it didn't. however the app_button_init is defined differently in the current sdk then it was from when the app note was written. I would seriously consider re-writing that app note. Thanks for allowing me to identify the answer.

The Correct solution was to replace

APP_BUTTON_INIT(buttons, sizeof(buttons) / sizeof(buttons[0]), BUTTON_DETECTION_DELAY, true);

with

app_button_init(buttons, sizeof(buttons) / sizeof(buttons[0]), BUTTON_DETECTION_DELAY);

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

Stian Røed Hafskjold — Thu, 12 Mar 2015 09:50:42 GMT

APP_BUTTON_INIT() is defined on line 92 in app_button.h if you are using SDK 7.1. If it is not there, where did you download the SDK? Are you sure you haven't altered the file in another project or something? It should compile right out of the box.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

bulbse — Thu, 12 Mar 2015 02:07:21 GMT

I also tried replacing the call, you included an additional parameter that was not defined in the header file definition ..., app_button_evt_schedule)... I removed that last parameter and it compiled.

the app note calls out a function that is not defined in my header why is that?

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

bulbse — Thu, 12 Mar 2015 02:02:02 GMT

Yes the header file is correctly mapped, it seems like the header doesn't define APP_BUTTON_INIT I need to find a header file in which this is defined or will need to define it myself. I still have to verify the proposed solution.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

bulbse — Thu, 12 Mar 2015 01:59:40 GMT

I couldn't find "#define APP_BUTTON_INIT()" in my app_button.h, Am i supposed to define it? I didn't touch any files other than what the app note instructs.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

bulbse — Thu, 12 Mar 2015 01:53:41 GMT

No the code does not compile.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

Chris — Thu, 12 Mar 2015 01:51:37 GMT

If you right click on "app_button.h" does it open the header file? If not maybe there is a typo in c_user_include_directories. If you are using macro expansion maybe try and use the full path and see if that fixes the problem. This happened to me recently using CrossWorks on OSX.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

RK — Thu, 12 Mar 2015 01:10:53 GMT

Does the code compile even though Keil shows this problem? There's been a few posts here recently where people have had similar issues, code showing errors on the preview screen but actually compiling correctly.

RE: nAN-36 + nRF51822 PCA10028 APP_BUTTON_INIT [SOLVED]

Stian Røed Hafskjold — Wed, 11 Mar 2015 12:32:30 GMT

The only way I can reproduce this is if I deliberately delete the #define APP_BUTTON_INIT(...) definition in app_button.h.

Are you sure there are no typos or something in app_button.h? Is the file added to the include paths?

If you download the example from git again (github.com/.../nrf51-ble-app-lbs), and put it in the SDK 7.1 examples/ble_peripheral folder without modifying it, does it compile?

What happens if you replace the call to the APP_BUTTON_INIT macro in main.c with a direct call to app_button_init()? like this:

app_button_init(buttons, sizeof(buttons) / sizeof(buttons[0]), BUTTON_DETECTION_DELAY, app_button_evt_schedule);