Skip to content

Examples stuff: fix examples/rules, extend rules and add f7 examples #193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
h2obrain wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Examples stuff: fix examples/rules, extend rules and add f7 examples #193

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
877749f
lpc43xx: Fixed lpc43xx examples (untested) and reenabled compilation
h2obrain May 19, 2019
2a7e4fa
stm32l4: added missing l4 target to Makefile and added stm32/l4/Makef…
h2obrain May 19, 2019
72027c9
rules.mk: define missing LIBDEPS for old style compilation
h2obrain May 23, 2019
6aec4eb
rules.mk: fix build dependencies
h2obrain May 23, 2019
6c02f9d
rules.mk: removed unnecessary recipe switch
h2obrain May 23, 2019
16d6b24
rules.mk: build only for the required libopencm3 target
h2obrain May 23, 2019
e20be5f
rules.mk: pass CFLAGS variable to libopencm3
h2obrain May 23, 2019
15a0ebf
rules.mk: avoid double slashes in paths
h2obrain May 23, 2019
e54f872
stm32f7: added demo Makefiles
h2obrain May 19, 2019
4e6cb4f
stm32f769i-disco: added blinkled demo
h2obrain May 16, 2019
7c432aa
stm32f769i-disco: added sdram demo
h2obrain May 16, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 17 additions & 1 deletion examples/rules.mk
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,9 +168,25 @@ else
include $(OPENCM3_DIR)/mk/genlink-rules.mk
endif

# Find libopencm3 library folder
# this tries to match LIBNAME to manufacturer or manufacturer/series folder in lib/
# eg. LIBNAME=lm3s LIBFOLDER=lib/lm3s or LIBNAME=stm32f7 LIBFOLDER=lib/stm32/f7
DIR=$(notdir $(subst /.,,$(wildcard $1/*/.)))
LIBFOLDER=lib/$(strip \
$(foreach M,$(filter-out usb ethernet dispatch,$(call DIR,$(OPENCM3_DIR)/lib)), \
$(if $(subst opencm3_$M,,$(LIBNAME)),\
$(foreach S,$(call DIR,$(OPENCM3_DIR)/lib/$(M)), \
$(if $(subst opencm3_$M$S,,$(LIBNAME)),,$M/$S)), \
$M)))
# exceptions
Copy link
Copy Markdown
Member

@karlp karlp May 25, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

definitely not interested in this one. I've had enough rpoblems with people trying to pick and choose things, and reportings bugs when all they really needed to do was just run "make" once in the top and everythign would have been fine. This relies far too much on knowledge of the build system and file structure within the library, which should be treated as a simple .a dependency. It's a submodule for exactly that reason, just build it once, move on.

ifeq ($(LIBFOLDER),lib/lpc43xx)
LIBFOLDER=lib/lpc43xx/m4
endif

# Build libopencm3-lib if it does exists
$(OPENCM3_DIR)/lib/lib$(LIBNAME).a:
$(warning $(LIBNAME).a not found, attempting to rebuild in $(OPENCM3_DIR))
$(MAKE) -C $(OPENCM3_DIR)
$(MAKE) -C $(OPENCM3_DIR) $(LIBFOLDER)
$(OPENCM3_DIR)/include/%.h: $(OPENCM3_DIR)/lib/lib$(LIBNAME).a;

# Define a helper macro for debugging make errors online
Expand Down