Timer 0 und ich seh den Fehler nicht.

[Christian (Gast)]

Leute ich seh den Fehler nicht,
ich will an Pin A3 eine PWM erzeugen aber der Timer läuft irgendwie 
nicht an ...

hier mal der eigentlich kurze Code, sieht da jemand einen Fehler?

Eigentlich sollte der doch laufen, hat sonst noch jemand eine idee?

#include <stdlib.h>
#include <avr/io.h>
#include <avr/interrupt.h>
//#include <avr/pgmspace.h>
//#include <inttypes.h>
//#include <avr/eeprom.h>
//#include <string.h>
//#include <util/delay.h>
//#include <avr/wdt.h>

/* define CPU frequency in Mhz here if not defined in Makefile */
#ifndef F_CPU
#define F_CPU 14745600UL
#endif




#define LED_DDR         DDRA
#define LED_PORT        PORTA

#define LED1            0
#define LED2            1

#define P_PORT      PORTA
#define P_DDR      DDRA
#define  P1_PIN      3
#define  P2_PIN      4




ISR(TIMER0_OVF_vect)
{
  P_PORT |= (1<<P1_PIN); 
  
}

ISR(TIMER0_COMP_vect)
{

  P_PORT &= ~(1<<P1_PIN);
}





int main(void)
{

  TCCR0 |= (1<<CS02) | (1<<CS00);              
  OCR0 = 0x80;                        

  P_DDR |= (1<<DDA3);
  P_DDR |= (1<<DDA4);


    sei();

  TIMSK |= (1<TOIE0) | (1<<OCIE0) ;
  P_PORT |= (1<<P1_PIN); 

while(1)
{

  return 0;

}
}

[Christian (Gast)]

Hab schon geschaut, jtag fuses sind aus. und auf Port A liegt kein JTAG.
Es ist ein Atmega 162

[Christian (Gast)]

Also in der Main wird der Pin gesetzt das kanni ch messen aber der Timer 
läuft nicht an

[Einer K. (Gast)]

Christian schrieb:
> while(1)
> {
>
>   return 0;
>
> }

Das Programm wird beendet und alle Interrupts deaktiviert.
Der absolute Stillstand tritt ein.

Christian schrieb:
> aber der Timer
> läuft nicht an
Doch, tut er (nehme ich mal an), das zeigt aber keine Wirkung. Weil dein 
Code die Interrupts abschaltet

[spess53 (Gast)]

Hi

Was ist mit den COM-Bits?

MfG Spess

[Christian (Gast)]

COM Bits?

[Christian (Gast)]

Ich nutze ja Softpwm in den Interrupts

[Thomas E. (thomase)]

Christian schrieb:
> Ich nutze ja Softpwm in den Interrupts

Du nutzt überhaupt keine Interrupts:

Arduino Fanboy D. schrieb:
> Das Programm wird beendet und alle Interrupts deaktiviert.
> Der absolute Stillstand tritt ein.

[Christian (Gast)]

Habe das so gändert, der Fehler bleibt:

int main()
{

  TCCR0 |= (1<<CS02) | (1<<CS00);              
  OCR0 = 0x80;                        

  P_DDR |= (1<<DDA3);
  P_DDR |= (1<<DDA4);


    sei();

  TIMSK |= (1<TOIE0)  ;
  //P_PORT |= (1<<P1_PIN); 

while(1)
{

  

};
return 0;
}

[Thomas E. (thomase)]

Christian schrieb:
> Habe das so gändert, der Fehler bleibt:

Welcher?

Der Portpin wird eingeschaltet:

Christian schrieb:
> TIMSK |= (1<TOIE0)  ;

Passt doch. Oder was fehlt da jetzt noch?



Christian schrieb:
> return 0;

Und streich diesen Quatsch. Wie kommst du auf diesen Unsinn?

[Christian (Gast)]

Der Timer läuft nicht eigentlich muss der ja laufen wenn ich den OVF 
Interrupt einschalte, habe gerade UART0 mal aktiviert, das senden klappt 
und er bleibt auch in der Endlosschleife, also das Programm wird nicht 
beendet.
aber der Pin toggelt einfach nicht in der ISR

[Christian (Gast)]

Vielleicht ist auch der AVR kaputt

[Thomas E. (thomase)]

Christian schrieb:
> aber der Pin toggelt einfach nicht in der ISR

In welcher?

Christian schrieb:
> ISR(TIMER0_OVF_vect)
> {
>   P_PORT |= (1<<P1_PIN);
>
> }

Da steht ja auch nichts von Toggeln.

Und für die andere:

Christian schrieb:
> ISR(TIMER0_COMP_vect)
> {
>
>   P_PORT &= ~(1<<P1_PIN);
> }

 ist der Interrupt  nicht aktiv:

Christian schrieb:
> TIMSK |= (1<TOIE0)  ;

[Christian (Gast)]

Entshculdigung ich habe zwischenzeitlich die routine so einfach wie 
möglich gemacht:

#include <stdlib.h>
#include <avr/io.h>
#include <avr/interrupt.h>
//#include <avr/pgmspace.h>
//#include <inttypes.h>
//#include <avr/eeprom.h>
//#include <string.h>
//#include <util/delay.h>
//#include <avr/wdt.h>

/* define CPU frequency in Mhz here if not defined in Makefile */
#ifndef F_CPU
#define F_CPU 14745600UL
#endif

/* 19200 baud Kessel */
#define BAUD2 9600UL          // Baudrate
#define UBRR_VAL_U0 ((F_CPU+BAUD2*8)/(BAUD2*16)-1)   // clever runden


#define LED_DDR         DDRA
#define LED_PORT        PORTA

#define LED1            0
#define LED2            1

#define P_PORT      PORTA
#define P_DDR      DDRA
#define  P1_PIN      3
#define  P2_PIN      4


void USART_Transmit( char data )
{
  /* Wait for empty transmit buffer */
  while ( !( UCSR0A & (1<<UDRE0)) )
  ;
  /* Put data into buffer, sends the data */
  UDR0 = data;
}

ISR(TIMER0_OVF_vect)
{
  P_PORT ^= (1<<P1_PIN); 
  
}

ISR(TIMER0_COMP_vect)
{

  //P_PORT &= ~(1<<P1_PIN);
}





int main()
{

  TCCR0 |= (1<<CS02) | (1<<CS00);              
  OCR0 = 0x80;                        

  P_DDR |= (1<<DDA3);
  P_DDR |= (1<<DDA4);

 //USART0
     UBRR0H = UBRR_VAL_U0 >> 8;
     UBRR0L = UBRR_VAL_U0 & 0xFF;
  UCSR0B |= (1<<TXEN0);                // UART TX einschalten
 // UCSR0B |= (1<<RXEN0);
 // UCSR0B |= (1<<RXCIE0);
  UCSR0C |= (1<<URSEL0)|(1<<UCSZ00)| (1<<UCSZ01);    // Asynchron 8N1

    sei();

  TIMSK |= (1<TOIE0)  ;
  //P_PORT |= (1<<P1_PIN); 
USART_Transmit('L');  
while(1)
{



};
return 0;
}

da wird in der OVF Routine getoggelt, eigentlich...

[Christian (Gast)]

komisch ist ja das Timer 1 auch nicht läuft, hatte das jetzt auch mal 
getestet.

[Christian (Gast)]

usart empfangen und wieder raus senden funktioniert auch...

[Thomas E. (thomase)]

Christian schrieb:
> komisch ist ja das Timer 1 auch nicht läuft, hatte das jetzt auch mal
> getestet.

Das dauert aber auch 256x länger bis zum Overflow. Da kann man schnell 
die Geduld verlieren.

Damit du siehst, ob der Port überhaupt toggelt, mach mal das hier:

while(1)
{
 P_PORT ^= (1<<P1_PIN); 
 _delay_ms(100);
}

Und kommentier das Toggeln in der ISR aus.

[Christian (Gast)]

Thomas E. schrieb:
> Damit du siehst, ob der Port überhaupt toggelt, mach mal das hier:
>
> while(1)
> {
>  P_PORT ^= (1<<P1_PIN);
>  _delay_ms(100);
> }
>
> Und kommentier das Toggeln in der ISR aus.

Jo da kommt ein schönes Rechteck bei raus :)

[Thomas E. (thomase)]

Christian schrieb:
> Jo da kommt ein schönes Rechteck bei raus

Na, dann ist der Controller ja schon mal in Ordnung.

Mit dem Rest der Fehlersuche lasse ich dich allerdings in Anbetracht der 
Uhrzeit alleine. Wobei die Erfahrung zeigt, daß es oft sinnvoller ist, 
das ganze Zeugs einfach liegen zu lassen und erst später wieder dabei zu 
gehen.

[Christian (Gast)]

Ja ich mach jetzt auch schluss für heute, erstmal drüber schlafen

[S. Landolt (Gast)]

>   TIMSK |= (1<TOIE0)  ;

!: '<<'

[Einer K. (Gast)]

S. Landolt schrieb:
> <<

Fein!
(Ist mir doch glatt durch die Lappen gegangen)

[Wolfgang (Gast)]

Christian schrieb:
> Ja ich mach jetzt auch schluss für heute, erstmal drüber schlafen

Und dann schmeiß mal den Simulator an ...

Christian schrieb:
> TIMSK |= (1<TOIE0)  ;

Bist du sicher, dass du da nicht schieben möchtest?
Ein bisschen Trennung zwischen Operanden und Operator erhöht die 
Lesbarkeit ;-)

[Thomas E. (thomase)]

Arduino Fanboy D. schrieb:
> Fein!
> (Ist mir doch glatt durch die Lappen gegangen)

Nicht nur dir. Da guckt man später nochmal drüber und schon fällt es 
einem wie Schuppen aus den Augen.

[M. K. (sylaina)]

Christian schrieb:
> TIMSK |= (1<TOIE0)  ;
>   //P_PORT |= (1<<P1_PIN);

P1 ist immer noch Input-Pin und ein < ist was anderes als ein <<

[S. Landolt (Gast)]

> P1 ist immer noch Input-Pin

Christian versteht PIN nicht als 'Input', sondern als pin (Anschluss).

[M. K. (sylaina)]

S. Landolt schrieb:
>> P1 ist immer noch Input-Pin
>
> Christian versteht PIN nicht als 'Input', sondern als pin (Anschluss).

Ändert nichts. Ich sehe nicht, dass der entsprechende Pin als Ausgang 
definiert wird, wäre aber schon wichtig in der Anwendung hier.

[S. Landolt (Gast)]

Wird als Ausgang definiert; mit dem '<<' läuft das Programm, ist 
getestet.

[M. K. (sylaina)]

Ach ja, da bei den Timereinstellungen...hatte ich nicht gesehen.

[Christian (Gast)]

Leute ich danke euch !!!
War echt zu spät gestern und ich habs echt übersehen, so ein Mist, ich 
hab echt an mir gezweifelt weil das ja eigentlich nicht viele Befehle 
sind.

So ein Mist, und dafür wertvollen Speicherplatz im Internet verwendet...

Vielen Dank, läuft...