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

Change Fin WiFi/BT pins drive strength to 6mA #934

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

majorz
Copy link
Contributor

@majorz majorz commented Oct 27, 2022

Currently pins 34-39 that are used for the wireless chip on balenaFin boards are not being specified in the Fin dt-blob. This gives them a default drive strength of 2mA, which causes signal integrity issues.

This commit changes them to 6mA, which is two steps above the current setting.

Change-type: patch
Signed-off-by: Zahari Petkov zahari@balena.io

@rcooke-warwick
Copy link

@balena-ci rebase

@ghost ghost force-pushed the majorz/fin-wifi-pin-drive-strength branch from 8da2f28 to 7d9e6ce Compare October 31, 2022 10:41
Currently pins 34-39 that are used for the wireless chip on balenaFin
boards are not being specified in the Fin dt-blob. This gives them
a default drive strength of 2mA, which causes signal integrity issues.

This commit changes them to 6mA, which is two steps above the current
setting.

Change-type: patch
Signed-off-by: Zahari Petkov <zahari@balena.io>
@majorz majorz force-pushed the majorz/fin-wifi-pin-drive-strength branch from 7d9e6ce to e42ee75 Compare October 31, 2022 15:53
@majorz
Copy link
Contributor Author

majorz commented Oct 31, 2022

@balena-ci rebase

@ghost ghost force-pushed the majorz/fin-wifi-pin-drive-strength branch from e42ee75 to db1e17c Compare October 31, 2022 15:54
@majorz
Copy link
Contributor Author

majorz commented Nov 14, 2022

@balena-ci rebase

@ghost ghost force-pushed the majorz/fin-wifi-pin-drive-strength branch from db1e17c to 9dbb0d8 Compare November 14, 2022 10:28
@floion
Copy link
Collaborator

floion commented Feb 14, 2023

Hi @majorz is this still in draft?

@majorz
Copy link
Contributor Author

majorz commented Feb 14, 2023

@floion we started testing this with @hraftery and Zisis, but it turned out it was blocked on a couple of other PRs. The blockers are all merged now, but we never resumed testing afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants