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

Pins definition #1

Open
ssombra opened this issue Sep 3, 2018 · 4 comments
Open

Pins definition #1

ssombra opened this issue Sep 3, 2018 · 4 comments

Comments

@ssombra
Copy link

ssombra commented Sep 3, 2018

Looking forward to build your project. After just having a look at the code, in the section where the Pins are defined, shouldn't be CHOut instead of CEOut?

@DavidMarzocca
Copy link
Owner

Hi, yes thanks a lot! I corrected this on my local version when I compiled the code but forgot to update it here. Now the variable is defined as CEout (Charge Enabler output).
The variable controls when to turn on or off the TP4056. I set it up so that the lipo gets charged at max ~4.0V, which should prolong its lifetime if it is left for long periods without use, at the price of a slightly smaller autonomy (it's extremely long in any case..).

@ssombra
Copy link
Author

ssombra commented Sep 3, 2018

Good. And talking about the TP4056, do I need to change something in the code if I am not using it?

@DavidMarzocca
Copy link
Owner

No, you can leave it as it is. Simply in that case it won't matter if the CEout pin is enabled or disabled, you can also remove those lines if you want.

@ssombra
Copy link
Author

ssombra commented Sep 3, 2018

Thanks. I'll give it a try.

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

2 participants