Optimizing F7 template

04c5f18d · Sebastian Renner · 26d6015e · 04c5f18d · 04c5f18d · 04c5f18d
Commit 04c5f18d authored Jul 22, 2019 by Sebastian Renner
Hide whitespace changes
Inline Side-by-side

Showing with 201 additions and 8 deletions

templates/f7/Makefile
+3 -2

templates/f7/NIST_Makefile
+192 -0

templates/f7/Src/main.c
+5 -5

templates/f7/middleware.py
+1 -1

No files found.
--- a/templates/f7/Makefile
+++ b/templates/f7/Makefile
@@ -20,9 +20,9 @@ TARGET = f7
 # building variables
 ######################################
 # debug build?
-DEBUG = 1
+DEBUG = 0
 # optimization
-OPT = -Og
+OPT = -O2
 #######################################
@@ -130,6 +130,7 @@ endif
 # Generate dependency information
 CFLAGS += -MMD -MP -MF"$(@:%.o=%.d)"
+#CFLAGS +=-std=c99 -Wextra -Wshadow
 #######################################

--- a/templates/f7/NIST_Makefile
+++ b/templates/f7/NIST_Makefile
+##########################################################################################################################
+# File automatically-generated by tool: [projectgenerator] version: [3.3.0] date: [Sun Jul 14 11:29:48 CEST 2019]
+##########################################################################################################################
+# ------------------------------------------------
+# Generic Makefile (based on gcc)
+#
+# ChangeLog :
+#	2017-02-10 - Several enhancements + project update mode
+#   2015-07-22 - first version
+# ------------------------------------------------
+######################################
+# target
+######################################
+TARGET = f7
+######################################
+# building variables
+######################################
+# debug build?
+DEBUG = 0
+# optimization
+OPT = -O2
+#######################################
+# paths
+#######################################
+# Build path
+BUILD_DIR = build
+######################################
+# source
+######################################
+# C sources
+C_SOURCES =  \
+Src/main.c \
+Src/stm32f7xx_it.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_hal_cortex.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_gpio.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_usart.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_dma.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_rcc.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_utils.c \
+Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_ll_exti.c \
+Src/system_stm32f7xx.c \
+Src/RTT/SEGGER_RTT.c \
+Src/RTT/SEGGER_RTT_printf.c \
+$(SRC_FILES)
+# ASM sources
+ASM_SOURCES =  \
+startup_stm32f746xx.s
+#######################################
+# binaries
+#######################################
+PREFIX = arm-none-eabi-
+# The gcc compiler bin path can be either defined in make command via GCC_PATH variable (> make GCC_PATH=xxx)
+# either it can be added to the PATH environment variable.
+ifdef GCC_PATH
+CC = $(GCC_PATH)/$(PREFIX)gcc
+AS = $(GCC_PATH)/$(PREFIX)gcc -x assembler-with-cpp
+CP = $(GCC_PATH)/$(PREFIX)objcopy
+SZ = $(GCC_PATH)/$(PREFIX)size
+else
+CC = $(PREFIX)gcc
+AS = $(PREFIX)gcc -x assembler-with-cpp
+CP = $(PREFIX)objcopy
+SZ = $(PREFIX)size
+endif
+HEX = $(CP) -O ihex
+BIN = $(CP) -O binary -S
+#######################################
+# CFLAGS
+#######################################
+# cpu
+CPU = -mcpu=cortex-m7
+# fpu
+FPU = -mfpu=fpv5-sp-d16
+# float-abi
+FLOAT-ABI = -mfloat-abi=hard
+# mcu
+MCU = $(CPU) -mthumb $(FPU) $(FLOAT-ABI)
+# macros for gcc
+# AS defines
+AS_DEFS = 
+# C defines
+C_DEFS =  \
+-DUSE_FULL_LL_DRIVER \
+-DUSE_HAL_DRIVER \
+-DSTM32F746xx \
+-DUSE_FULL_LL_DRIVER \
+-DUSE_HAL_DRIVER \
+-DUSE_HAL_DRIVER \
+-DSTM32F746xx
+# AS includes
+AS_INCLUDES = 
+# C includes
+C_INCLUDES =  \
+-IInc \
+-IDrivers/STM32F7xx_HAL_Driver/Inc \
+-IDrivers/STM32F7xx_HAL_Driver/Inc/Legacy \
+-IDrivers/CMSIS/Device/ST/STM32F7xx/Include \
+-IDrivers/CMSIS/Include \
+# compile gcc flags
+ASFLAGS = $(MCU) $(AS_DEFS) $(AS_INCLUDES) $(OPT) -Wall -fdata-sections -ffunction-sections 
+CFLAGS = $(MCU) $(C_DEFS) $(C_INCLUDES) $(OPT) -Wall -fdata-sections -ffunction-sections 
+ifeq ($(DEBUG), 1)
+CFLAGS += -g -gdwarf-2
+endif
+# Generate dependency information
+CFLAGS += -MMD -MP -MF"$(@:%.o=%.d)"
+CFLAGS +=-std=c99 -Wall -Wextra -Wshadow
+LFLAGS=-lm
+#######################################
+# LDFLAGS
+#######################################
+# link script
+LDSCRIPT = STM32F746ZGTx_FLASH.ld
+# libraries
+LIBS = -lc -lm -lnosys 
+LIBDIR = 
+LDFLAGS = $(MCU) -specs=nano.specs -T$(LDSCRIPT) $(LIBDIR) $(LIBS) -Wl,-Map=$(BUILD_DIR)/$(TARGET).map,--cref -Wl,--gc-sections
+# default action: build all
+all: $(BUILD_DIR)/$(TARGET).elf $(BUILD_DIR)/$(TARGET).hex $(BUILD_DIR)/$(TARGET).bin
+#######################################
+# build the application
+#######################################
+# list of objects
+OBJECTS = $(addprefix $(BUILD_DIR)/,$(notdir $(C_SOURCES:.c=.o)))
+vpath %.c $(sort $(dir $(C_SOURCES)))
+# list of ASM program objects
+OBJECTS += $(addprefix $(BUILD_DIR)/,$(notdir $(ASM_SOURCES:.s=.o)))
+vpath %.s $(sort $(dir $(ASM_SOURCES)))
+$(BUILD_DIR)/%.o: %.c Makefile | $(BUILD_DIR) 
+	$(CC) -c $(CFLAGS) -Wa,-a,-ad,-alms=$(BUILD_DIR)/$(notdir $(<:.c=.lst)) $< -o $@
+$(BUILD_DIR)/%.o: %.s Makefile | $(BUILD_DIR)
+	$(AS) -c $(CFLAGS) $< -o $@
+$(BUILD_DIR)/$(TARGET).elf: $(OBJECTS) Makefile
+	$(CC) $(OBJECTS) $(LDFLAGS) -o $@
+	$(SZ) $@
+$(BUILD_DIR)/%.hex: $(BUILD_DIR)/%.elf | $(BUILD_DIR)
+	$(HEX) $< $@
+$(BUILD_DIR)/%.bin: $(BUILD_DIR)/%.elf | $(BUILD_DIR)
+	$(BIN) $< $@	
+$(BUILD_DIR):
+	mkdir $@		
+#######################################
+# clean up
+#######################################
+clean:
+	-rm -fR $(BUILD_DIR)
+#######################################
+# dependencies
+#######################################
+-include $(wildcard $(BUILD_DIR)/*.d)
+# *** EOF ***
--- a/templates/f7/Src/main.c
+++ b/templates/f7/Src/main.c
@@ -24,7 +24,7 @@
 #include "api.h"
 #include <main.h>
 //#define DEBUG
-#define MAX_LEN 1400
+#define MAX_LEN 100
 #ifdef DEBUG
  #include "SEGGER_RTT.h"
@@ -213,9 +213,9 @@ int main(void)
        dbg_printf("p: "); for (int i = 0; i < nplen; i++) dbg_printf("%02x", npub[i]); dbg_printf("\n");
        dbg_printf("k: "); for (int i = 0; i < klen; i++) dbg_printf("%02x", k[i]); dbg_printf("\n");
        #endif
-        LL_GPIO_TogglePin(GPIOB, LL_GPIO_PIN_7);
+        LL_GPIO_ResetOutputPin(GPIOD, LL_GPIO_PIN_7);
        res = crypto_aead_encrypt(c, &clen, m, mlen, ad, adlen, nsec, npub, k);
-        LL_GPIO_TogglePin(GPIOB, LL_GPIO_PIN_7);
+        LL_GPIO_SetOutputPin(GPIOD, LL_GPIO_PIN_7);
 				break;
      case 'd':
@@ -226,9 +226,9 @@ int main(void)
        dbg_printf("k: "); for (int i = 0; i < klen; i++) dbg_printf("%02x", k[i]); dbg_printf("\n");
        #endif
-        LL_GPIO_TogglePin(GPIOB, LL_GPIO_PIN_12);
+        LL_GPIO_ResetOutputPin(GPIOD, LL_GPIO_PIN_7);
        res = crypto_aead_decrypt(m, &mlen, nsec, c, clen, ad, adlen, npub, k);
-        LL_GPIO_TogglePin(GPIOB, LL_GPIO_PIN_12);
+        LL_GPIO_SetOutputPin(GPIOD, LL_GPIO_PIN_7);
        break;
      default:

--- a/templates/f7/middleware.py
+++ b/templates/f7/middleware.py
@@ -55,7 +55,6 @@ def submit(channel, action, data):
        h = struct.pack("<BI", ord(action), len(data))
    write(channel, h)
    write(channel, data)
-    time.sleep(0.1)
 def main(argv):
    if DBG:
@@ -78,6 +77,7 @@ def main(argv):
        elif action == b'C' or action == b'M' or action == b'A' or action == b'K' or action == b'S' or action == b'P':
            write('ser', action)
+            time.sleep(0.1)
            data = obtain('ser')
            if DBG:
                print("Obtained following data from serial: %s\n" % data, file=sys.stderr)