xmos / fwk_rtos

RTOS Framework
Other
8 stars 20 forks source link

Re-enable QSPI RTOS driver test #142

Closed GitJRS closed 1 year ago

GitJRS commented 1 year ago

There is a fix in progress which addresses a flash issue with 2v0 boards. Once that is merged, the test/rtos-drivers/hil qspi test should be re-enabled in CI.

GitJRS commented 1 year ago

blocked by #46

xmos-jmccarthy commented 1 year ago

The flash test is currently broken.

It appears that since the lib_qspi_fast_read change that the qspi test was silent failing. There are 2 reasons for this, firstly there is no pytest checker in hil_add to verify that QSPI passed, hiding that it has been failing. Second, the flashing of the calibration pattern is missing in the CI test portion.

For #157 the qspi test has been temporarily disabled for the CI. This PR was verified locally.

Please reenable: https://github.com/xmos/fwk_rtos/blob/2ee97ef1bd59863ebcec3cbd6618d56b50f7b7d4/test/rtos_drivers/hil_add/hil_add.cmake#L6 and update accordingly.