Upstream sync#439
Open
VIXIVIXIV wants to merge 11 commits into
Open
Conversation
Update the github workflow lock-threads action in hope that it will fix run failure reports. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
STM32H750 only has 128kb of storage so the STM32H7 typical offset of 128kb would be too large, I added it to the 32kb bootloader selections. Signed-off-by: Bill Horan <whoran83@gmail.com> Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Signed-off-by: Bill Horan <whoran83@gmail.com> Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Signed-off-by: Dmitry Butyugin <dmbutyugin@google.com>
Signed-off-by: Dmitry Butyugin <dmbutyugin@google.com>
Signed-off-by: Dmitry Butyugin <dmbutyugin@google.com>
Some SD cards will fail to initialize at the 400KHz clock documented in the official specification. Add a "fast speed" option that runs at 4MHz. Signed-off-by: Eric Callahan <arksine.code@gmail.com>
Signed-off-by: Eric Callahan <arksine.code@gmail.com>
Also remove unused "utils" import. Signed-off-by: Eric Callahan <arksine.code@gmail.com>
Move the low-level least squares math to a new TapBestFit class and add additional code comments. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
The CRC helper took a 16-bit integer and split it into bytes with a _split_bytes() helper based on bit_length(). When the high byte was zero (a common case for in-range temperature and humidity readings) bit_length() reported 8 bits or fewer, so _split_bytes() returned a single byte and the CRC was computed over one byte instead of the two the sensor checksums. The computed CRC then mismatched the sensor's and the reading was logged as a checksum error. Replace the variable-length split with a fixed two-byte split so the checksum is always computed over the full two-byte value, matching the SHT3x datasheet. This also drops the now-unused _split_bytes() helper. Signed-off-by: Kevin Blackburn-Matzen <kmatzen@gmail.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
No description provided.