pkg/cryptoauthlib: Pass ATCA_NO_HEAP Flag to library

[Einhornhool]

Contribution description

ATCA_NO_HEAP must be passed to cryptoauthlib build system in order to not use dynamic memory allocation.
Build fails when using KCONFIG for dependency resolution, because ATCA_NO_HEAP is not defined.
This fixes the problem.

Testing procedure

Build tests/pkg_cryptoauthlib_compare_sha256/ using KCONFIG dependency resolution.

[riot-ci]

Murdock results

✔️ PASSED

2be8b22 pkg/cryptoauthlib: Pass ATCA_NO_HEAP Flag to library

Success	Failures	Total	Runtime
1980	0	1980	06m:51s

Artifacts

• Documentation preview

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

[MrKevinWeiss]

ACK

Running:

BOARD=acd52832 TEST_KCONFIG=1 make -C tests/pkg_cryptoauthlib_compare_sha256/

now passes, but fails in master. Also it makes sense, the only thing I didn't understand is why it works with regular make. Oh well...

[Einhornhool]

ACK

Running:

BOARD=acd52832 TEST_KCONFIG=1 make -C tests/pkg_cryptoauthlib_compare_sha256/

now passes, but fails in master. Also it makes sense, the only thing I didn't understand is why it works with regular make. Oh well...

I took a look at it and what I've found is, that cmake (which is being used to build the cryptoauthlib), generates an atca_config.h file in the test application pkg-build folder, in which some symbols for the build are defined.
If we pass ATCA_NO_HEAP=TRUE, it is also defined in this config file.

If we don't pass it and build with make, it does not define ATCA_NO_HEAP, but it contains the lines
#define ATCA_PLATFORM_MALLOC malloc and #define ATCA_PLATFORM_FREE free.
If we build with Kconfig, it does not contain the malloc and free defines.

I don't know why that is, but since we don't want to use malloc and free, I think this is the best fix, because it actually leads to the correct library configuration.