Mantis Bug Tracker

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000213FirmwareGeneral Featurespublic2011-07-18 07:042014-02-18 23:47
ReporterFred 
Assigned ToFred 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.2.0-SNAPSHOT 
Target Version0.2.0Fixed in Version0.2.0 
Summary0000213: Better And More Robust Handling Of PLL
DescriptionAttempt to ensure that the EMS can still function in a sort of low performance limp mode should PLL be lost, and also ensure that it tries to regain PLL. I'm unsure what currently happens when PLL is lost, but probably bad things, such as nothing.
TagsNo tags attached.
FirmwareVersion
Issue TypeImprovement
Risk of Breakagemedium
Attached Files

- Relationships
related to 0000844closedFred Better And More Robust Handling Of Main Clock 

-  Notes
User avatar (0002648)
Fred (administrator)
2013-05-16 19:40

I've spent all day working on this and related stuff. Much of it reading the fucking manual like a good boy.

It looks like loss of PLL should result in normal low speed operation. Likewise loss of main clock should fall back to self clock mode.

Currently it appears as if that doesn't happen because the main loop fails to service the COP if configured when the main clock is glitched. It therefore must be:

Frozen/fucked completely
Stuck executing an ISR repeatedly
Stuck in bad code in the SM due to bad vector redirection.

Tested today with a modified FW build.
User avatar (0002649)
Fred (administrator)
2013-05-17 20:35

More results here: http://forum.diyefi.org/viewtopic.php?f=8&p=34160#p34160 [^]

Summary: PLL successfully re-enables itself every time without issue. Comms and other time based things are likely corrupted/wrong in the mean time.

I should enable and catch the interrupt and force a sync loss, then disable the timer inputs/outputs until it is brought back to life, then restore normal operation.

The main clock needs to be handled too, but that's a separate issue.
User avatar (0002652)
Fred (administrator)
2013-05-18 14:47

Update target version.
User avatar (0002653)
Fred (administrator)
2013-05-18 14:49

Available in hash 3959ba8 from dev branch. Please code review and close if happy, and if you see that Andy is happy running it on his car.
User avatar (0002749)
Fred (administrator)
2014-02-18 23:47

Closing as this seems to function as designed.


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker