sdk_config.h inconsistency

RE: sdk_config.h inconsistency

tesc — Wed, 12 Oct 2022 13:37:27 GMT

Hi,

You are correct that focus has not been on nRF5 SDK over the past couple of years, since that SDK is now in maintenance mode. Instead we are focusing on the scalable and future-proof nRF Connect SDK. See our nRF Connect SDK and nRF5 SDK statement for details.

Regards,
Terje

RE: sdk_config.h inconsistency

__Tom — Wed, 12 Oct 2022 09:03:13 GMT

It is 2022.10.12 and I'm using SDK17.1.0. sdk_config.h in folder "nRF5_SDK_17.1.0_ddde560\config\nrf52840\config" is still not generic as it's designed for. I wonder whether you guys really have been focusing on improving your sdk. Such issue is driving our company away from your ICs.

RE: sdk_config.h inconsistency

tesc — Fri, 16 Oct 2020 09:36:12 GMT

Hi,

Thank you for the suggestion, it is always appreciated! That would be one way to do it, yes. I have forwarded the suggestion to the SDK team.

What we are talking about is the second option for overriding the sdk_config.h configuration file, as documented here: Overriding the sdk_config.h configuration.

Please note that this could still mean mulitple "standard" sdk_config.h files, as there are some differences with regards to e.g. what peripherals are available and in what quantities for the different SoCs.

Regards,
Terje

RE: sdk_config.h inconsistency

JonasJ — Thu, 15 Oct 2020 19:42:41 GMT

tesc

I count 622 sdk_config.h in the 7.0.2 release. Wouldn't it make more sense to only have one sdk_config.h in the config/nrfXXXX/config/ directory (why the extra config dir?) and make all the examples use the app_config.h feature.

This way it's very obvious what changes the examples has made to the default configuration.

Thanks.

RE: sdk_config.h inconsistency

tesc — Mon, 18 May 2020 13:16:18 GMT

Hi,

There will be future releases of the nRF5 SDK yes. Checking with the SDK team on the status of this issue with regards to next release.

Regards,
Terje

RE: sdk_config.h inconsistency

chroche — Wed, 06 May 2020 11:53:43 GMT

Hi,

about this topic, is there a fix/update planned for the next SDK release?

Btw. Is a new release SDK already scheduled?

RE: sdk_config.h inconsistency

tesc — Mon, 27 Jan 2020 09:40:39 GMT

Hi,

I see that those are defined in many of the sdk_config.h files throughout the examples, so they are definitely among the defines that should have been in all sdk_config.h.

Regards,
Terje

RE: sdk_config.h inconsistency

RVM — Fri, 24 Jan 2020 20:11:24 GMT

SDK v15.3.0 does NOT contain NRF_BLE_CONN_PARAMS_ENABLED, NRF_BLE_GATT_ENABLED among others.

I had to add them to an external file (e.g. app_config.h) in order for the build system to pull in the definitions of functions such as nrf_ble_gatt_init() etc.

RE: sdk_config.h inconsistency

tesc — Thu, 12 Dec 2019 15:59:28 GMT

Hi,

Thank you for the detailed reporting. I have notified the developers.

Regards,
Terje

RE: sdk_config.h inconsistency

Benjamin — Tue, 10 Dec 2019 23:35:13 GMT

Please fix this sooner rather than later. It's obviously been an issue for a long time, and is still a mess in version 16.

The 'generic' sdk_config.h for the nRF52840 has 1149 unique defines. The 'template' (examples/peripherals/template) sdk_config.h has 968 unique defines. Together, they have a combined 1318 unique defines. Adding in ble_app_template/../sdk_config.h brings the total to 1352 unique defines.

Across all the config files for the nRF 52840 DK, it appears there are 1735 defined parameters:

$ find . -path '*/pca10056/*' -name sdk_config.h -exec grep "#define" {} + | cut -d ' ' -f 2 | sort | uniq | wc -l
> 1735

Also, FWIW, roughly 1/3 of that is logging:

$ grep -v -e LOG -e DEBUG_COLOR -e INFO_COLOR nrf52840_sdk_config.h | wc -l
> 1169
$ grep -e LOG -e DEBUG_COLOR -e INFO_COLOR nrf52840_sdk_config.h | wc -l
> 566

For the nRF52 DK there are 1815 defined parameters in total, but only 1254 in the 'generic' sdk_config.h

Now I recognize that the 'generic' config might not contain parameters specific to the DK boards intentionally, but then please update the 'template' examples appropriately, as they are most definitely targeting the dev kits.

RE: sdk_config.h inconsistency

tesc — Tue, 10 Dec 2019 18:49:23 GMT

Hi,

The file at <SDK root folder>/config/<device>/config/sdk_config.h should contain all settings.

Regarding what is (and is not) included in the various examples, as well as the point on ordering, I will tell the developers.

Regards,
Terje

RE: sdk_config.h inconsistency

chroche — Tue, 10 Dec 2019 17:24:44 GMT

Hi,

I think with SDK 16 this is still inconsistent, which is very irritating and confusing. I am just trying to find a "full" sdk_config.h for a new project, but already the ble_app_template for pca10040 (=nrf52832) has a different one compared to top level config for nrf52832. See nRF5_SDK_16.0.0_98a08e2\examples\ble_peripheral\ble_app_template\pca10040\s132\config\sdk_config.h vs. nRF5_SDK_16.0.0_98a08e2\config\nrf52832\config\sdk_config.h. The first contains additionally e.g. NRF_BLE_CONN_PARAMS_ENABLED, NRF_BLE_GATT_ENABLED and NRFX_*, nRF_Segger_RTT blocks, the latter has a lot of ANT blocks and misses some NFC settings.

When will this be cleaned up, so that all examples and "top level" sdk_config.h are content-wise really "full" i.e. complete regards all possibilities (with default=off) and diff-able (same content, keep same order)? This would make comparing and merging examples much much more easier ...

Can you name me the actual "full" sdk_config.h for nrf52832 and S132 (if this makes a difference) I should use for a new project?

RE: sdk_config.h inconsistency

tesc — Thu, 04 Apr 2019 06:30:02 GMT

Hi,

Thank you for the pointer to the new thread. We are aware of the situation, and hopefully we can sort it out sooner rather than later. Our intention is that at least the generic sdk_config.h contains the full set of defines, and that should preferably be the case also for the ones used in the examples.

Regards,
Terje

RE: sdk_config.h inconsistency

awneil — Wed, 03 Apr 2019 08:41:57 GMT

(I know this is old, but it just came up as a 'Related' topic)

[quote userid="8164" url="~/f/nordic-q-a/36279/sdk_config-h-inconsistency/139730"]Our goal is that all the examples have all the options in their sdk_config.h.[/quote]

They most certainly do not:

https://devzone.nordicsemi.com/f/nordic-q-a/45366/missing-defines-in-generic-sdk_config-h

RE: sdk_config.h inconsistency

tesc — Wed, 11 Jul 2018 13:17:08 GMT

Hi,

(1) There are sdk_config.h template files in SDK 15, found at <sdk folder>/config/<nrf variant>/config/sdk_config.h.

(2) Our goal is that all the examples have all the options in their sdk_config.h. If not then it is a bug. Note however that what those settings are set to will differ, depending on the needs of the particular example. If you merge functionality from separate examples, or add a module to the project, you must change the settings accordingly.

I will register the suggestion of putting log configuration in a separate file, although one might argue that it would defeat the purpose of having one configuration file for the project. Note that you can do this separation manually if you set the USE_APP_CONFIG flag and put log related settings in app_config.h. (All of the settings in sdk_config.h can be overridden, either by adding the setting as a project flag/define, or by the method of setting USE_APP_CONFIG and using an app_config.h file.)

Regards,
Terje