Fler problem med avbrottsrutin * LÖST VIDEO BIFOGAD*

[kimmen]

Behöver man ha if-satsen alls? Är problemet att displayen flimrar när man anropar Print_RPM(); även utan att ändra vilket värde som skrivs?

Och det är sant att det finns ett läge där de två funktioner kan komma i konflikt med varandra, i detta fall kan det vara vid Capture-värden runt 0xFFFF och möjligen några steg innan. Jag har dock bedömd att för att visa RPM ville det vara likgiltigt då det bara är en visning.

Ja visst är det så att det kanske inte spelar någon större roll i det här fallet. I min tillämpning gjordes mätningen på signaler upp till 20 kHz så det var ganska vanligt att det mätte fel.

Fast skall man vara petig borde det väl problemet fortfarande finnas kvar i din tidigare kod fast tvärt om. Om capture kommer efter overflow men före if(!PIR1.CCP1IF) körs mäter man 65536 för få timerticks i differens. (Istället för att mäta 65536 för många i det motsatta fallet)

[SeniorLemuren]

Som sagt var. I denna applikation, som tråden gäller så finns inget behov av ändringar av programmet som det ser ut nu. Inget flicker förekommer. Varvtalsvisningen är stabil inom det varvtalsområde som är aktuellt.

[Icecap]

Ändå så retar detta mig lite...

Kod: Markera allt

void interrupt(void)
  {
  if(PIR1.CCP1IF)
    { // Save data
    Time_Now.Byte[0]  = CCPR1L; // Transfer the value, low byte
    Time_Now.Byte[1]  = CCPR1H; // Transfer the value, high byte
    if(PIR1.TMR1IF) if(Time_Now.Word[0] <= 0x0002) Overflow++; // Catch the very rare problem, 0x0002 should be calculated better
    if(!Flag.Running)
      {
      Time_Diff.Dword = Time_Now.Dword - Time_Then.Dword;
      }
    Time_Then.Word[0] = Time_Now.Word[0]; // Save only the lower word
    Overflow          = 0;     // Start from known state
    Flag.Running      = true;
    PIR1.CCP1IF       = false; // Acknowledge interrupt
    }
  if(PIR1.TMR1IF)
    { // ~15,26Hz
    PIR1.TMR1IF = false; // Acknowledge interrupt
    if(Overflow < TIMEOUT_LIMIT) if(++Overflow >= TIMEOUT_LIMIT)
      { // Here after ~3,26 sek
      Flag.Timeout = true;  // Set the timed-out flag
      Flag.Running = false; // OK, a measurement is made - sort of...
      }
    }
  }

Jag tror att jag skulle ha adresserat problemet här, i just detta projekt kanske det inte är så himla viktigt men i andra projekt kan det ställa till med en del strul som kan vara mycket svårt att hitta och detta är ju tänkt för andra som kan behöva detta.

Petig som jag är har jag även ändrat lite med utskriften enl. vad som Senior önskar:

Kod: Markera allt

#define DEVIATION_MIN   5

#if DEVIATION_MIN
WORD RPM_Old;
#endif // DEVIATION_MIN

void main(void)
  {
  Initiate_Hardware();
  RPM.Dword    = 0;     // Initial value
  Flag.Running = false;
  Flag.Update  = true;  // Forces a initial readout
  Flag.Timeout = false;
#if SHIFTS
  Intermediate = 0;
#endif // SHIFTS
#if DEVIATION_MIN
  RPM_Old      = 0;
#endif // DEVIATION_MIN
  while(true)
    {
    if(Flag.Running)
      {
      RPM.Dword    = (MAGIC_CONSTANT / Time_Diff.Dword); // Calculate the RPM
      Flag.Running = false; // Allow further updates
      Flag.Update  = true;
#if SHIFTS
      Intermediate = (Intermediate - (Intermediate >> SHIFTS)) + RPM.Dword;
      if(!Intermediate) Intermediate = RPM.Word[0] << SHIFTS; // Gives fast response from zero
      RPM.Dword    = Intermediate >> SHIFTS;
#endif // SHIFTS
      }
    if(Flag.Timeout)
      {
      Flag.Timeout = false;
      Flag.Update  = true;
      RPM.Dword    = 0;
      Intermediate = 0;
      }
    if(Flag.Update)
      { // Print out RPM as needed
      // Här kan man skriva ut RPM om man vill det, en mätning är iaf. gjort
#if DEVIATION_MIN
      if(abs(RPM.Word[0] - RPM_Old) >= DEVIATION_MIN)
        {
        RPM.Word[0] -= (RPM.Word[0] % DEVIATION_MIN);
        RPM_Old = RPM.Word[0];
        Print_RPM();
        }
#else
      Print_RPM();
#endif
      Flag.Update = false; // Mark that measurement as used
      }
    }
  }