Skip to content
This repository has been archived by the owner on Jan 31, 2022. It is now read-only.

Add a wait time between link reset and VFAT link checks #103

Open
1 of 2 tasks
bdorney opened this issue Mar 12, 2019 · 1 comment
Open
1 of 2 tasks

Add a wait time between link reset and VFAT link checks #103

bdorney opened this issue Mar 12, 2019 · 1 comment

Comments

@bdorney
Copy link
Contributor

bdorney commented Mar 12, 2019

Brief summary of issue

Based on a discussing that @evka85 and I had we should increase the time between when a link reset is performed and when VFAT sync error counters are checked. These counters are checked once per orbit so we should allow one or more orbits to occur by before checking those counters after a link reset.

An orbit is 9.09091E-05 seconds.

Types of issue

  • Bug report (report an issue with the code)
  • Feature request (request for change which adds functionality)

Expected Behavior

After line 801:

//Reset Requested?
if (doReset)
{
writeReg(la, "GEM_AMC.GEM_SYSTEM.CTRL.LINK_RESET", 0x1);
}

we should insert a sleep statement for at least N orbits (N=1?).

We could consider also inserting a sleep statement in between each OH (e.g. between lines 806 & 807 but I'm not sure that's necessary (as 24 register reads should take at least some 200 micro-seconds?).

Current Behavior

Right after a link reset is sent we immediately start checking the SYNC_ERR_CNT registers.

Context (for feature requests)

Returned results by monitoring getmonVFATLinkLocal may not be accurate because we aren't waiting enough time for the sync error counters to increment.

Your Environment

@jsturdy
Copy link
Contributor

jsturdy commented Mar 29, 2019

Partially addressed by #110

bdorney pushed a commit to bdorney/ctp7_modules that referenced this issue Apr 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants