.murdock: Add same54-xpro to kconfig tests#17255
Conversation
|
Just so this is documented somewhere: Currently the same54-xpro board has 2 mdt based interfaces, the spi_nor and the at24mac... This should be described by the board somehow but maybe abstracting the a HAVE_ is not the best way since , only for this board we need to select the spi_on_qspi for the spi_nor. With an abstracted HAVE_MTD_SPI_NOR this should normally bring in just the periph_spi but not all boards will need this extra condition. |
|
Thanks for the review @benpicco I adapted your changes and hopefully cleared up the I also fixed the structure of the mtd and eui by resolving them in the modules based on the |
9f352ef to
2207b8b
Compare
|
rebased to fix the atmac conflict thing. Also running the whole test suite to see if I broke other things. |
|
doh, I didn't test the right board... Working on it |
209792d to
26e8eff
Compare
|
Things we getting a bit hairy so a rebased and squashed. This PR will only add the missing modules needed for the That being said, the |
cpu/samd5x/Kconfig
Outdated
| select HAS_PERIPH_HWRNG | ||
| select HAS_PERIPH_RTC_MEM | ||
| select HAS_PERIPH_SPI_ON_QSPI | ||
| select MODULE_PERIPH_RTC_RTT if MODULE_PERIPH_GPIO_TAMPER_WAKE |
There was a problem hiding this comment.
Hmmmm I am not actually too happy with this.
|
Looks good from my side, feel free to squash |
bc2b58b to
1b3374c
Compare
|
Done! Should we try the automerge? |
|
seems like some memory was full in the CI, rerunning. |
|
Thanks! |
Contribution description
mtd_spi_norkconfig modelling on all affected boardsavr-rss2viaat24macmodule as it contains chip on the boardMODULE_PERIPH_GPIO_TAMPER_WAKEMODULE_PERIPH_SPI_ON_QSPITesting procedure
CI passes
Issues/PRs references
Related to #16875