Skip to content

Commit 822ead2

Browse files
committed
Further simplify makefile and move background execution to the bash script for more robustness
1 parent 0b8b023 commit 822ead2

File tree

2 files changed

+14
-20
lines changed

2 files changed

+14
-20
lines changed

Makefile

+12-18
Original file line numberDiff line numberDiff line change
@@ -140,36 +140,30 @@ fpga-stream: $(HOST_FILE)
140140
%.aocx: %.cl
141141
mkdir -p $(FOLDER)
142142
-ln -sfn $(FOLDER)/$(KERNEL_BINARY).aocx $(KERNEL).aocx
143-
cd $(FOLDER) && \
144-
rm -rf $(KERNEL_BINARY)* && \
145-
sh $(SRC_FOLDER)/override_fmax.sh $(KERNEL_BINARY) $(FMAX) & \
146-
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(KERNEL_BINARY) && \
147-
rm -rf $(KERNEL_BINARY).aoc* && \
148-
cd ..
143+
rm -rf $(FOLDER)/$(KERNEL_BINARY)*
144+
sh $(SRC_FOLDER)/override_fmax.sh $(FOLDER)/$(KERNEL_BINARY) $(FMAX)
145+
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(FOLDER)/$(KERNEL_BINARY)
146+
rm -rf $(FOLDER)/$(KERNEL_BINARY).aoc*
149147

150148
fpga_1: KERNEL_FLAGS += -DFPGA_1
151149
fpga_1: KERNEL_BINARY = $(KERNEL)-sch_$(KERNEL_CONFIG)$(EXTRA_CONFIG)_FPGA_1
152150
fpga_1: $(KERNEL)-sch.cl
153151
mkdir -p $(FOLDER)
154152
-ln -sfn $(FOLDER)/$(KERNEL_BINARY).aocx $(KERNEL)_FPGA_1.aocx
155-
cd $(FOLDER) && \
156-
rm -rf $(KERNEL_BINARY)* && \
157-
sh $(SRC_FOLDER)/override_fmax.sh $(KERNEL_BINARY) $(FMAX) & \
158-
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(KERNEL_BINARY) && \
159-
rm -rf $(KERNEL_BINARY).aoc* && \
160-
cd ..
153+
rm -rf $(FOLDER)/$(KERNEL_BINARY)*
154+
sh $(SRC_FOLDER)/override_fmax.sh $(FOLDER)/$(KERNEL_BINARY) $(FMAX)
155+
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(FOLDER)/$(KERNEL_BINARY)
156+
rm -rf $(FOLDER)/$(KERNEL_BINARY).aoc*
161157

162158
fpga_2: KERNEL_FLAGS += -DFPGA_2
163159
fpga_2: KERNEL_BINARY = $(KERNEL)-sch_$(KERNEL_CONFIG)$(EXTRA_CONFIG)_FPGA_2
164160
fpga_2: $(KERNEL)-sch.cl
165161
mkdir -p $(FOLDER)
166162
-ln -sfn $(FOLDER)/$(KERNEL_BINARY).aocx $(KERNEL)_FPGA_2.aocx
167-
cd $(FOLDER) && \
168-
rm -rf $(KERNEL_BINARY)* && \
169-
sh $(SRC_FOLDER)/override_fmax.sh $(KERNEL_BINARY) $(FMAX) & \
170-
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(KERNEL_BINARY) && \
171-
rm -rf $(KERNEL_BINARY).aoc* && \
172-
cd ..
163+
rm -rf $(FOLDER)/$(KERNEL_BINARY)*
164+
sh $(SRC_FOLDER)/override_fmax.sh $(FOLDER)/$(KERNEL_BINARY) $(FMAX)
165+
$(KERNEL_COMPILER) $(KERNEL_FLAGS) $(SRC_FOLDER)/$< -o $(FOLDER)/$(KERNEL_BINARY)
166+
rm -rf $(FOLDER)/$(KERNEL_BINARY).aoc*
173167

174168
clean:
175169
rm -f $(HOST_BINARY)

override_fmax.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ fi
1919

2020
if [[ $skip -eq 0 ]]
2121
then
22-
if [[ -z `quartus_fit --version | grep "Pro"` ]]
22+
(if [[ -z `quartus_fit --version | grep "Pro"` ]]
2323
then
2424
file=`echo $path/scripts/post_flow.tcl`
2525
else
@@ -55,5 +55,5 @@ then
5555
new_fix=$(printf '%s\n' "$new" | sed 's:[\/&]:\\&:g;$!s/$/\\/')
5656

5757
sed -i "s/$orig_fix/$new_fix/" $file
58-
echo "Overriding Fmax: SUCCESS!!"
58+
echo "Overriding Fmax: SUCCESS!!") &
5959
fi

0 commit comments

Comments
 (0)