[syntax=c]//funkcja przesuwająca bit - kolejne zapalanie następnej diody LED
void przesuwanie_lewo(volatile uint8_t *KPORT){
uint8_t bajt = 1;
//po krotkim nacisnieciu klawisza zmienna akcja1 przyjmuje wartosc 1 oraz sprawdzenie czy Timer2 ==0;
if(klawisz1.akcja1 && !Timer2){
Timer2 = 50;
*KPORT = ~bajt;
bajt = (bajt<<1) | (bajt>>7);
if(bajt > 8) bajt = 1;
}
}[/syntax]
gdzie zmienna akcja1 z obiektu struktury klawisz1 zmienia się cyklicznie z 0 na 1 po każdym naciśnięciu klawisza KEY:
[syntax=c]//funkcja zmieniająca stan zmiennej "akcja" na przeciwny
void wlacz_wylacz(uint8_t *akc){
*akc ^= 1;
}[/syntax]
I teraz pojawia się problem ponieważ po wciśnięciu klawisza KEY zapala się tylko pierwsza dioda LED1. Jeśli w pierwszym listingu usunąć Timer2 i zastosować delay_ms to uzyskuje zamierzony efekt lecz nie mogę już wyłączyć diod po kolejnym wciśnięciu.
Proszę o pomoc w rozwiązaniu tych dwóch problemów. Niżej wstawiam kod całego programu.
Z GÓRY DZIĘKUJĘ ZA POMOC!
[syntax=c]/*
* main.c
*
* Created on: 11 paź 2015
* Author: damia
*/
//F_CPU 8000000 MHz
//dołączenie plików nagłówkowych
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#define LED1 (1<<PD0)
#define LED2 (1<<PD1)
#define LED3 (1<<PD2)
#define KEY (1<<PC0)
//definicja struktury SKLAWISZ
typedef struct{
volatile uint8_t *KPIN;//zmienna do przechowywania PINx
uint8_t key_mask;//zmienna przechowująca maskę klawisza
uint8_t wait_time_s;//zmienna przechowująca czas oczekiwania
void (*kfun1)(uint8_t *akc);//wskaznik do funkcji
uint8_t klock;
uint8_t flag;
uint8_t akcja1;//zmienna pomocnicza do włączania i wyłączania diod - krotkie wcisnięcie
uint8_t akcja2;//zmienna pomocnicza do wlaczania i wylączania diod - dlugie przytrzymanie
}SKLAWISZ;
//timery programowe
volatile uint8_t Timer1, Timer2, Timer3;
//deklaracje funkcji
void przesuwanie_lewo(volatile uint8_t *KPORT);
void wlacz_wylacz(uint8_t *akc);
void wcisniecie(SKLAWISZ *kla);
//utworzenie nowego klawisza
SKLAWISZ klawisz1;
int main(void){
//kierunek wyjsciowy pinów portu D oraz podciagniecie do VCC bo dioda podlaczona katoda do VCC
DDRD |= (LED1 | LED2 | LED3);
PORTD |= (LED1 | LED2 | LED3);
//kierunek wejściowy pinu PC0 portu C oraz podciagniecie do VCC
DDRC &= KEY;
PORTC |= KEY;
//******Inicjalizacja timera Timer0*********
TCCR0 |= (1<<WGM01) | (1<<WGM00);//tryb pracy CTC
TCCR0 |= (1<<CS01) | (1<<CS00);//preskaler 1024
OCR0 = 77;//podział przez 78 - podstawa 100 Hz
TIMSK |= (1<<OCIE0);//CompareMatch
//przypisanie do zmiennych struktury poszczególnych danych
klawisz1.KPIN = &PINC;
klawisz1.key_mask = KEY;
klawisz1.kfun1 = wlacz_wylacz;
klawisz1.wait_time_s = 3;
sei();
while(1){
wcisniecie(&klawisz1);//sprawdzenie jak zostal wcisniety klawisz
przesuwanie_lewo(&PORTD);//zapalanie kolejnych diod
}
}
//funkcja zmieniająca stan zmiennej "akcja" na przeciwny
void wlacz_wylacz(uint8_t *akc){
*akc ^= 1;
}
//funkcja przesuwająca bit - kolejne zapalanie następnej diody LED
void przesuwanie_lewo(volatile uint8_t *KPORT){
uint8_t bajt = 1;
//po krotkim nacisnieciu klawisza zmienna akcja1 przyjmuje wartosc 1 oraz sprawdzenie czy Timer2 ==0;
if(klawisz1.akcja1 && !Timer2){
Timer2 = 50;
*KPORT = ~bajt;
bajt = (bajt<<1) | (bajt>>7);
if(bajt > 8) bajt = 1;
}
}
//funkcja sprawdzająca rodzaj wcisniecia klawisza
void wcisniecie(SKLAWISZ *kla){
register uint8_t wcisniety = (*kla->KPIN & kla->key_mask);
if(!kla->klock && !(wcisniety)){
kla->klock = 1;
if(kla->kfun1) kla->kfun1(&kla->akcja1);//tutaj przekazana jest zmienna akcja1 po nacisnieciu klawisza w celu jej zmiany na przeciwny
//powoduje to kolejne zapalanie diod w funkcji "przesuwanie_lewo"
kla->flag = 1;
Timer1 = kla->wait_time_s * 100;
}else if(kla->klock && wcisniety){
(kla->klock)++;
if(!kla->klock){
kla->flag = 0;
Timer1 = 0;
}
//tutaj przekazanie zmiennaj akcja do innego działania
}else if(kla->flag && !(Timer1)){
if(kla->kfun1) kla->kfun1(&kla->akcja2);
kla->flag = 0;
}
}
//Timery
ISR(TIMER0_COMP_vect){
uint8_t n;
n=Timer1;
if(n) Timer1 = --n;// 100Hz
n=Timer2;
if(n) Timer2 = --n;// 100Hz
}[/syntax]
------------------------ [ Dodano po: 57 minutach ]
Znalazłem już błąd.
1. Po pierwsze w funkcji :
[syntax=c]void przesuwanie_lewo(volatile uint8_t *KPORT);[/syntax]
trzeba utworzyć zmienną statyczną. Po każdym wejściu w funkcję kompilator na nowo definiował zmienną bajt wartością 1 zamiast zapamiętać ostatnią wartość. Rozwiązuje to dodanie słowa static:
[syntax=c]static uint8_t bajt = 1;[/syntax]
2. Timer był źle zainicjalizowany, ale nie miało to większego znaczenia w tym przypadku.Statystyki: Napisane przez drewpol — 12 paź 2015, o 19:31
]]>