-
Notifications
You must be signed in to change notification settings - Fork 122
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
Enable pins of CAN bus is reversed #32
Comments
Cool, but you are reporting the issue to the wrong repository as this library does not deal with enable pins. You probably meant this one https://github.com/collin80/FlexCAN_Library |
Thanks Pawelsky, there is no place to report issues at collin80. And all the examples here also have the above code to do the activation. |
First of all the fact thet there is no place to report issues in Collin's repository does not mean you should report in to other random repository. Secondly, no, there are no examples here that have the mentioned code. Finally, even if there were this is not even a bug in the library, as the transcievers that you use do not have the ENABLE pins. They have Rs pins which for this particular chip (SN65HVD230D) enables the High-speed mode when pulled low, low-power mode when pulled high, and manages slope control when pulled down with a 10kohm-100kohm resistor. If you really want to argue more contact colin80 directly (e.g. via the pjrc forum: https://forum.pjrc.com/threads/39867-Another-fork-of-FlexCAN) or create a pull request in HIS repo. |
Sorry for the mix up involving my repo. I have fixed the problem and if you have trouble you can now directly create an issue and yell at me for it. |
Sorry Pawelsky. I am asking question not reporting a bug of the library. Your explanation make many sense. I will check the spec of SN65HVD230D. As you clearly mentioned about the pull down to control the slope, do you think I should put one, as I got very unstable CAN bus reading. |
I am working with Teensy 3.6 with the "Dual CAN-Bus adapter for Teensy 3.5, 3.6". Though in all examples, the pins are set to "HIGH", but I must set them to "LOW" to get it work with the example CANTest.
//if using enable pins on a transceiver they need to be set on
pinMode(2, OUTPUT);
pinMode(35, OUTPUT);
//digitalWrite(2, HIGH);
// digitalWrite(35, HIGH);
digitalWrite(2, HIGH);
digitalWrite(35, HIGH);
The text was updated successfully, but these errors were encountered: