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

Enable pins of CAN bus is reversed #32

Open
wt70707 opened this issue Jan 30, 2019 · 5 comments
Open

Enable pins of CAN bus is reversed #32

wt70707 opened this issue Jan 30, 2019 · 5 comments

Comments

@wt70707
Copy link

wt70707 commented Jan 30, 2019

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);

@pawelsky
Copy link

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

@wt70707
Copy link
Author

wt70707 commented Jan 30, 2019

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.

@pawelsky
Copy link

pawelsky commented Jan 30, 2019

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.

@collin80
Copy link

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.

@wt70707
Copy link
Author

wt70707 commented Jan 30, 2019

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 will put my question on Collin80.

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

No branches or pull requests

3 participants