Disable safety timer in PMICClass::disableWatchdog

[Stefanhg]

Creating a Pull request regarding issue 11
Changed the code as there was some typo's in the code
#11

[facchinm]

LGTM!
@Rocketct can you test it on a board (and also check why some examples are not compiling anymore to make the CI green again) ?

[Rocketct]

yep sure

[Rocketct]

i remember why we leaved the charge safety timer enabled, disable this timer could results in a dangerous event if the charge time is too long, maybe more than disable it permanently could be better allows for manually disable it by the user?
i also found why the others examples doesn't compile i'll fix them and push in another PR

[Stefanhg]

@Rocketct
In my case i actually wanted to disable charging permanently.
It could also be considered if this should be in disableCharging instead of disableWatchdog.
AS it is kind of pointless of having a disableCharging if it enables after 1 hour anyways

[Rocketct]

yes in that case have a lot more sense, so the best thing could be modify the disableWatchdog in order that just disable the watchdog and inside the disableCharge add the disable safety timer, in this case shall be also added the enable charge timer in the enableCharge, i'll made the pr asap

[s0rent]

Who can press the merge button @per1234 @Rocketct ? Whats the holdup?

Side note, what is the difference between disableCharge and disableCharging ?
They do two different things, both are described as "Disable the battery charger", none of them disable the battery LED which will start blinking if no battery is connected.

Another side note, imagine how many people do not use battery charging function at all (MKR WiFi 1010 and others), yet they will have to stare at a stupid blinking LED because it is on per default. Even if this merge request eventually manages to get in, all these users will have to include a whole charging lib just to disable it.