sys/defs; add default definitions for sec_XXX_core. New section

definitions for sec_XXX_secret. Place enc_flash structure within
the secret section.

Author: Marko Kiiskila

hw/bsp/native/src/hal_bsp.c

@@ -30,6 +30,7 @@
 #include "mcu/native_bsp.h"
 #include "mcu/mcu_hal.h"
 #include "hal/hal_i2c.h"
+#include "defs/sections.h"
 #include "ef_tinycrypt/ef_tinycrypt.h"
 
 #if MYNEWT_VAL(SIM_ACCEL_PRESENT)
@@ -40,7 +41,7 @@ static struct sim_accel os_bsp_accel0;
 static struct uart_dev os_bsp_uart0;
 static struct uart_dev os_bsp_uart1;
 
-static struct eflash_tinycrypt_dev ef_dev0 = {
+static sec_data_secret struct eflash_tinycrypt_dev ef_dev0 = {
     .etd_dev = {
         .efd_hal = {
             .hf_itf = &enc_flash_funcs,

hw/bsp/nrf52dk/src/hal_bsp.c

@@ -29,6 +29,7 @@
 #include "mcu/nrf52_hal.h"
 #include "mcu/nrf52_periph.h"
 #include "bsp/bsp.h"
+#include "defs/sections.h"
 #if MYNEWT_VAL(SOFT_PWM)
 #include "pwm/pwm.h"
 #include "soft_pwm/soft_pwm.h"
@@ -63,7 +64,7 @@ static const struct hal_bsp_mem_dump dump_cfg[] = {
 };
 
 #if MYNEWT_VAL(ENC_FLASH_DEV)
-struct eflash_nrf5x_dev enc_flash_dev0 = {
+static sec_data_secret struct eflash_nrf5x_dev enc_flash_dev0 = {
     .end_dev = {
         .efd_hal = {
             .hf_itf = &enc_flash_funcs,

sys/defs/include/defs/sections.h

@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+#ifndef H_DEFS_SECTIONS_
+#define H_DEFS_SECTIONS_
+
+#include <bsp/bsp.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/**
+ * BSP/MCU can specify special sections it wants to use.
+ * But it does not have to, and if it doesn't define them, then here is the
+ * default.
+ */
+/*
+ * XXX explanation of when to use these 3?
+ */
+#ifndef sec_data_core
+#define sec_data_core
+#endif
+#ifndef sec_bss_core
+#define sec_bss_core
+#endif
+#ifndef sec_bss_nz_core
+#define sec_bss_nz_core
+#endif
+
+/**
+ * Sensitive data (eg keys), which should be excluded from corefiles.
+ */
+#ifndef sec_bss_secret
+#define sec_bss_secret
+#endif
+#ifndef sec_data_secret
+#define sec_data_secret
+#endif
+
+#ifdef __cplusplus
+}
+#endif
+
+
+#endif /* H_DEFS_SECTIONS_ */