Atmega8 interrupt signal erzeugen

[HaraldB (Gast)]

Hallo Zusammen,

Ich steh gerade irgendwie auf dem Schlauch.

Ich möchte mittels Funkmodul und Atmega8 diverse Funksteckdosen 
ansteuern was auch soweit mit Delays schon funktioniert wobei das mit 
den Delays nicht sehr genau ist.

Nun habe ich es schon geschafft mit dem 8bit Timer0 alle 500us einen Pin 
zu togglen ich steh nun aber gerade voll auf dem schlauch.

Vllt. kann mir jemand auf die Sprünge helfen wie kann ich nun z.B. einen 
High Puls mit 500us einfach mehrmals hintereinander senden?

Gruß

Chris

[Karl H. (kbuchegg)]

HaraldB schrieb:

> Vllt. kann mir jemand auf die Sprünge helfen wie kann ich nun z.B. einen
> High Puls mit 500us einfach mehrmals hintereinander senden?

Ich versteh, wo dein Problem liegt.

Wenn du es doch schon geschafft hast, eine ISR alle 500µs aurufen zu 
lassen, dann hast du doch deinen gewünschten Puls. Wenn du x Pulse 
brauchst, dann zählst du halt einfach in einer Variablen mit und wenn 
die ISR dann eben 2*x mal aufgerufen wurde, dann machst du einfach 
nichts mehr am Pin.

[HaraldB (Gast)]

Das dacht ich eben auch hat aber nicht funktioniert deshalb dacht ich 
mir evtl. geht es doch nicht so einfach.

Hatte die ganze Zeit schon viele mögliche Varianten Ausprobiert und ein 
"volatile int count" hat mein Problem nun gelöst denn es funktioniert 
nun :)

Alle 8 Bit werden nun korrekt Übertragen.

Auszug Main:

volatile int count = 0;
volatile int send;
volatile int data[10];

int main(void)
{

// PortB1 Output
PORTB &= ~(1<<PB1); // Low
DDRB = (1 << PB1);

// Prescaler 64
TCCR0 |= (1<<CS01) | (1<<CS00);

// Enable Timer0 Overflow Interrupt
TIMSK |= (1<<TOIE0);

// Allow Interrupts
sei();

// set prescaler to 64 and start timer
TCCR2  |= (1<<CS21);

data[0] = 1;
data[1] = 0;
data[2] = 1;
data[3] = 0;
data[4] = 0;
data[5] = 1;
data[6] = 0;
data[7] = 1;

while(1)
{
    if(data[count] == 1)
    {
      send = 1;
    } else if(data[count] == 0)
    {
      send = 0;
    }
    
    if(count == 9)
    {
      count = 0;
    }
}

ISR:

ISR(TIMER0_OVF_vect)
{  
  TCNT0 = 131; // 500us

  if(send == 1)
  {
      PORTB |= (1<<PB1); // High
  } else if(send == 0)
  {
      PORTB &= ~(1<<PB1); // Low
  }

  count++;
  
  //PORTB ^= (1<<PB1);
}

[Hmm (Gast)]

Ich halte das für unsauber, count an zwei Stellen zu verändern. Nimm den 
modulo 10 besser noch in den Interrupt.
Im übrigen spräche aus meiner Sicht nichts dagegen die Ausgabe und den 
Zugriff auf data auch im Interrupt zu machen.

[HaraldB (Gast)]

Ich hab nun alles in die ISR gepackt jetzt ist die Pulselänge immer 1-2 
us zu lang/kurz.

Evtl. doch nicht ganz so Optimal.

[Karl H. (kbuchegg)]

HaraldB schrieb:

> Auszug Main:

Tu dir selbst ein paar Gefallen

> volatile int count = 0;
> volatile int send;
> volatile int data[10];

wenn Variablen per Design keine größeren Werte als 255 haben können und 
auch nie haben werden, dann bürde dem µC keine 16-Bit Airthmetik auf.
Mal abgesehen davon, löst sich dann auch das Problem des atomaren 
Zugriffs in Luft auf, sobald ISR mit dieser Variable im Spiel sind. 8 
Bit pro Variable tuns auch:

volatile uint8_t count = 0;
volatile uint8_t send;
volatile uint8_t data[10];


>     if(data[count] == 1)
>     {
>       send = 1;
>     } else if(data[count] == 0)
>     {
>       send = 0;
>     }

tus nicht.
data KANN bei dir per Design nur 0 oder 1 sein. Andere Werte gibt es 
nicht. Wenn data[count] nicht 1 ist, dann KANN es daher nur 0 sein. Das 
muss man nicht testen. Jedes Wort, dass du unnötig schreibst, ist eine 
potentielle Fehlerquelle.


>   if(send == 1)
>   {
>       PORTB |= (1<<PB1); // High
>   } else if(send == 0)
>   {
>       PORTB &= ~(1<<PB1); // Low
>   }

selbiges.
Wenn send nicht 1 ist, dann kann es per Definition nur 0 sein.



Wozu eigentlich der ganze komplizierte Umweg über send? Wickle doch 
gleich alles komplett in der ISR ab, anstatt die Logik quer über das 
ganze Programm zu verstreuen.

[Hmm (Gast)]

Oops. Sorry. Aber gleich 2us? Hmm. Naja. Dann lass es lieber. Vielleicht 
nur den Modulus im Int.

[Karl H. (kbuchegg)]

HaraldB schrieb:
> Ich hab nun alles in die ISR gepackt jetzt ist die Pulselänge immer 1-2
> us zu lang/kurz.
>
> Evtl. doch nicht ganz so Optimal.

Kann nicht sein.

ISR( ... )
{
  TCNT0 = 131; // 500us

  if( data[count] )
    PORTB |= (1<<PB1); // High
  else
    PORTB &= ~(1<<PB1); // Low

  count++;
  if( count == sizeof( data ) )
    count = 0;
}

und mach deine Variablen uint8_t

[Hmm (Gast)]

Wenn Du da noch 1-2us Verzögerung hast, dann ist was anderes faul.

[HaraldB (Gast)]

Danke euch :)