ATNEL tech-forum 2012-10-04T14:57:10+01:00 https://forum.atnel.pl/feed.php?f=4&t=1622&mode 2012-10-04T14:57:10+01:00 2012-10-04T14:57:10+01:00 https://forum.atnel.pl/viewtopic.php?t=1622&p=16421#p16421 <![CDATA[Re: ADC + PWM]]> kidcowboy napisał(a):

Kurde !!! Wielkie dzięki Mirku nareszcie w pełni zrozumiałem idee mapowania.
P.S. 30 pompek leci


maskowania a nie mapowania ;) więc jeszcze 5 pompek dodatkowo za literówkę ;)

Statystyki: Napisane przez mirekk36 — 4 paź 2012, o 14:57

]]> 2012-10-04T14:32:27+01:00 2012-10-04T14:32:27+01:00 https://forum.atnel.pl/viewtopic.php?t=1622&p=16420#p16420 <![CDATA[Re: ADC + PWM]]> P.S. 30 pompek leci

Statystyki: Napisane przez kidcowboy — 4 paź 2012, o 14:32

]]> 2012-10-04T09:48:48+01:00 2012-10-04T09:48:48+01:00 https://forum.atnel.pl/viewtopic.php?t=1622&p=16404#p16404 <![CDATA[Re: ADC + PWM]]>
pomiar() i pomiar1()

jak można to zrobić w ramach jednej funkcji ?

poza tym już dawno pamiętam że tłumaczyliśmy koledze że popełnia poważny błąd (kiedyś tam w innym poście)

Code:
ADMUX |= (ADMUX & 0xF8) | kanal;


i mówiliśmy jak to rozwiązać - a kolega zdaje się już wtedy (o ile pamiętam zaczął pisać oddzielne funkcje) zamiast poprawić ten błąd. Proszę poczytać:

Obrazek
(warto tu zaglądać)

przecież tam jest WYRAŹNIE napisane o tym błędzie - wystarczy poprawić powyższą linię na:

Code:
ADMUX = (ADMUX & 0xF8) | kanal;


i wystarczy jedna funkcja .... (bo teraz nawet mając te dwie funkcje i tak ten BŁĄD opisany w linku nadal u ciebie występuje)



pomijam oczywiście ten dodatek że

Code:
return 127;


a nie działa to dlatego że jak przetwornik działa to wcale nie zawsze jest tam 0 gdy z przetwornika nic nie "leci" jak kolega pisze .... tam jest szum - więc te wartości zmieniają się w okolicy zera (błąd 1-2 bitów przetwornika)

------------------------ [ Dodano po: 2 minutach ]

Zrób sobie jedną funkcję:

[syntax=c]uint8_t pomiar(uint8_t kanal)
{
// USTAWIENIE WYBRANEGO KANA£U ADC W REJESTRZE ADMUX

// ADMUX&=~(1<<MUX0);
// ADMUX&=~(1<<MUX1);
ADMUX = (ADMUX & 0xF8) | kanal; // <---- tu poprawka
ADCSRA|=(1<<ADSC);
while(ADCSRA&(1<<ADSC));
if (ADCH < 10 )return ADCH; // <--- tu proponuję tak sprawdzać
else return 127;
}[/syntax]

i to ci wystarczy zamiast tych dwóch - bo pomyśl sobie - jakbyś chciał na 8 kanałach mierzyć to byś 8 takich funkcji napisał ? a na dodatek gdybyś w każdej zostawił ten sam błąd - to dopiero byłaby kaszana ;) .... po prostu ta jedna operacja OR jest niepotrzebna bo nawet jak masz te dwie funkcje to i tak wciąż pozostają bity kanałów pomieszane.

------------------------ [ Dodano po: 3 minutach ]

a nie przepraszam ty sobie w tych funkcjach wymyśliłeś:

Code:
ADMUX&=~(1<<MUX0);
        ADMUX&=~(1<<MUX1);


;) ale to nie tędy droga bo nie dość że nadal dwie funkcje niepotrzebne to te dwie linie kodu załatwia jedna:

Code:
ADMUX = (ADMUX & 0xF8) | kanal;


dobra koniec - bo się rozpisałem ;)

Statystyki: Napisane przez mirekk36 — 4 paź 2012, o 09:48

]]> 2012-10-04T11:15:52+01:00 2012-10-04T09:20:35+01:00 https://forum.atnel.pl/viewtopic.php?t=1622&p=16403#p16403 <![CDATA[ADC + PWM]]> [syntax=c]
/*
* main.c
*
* Created on: 2012-07-12
* Author: Józef Bąk - syn łaskawie panującego nam premiera
*/

#include <avr/io.h>
#include <util/delay.h>
#include <stdlib.h>



#define ADC_PIN1 (1<<PA0)
#define ADC_PIN2 (1<<PA1)
#define PWM1 (1<<PB3)
#define PWM2 (1<<PB4)


uint8_t pomiar (uint8_t kanal);
uint8_t pomiar1 (uint8_t kanal1);



int main(void)
{
// PODCIAGNIECIE DO VCC
DDRA|=ADC_PIN1 |ADC_PIN1
// PWM KIERUNEK WYJŒCIE
DDRB|=PWM1|PWM2;
//PODCI¥GNIÊCIE DO VCC
PORTB|=PWM1|PWM2;

//INICJALIZACJA TIMERA0 PORT PB3/OC0A
TCCR0A|=(1<<WGM01)|(1<<WGM00); // TRYB FAST PWM
TCCR0A|=(1<<COM0A1)|(1<<COM0B1)|(1<<COM0B0);// CLEAR AT TOP DLA A1 I SET AT TOP DLA B2
TCCR0B|=(1<<CS01);//PRESKALER 8 => 1MHz


// INICJALIZACJA PRZETWORNIKA
ADMUX|=(1<<REFS0)|(1<<ADLAR); //WEW.NAPIÊCIE ZASILANIA 5V I WYROWNANIE DO LEWEJ
ADCSRA|=(1<<ADEN)|(1<<ADPS1);// ROZDZIELCZOSÆ 8(ADEN) BITOWA PRESKALAER 2
//FLAGA SPRAWDZAJ¥CA CZY F(1) JEST PIERWSZ¥ FUNKCJ¥ PO ZA£¥CZENIU ZASILANIA



while(1)
{


pomiar(0);
uint8_t a=ADCH;
OCR0A=a;

pomiar1(1);
uint8_t b=ADCH;
OCR0B=b;


}

}

uint8_t pomiar(uint8_t kanal)
{
// USTAWIENIE WYBRANEGO KANA£U ADC W REJESTRZE ADMUX

ADMUX&=~(1<<MUX0);
ADMUX&=~(1<<MUX1);
ADMUX|=(ADMUX )|kanal;
ADCSRA|=(1<<ADSC);
while(ADCSRA&(1<<ADSC));
if (ADCH!=0)return ADCH;
else return 127;
}

uint8_t pomiar1(uint8_t kanal1)
{
// USTAWIENIE WYBRANEGO KANA£U ADC W REJESTRZE ADMUX
ADMUX|=(1<<MUX0);
ADMUX&=~(1<<MUX1);
ADMUX|=(ADMUX)|kanal1;
ADCSRA|=(1<<ADSC);
while(ADCSRA&(1<<ADSC));
if (ADCH!=0)return ADCH;
else return 127;
}



[/syntax]

[ panie kolego ale zdaje się że już nie raz prosiłem i ja i SunRiver, żeby kody programów otaczać znacznikami [ syntax = c ] a nie [ code ], więc proszę się do tego stosować OK ? jak kolega nie wie jak to proszę edytować ten post i sprawdzić jak to wygląda. mirekk36]

OK Przepraszam - zapomniałem się

Statystyki: Napisane przez kidcowboy — 4 paź 2012, o 09:20

]]>