Skip to content
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

FuriHal: fix start duration furi_hal_subghz_async_tx #3230

Merged
merged 26 commits into from
Jan 11, 2024

Conversation

Skorpionm
Copy link
Member

@Skorpionm Skorpionm commented Nov 19, 2023

What's new

  • FuriHal: add check min duration arr for the first level

Verification

  • Enable data output from GD0 to the external comb. attach a logic analyzer. Play the attached file, the levels must correspond exactly to 1700 -900 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900 900 -1700 900 -1 700 900 -1700 900 - 1700 1700 -900 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 - 900 1700 -900 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 900 -1700 900 -1700 900 -1700 900 -1700 1700 -900 900 -1700 1700 -900 1700 -900 1700 -900 1700 -900 900 -1700 1700 -900

example.sub.txt
image

  • Files for testing. Limitations: internal support for CC1101, duration from 2us, external from 4us. all durations below these values ​​will be increased to them
    subghz_test.zip

  • When transmitting, there is now no need to worry about the sequence of high/low durations. Or you can simply feed the required durations with levels, the driver itself will glue everything together correctly and transmit it. Example RAW file with Manchester encoding
    image

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

Copy link

github-actions bot commented Nov 19, 2023

Compiled f7 firmware for commit e209c02d:

@Skorpionm Skorpionm marked this pull request as ready for review November 20, 2023 08:22
@skotopes
Copy link
Member

skotopes commented Dec 2, 2023

subghz_test_files.zip
@Skorpionm FURI_HAL_SUBGHZ_ASYNC_MIRROR_GPIO feature is broken

@skotopes skotopes marked this pull request as draft December 4, 2023 05:45
@skotopes
Copy link
Member

skotopes commented Dec 4, 2023

Double negatives handling is broken

@hedger hedger added the Sub-GHz Sub-GHz-related label Dec 12, 2023
@Skorpionm Skorpionm marked this pull request as ready for review December 29, 2023 05:25
@skotopes skotopes merged commit 34539cd into dev Jan 11, 2024
9 checks passed
@skotopes skotopes deleted the skorp/fix_furi_hal_subghz_async_tx branch January 11, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sub-GHz Sub-GHz-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants