Problem mit C Programm

[Alien (Gast)]

Hallo zusammen,

um es vorweg zu nehmen, ich bin relativ neu auf dem Gebiet 
Mikrocontroller und C... Momentan schreibe ich an einer Uhr, das hat 
auch alles bisher wunderbar geklappt.
Die Ausgabe hat auch schon geklappt, jetzt habe ich allerdings noch 
versucht einen Teil einzubauen, seither wird aber nichts mehr 
dargestellt auf den Ausgängen auch nich wenn ich auf den alten Zustand 
zurückgeh. Ich habe extra noch ein anders Programm zum Ausgänge testen 
gemacht, un die funktionieren noch alle.

Den Code häng ich jetzt mal an.

#include <avr/io.h>
#include <stdint.h>
#include <avr/iom16.h>
#include <avr/interrupt.h>


// CTC Funktionalität herstellen Clear on Compare/Match

#define t1ctc 0x04
#define t0ctc 0x01

// Maskierung das nur der TIMSK Teil von Timer 1 geändert wird

#define t1mask 0b00111100


//Maskierung für TIMSK von Timer0

#define t0mask 0b00000011


//Vorladewert da Timer 1ms darstellen soll

#define t0vor  124    // -1 weil bei 0 angefangen wird zu zählen

//CTC Vektor von Timer 0

#define ctc_vek_t0 TIMER0_COMP_vect


//Variablendefinition

volatile uint16_t millisek;
volatile uint8_t sek;
volatile uint8_t min,std;

//Funktionsdefinition

void timer0_config(void)
{

TCCR0 = (0b00000011 & 0b00000111);   // Prescaler von 64 daraus ergibt sich 8000000/64=125000

OCR0  =  t0vor;   // t0 vor = 124 d.h. bei jedem erreichen von t0vor wird das Interrupt flag des ISR Vectors gesetzt (8000000/64)/f(1000Hz) =125


TIMSK = (t0ctc & t0mask);      // CTC Funktionen herstellen



}


void main(void)
{
//Datenrichtungsregister

DDRA=0xFF;
DDRB=0xFF;
DDRC=0xFF;
DDRD=0xF8;  //Port D 0-2 als Eingang alle anderen Ausgang für Darstellung der Uhrzeit mit LED's


//Interruptroutine

timer0_config();

sei();

while(1)
{
/* uint8_t set; //auskommentiert da Funktion momentan nicht gegeben, und ohne Switch war Funktion vorhanden, jetzt nicht mehr.

set=PORTD;

switch(set)
{
case 3:
{
min++;
PORTB=min;
}

case 5:
{
std++;
PORTA=std;
}

default:
{
*/
PORTA=std;
PORTB=min;
PORTC=sek;
//}

//}
}

}


ISR(TIMER0_COMP_vect)
{
millisek=millisek+1; //Alternativ millisek++

if(millisek==1000)
{
sek++;
millisek=0;
}
if(sek==60)
{
min++;
sek=0;
}

if(min==60)
{
std++;
min=0;
if(std==24)
{
std=0;
}
}
}

[Peter (Gast)]

> TCCR0 = (0b00000011 & 0b00000111);

wie kommst man denn auf soetwas, was erwartest du dir davon?

[Timmo H. (masterfx)]

Vielleicht gehts nicht weil du die "break;"'s in der Switch-Anweisung 
vergessen hast.

[Floh (Gast)]

Alien schrieb:
> /* uint8_t set; //auskommentiert da Funktion momentan nicht gegeben, und ohne 
Switch war Funktion vorhanden, jetzt nicht mehr.
>
> set=PORTD;

Wenn du damit die Taster einlesen willst, ist das falsch.
Das geht mit PIND.
:-)

[Alien (Gast)]

Peter schrieb:
> wie kommst man denn auf soetwas, was erwartest du dir davon?

das war einfach für mich eine gedankenstütze welche Pins in dem Register 
überhaupt dazu gehören, sozusagen als "Sicherheit"

Habe jetzt die Breaks eingefügt und das ganze auch auf PIND abgeändert.

Aber momentan immernoch ohne Erfolg, leider. :(

Weiß noch jemand Rat?

[Timmo H. (masterfx)]

Mach mal das "#include <avr/iom16.h>" raus, die wird eh durch io.h 
eingebunden, steht auch in der Quelle:

/* This file should only be included from <avr/io.h>, never directly. */

Welchen AVR hast du? Ist er auch richtig beim Compiler/AVR Studio 
eingestellt?

[Klaus W. (mfgkw)]

Mit deinem unübersichtlichen Bitgemurkse setzst du in TIMSK
das unterste Bit.
Dadurch wird ggf. ein Overflow-Interrupt ausgelöst. Den hast du aber
gar nicht definiert, sondern den Compare-Interrupt.
Also wird regelmäßig ein Reset ausgelöst.

Vorschlag: sauber neu schreiben.

[Karl H. (kbuchegg)]

Klaus Wachtler schrieb:

> Vorschlag: sauber neu schreiben.

Das war auch mein erster Gedanke.
Das ist klassischer Code mit dem man sich selber austrickst.
Schmeiss als erster die ganzen #define raus und schreib das sauber.

Sowas

#define t1ctc 0x04
#define t0ctc 0x01

...

#define t0mask 0b00000011

....

TIMSK = (t0ctc & t0mask);      // CTC Funktionen herstellen

hat nur einen Zweck: maximale Konfusion zu erzeugen. Und zwar bei dir 
genaus wie bei deinen Lesern.

Und rück deinen Code ein!