google / CFU-Playground

Want a faster ML processor? Do it yourself! -- A framework for playing with custom opcodes to accelerate TensorFlow Lite for Microcontrollers (TFLM). . . . . . Online tutorial: https://google.github.io/CFU-Playground/ For reference docs, see the link below.
http://cfu-playground.rtfd.io/
Apache License 2.0
458 stars 117 forks source link

Conflicting uses of TARGET #124

Closed tcal-x closed 3 years ago

tcal-x commented 3 years ago

This is the original message; see also my following replies.

Hi, @alanvgreen :

After some testing with this update, I met a problem. when compile tflm library, it complained no found to "qmtech_wukong_makefile.inc" wihch the board I used is wukong board.

Would you like to share how to solve this problem? or just simple copy the mcu_riscv_makefile.inc to digilet_arty_makefile.inc?

Many thanks.

BR, Akio

Originally posted by @akioolin in https://github.com/google/CFU-Playground/issues/118#issuecomment-863387163

tcal-x commented 3 years ago

My followup comment:

I think I see the issue. It is due to the interaction of this PR and PR #105. #105 added the use of TARGET to specify which LiteX board we're using. This PR causes more of the TFLite-Micro system to be used for generating the build/src/tensorflow/lite/micro/ directory, and that also uses TARGET but for a different purpose. Hopefully the solution is as easy as just unsetting TARGET before calling the TFLM generation.

tcal-x commented 3 years ago

To reproduce:

make TARGET=digilent_arty software

output:

/home/tim/google/CFU-Playground/third_party/tflite-micro/tensorflow/lite/micro/tools/make/Makefile:544: tensorflow/lite/micro/tools/make/targets/digilent_arty_makefile.inc: No such file or directory
make[1]: *** No rule to make target 'tensorflow/lite/micro/tools/make/targets/digilent_arty_makefile.inc'.  Stop.
make[1]: Leaving directory '/home/tim/google/CFU-Playground/third_party/tflite-micro'
make: *** [../proj.mk:209: tflite-micro-src] Error 2
akioolin commented 3 years ago

Hi, @tcal-x :

Yes! You are right. CFU is a very young baby, but It will be a Giant in some day. after some test last night, I found the simple work around way is copy mcu_riscv_makefile.inc to qmtech_wukong_makefile.inc, and the build process will be continued. the outcome seems work fine.

BR, Akio

akioolin commented 3 years ago

Hi, @tcal-x :

after follow PR #132, add the unset into proj.mk. the build process is very smooth. Many Thanks.

BR, Akio

tcal-x commented 3 years ago

Thanks for reporting the issue, @akioolin !