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

CAN options #42

Merged
merged 1 commit into from
Jan 13, 2022
Merged

CAN options #42

merged 1 commit into from
Jan 13, 2022

Conversation

rusefillc
Copy link
Contributor

was NOT able to see happy CAN bus but does it look good or bad in terms of timing?

@rusefillc
Copy link
Contributor Author

@mck1117 @dron0gus @andreika-git do you see obvious issues by any chance?

@mck1117
Copy link
Owner

mck1117 commented Jan 13, 2022

Is your bus terminated? You won't see anything without a terminator.

@rusefillc
Copy link
Contributor Author

rusefillc commented Jan 13, 2022

I am connecting to https://canhacker.com/ device which has 120 OHm option checked in settings.

Or do you mean I need some changes on my WB board @mck1117 ?

@mck1117
Copy link
Owner

mck1117 commented Jan 13, 2022

hah, we're not running at 72MHz, please hold

@mck1117
Copy link
Owner

mck1117 commented Jan 13, 2022

Please merge in master, we now run at 64MHz. It's not actually possible to run at 72 on HSI because of the PLL's limited configuration options, so 64 is the best we can do.

@mck1117
Copy link
Owner

mck1117 commented Jan 13, 2022

update: back to 48mhz with 660dad4

@rusefillc
Copy link
Contributor Author

@mck1117 PR adjusted to 48MHz - CAN settings actually same between F0 and F1 - still failing my real HW test but maybe it's mergeable as is at least for the code comments and attempt?

@mck1117 mck1117 merged commit d9b8337 into mck1117:master Jan 13, 2022
@rusefillc rusefillc deleted the can2 branch January 13, 2022 19:27
@rusefillc
Copy link
Contributor Author

logic analyzer on CAN tx

image

wb-can-tx.zip

@mck1117
Copy link
Owner

mck1117 commented Jan 13, 2022

Looks like CAN peripheral is running at half speed (divider somewhere we forgot about?), pushed 8603450

@rusefillc
Copy link
Contributor Author

Let's continue at #44

rusefillc added a commit to dron0gus/wideband that referenced this pull request Nov 10, 2022
Co-authored-by: rusefillc <sdfsdfqsf2334234234>
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

2 participants