[riot-notifications] [RIOT-OS/RIOT] cpu/qn908x: Initial minimal support for NXP QN908x CPUs. (#13855)
notifications at github.com
Sun Nov 29 16:51:26 CET 2020
@benpicco Hey, sorry for being unresponsive with this for so long. The current situation didn't leave me much time to work on this but I'll dedicate more time to this now.
I updated the patches to work on top of master and added a Kconfig. This pull request now has 3 commits:
* one for the cpu/qn908x
* one for the boards/common/qn908x (there's a fair amount of things to deal with at the board level regardless of the actual board you use)
* a new one for boards/qn9080dk for the base board of the development kit.
This being the minimum possible to make things compile/work together it only implements GPIO and WDT. I have more commits in the pipeline to add UART, I2C and SPI, but I will add those as follow up commits since this is already large.
Would this be enough for the CI to run? I tested it with:
make BOARD=qn9080dk -C examples/hello-world/
(and the examples/saul too)
Regarding the choice of board, there's a "QN9080-001-M17Z" "module" (it looks like epoxied together so somewhere between a board and stacked chip) but that one is for the QN9080SIP which includes NFC. I don't have one of those and I'm not sure if it makes more sense to handle that little board as a cpu_model or as a board. Anyway, the qn9080dk fits better the description of a board and would be a better example if you want to make your own board based on this or stack a module on top of the DK GPIOs.
Finally, I'm getting this error in the `make static-test` but I don't know how to solve it, other cortex MCUs have the same code:
Running "./dist/tools/cppcheck/check.sh" x
cpu/qn908x/vectors.c:30: error (unknownMacro): There is an unknown macro here somewhere. Configuration is required. If ISR_VECTOR is a macro then please configure it.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications