ATNEL tech-forum

Re: pwm zmiana za pomoca przycisku

2012-12-23T15:40:21+01:00

jrq napisał(a):

A co do _BV dlaczego nie warto tego używać?

Tak jak napisał kolega wyżej .... ja dodam tylko, że każdy początkujący, który zamiast przesunięć korzysta z kociego makra _BV() ..... to widać od razu na kilometr - że nie do końca zrozumiał działanie przesunięć i stara się przed tym uciekać - a jak widać to się szybko mści

dlatego dobra solucja - na naszym forum - jak prezentujemy kody to używamy przesunięć ..... poza tym jak sam się przyzwyczaisz do ich używania - to szybko też dojdziesz do wniosku że kod źródłowy o 100x lepiej wygląda

wesołych świąt

Statystyki: Napisane przez mirekk36 — 23 gru 2012, o 15:40

Re: pwm zmiana za pomoca przycisku

2012-12-23T13:58:48+01:00

Nie tyle, co nie warto - po prostu Mirek preferuje ustawianie konkretnych bitów za pomocą przesunięć, a np. dla porównania T. Francuz (autor innej książki do AVRów) wykorzystuje makro _BV(PBX).
Na forum przyjęły się przesunięcia bitowe, bo faktycznie kod jest wtedy czytelniejszy i łatwiej znaleźć ewentualną pomyłkę.

Statystyki: Napisane przez Wrangler — 23 gru 2012, o 13:58

Re: pwm zmiana za pomoca przycisku

2012-12-23T13:26:14+01:00

Dzięki już chodzi , właśnie jestem w trakcie przerabiania tej książki niestety to mi umknęło.
A co do _BV dlaczego nie warto tego używać?

Statystyki: Napisane przez jrq — 23 gru 2012, o 13:26

Re: pwm zmiana za pomoca przycisku

2012-12-23T13:17:10+01:00

[syntax=c]#include
#include
#include

#define przycisk _BV(PB0)
void pwmlow(void);
void pwmhigh(void);
uint8_t klawisz_wcisniety(uint8_t klawisz);

int main(void)
{
DDRB &= ~przycisk;
PORTB |= przycisk;
DDRD |= _BV(PD2); // kierunek ruchu silnika
PORTD |= _BV(PD2);//

if(klawisz_wcisniety(przycisk))
void pwmlow(void); ///////miejsce błędu <- te oba void'y są nie potrzebne!
else
void pwmhigh(void); ////// miejsce błędu <- j.w
return 0;

}

void pwmlow(void)
{

DDRB |= _BV(PB1);
TCCR1A |= _BV(COM1A1) | _BV(WGM11);
TCCR1B |= _BV(WGM13) | _BV(WGM12) | _BV (CS12) | _BV(CS10);

ICR1=1000;
OCR1A=800;
for(;;);

}
void pwmhigh(void)
{

DDRB |= _BV(PB1);
TCCR1A |= _BV(COM1A1) | _BV(WGM11);
TCCR1B |= _BV(WGM13) | _BV(WGM12) | _BV (CS12) | _BV(CS10);

ICR1=1000;
OCR1A=900;
for(;;);

}

uint8_t klawisz_wcisniety(uint8_t klawisz)
{
if(!(PINB & przycisk))
{
_delay_ms(80);
if(!(PINB & przycisk))
{
return 1;
}
return 0;
}
}[/syntax]

Te void służy do zadeklarowania i zdefiniowania funkcji która niczego nie zwraca w programie. Funkcję należy wywoływać w ten sposób:
pwmlow() itp.
Swoją drogą używaj operacji bitowych(tutaj masz to wyjaśnione: http://mirekk36.blogspot.com/2012/01/av ... stawy.html) a nie mark _BV a najlepiej będzie jak się wyposażysz w tą pozycję: http://atnel.pl/mikrokontrolery-avr-jezyk-c.html bo widać u ciebie spore braki w rozumieniu języka C.

Jeszcze jedna uwaga otóż widzę że zapomniałeś o klamrach w instrukcji warunkowej a wykonujesz więcej niż jedną operację więc te klamry powinny być.

Pozdrawiam i życzę wesołych świąt!

Statystyki: Napisane przez MichalXY — 23 gru 2012, o 13:17

Re: pwm zmiana za pomoca przycisku

2012-12-23T13:11:55+01:00

Pierwsze spostrzeżenia:
1. Jak dodajesz kod umieść go w znacznikach coda dla C.
2. Źle zdefiniowałeś przycisk.

Statystyki: Napisane przez lukas12 — 23 gru 2012, o 13:11

pwm zmiana za pomoca przycisku

2012-12-23T13:04:27+01:00

Dzień dobry,

Chce stworzyć program, który zmienia pwm w atmega8 pod wpływem przyciśnięcia przycisku na wejściu PB0.
Steruje silnikiem przez scalaka l293d, niestety po drobnych zmianach program nawet nie chce się skompilować.
Wywala błąd "expected expression before 'void'" w miejscu zaznaczonym w programie. Proszę o pomoc w zrealizowaniu tego pomysłu.

[syntax=c]
#include
#include
#include

#define przycisk _BV(PB0)
void pwmlow(void);
void pwmhigh(void);
uint8_t klawisz_wcisniety(uint8_t klawisz);

int main(void)
{
DDRB &= ~przycisk;
PORTB |= przycisk;
DDRD |= _BV(PD2); // kierunek ruchu silnika
PORTD |= _BV(PD2);//

if(klawisz_wcisniety(przycisk))
void pwmlow(void); ///////miejsce błędu
else
void pwmhigh(void); ////// miejsce błędu
return 0;

}

void pwmlow(void)
{

DDRB |= _BV(PB1);
TCCR1A |= _BV(COM1A1) | _BV(WGM11);
TCCR1B |= _BV(WGM13) | _BV(WGM12) | _BV (CS12) | _BV(CS10);

ICR1=1000;
OCR1A=800;
for(;;);

}
void pwmhigh(void)
{

DDRB |= _BV(PB1);
TCCR1A |= _BV(COM1A1) | _BV(WGM11);
TCCR1B |= _BV(WGM13) | _BV(WGM12) | _BV (CS12) | _BV(CS10);

ICR1=1000;
OCR1A=900;
for(;;);

}

uint8_t klawisz_wcisniety(uint8_t klawisz)
{
if(!(PINB & przycisk))
{
_delay_ms(80);
if(!(PINB & przycisk))
{
return 1;
}
return 0;
}
}

[/syntax]

Wstawiłem znaczniki - SunRiver
Następnym razem do lochu .....

Statystyki: Napisane przez jrq — 23 gru 2012, o 13:04