-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Revert "Make w25n01g FLASH driver non-blocking for SPI (#13555)" #13560
Conversation
)" This reverts commit f4d6a2c.
Do you want to test this code? You can flash it directly from Betaflight Configurator:
WARNING: It may be unstable. Use only for testing! |
IMHO, this PR is the right way to go as #13555 feels like a new feature, changes a lot and has already had unintended side effects, creates a mismatch between SPI and QSPI and feels incomplete and lacked sufficient testing on all the affected targets. |
|
Of course you’d say that as it doesn’t benefit your FCs, however #13555 makes 30 other FCs usable. You’re free to suggest any changes you’d like in a follow on PR. |
Actually it does brick H7NANO.. |
and did #13562 fix or retain the breakages? EDIT: found : #13562 (comment) |
@nerdCopter : H7EVO and H7RF are still broken |
If @ledvinap Could you try the following patch.
I don't believe the only other change to |
@SteveCEvans : #13562 (comment) Problem is in |
But that's only called if
Are you testing current master? |
and USE_OCTOSPI? The hack to fix it is trivial: w25n01g_deviceInit / w25n01g_configure is better place |
Actually, I prefer reading the code ;-) |
O damn of course - will also need |
that is correct, the H7EF (H730) has a single QuadSPI chip in memory mapped mode and an SPI flash for logging. |
No, I say it for professional development reasons, accidental merges cause unexpected breakages which then has all the developers running around fire fighting and causes undue stress instead of operating in a nice relaxed 'change-test-review' cycle. |
ok noted. @SteveCEvans / @haslinghuis The PR didn't link to any bug that it was attempting to fix, so it wasn't at all clear to me that there was a bug and that it was a bug fix, it just talked about making the spi flash drivers more deterministic, which felt like an improvement not a fix. |
This reverts commit f4d6a2c. (
git revert -m 1 f4d6a2ce4
)