Skip to content
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.

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

AI85 Fix Build Errors #7

Merged
merged 10 commits into from
Jul 18, 2022
Merged

AI85 Fix Build Errors #7

merged 10 commits into from
Jul 18, 2022

Conversation

Jake-Carter
Copy link
Contributor

This PR fixes misc. build errors for the AI85 projects.

All examples with the exception of "mnist-streaming" (see https://jira.maxim-ic.com/browse/MSDK-746) should build cleanly.

Resolves #6 - The Github migration missed some updates to a few projects. "config.h" has been renamed "example_config.h" to avoid conflicts with external libraries

- Add "override BOARD=FTHR_RevA" into project.mk to force projects that are only
  compatible with the FTHR board to compile correctly.
- Added warning when this is the case
- Add project.mk for UNet-highres-demo & digit-detection-demo
- Restructure TFT sub-folders for new auto-generated Makefile, mainly
  moving unused fonts and img C files for "fthr".
- Reference "example_config.h" instead of "config.h" to avoid conflicts
  with other libraries
- MSDK-692
.gitmodules Outdated Show resolved Hide resolved
@Jake-Carter Jake-Carter merged commit e1c39d0 into analogdevicesinc:main Jul 18, 2022
@kevin-gillespie
Copy link
Contributor

@Jake-Carter I'm still getting some build errors.

make[1]: Entering directory '/home/kgills/Workspace/msdk-github/Examples/MAX78000/CNN/mnist-streaming'
make[1]: Leaving directory '/home/kgills/Workspace/msdk-github/Examples/MAX78000/CNN/mnist-streaming'
make[1]: Entering directory '/home/kgills/Workspace/msdk-github/Examples/MAX78000/CNN/mnist-streaming'
  CC    main_riscv.c
main_riscv.c:70:10: fatal error: tornadocnn.h: No such file or directory
 #include "tornadocnn.h"
          ^~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [../../../../Libraries/CMSIS/Device/Maxim/MAX78000/Source/GCC/gcc_riscv.mk:225: /home/kgills/Workspace/msdk-github/Examples/MAX78000/CNN/mnist-streaming/buildrv/main_riscv.o] Error 1
make[1]: Leaving directory '/home/kgills/Workspace/msdk-github/Examples/MAX78000/CNN/mnist-streaming'
make: *** [Makefile:59: riscv] Error 2

This is the script I'm running from Examples/MAX78000

#/bin/bash

# Find all of the examples
SUBDIRS=$(find . -mindepth 2 -name "?akefile" -printf '%h\n')

for dir in ${SUBDIRS}
do
    make -C ${dir} clean
    make -C ${dir} -j8
done

@Jake-Carter
Copy link
Contributor Author

Hey @kevin-gillespie, mnist-streaming is the only broken project. See https://jira.maxim-ic.com/browse/MSDK-746

It's taking a bit more work to get updated. For some reason the CNN is embedded inside of the main source files with an older kernel format, so it's not trivial to get it updated with the latest missing CNN files. Working on fixing it

@kevin-gillespie
Copy link
Contributor

Ok I'll skip that example for now. I figured it was just a simple problem with missing files.

kevin-gillespie added a commit that referenced this pull request Nov 30, 2022
* Using 32kHz clock for pal_timer.

* Accounting for short timeouts.

* Update pal_timer.c

Correcting PalTimerGetExpTime conversion.

* Increasing setup time.

* Tuning setup time.

* Updating .yml file to match msdk.

* Fixing .yml syntax.

Co-authored-by: Kevin Gillespie <Kevin.Gillespie@analog.com>
kevin-gillespie added a commit that referenced this pull request Dec 1, 2022
* Original move from gerrit repot: https://gerrit.maxim-ic.com:8443/#/admin/projects/cordio-open

    2960f7917b43be03c60b7882a48424185a3875fd

* removes byte swapping, in turn fixing pairing failures

* Fixing build errors.

* Changes made to Cordio files to Support IAR/Keil Effort (#2)

* Changes made to Cordio files to Support IAR/Keil Effort

* updated pal_flash.c and wsf_heap_sbrk.c based on PR comments

Co-authored-by: SRV-JenkinsIDE <SRV-JenkinsIDE@maximintegrated.com>

* added missing argument fix to PalCfgLoadData() for max32655 and max32665

* Adding BLE_Examples_Tests.yml

* Deleting Libraries/Cordio before checkout.

* Updating nordic pal_cfg.c

* Fixing sign of trace value.

* Adding VS commands.

* Recovering from decryption failure due to counter offset.

* Fixing HCI bug with LHCI_OPCODE_VS_RESET_CONN_STATS.

* Making default Host and Controller definitions.

* update yaml to use workflow from .github folder

* testing

* Update BLE_Examples_Tests.yml

* Added new command 'cmd' in terminal.

* Corrected the terminal cmd example.

* Using 32kHz clock for pal_timer. (#7)

* Using 32kHz clock for pal_timer.

* Accounting for short timeouts.

* Update pal_timer.c

Correcting PalTimerGetExpTime conversion.

* Increasing setup time.

* Tuning setup time.

* Updating .yml file to match msdk.

* Fixing .yml syntax.

Co-authored-by: Kevin Gillespie <Kevin.Gillespie@analog.com>

* removes memset workaround in ME18 PalFlashRead

* Removing Cordio submodules.

* Merge prep.

* clang-format bot reformatting.

* Adding Libraries/Cordio to ignore list

* Updating ignore directories.

* Revert "clang-format bot reformatting."

This reverts commit 6db1af9.

* Ignoring Libraries/Cordio

* Removing submodules.

* Adding Libraries/Cordio to watch list.

Co-authored-by: Kevin Gillespie <Kevin.Gillespie@analog.com>
Co-authored-by: Eddie <eddie.amaya@analog.com>
Co-authored-by: Eddie <62710807+EdwinFairchild@users.noreply.github.com>
Co-authored-by: ftariqAnalogTx <109228171+ftariqAnalogTx@users.noreply.github.com>
Co-authored-by: SRV-JenkinsIDE <SRV-JenkinsIDE@maximintegrated.com>
Co-authored-by: Ying Cai <110848915+yc-adi@users.noreply.github.com>
Co-authored-by: Ying Cai <ying.cai@analog.com>
Co-authored-by: kevin-gillespie <kevin-gillespie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Examples/MAX78000/CNN/asl_demo not building
3 participants