Change the apps directory structure to follow the structure used for modules.
authorNathael Pajani <nathael.pajani@ed3l.fr>
Mon, 21 Sep 2015 10:28:57 +0000 (12:28 +0200)
committerNathael Pajani <nathael.pajani@ed3l.fr>
Mon, 21 Sep 2015 10:28:57 +0000 (12:28 +0200)
Makefile
apps/dtplug/usb/Makefile [new file with mode: 0644]
apps/dtplug/usb/README [new file with mode: 0644]
apps/dtplug/usb/main.c [new file with mode: 0644]
apps/usb/Makefile [deleted file]
apps/usb/README [deleted file]
apps/usb/main.c [deleted file]

index ffd375c..2093f49 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,6 @@
 # Makefile for GPIO Demo Module
 
-TARGET_DIR = apps/$(NAME)
+TARGET_DIR = apps/$(MODULE)/$(NAME)
 
 LPC = lpc1764
 CPU = cortex-m3
@@ -16,7 +16,7 @@ LINKOPTS = -static -nostartfiles -nostdlib \
                   -Wl,-Map=$(TARGET_DIR)/lpc_map_$(LPC).map -Tlpc_link_$(LPC).ld
 
 
-APPS = $(subst apps/,,$(wildcard apps/*))
+APPS = $(subst apps/,,$(wildcard apps/*/*))
 
 .PHONY: all $(APPS)
 all: $(APPS)
@@ -26,6 +26,7 @@ TARGET_INCLUDES = $(TARGET_DIR)/
 OBJDIR = objs
 
 SRC = $(wildcard */*.c)
+SRC += $(wildcard lib/*/*.c)
 OBJS = ${SRC:%.c=${OBJDIR}/%.o}
 DEPS = ${OBJS:%.o=$(OBJDIR)/%.d}
 
@@ -38,13 +39,14 @@ NAME_DEPS = ${NAME_OBJS:%.o=$(OBJDIR)/%.d}
 .SECONDARY: $(OBJS) $(NAME_OBJS)
 .PRECIOUS: %.elf
 %.elf: $(OBJS) $(NAME_OBJS)
-       @echo "Linking $(NAME) ..."
+       @echo "Linking $(MODULE)/$(NAME) ..."
        @$(CC) $(LINKOPTS) $(OBJS) $(NAME_OBJS) -o $@
 
 %.bin: %.elf
        @echo "Creating image : \e[32m$@\e[39m"
        @$(CROSS_COMPILE)objcopy -R .stack -R .bss -O binary $^ $@
        @ls -l $@
+       @$(CROSS_COMPILE)size $^
        @echo Done.
 
 ${OBJDIR}/%.o: %.c
@@ -54,7 +56,7 @@ ${OBJDIR}/%.o: %.c
 
 
 $(APPS):
-       @make --no-print-directory NAME=$@ apps/$@/$@.bin
+       @make --no-print-directory MODULE=$(shell dirname $@) NAME=$(shell basename $@) apps/$(shell dirname $@)/$(shell basename $@)/$(shell basename $@).bin
 
 all_apps: $(APPS)
 
@@ -62,18 +64,17 @@ clean:
        rm -rf $(OBJDIR)
 
 mrproper: clean
-       rm -f apps/*/*.bin apps/*/*.elf apps/*/*.map
+       rm -f apps/*/*/*.bin apps/*/*/*.elf apps/*/*/*.map
 
 
 # Some notes :
-# The command "make -f /path/to/here/Makefile app_name" does not work, as well
-# as the command "make -f /path/to/here/apps/app_name/Makefile".
+# The command "make -f /path/to/here/Makefile module/app_name" does not work, as well
+# as the command "make -f /path/to/here/apps/module/app_name/Makefile".
 # This could be solved in the app Makefiles by replacing
 #   "NAME = $(shell basename $(CURDIR))"
 # with
 #   "NAME = $(shell basename $(shell dirname $(realpath $(lastword $(MAKEFILE_LIST)))))"
 # and possibly some similar trick for the base Makefile but this is just
 # unreadable and moslty unusefull, so it won't be supported.
-# Use "make -C /path/to/here/ app_name" or "make -C /path/to/here/apps/app_name"
+# Use "make -C /path/to/here/ module/app_name" or "make -C /path/to/here/apps/module/app_name"
 # instead.
-
diff --git a/apps/dtplug/usb/Makefile b/apps/dtplug/usb/Makefile
new file mode 100644 (file)
index 0000000..d2ae3a7
--- /dev/null
@@ -0,0 +1,11 @@
+# Makefile for DTPlug apps
+
+MODULE = $(shell basename $(shell cd .. && pwd && cd -))
+NAME = $(shell basename $(CURDIR))
+
+.PHONY: $(NAME).bin
+$(NAME).bin:
+       @make -C ../../.. --no-print-directory NAME=$(NAME) MODULE=$(MODULE) apps/$(MODULE)/$(NAME)/$@
+
+clean mrproper:
+       @make -C ../../.. --no-print-directory $@
diff --git a/apps/dtplug/usb/README b/apps/dtplug/usb/README
new file mode 100644 (file)
index 0000000..7bb5d5d
--- /dev/null
@@ -0,0 +1,23 @@
+USB Example
+
+Copyright 2013-2015 Nathael Pajani <nathael.pajani@ed3l.fr>
+
+
+/* ****************************************************************************
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ *************************************************************************** */
+
+This example shows the support of the USB device interface for the DTPlug.
+
diff --git a/apps/dtplug/usb/main.c b/apps/dtplug/usb/main.c
new file mode 100644 (file)
index 0000000..4d557b5
--- /dev/null
@@ -0,0 +1,103 @@
+/****************************************************************************
+ *   apps/adc/main.c
+ *
+ * ADC example
+ *
+ * Copyright 2013-2014 Nathael Pajani <nathael.pajani@ed3l.fr>
+ *
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ *************************************************************************** */
+
+
+#include <stdint.h>
+#include "core/lpc_regs_17xx.h"
+#include "core/lpc_core_cm3.h"
+#include "core/pio.h"
+#include "core/system.h"
+#include "core/systick.h"
+#include "drivers/serial.h"
+#include "drivers/gpio.h"
+#include "drivers/usbcore.h"
+#include "lib/stdio.h"
+#include "lib/usb.h"
+
+
+#define BOARD_VERSION    0x11
+#define BOARD_NAME      "DTPlug"
+
+
+#define SELECTED_FREQ  FREQ_SEL_100MHz
+
+/***************************************************************************** */
+/* Pins configuration */
+/* pins blocks are passed to set_pins() for pins configuration.
+ * Unused pin blocks can be removed safely with the corresponding set_pins() call
+ * All pins blocks may be safelly merged in a single block for single set_pins() call..
+ */
+const struct pio_config common_pins[] = {
+       /* UART 0 */
+       { LPC_UART0_TX_PIO_0_2, 0 },
+       { LPC_UART0_RX_PIO_0_3, 0 },
+       ARRAY_LAST_PIO,
+};
+
+
+/***************************************************************************** */
+void system_init()
+{
+       /* Stop the watchdog */
+       stop_watchdog(); /* Do it right now, before it gets a chance to break in */
+
+       /* Note: Brown-Out detection must be powered to operate the ADC. adc_on() will power
+        *  it back on if called after system_init() */
+       system_brown_out_detection_config(0);
+       system_set_default_power_state();
+       clock_config(SELECTED_FREQ);
+       set_pins(common_pins);
+       /* System tick timer MUST be configured and running in order to use the sleeping
+        * functions */
+       systick_timer_on(1); /* 1ms */
+       systick_start();
+}
+
+/* Define our fault handler. This one is not mandatory, the dummy fault handler 
+ * will be used when it's not overridden here.
+ * Note : The default one does a simple infinite loop. If the watchdog is deactivated
+ * the system will hang.
+ */
+void fault_info(const char* name, uint32_t len)
+{
+       serial_write(0, name, len);
+       /*FIXME : wait for end of Tx and perform soft reset of the micro-controller ! */
+}
+
+
+
+/***************************************************************************** */
+int main(void) {
+       system_init();
+       uart_on(0, 115200, NULL);
+
+       usb_on();
+       usb_start(USB_MAX_PACKET_EP0_SIZE);
+
+       while (1) {
+       }
+       return 0;
+}
+
+
+
diff --git a/apps/usb/Makefile b/apps/usb/Makefile
deleted file mode 100644 (file)
index 91407f3..0000000
+++ /dev/null
@@ -1,10 +0,0 @@
-# Makefile for GPIO Demo Module apps
-
-NAME = $(shell basename $(CURDIR))
-
-.PHONY: $(NAME).bin
-$(NAME).bin:
-       @make -C ../.. --no-print-directory NAME=$(NAME) apps/$(NAME)/$@
-
-clean mrproper:
-       @make -C ../.. --no-print-directory $@
diff --git a/apps/usb/README b/apps/usb/README
deleted file mode 100644 (file)
index 7bb5d5d..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-USB Example
-
-Copyright 2013-2015 Nathael Pajani <nathael.pajani@ed3l.fr>
-
-
-/* ****************************************************************************
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- *
- *************************************************************************** */
-
-This example shows the support of the USB device interface for the DTPlug.
-
diff --git a/apps/usb/main.c b/apps/usb/main.c
deleted file mode 100644 (file)
index 4d557b5..0000000
+++ /dev/null
@@ -1,103 +0,0 @@
-/****************************************************************************
- *   apps/adc/main.c
- *
- * ADC example
- *
- * Copyright 2013-2014 Nathael Pajani <nathael.pajani@ed3l.fr>
- *
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- *
- *************************************************************************** */
-
-
-#include <stdint.h>
-#include "core/lpc_regs_17xx.h"
-#include "core/lpc_core_cm3.h"
-#include "core/pio.h"
-#include "core/system.h"
-#include "core/systick.h"
-#include "drivers/serial.h"
-#include "drivers/gpio.h"
-#include "drivers/usbcore.h"
-#include "lib/stdio.h"
-#include "lib/usb.h"
-
-
-#define BOARD_VERSION    0x11
-#define BOARD_NAME      "DTPlug"
-
-
-#define SELECTED_FREQ  FREQ_SEL_100MHz
-
-/***************************************************************************** */
-/* Pins configuration */
-/* pins blocks are passed to set_pins() for pins configuration.
- * Unused pin blocks can be removed safely with the corresponding set_pins() call
- * All pins blocks may be safelly merged in a single block for single set_pins() call..
- */
-const struct pio_config common_pins[] = {
-       /* UART 0 */
-       { LPC_UART0_TX_PIO_0_2, 0 },
-       { LPC_UART0_RX_PIO_0_3, 0 },
-       ARRAY_LAST_PIO,
-};
-
-
-/***************************************************************************** */
-void system_init()
-{
-       /* Stop the watchdog */
-       stop_watchdog(); /* Do it right now, before it gets a chance to break in */
-
-       /* Note: Brown-Out detection must be powered to operate the ADC. adc_on() will power
-        *  it back on if called after system_init() */
-       system_brown_out_detection_config(0);
-       system_set_default_power_state();
-       clock_config(SELECTED_FREQ);
-       set_pins(common_pins);
-       /* System tick timer MUST be configured and running in order to use the sleeping
-        * functions */
-       systick_timer_on(1); /* 1ms */
-       systick_start();
-}
-
-/* Define our fault handler. This one is not mandatory, the dummy fault handler 
- * will be used when it's not overridden here.
- * Note : The default one does a simple infinite loop. If the watchdog is deactivated
- * the system will hang.
- */
-void fault_info(const char* name, uint32_t len)
-{
-       serial_write(0, name, len);
-       /*FIXME : wait for end of Tx and perform soft reset of the micro-controller ! */
-}
-
-
-
-/***************************************************************************** */
-int main(void) {
-       system_init();
-       uart_on(0, 115200, NULL);
-
-       usb_on();
-       usb_start(USB_MAX_PACKET_EP0_SIZE);
-
-       while (1) {
-       }
-       return 0;
-}
-
-
-