-
Notifications
You must be signed in to change notification settings - Fork 74
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
Improve reliability of SmartPort packet receive #794
Conversation
FozzTexx
commented
Sep 17, 2024
- Prepare linked list for SmartPort command packet and block packet in advance
- Directly access SPI registers to start receive and avoid overhead/latency of ESPIDF SPI driver
* Prepare linked list for SmartPort command packet and block packet in advance * Directly access SPI registers to start receive and avoid overhead/latency of ESPIDF SPI driver
248d56d
to
58c26a7
Compare
Simplest test procedure: Setup a ProDOS disk with BASIC.SYSTEM and save a BASIC program called STARTUP with the contents:
Replace the Pre-built firmware for Apple FujiNet from commit ID 58c26a7: |
You can pull down the test images from FT here: TNFS://fujinet.diller.org/APPLE/TEST/PR794/hello.dsk |
Issue report: running bounce world client on this fw- it stops working in about 3 min. Before on 1.4 release you could run bouncy client for 12 hours +. |
I have the original spi-fast-start branch cloned from FozzTexx's github still here locally before the 'cleanup'. When I load that into my FN its works very well for the looping boot test. If I take the current Fujinet Master and add this PR 794 to make a branch locally for testing. That does not work any where near as well. It stops with the request timeouts after only a few mins. This is testing on a IIe with DB19 disk controller and Grapper Minus card with v6a firmware. |
Can you make a bundle with that older commit and send me the bundle? |
Here is a tar of the lib directory. Let me know if that's ok. |
What's the commit ID of HEAD~1 so I can check that out and then extract the tar on top of it? |
95140df (HEAD -> spi-fast-start, origin/spi-fast-start) Wait for transaction to finish after receiving packet. |
Cool, I'll see if I can reproduce the problem and see what the difference is. |
Where is bounce world client disk image? |