tmedicci commented on PR #16914:
URL: https://github.com/apache/nuttx/pull/16914#issuecomment-3234652770

   Hi @cederom !
   
   I'm sorry. I forgot to mention that the general I2S driver is currently 
broken for our devices (at least, I tested it on ESP32-S3 and ESP32-C6 using 
the `i2schar` driver). This PR was part of the effort to have a reliable 
testing mechanism. I have already provided a fix for the RISC-V device, but I 
haven't submitted it yet. In fact, this assertion is related to the lack of 
such a fix. It still requires some steps to be fully functional for all 
devices, so that's why I haven't submitted the fix yet.
   
   > ps/2: you may want to update apps repo name and remove incubator 
(https://github.com/tmedicci/incubator-nuttx-apps.git) ? :-)
   
   Yes, I know... I keep postponing it, haha. I'll make the change ;)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to