Ralim / IronOS

Open Source Soldering Iron firmware
https://ralim.github.io/IronOS/
GNU General Public License v3.0
7.21k stars 713 forks source link

wrong voltage #609

Closed discip closed 4 years ago

discip commented 4 years ago
voltage amperage
3.6 - 6.5V 3A
6.5 - 9V 2A
9 - 12V 1.5A

Thank you in advance kind regards

Ralim commented 4 years ago

This is just down to the general shittyness of QC. Most likely your chargers implementation of QC 3.0 is marginal for how the negotiation is coded (which follows the spec, but then was modified to be more tolerant).

Also note that your exceeding your chargers rating at peaks using a 1.5A charger at 12V (12V spikes at ~2.5A)

discip commented 4 years ago

Does that mean there is no solution for that? I do not recall facing such behavior with the previous versions.

Ralim commented 4 years ago

Generally yes, not much that can be done. Doubly so in that often if we fix this for one power supply, we break maybe 10-20 other models :(

This is often the case if you replug quickly into some power supplies, depending how fussy they are some don't like it and go into a safety lockout (5V) as well.

discip commented 4 years ago

Ah ok, so I will have to replug the iron in such cases. :) Thank you for clarification.

ps: There are some minor issues about the scrolling description text. e.g.:

Ralim commented 4 years ago

For all text issues with the descriptions, these are in the json files and you are 1000% welcome to open pull requests to adjust this. :)

discip commented 4 years ago

Ok. Thanks. :) Then I will try to do so.

discip commented 4 years ago

@Ralim Sorry to bother, but I am struggling to open a pull request. Would you please provide some guidance?

thank you in advance

Ralim commented 4 years ago

@discip you should be able to go to the language file and click the edit button up the top right and edit it in browser, and then at the bottom propose the changes as a PR as far as I'm aware?

You can also fork the repo to your account, edit there, and then open a pull request from that fork into this main one as well 😀

discip commented 4 years ago

@Ralim Thanks! 👍