Skip to content

Patch 3 #21

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.

Sign up for GitHub

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

Jump to bottom
Open
H4nky84 wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Patch 3 #21

H4nky84 wants to merge 16 commits into from

Conversation

@H4nky84
Copy link

@H4nky84 H4nky84 commented Oct 24, 2016

Added support for RTR bits in can packet as well as another tested transceiver.

@Alex-2-5
Copy link

Alex-2-5 commented Oct 26, 2016 •
edited
Loading

I downloaded the patch3 when it was commited but i have a problem.
As soon as i start "CANbus1.begin(myMask1);" the program freezes.
Nothing goes on.

Any Idea what it could be?

EDIT:
Sorry, i think i found it.
I had "old style"
FlexCAN CANbus0(500000);
FlexCAN CANbus1(500000);

But it should be (as seen in the example):
FlexCAN CANbus0(500000, 0);
FlexCAN CANbus1(500000, 1);

I hope that was it.

@H4nky84
Copy link
Author

H4nky84 commented Oct 26, 2016

Hopefully that is it.

Make sure you download the current one on my list as I made some changes earlier today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@H4nky84 @Alex-2-5 @pawelsky