ATNEL tech-forum 2014-08-26T09:38:35+01:00 https://forum.atnel.pl/feed.php?f=58&t=8263&mode 2014-08-26T09:38:35+01:00 2014-08-26T09:38:35+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92914#p92914 <![CDATA[Re: Obsługa dwóch timerów]]> , bo dla mnie nie które wypowiedzi brzmią jak zaklęcia voodoo.
Tj. Mam na myśli, że Mirek mimo wszystko podaje to bardziej strawnie. Ciężko na początku drogi jeszcze ogarniać te wszystkie modularyzacje.
Sam jestem jak najbardziej za czytelnym kodem i trzymaniem się standardów, niestety czasem trzeba pochylić się nad maluczkimi (np. mną, bom tempy niczym gerlach z zastawy babci).

Statystyki: Napisane przez Gość — 26 sie 2014, o 09:38

]]> 2014-08-26T08:38:24+01:00 2014-08-26T08:38:24+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92907#p92907 <![CDATA[Re: Obsługa dwóch timerów]]> mokrowski napisał(a):

Rzeczywiście napisałeś że Atmega162, przepraszam umknąło mi....

Nie rozumiem intencji init_leds() gdzie LED_PORT jest zerowany a później ,,tłuczesz" 0xFF na port w przerwaniu... O co tu chodzi?
Kto czyta tę zmienną k którą tak pracowicie inkrementujesz i zerujesz w przerwaniu?

Proszę podziel kod na oddzielne pliki gdzie klawiatura będzie miała plik *.c *.h a wyświetlacze segmentowe oddzielne. Wtedy będzie widać intencje...
NIestety jestem ,,upierdliwcem" co do elegancji kodu :-)

Ma być coś w stylu init_led_port(), init_led_timer(), init_key_port(), init_key_timer(). A w main.c tylko użycie... Już część masz ale reszta przemieszana.
Do oddzielnych plików i wiele się wyjaśni.

Rozumiem że chcesz sprawdzić czy działa timer2? Jeśli tak i ma zamrugać diodą/diodami to zmień kod na:
[syntax=c]ISR(TIMER2_COMP_vect)
{
/* LED_PORT = 0xFF; */
k++;
if(k==100)
{
LED_PORT ^=0xFF;
k = 0;
}
}[/syntax]
wtedy będziesz wiedział czy przerwania timer2 pracują.

Cześć.
Skupiłeś na elemencie kodu, który nie działał, a 0xFF umieściłem po to by sprawdzić, czy w ogóle następuje procedura przerwania. Inicjalizacja diod jest śmieszna, ale chciałem przećwiczyć ten element poprzez funkcję, która nie zwraca wartości. Dobrze zwrócić uwagę na nazwy zmiennych, funkcji, tak jak to pokazałeś w swoim komentarzu. Masz rację odnośnie elegancji kodu, gdyż taki kod czyta się dużo łatwiej i analizuje. Obiecuję, że postaram się nad tym popracować. Zmienna k miała służyć do zliczania ilości przerwań i na jej podstawie wyliczyć, jakąś przerwę czasową. Na pewno jej użyję, ale póki, co skupiam się na wykorzystaniu sprzętowych zasobów mikroprocka i sprawdzam, czy blokuje to program. Ot tak, jak dziecko sprawdza, czy płomień parzy ;)
Odnośnie tego błędu, byłem pewny, że wpisałem OCIE2, a tymczasem.... :oops:
Dziękuję za wszystko.

Statystyki: Napisane przez repro — 26 sie 2014, o 08:38

]]> 2014-08-25T18:37:43+01:00 2014-08-25T18:37:43+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92847#p92847 <![CDATA[Re: Obsługa dwóch timerów]]> repro napisał(a):

[syntax=c]//utawienia timera0
        TCCR0 |= (1<<WGM01);
        TCCR0 |= (1<<CS02) | (1<<CS00);
        OCR0 = 78;
        TIMSK |= (1<<OCIE0);[/syntax]

repro napisał(a):

[syntax=c]////////timer2///////////////
        TCCR2 |= (1<<WGM21); //tryb CTC
        TCCR2 |= (1<<CS20) | (1<<CS21) | (1<<CS22); //preskaler 1024
        OCR2 = 156; //100Hz,10ms
        TIMSK |= (1<<TOIE2); //zezwolenie na CTC[/syntax]

W drugim przypadku masz TOIE2.

Statystyki: Napisane przez Krauser — 25 sie 2014, o 18:37

]]> 2014-08-25T12:56:39+01:00 2014-08-25T12:56:39+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92802#p92802 <![CDATA[Re: Obsługa dwóch timerów]]>

Statystyki: Napisane przez repro — 25 sie 2014, o 12:56

]]> 2014-08-25T11:27:41+01:00 2014-08-25T11:27:41+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92788#p92788 <![CDATA[Re: Obsługa dwóch timerów]]> Głupia gafa o której zapomniałem. Rzeczywiście nazwy nie mogą się rozpoczynać od cyfr. Chciałem obsłużyć 2 timery ;-) Oczywiście można zrobić tak, jak napisałeś, ale chcę korzystać z dwóch timerów.. W przyszłości będę chciał obsługiwać kilka rzeczy "na raz", tzn. licznik na 7seg, animacja na diodach,za pomocą microswitch zmiana animacji itd. Coś się jeszcze znajdzie. Po waszych poprawkach wszystko działa, jak powinno i bez błędów. Jak będzie z tym drugim timerem? Dlaczego nie zlicza?

Statystyki: Napisane przez repro — 25 sie 2014, o 11:27

]]> 2014-08-25T07:57:20+01:00 2014-08-25T07:57:20+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92773#p92773 <![CDATA[Re: Obsługa dwóch timerów]]> Po kiego grzyba ci
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>[/syntax]
w pliku 7SEG.h ?

Statystyki: Napisane przez perlon — 25 sie 2014, o 07:57

]]> 2014-08-25T01:44:22+01:00 2014-08-25T01:44:22+01:00 https://forum.atnel.pl/viewtopic.php?t=8263&p=92768#p92768 <![CDATA[Obsługa dwóch timerów]]> chciałbym używać dwóch timerów w "locie". Jeden do multipleksowania, a drugi do obsługi klawiatury matrycowej 4x4 i w międzyczasie puścić animację na drabince diodowej. Drugi timer nie chce mi działać. Myślałem, że można obsługiwać dwa timery TIMER0 i TIMER2.
Plik 7seg.c z obsługą Timera0
[syntax=c]/*
* PB0 = A
* PB1 = B
* PB2 = C
* PB3 = D
* PB4 = E
* PB5 = F
* PB6 = G
*
*
* PD2 = LED1
* PD3 = LED2
* PD4 = LED3
* PD5 = LED4
*
* OCR = 78 // 200MHz dla preskalera 1024
*
*/

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

#include "7seg.h"


//definicja zmiennych globalnych

volatile uint8_t cy1;
volatile uint8_t cy2;
volatile uint8_t cy3;
volatile uint8_t cy4;

uint8_t cyfry[15] = {

~( SEG_A | SEG_B | SEG_C | SEG_D | SEG_E | SEG_F ),//0
~( SEG_B | SEG_C ),//1
~( SEG_A | SEG_B | SEG_D | SEG_E | SEG_G ),//2
~( SEG_A | SEG_B | SEG_C | SEG_D | SEG_G ),//3
~( SEG_B | SEG_C | SEG_F | SEG_G ),//4
~( SEG_A | SEG_C | SEG_D | SEG_G | SEG_F ),//5
~( SEG_A | SEG_C | SEG_D | SEG_E | SEG_F | SEG_G ),//6
~( SEG_A | SEG_B | SEG_C | SEG_F),//7
~( SEG_A | SEG_B | SEG_C | SEG_D | SEG_E | SEG_F | SEG_G ), //8
~( SEG_A | SEG_B | SEG_C | SEG_D | SEG_F | SEG_G ),//9
0xFF//10

};



void d_led_init(void)
{
LED_DATA_DIR = 0xFF; // wszystkie portu B jako wyjscia(katody)
LED_DATA = 0xFF; // wygaszenie wszystkihc katod - stan wysoki
// 4 piny portu D jako wyjscia(anody)
ANODY_DIR |= CD1 | CD2 | CD3 | CD4;
// wygaszenie wszystkich wyswietlaczy
ANODY_PORT |= CD1 | CD2 | CD3 | CD4;

//utawienia timera0

TCCR0 |= (1<<WGM01);
TCCR0 |= (1<<CS02) | (1<<CS00);
OCR0 = 78;
TIMSK |= (1<<OCIE0);
}

// ============== PROCEDURA OBSŁUGI PRZERWANIA - COMPARE MATCH =====

ISR(TIMER0_COMP_vect)
{
//zmienna do przelaczania kolejno anod wyswietlacza
static uint8_t licznik = 4;
// cykliczne przelaczanie w kazdym przerwaniu
// anoud kazdego wyswietlacza ( wartosc negowana, aby krazylo zero )
ANODY_PORT = (ANODY_PORT | MASKA_ANODY);

//gdy zapalony wysw 1 podaj stan zmiennej c4
if( licznik == 4 ) LED_DATA = cyfry[cy4];
// gdy zapalony wysw 2 podaj stan zmiennej c3
else if( licznik == 8 ) LED_DATA = cyfry[cy3];
// gdy zapalony wysw 3 podaj stan zmiennej c2
else if( licznik == 16 ) LED_DATA = cyfry[cy2];
// gdy zapalony wysw 4 podaj stan zmiennej c1
else if( licznik == 32 ) LED_DATA = cyfry[cy1];

ANODY_PORT = (ANODY_PORT & ~MASKA_ANODY) | (~licznik & MASKA_ANODY);


//operacje cyklicznego przesuwania bity zapalaacego anody w zmiennej licznik
licznik <<= 1; // przesuniecie zawartosci bitow licznika o 1 w lewo
if( licznik > 32 ) licznik = 4; // jesli licznik wiekszy niz 8 to ustaw na 1
}[/syntax]
Plik nagłówkowy 7seg.h
[syntax=c]/*
* 7seg.h
*

*/
//#ifndef 7SEG_H_
//#define 7SEG_H_

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

#define LED_DATA PORTB
#define LED_DATA_DIR DDRB

#define ANODY_PORT PORTD
#define ANODY_DIR DDRD

#defineCD1(1<<PD2)
#define CD2(1<<PD3)
#define CD3(1<<PD4)
#define CD4(1<<PD5)

#define MASKA_ANODY ( CD1 | CD2 | CD3 | CD4 )

#define SEG_A (1<<0)
#define SEG_B (1<<1)
#define SEG_C (1<<2)
#define SEG_D (1<<3)
#define SEG_E (1<<4)
#define SEG_F (1<<5)
#define SEG_G (1<<6)
#define NIC 10

extern volatile uint8_t cy1;
extern volatile uint8_t cy2;
extern volatile uint8_t cy3;
extern volatile uint8_t cy4;


void d_led_init(void);

//#endif /* 7SEG_H_ */[/syntax]
Jak odkomentuję:
[syntax=c]#ifndef 7SEG_H_
#define 7SEG_H_
....
#endif /* 7SEG_H_ */[/syntax]
To mam taką akcję i pierwszy raz coś takiego mi się zdarzyło.
Obrazek
Dlatego zakomentowałem i problem znikł.
Plik main.c
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/delay.h>
#include "7seg.h"

///deklaracje klawiszy
#define KEY_PIN PINC
#define KEY_PORT PORTC
#define KEY_DIR DDRC

#define SW1 (1<<PC7)
#define SW2 (1<<PC6)
#define SW3 (1<<PC5)
#define SW4 (1<<PC4)
///////////////////////////////

//diody
#define LED_PORT PORTA
#define LED_DDRDDRA

void init_keys();
void init_leds();
uint8_t key_press(uint8_t key);

volatile uint8_t k = 0;

int main()
{
/*inicjalizacja*/
d_led_init();
init_keys();
init_leds();
////////timer2///////////////
TCCR2 |= (1<<WGM21); //tryb CTC
TCCR2 |= (1<<CS20) | (1<<CS21) | (1<<CS22); //preskaler 1024
OCR2 = 156; //100Hz,10ms
TIMSK |= (1<<TOIE2); //zezwolenie na CTC



//testowa inicjalizacja liczb na wyswietlaczu led

cy1=NIC; //2 c,g
cy2=NIC;
cy3=NIC;
cy4=NIC;

sei(); //zezwolenie na globalne przerwania

_delay_ms(1000);


/***petla glowna programu***/

while(1)
{
cy1,cy2 = NIC;
//liczenie do 99 i rest, i - jednosci, j - dziesiatki
for( uint8_t i = 1, j=0; ;i++)
{

if(i > 9)
{
i = 0;
j++;
cy2 = j;

if(j > 9)
{
j = 0;
break;
}
}

cy1 = i;
_delay_ms(200);
}

}

}


void init_keys()
{
///inicjalizacja klawiszy
KEY_DIR &= ~( SW1 | SW2 | SW3 | SW4 );
KEY_PORT |= SW1 | SW2 | SW3 | SW4;

}

void init_leds()
{
LED_DDR = 0xFF;
LED_PORT = 0;
}

uint8_t key_press(uint8_t key)
{
if( !( key & KEY_PIN))
{
_delay_ms(80);
if( !( key & KEY_PIN ) )
{
return 1;
}
}
return 0;
}
ISR(TIMER2_COMP_vect)
{
LED_PORT = 0xFF;
k++;
if(k==100)
{
k = 0;
}
}[/syntax]
Przerwanie Timer2 nie następuje. Gdzie popełniam błąd, albo co mi umyka?
uC: Atmega162, 16MHz.

Statystyki: Napisane przez repro — 25 sie 2014, o 01:44

]]>