From 48625988bfaa75289a4895f503505efb05d932cc Mon Sep 17 00:00:00 2001
From: Michal Sojka <sojkam1@fel.cvut.cz>
Date: Wed, 8 Jul 2015 19:13:10 +0200
Subject: [PATCH] Generate documentation correctly depending on the selected
 target

---
 rpp-lib                           |  2 +-
 rpp-test-sw/Makefile.inc          | 38 +------------------------------
 rpp-test-sw/Makefile.var          | 36 +++++++++++++++++++++++++++++
 rpp-test-sw/commands/cmd.c        |  2 ++
 rpp-test-sw/commands/doc/Makefile | 17 ++++++--------
 5 files changed, 47 insertions(+), 48 deletions(-)
 create mode 100644 rpp-test-sw/Makefile.var

diff --git a/rpp-lib b/rpp-lib
index ab932b4..bcc51b0 160000
--- a/rpp-lib
+++ b/rpp-lib
@@ -1 +1 @@
-Subproject commit ab932b46eaec6a8a4253c36454d75ee4fd0a00ff
+Subproject commit bcc51b05171e724172a568ca7e76093ddb117c66
diff --git a/rpp-test-sw/Makefile.inc b/rpp-test-sw/Makefile.inc
index 3b57a8e..8437080 100644
--- a/rpp-test-sw/Makefile.inc
+++ b/rpp-test-sw/Makefile.inc
@@ -12,43 +12,7 @@ endif
 
 include $(makefile_inc_dir)/../rpp-lib/build/Makefile.rules
 include $(makefile_inc_dir)/../rpp-lib/common.mk
-
-SOURCES = \
-	cmdproc/src/cmdproc.c \
-	cmdproc/src/cmdproc_freertos.c \
-	cmdproc/src/cmdproc_io.c \
-	cmdproc/src/cmdproc_io_line.c \
-	cmdproc/src/cmdproc_io_std_line.c \
-	cmdproc/src/cmdproc_io_tisci.c \
-	cmdproc/src/cmdproc_run.c \
-	cmdproc/src/cmdproc_utils.c \
-	cmdproc/src/i2str.c \
-	commands/cmd.c \
-	commands/cmd_adc.c \
-	commands/cmd_can.c \
-	commands/cmd_pin.c \
-	commands/cmd_port.c \
-	commands/main.c
-
-SOURCES_tms570_rpp = \
-	commands/cmd_dac.c \
-	commands/cmd_din.c \
-	commands/cmd_emac.c \
-	commands/cmd_fray.c \
-	commands/cmd_fr_basic_test.c \
-	commands/cmd_hbr.c \
-	commands/cmd_hout.c \
-	commands/cmd_lin.c \
-	commands/cmd_lout.c \
-	commands/cmd_motor_example.c \
-	commands/cmd_nc.c \
-	commands/cmd_netstats.c \
-	commands/cmd_sdram.c \
-	commands/cmd_spi.c \
-	commands/cmd_vbat.c
-
-
-SOURCES += $(SOURCES_$(TARGET))
+include $(makefile_inc_dir)/Makefile.var
 
 OBJS = $(SOURCES:%.c=%.obj)
 
diff --git a/rpp-test-sw/Makefile.var b/rpp-test-sw/Makefile.var
new file mode 100644
index 0000000..8ce88a5
--- /dev/null
+++ b/rpp-test-sw/Makefile.var
@@ -0,0 +1,36 @@
+SOURCES = \
+	cmdproc/src/cmdproc.c \
+	cmdproc/src/cmdproc_freertos.c \
+	cmdproc/src/cmdproc_io.c \
+	cmdproc/src/cmdproc_io_line.c \
+	cmdproc/src/cmdproc_io_std_line.c \
+	cmdproc/src/cmdproc_io_tisci.c \
+	cmdproc/src/cmdproc_run.c \
+	cmdproc/src/cmdproc_utils.c \
+	cmdproc/src/i2str.c \
+	commands/cmd.c \
+	commands/cmd_adc.c \
+	commands/cmd_can.c \
+	commands/cmd_pin.c \
+	commands/cmd_port.c \
+	commands/main.c
+
+SOURCES_tms570_rpp = \
+	commands/cmd_dac.c \
+	commands/cmd_din.c \
+	commands/cmd_emac.c \
+	commands/cmd_fray.c \
+	commands/cmd_fr_basic_test.c \
+	commands/cmd_hbr.c \
+	commands/cmd_hout.c \
+	commands/cmd_lin.c \
+	commands/cmd_lout.c \
+	commands/cmd_motor_example.c \
+	commands/cmd_nc.c \
+	commands/cmd_netstats.c \
+	commands/cmd_sdram.c \
+	commands/cmd_spi.c \
+	commands/cmd_vbat.c
+
+
+SOURCES += $(SOURCES_$(TARGET))
diff --git a/rpp-test-sw/commands/cmd.c b/rpp-test-sw/commands/cmd.c
index a53e6a2..5a01bb6 100644
--- a/rpp-test-sw/commands/cmd.c
+++ b/rpp-test-sw/commands/cmd.c
@@ -66,6 +66,7 @@ cmd_des_t const **cmd_list;
 #ifdef TARGET_TMS570_RPP
 int cmd_do_sleep(cmd_io_t *cmd_io, const struct cmd_des *des, char *param[])
 {
+#ifndef DOCGEN
 	hal_gpio_pin_set_value(PIN_DSC_CANNSTB, 1);
 	hal_gpio_pin_set_value(PIN_DSC_CANEN, 1);
 	hal_gpio_pin_set_value(PIN_DSC_LIN2NSLP, 1);
@@ -74,6 +75,7 @@ int cmd_do_sleep(cmd_io_t *cmd_io, const struct cmd_des *des, char *param[])
 	hal_gpio_pin_set_value(PIN_DSC_LIN2NSLP, 0);
 	hal_gpio_pin_set_value(PIN_DSC_LIN1NSLP, 0);
 	hal_gpio_pin_set_value(PIN_DSC_CANNSTB, 0);
+#endif
 	return 0;
 }
 
diff --git a/rpp-test-sw/commands/doc/Makefile b/rpp-test-sw/commands/doc/Makefile
index 6f1d32c..bdd65b5 100644
--- a/rpp-test-sw/commands/doc/Makefile
+++ b/rpp-test-sw/commands/doc/Makefile
@@ -1,10 +1,12 @@
 include ../../../rpp-lib/Makefile.config
+include ../../../rpp-lib/common.mk # for UPPERCASE
+include ../../Makefile.var
 
-COMMANDS=$(wildcard ../cmd*.c)
-COMMANDS+=$(wildcard ../_$(TARGET)/cmd*.c)
-OBJS=$(COMMANDS:../%.c=%.o) docgen.o
+COMMANDS=$(notdir $(filter commands/cmd%,$(SOURCES)))
+OBJS=$(COMMANDS:%.c=%.o) docgen.o
 
-CFLAGS=-Wall -std=c99 -I../.. -I../../cmdproc/include -I../../rpp-lib/rpp/include -DDOCGEN -DNULL=0 -D'CMD_HANDLER(x)=0'
+CFLAGS = -Wall -std=c99 -I../.. -I../../cmdproc/include -I../../rpp-lib/rpp/include -DDOCGEN -DNULL=0 -D'CMD_HANDLER(x)=0'
+CFLAGS += -DTARGET_$(call UPPERCASE,$(TARGET))
 
 
 all:print-doc
@@ -17,13 +19,8 @@ docgen: $(OBJS)
 %.o: ../%.c
 	$(CC) $(CFLAGS) -c -o $@ $<
 
-_$(TARGET)/%.o: ../_$(TARGET)/%.c
-	mkdir -p _$(TARGET)
-	$(CC) $(CFLAGS) -c -o $@ $<
-
-clean:
+clean::
 	rm -f $(OBJS) docgen
-	rm -rf _$(TARGET)
 
 markdown = $(shell if pandoc -h|grep -q markdown_strict; then echo markdown_strict; else echo markdown; fi)
 
-- 
2.39.2