Update oled.c to reduce possible interference with motor PWM harmonics #20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
changed frequency from 0d80 (80) to 0x00 (0) to reduce speed for OLED communication in order to avoid motor PWM harmonics (EMI)
Use SWDO pin (silkscreen on PCB) of S42B communication pin header as active low signal to trigger endstop based on angular deviation. Mind that connecting SWDIO to an endstop pin on the mainboard is a non-buffered direct connection to the STM32 controller on the S42B Adjust the sensitivty for sensorless homing with #define sensorless_sensitivty. And generally do not home too fast during testing. Also note that the repurposing of the SWDIO Pin (PA13) as sensorless homing endstop output might break the on-the-fly SWD programming. Pressing RES(ET) on the S42B once and retry programming normally works fine. The triggering of the sensorless homing sensitivty threshold kills the on-the-fly SWD interface!). If it does not, you will have to hold the RES(ET) button manually until "hla_swd" is shown during openOCD programming. Releasing RES directly at this point will start the normal programming. Holding RES too long will result in a timeout.
Added sensorless homing functionality.
Oled stability optimization
|
The display issue seems to have two root-causes.
Regarding 1. increasing the F_OSC or internal clock of the OLED display seems to help. Testing with a bigger sample size is now required. |
changed comment for OLED F_OSC selection
…to recover from any glitches. Screen goes blank for one second during re-init.
|
testing by other contributers still showed issues with display glitches. |
with the optimized NOPs in the SPI handler, my testing shows a display issue after 3 days sitting on my bench. As the occurence seems to lowered now, a higher OLED_reset_counter value might be more appropriate, e.g. once per minute (300k) or even once per 10min (3mio).
|
changed re-init of OLED to ~60 seconds. |
This was referenced Nov 6, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
changed frequency from 0d80 (80) to 0x00 (0) to reduce speed for OLED communication in order to avoid motor PWM harmonics (EMI).
Also disccused here: #16