crypto/mbedtls: Use config defining flag globally

Now -DMBEDTLS_USER_CONFIG_FILE=<mbedtls/config_mynewt.h> flag is
used globally for each build that depends on mbedtls package.
This way we won't have to add this flag in each package
that uses mbedtls.

Author: m-gorecki

apps/crypto_test/pkg.yml

@@ -28,5 +28,3 @@ pkg.deps:
     - "@apache-mynewt-core/sys/log"
     - "@apache-mynewt-core/crypto/mbedtls"
     - "@apache-mynewt-core/crypto/tinycrypt"
-
-pkg.cflags: '-DMBEDTLS_USER_CONFIG_FILE="mbedtls/config_mynewt.h"'

apps/hash_test/pkg.yml

@@ -28,5 +28,3 @@ pkg.deps:
     - "@apache-mynewt-core/sys/log"
     - "@apache-mynewt-core/crypto/mbedtls"
     - "@apache-mynewt-core/crypto/tinycrypt"
-
-pkg.cflags: '-DMBEDTLS_USER_CONFIG_FILE="mbedtls/config_mynewt.h"'

crypto/mbedtls/pkg.yml

@@ -26,11 +26,14 @@ pkg.keywords:
     - tls
 pkg.type: sdk
 
-pkg.cflags:
+app.cflags:
     - '-DMBEDTLS_USER_CONFIG_FILE=<mbedtls/config_mynewt.h>'
+app.cflags.TEST:
+    - '-DTEST'
+
+pkg.cflags:
     - -Wno-maybe-uninitialized
     - -Wno-unknown-warning-option
-pkg.cflags.TEST: -DTEST
 
 pkg.include_dirs:
     - "include"

crypto/mbedtls/syscfg.yml

@@ -119,8 +119,10 @@ syscfg.defs:
     value: 0
   MBEDTLS_CIPHER_MODE_CFB:
     value: 0
+  # XXX: This should be 0 on default, but mcuboot's test needs this enabled. Until a new mcuboot version with this
+  # config enabled is released we have to enable this by default.
   MBEDTLS_CIPHER_MODE_CTR:
-    value: 0
+    value: 1
   MBEDTLS_CIPHER_MODE_OFB:
     value: 0
   MBEDTLS_CIPHER_MODE_XTS: