Makefile not including files from Makefile.common

RE: Makefile not including files from Makefile.common

Anton Maes — Sun, 23 Feb 2014 12:40:58 GMT

Unfortunately when I try to run the flash download the same error causes it to launch a popup saying a problem has occured. Should this also be ignored?

RE: Makefile not including files from Makefile.common

Anton Maes — Sun, 23 Feb 2014 12:27:32 GMT

Yep that looks familiar.

I'm down to this error: undefined reference to `softdevice_ble_evt_handler_set'

So i guess it should be ok. I there any other check i can do to make sure the build was ok? I do see that the build folder has been populated.

RE: Makefile not including files from Makefile.common

Mike — Thu, 13 Feb 2014 19:12:07 GMT

Are you really talking about a gcc error or an error in eclipse. For the eclipse error check this: https://devzone.nordicsemi.com/index.php/eclipse-kepler-as-ide-for-nordic-sdk-stucks-on-__inline#respond

RE: Makefile not including files from Makefile.common

Ole Morten — Thu, 23 Jan 2014 09:20:40 GMT

Have you verified that the path C:/Program Files (x86)/GNU Tools ARM Embedded/4.8 2013q4/lib/gcc/arm-none-eabi/4.8/armv6-m actually exists? This version number will usually be on the format x.y.z, not just x.y, and on my computer only the folder with 4.8.3 exists. I'd hence recommend you to try setting GNU_VERSION to 4.8.3.

Based on your build output, you should also include app_button.c, app_gpiote.c and set the define BLE_STACK_SUPPORT_REQD.

RE: Makefile not including files from Makefile.common

Anton Maes — Wed, 22 Jan 2014 22:08:16 GMT

Makefile.windows looks ok. I also tried it with 4.7 2013q1 but no difference.

ifeq ($(findstring 86, $(ProgramFiles)), ) PROGFILES := C:/Program Files else PROGFILES := C:/Program Files (x86) endif

GNU_INSTALL_ROOT := $(PROGFILES)/GNU Tools ARM Embedded/4.8 2013q4 GNU_VERSION := 4.8 GNU_PREFIX := arm-none-eabi

build_console_output.txt

RE: Makefile not including files from Makefile.common

Ole Morten — Wed, 22 Jan 2014 15:58:12 GMT

Can you please make sure that the GCC version is set correctly in Makefile.windows? If it isn't, it won't find the toolchain includes, which seems to be what you're describing.

If you still have trouble, can you please try uploading the entire build output as a text file, so I can see exactly what it does?

RE: Makefile not including files from Makefile.common

Anton Maes — Tue, 21 Jan 2014 20:31:50 GMT

Ok I included all the c_source files and now it seems to get trough building battery.c but I still have tons of undefined symbols in main.c

eg:

Symbol 'PSTORAGE_FLASH_PAGE_END' could not be resolved main.c /ble_app_hrs_PCA10001 line 514 Symbol 'PSTORAGE_FLASH_PAGE_END' could not be resolved main.c /ble_app_hrs_PCA10001 line 515 Symbol 'NULL' could not be resolved main.c /ble_app_hrs_PCA10001 line 383 Symbol 'RADIO_CRCCNF_LEN_Msk' could not be resolved radio_config.c /LED line 57

These seem to all be in pstorage_platform.h which is in the project.

Looking in the file I see <stdint.h> is listed as an unresolved inclusion. Could this be a toolchain problem? I also see uint32_t cannot be resolved.

RE: Makefile not including files from Makefile.common

Anton Maes — Tue, 21 Jan 2014 20:12:43 GMT

Yes it's in the gcc subdirectory
It seems to find app_error.h but now app_error is looking for nrf_error which is in the Include/s110/ folder. I've added that as well to the makefile but it still can't seem to find it. [code type="xml"]In file included from ../battery.c:29:0: ../../../../../Include/app_common/app_error.h:27:23: fatal error: nrf_error.h: No such file or directory #include "nrf_error.h"[/code]

RE: Makefile not including files from Makefile.common

Ole Morten — Tue, 21 Jan 2014 09:11:21 GMT

I think that David are correct in that Makefile.common is missing an include path. If you take a look at the default Makefile for ble_app_hrs for nrf6310, you can see that it adds several includepaths:


TARGET_CHIP := NRF51822_QFAA_CA
BOARD := BOARD_NRF6310
CFLAGS += -DNRF51822_QFAA_CA -DBLE_STACK_SUPPORT_REQD -DBOARD_NRF6310

C_SOURCE_FILES += main.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_services/ble_bas.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_services/ble_dis.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_services/ble_hrs.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_services/ble_srv_common.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_error_log.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_advdata.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_sensorsim.c
C_SOURCE_FILES += ../../../../../Source/sd_common/softdevice_handler.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_conn_params.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_bondmngr.c
C_SOURCE_FILES += ../../../../../Source/app_common/pstorage.c
C_SOURCE_FILES += ../../../../../Source/app_common/crc16.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_radio_notification.c
C_SOURCE_FILES += ../../../../../Source/ble/ble_debug_assert_handler.c
C_SOURCE_FILES += ../../../../../Source/app_common/app_timer.c

INCLUDEPATHS += -I"../../../../../Include/ble"
INCLUDEPATHS += -I"../../../../../Include/s110"
INCLUDEPATHS += -I"../../../../../Include/app_common"
INCLUDEPATHS += -I"../../../../../Include/ble/ble_services"
INCLUDEPATHS += -I"../../../../../Include/sd_common"
SDK_PATH = ../../../../../
OUTPUT_FILENAME := ble_hrs_gcc

DEVICE_VARIANT := xxaa
#DEVICE_VARIANT := xxab

USE_SOFTDEVICE := S110
#USE_SOFTDEVICE := s210

include $(SDK_PATH)Source/templates/gcc/Makefile.common

Also, I see that you in your Makefile does not set the USE_SOFTDEVICE variable. If you compile ble_app_hrs, you most likely want to set this, if not, things won't work.

Finally, you may want to have a look at the GCC setup available here.

RE: Makefile not including files from Makefile.common

David Smoot — Mon, 20 Jan 2014 22:43:36 GMT

Is your makefile in a sub-directory of ble_app_hrs_gcc? If you followed the pattern from the blinky example, it should be. One possible problem is your relative path is wrong.
More likely is you are facing the same issue I was in that the common make file does not include all the paths you need. Try adding the line INCLUDEPATHS += -I"../../../../../../Include/app_common" and see if you get a different error. If so, you just need to add all the necessary include paths either in your local makefile or in makefile.common.