Merge pull request #42 from hathach/develop

add pca10056 to travis build

.travis.yml

@@ -20,4 +20,5 @@ script:
   - make -j2 -C examples/device/cdc_msc_hid BOARD=metro_m0_express
   - make -j2 -C examples/device/cdc_msc_hid BOARD=metro_m4_express
   - make -j2 -C examples/device/cdc_msc_hid BOARD=stm32f407g_disc1
+  - make -j2 -C examples/device/cdc_msc_hid BOARD=pca10056
 

README.md

@@ -24,6 +24,7 @@ TinyUSB is an open-source cross-platform USB Host/Device stack for embedded syst
 - Human Interface Device (HID): Keyboard, Mouse, Generic
 - Communication Class (CDC)
 - Mass Storage Class (MSC)
+- Musical Instrument Digital Interface (MIDI)
 
 ## Host Stack
 

examples/device/cdc_msc_hid/Makefile

@@ -12,6 +12,13 @@ else
   endif
 endif
 
+# Verbose mode (V=). 0: default, 1: print out CFLAG, LDFLAG 2: print all compile command
+ifeq ("$(V)","2")
+  QUIET =
+else
+  QUIET = @
+endif
+
 # If the build directory is not given, make it reflect the board name.
 BUILD ?= build-$(BOARD)
 
@@ -49,11 +56,13 @@ CFLAGS += \
 	-Wno-deprecated-declarations \
 	-Wnested-externs \
 	-Wunreachable-code \
-	-Wcast-align \
 	-Wno-error=lto-type-mismatch \
 	-ffunction-sections \
 	-fdata-sections
 
+# This causes lots of warning with nrf5x build due to nrfx code
+# CFLAGS += -Wcast-align
+
 #Debugging/Optimization
 ifeq ($(DEBUG), 1)
   CFLAGS += -O0 -ggdb
@@ -61,8 +70,17 @@ else
   CFLAGS += -flto -Os
 endif
 
-CFLAGS += $(INC) -Wall -Werror -std=gnu11 -nostdlib -DBOARD_$(shell echo $(BOARD) | tr a-z\- A-Z_)
-LDFLAGS += $(CFLAGS) -nostartfiles -fshort-enums -Wl,-T,$(TOP)/$(LD_FILE) -Wl,-Map=$@.map -Wl,-cref -Wl,-gc-sections -specs=nosys.specs -specs=nano.specs
+CFLAGS += $(INC) -Wall -Werror -std=gnu11 -DBOARD_$(shell echo $(BOARD) | tr a-z\- A-Z_)
+LDFLAGS += $(CFLAGS) -fshort-enums -Wl,-T,$(TOP)/$(LD_FILE) -Wl,-Map=$@.map -Wl,-cref -Wl,-gc-sections -specs=nosys.specs -specs=nano.specs
+
+ifeq ("$(V)","1")
+$(info CFLAGS  $(CFLAGS))
+$(info )
+$(info LDFLAGS $(LDFLAGS))
+$(info )
+$(info ASFLAGS $(ASFLAGS))
+$(info )
+endif
 
 LIBS = -lgcc -lc -lm -lnosys
 
@@ -91,6 +109,8 @@ SRC_C += $(LIB_SOURCE)
 # Assembly files can be name with upper case .S, convert it to .s 
 SRC_S := $(SRC_S:.S=.s)
 
+# Due to GCC LTO bug https://bugs.launchpad.net/gcc-arm-embedded/+bug/1747966
+# assembly file should be placed first in linking order
 OBJ += $(addprefix $(BUILD)/obj/, $(SRC_S:.s=.o))
 OBJ += $(addprefix $(BUILD)/obj/, $(SRC_C:.c=.o))
 
@@ -105,7 +125,7 @@ $(OBJ_DIRS):
 
 $(BUILD)/$(BOARD)-firmware.elf: $(OBJ)
 	@echo LINK $@
-	$(CC) -o $@ $(LDFLAGS) $^ -Wl,--start-group $(LIBS) -Wl,--end-group
+	$(QUIET)$(CC) -o $@ $(LDFLAGS) $^ -Wl,--start-group $(LIBS) -Wl,--end-group
 
 $(BUILD)/$(BOARD)-firmware.bin: $(BUILD)/$(BOARD)-firmware.elf
 	@echo CREATE $@
@@ -121,7 +141,7 @@ $(BUILD)/$(BOARD)-firmware.hex: $(BUILD)/$(BOARD)-firmware.elf
 vpath %.c . $(TOP)
 $(BUILD)/obj/%.o: %.c
 	@echo CC $(notdir $@)
-	@$(CC) $(CFLAGS) -c -MD -o $@ $<
+	$(QUIET)$(CC) $(CFLAGS) -c -MD -o $@ $<
 	@# The following fixes the dependency file.
 	@# See http://make.paulandlesley.org/autodep.html for details.
 	@# Regex adjusted from the above to play better with Windows paths, etc.
@@ -134,13 +154,13 @@ $(BUILD)/obj/%.o: %.c
 vpath %.s . $(TOP)
 $(BUILD)/obj/%.o: %.s
 	@echo AS $(notdir $@)
-	@$(CC) -x assembler-with-cpp $(ASFLAGS) -c -o $@ $<
+	$(QUIET)$(CC) -x assembler-with-cpp $(ASFLAGS) -c -o $@ $<
 
 # ASM sources upper case .S
 vpath %.S . $(TOP)
 $(BUILD)/obj/%.o: %.S
 	@echo AS $(notdir $@)
-	@$(CC) -x assembler-with-cpp $(ASFLAGS) -c -o $@ $<
+	$(QUIET)$(CC) -x assembler-with-cpp $(ASFLAGS) -c -o $@ $<
 
 # Flash binary using Jlink
 ifeq ($(OS),Windows_NT)

hw/bsp/metro_m0_express/board.mk

@@ -1,4 +1,4 @@
-CFLAGS = \
+CFLAGS += \
 	-DCFG_TUSB_MCU=OPT_MCU_SAMD21 \
 	-DCONF_DFLL_OVERWRITE_CALIBRATION=0 \
 	-D__SAMD21G18A__ \
@@ -6,7 +6,8 @@ CFLAGS = \
 	-mabi=aapcs-linux \
 	-mcpu=cortex-m0plus \
 	-msoft-float \
-	-mfloat-abi=soft
+	-mfloat-abi=soft \
+	-nostdlib
 
 # All source paths should be relative to the top level.
 LD_FILE = hw/bsp/metro_m0_express/samd21g18a_flash.ld

hw/bsp/metro_m4_express/board.mk

@@ -1,11 +1,14 @@
-CFLAGS = \
+CFLAGS += \
 	-DCFG_TUSB_MCU=OPT_MCU_SAMD51 \
 	-D__SAMD51J19A__ \
 	-mthumb \
 	-mabi=aapcs-linux \
 	-mcpu=cortex-m4 \
 	-mfloat-abi=hard \
-	-mfpu=fpv4-sp-d16
+	-mfpu=fpv4-sp-d16 \
+	-nostdlib
+
+CFLAGS += -Wno-error=undef
 
 # All source paths should be relative to the top level.
 LD_FILE = hw/bsp/metro_m4_express/samd51g19a_flash.ld

hw/bsp/pca10056/board.mk

@@ -2,13 +2,14 @@ CFLAGS += \
 	-DCFG_TUSB_MCU=OPT_MCU_NRF5X \
 	-DNRF52840_XXAA \
 	-mthumb \
-	-Wno-error=undef \
-	-Wno-error=cast-align \
 	-mabi=aapcs \
 	-mcpu=cortex-m4 \
 	-mfloat-abi=hard \
 	-mfpu=fpv4-sp-d16
 
+# nrfx issue undef _ARMCC_VERSION usage https://github.com/NordicSemiconductor/nrfx/issues/49
+CFLAGS += -Wno-error=undef 
+
 # All source paths should be relative to the top level.
 LD_FILE = hw/mcu/nordic/nrfx/mdk/nrf52840_xxaa.ld
 
@@ -18,6 +19,9 @@ SRC_C += \
 	hw/mcu/nordic/nrfx/drivers/src/nrfx_power.c \
 	hw/mcu/nordic/nrfx/mdk/system_nrf52840.c \
 
+# TODO remove later
+SRC_C += src/portable/$(VENDOR)/$(CHIP_FAMILY)/hal_$(CHIP_FAMILY).c
+
 INC += \
 	-I$(TOP)/hw/cmsis/Include \
 	-I$(TOP)/hw/mcu/nordic \
@@ -38,3 +42,7 @@ ASFLAGS += -DNRF52840_XXAA
 VENDOR = nordic
 CHIP_FAMILY = nrf5x
 
+JLINK_DEVICE = nRF52840_xxAA
+
+# flash using jlink
+flash: flash-jlink

hw/bsp/stm32f407g_disc1/board.mk

@@ -1,4 +1,4 @@
-CFLAGS = \
+CFLAGS += \
 	-DHSE_VALUE=8000000 \
 	-DCFG_TUSB_MCU=OPT_MCU_STM32F4 \
 	-DSTM32F407xx \
@@ -6,7 +6,8 @@ CFLAGS = \
 	-mabi=aapcs-linux \
 	-mcpu=cortex-m4 \
 	-mfloat-abi=hard \
-	-mfpu=fpv4-sp-d16
+	-mfpu=fpv4-sp-d16 \
+	-nostdlib
 
 # All source paths should be relative to the top level.
 LD_FILE = hw/bsp/stm32f407g_disc1/STM32F407VGTx_FLASH.ld

hw/mcu/nordic/nrfx

@@ -1 +1 @@
-Subproject commit 67710e47c7313cc56a15748e485079831ee6a3af
+Subproject commit 6f54f689e9555ea18f9aca87caf44a3419e5dd7a