ATNEL tech-forum

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-26T16:21:42+01:00

Dzięki za uwagi i pomoc Z Timerem jak najbardziej słuszna uwaga - też się do nich przekonałem, tutaj poszło tak na szybko z delay.

Co do kodu - pracuję nad nim

Statystyki: Napisane przez Lex_ — 26 cze 2014, o 16:21

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-25T22:22:56+01:00

widac pan mirek pomaga pełną parą, ale ja się dorzucę i fajnie by było rozbić te blokujące 750ms na jakiś timer, np. co 1ms i tyle.
Może to zboczenie, ale widząc kody pana mirka pisane od zera, to tam raczej dilejów nie ma. A ja 1wire pisałem sam od zera, więc podam malutki fragment kodu.
taki tam kod:
[syntax=c]enum convert_temperature_ds18b20 {ds18b20_start_mes=1,time_on_convert_temperature_ds18b20, ds18b20_read_meas};
enum convert_temperature_ds18b20 idx = ds18b20_start_mes;

void init_timer0_ctc(void);

int main(void)
{
static uint8_t temperature_sign;
static uint8_t temperature_decimal;
static uint8_t temperature_comma;

init_timer0_ctc();

if(init_1wire() == 0)
{
search_sensors(); //wyswietlenie ile jest widocznych czujnikow na magistrali
save_id_all_sensor();
}
sei();
while(1)
{

if(idx == ds18b20_start_mes && DS18X20_OK == DS18X20_start_meas(DS18X20_POWER_EXTERN,tab_id_sensor[0]))
{
idx = time_on_convert_temperature_ds18b20 ;
}

if(idx == ds18b20_read_meas)
{
DS18X20_read_meas(tab_id_sensor[0], &temperature_sign, &temperature_decimal, &temperature_comma);
setCurrentFont( &DefaultFontInfo );
tft_putint(10,50,temperature_decimal,white, black);
idx = ds18b20_start_mes;
}

}
return 0;
}

void init_timer0_ctc(void)
{
TCCR0A |= (1<TCCR0B |= (1<OCR0A = 173;
TIMSK0 |= (1<}

ISR( TIMER0_COMPA_vect )
{
static uint16_t time;
if(++time == 750 && idx == time_on_convert_temperature_ds18b20)
{
idx = ds18b20_read_meas;
time = 0;
}
}[/syntax]

Statystyki: Napisane przez Paul Dirac — 25 cze 2014, o 22:22

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-25T16:47:37+01:00

Lex_ napisał(a):

:D Spoko poradzę sobie

to mi się podoba - więc mała podpowiedź - słowo klucz to STRING, a więc numer DS'a musisz przekopiować jako string.

Statystyki: Napisane przez mirekk36 — 25 cze 2014, o 16:47

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-25T16:42:56+01:00

Spoko poradzę sobie

Statystyki: Napisane przez Lex_ — 25 cze 2014, o 16:42

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-25T04:39:35+01:00

ok tego ze wskaźnikami r i s nie zauważyłem bo to kolejne zresztą coraz większe mega zamieszanie w tym kodzie ...

Kłopot polega na tym, że ja tobie nie jestem w stanie napisać kodu ... to musisz zrobić sam, ja mogę tylko cię naprowadzać co robisz ewidentnie źle gdy uda mi się na szybko zobaczyć błędy (dzięki temu jak już sam dojdziesz co masz nie tak - to gwarantuję ci że zapamiętasz to rzeczywiście na zawsze a pewnie i czegoś nowego się nauczysz) ... Dlatego podpowiedziałem wyżej o tych twoich pętlach

to teraz kolejna wskazówka, jak masz tą linię:

[syntax=c]dat_p=fill_tcp_data(buf,dat_p,r); // r - przekazywanie wskaźnika jako argument funkcji[/syntax]

to powiedz mi analizowałeś działanie funkcji fill_tcp_data() ? bo coś mi się mocno wydaje, że nie ... bo gdybyś to zrobił to nigdy nie wpadłbyś na pomysł przesłania tam wskaźnika r, który wskazuje na twoją zmienną jednobajtową p albo q.

tymczasem ciało tej funkcji to zaledwie JEDNA linijka .... i powiedz mi co z niej wynika jeśli chodzi o przekazanie wskaźnika jako ostatniego argumentu ?

bo na razie to zadziałałeś z nią (tzn z tą funkcją) tak - że jeśli potrzebny jest tam wskaźnik - to dla ciebie nie jest ważne jaki wskaźnik ... tylko ważne żeby był jakiś , jakikolwiek, hmmm dowolny wskaźnik ...

Statystyki: Napisane przez mirekk36 — 25 cze 2014, o 04:39

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T23:18:56+01:00

Sedno:
Nie wiem więc jak poprawnie przekazać te dane do ramki.

Quote:

stworzyłeś sobie jakieś 3 zmienne p i q ... i co z tego ? gdzie później z nich korzystasz w kodzie ?

Być może nie zwrócił pan uwagi, ale 'p' i 'q' nie używam potem, a jedynie wskaźniki, które na te zmienne wskazują i potem są przekazywane w funkcji wysyłającej dane.
[syntax=c]dat_p=fill_tcp_data(buf,dat_p,r); // r - przekazywanie wskaźnika jako argument funkcji

dat_p=fill_tcp_data_p(buf,dat_p,PSTR("

"));

dat_p=fill_tcp_data(buf,dat_p,s); // s - przekazywanie wskaźnika jako argument funkcji[/syntax]

Quote:

po co w ogóle tworzyć jakieś dodatkowe byty w postaci innych tablic skoro można je przekazać wprost z tablicy sensors ?

Mógłby pan napisać w jaki sposób ?
Wprost przypisać wartość i wysłać?
Przykład:
[syntax=c]p=gSensorIDs[0][0];
q=gSensorIDs[1][0];[/syntax]

Rzecz jasna typ 'p' i 'q' do zmiany.

Dziękuję za odpowiedź.

Statystyki: Napisane przez Lex_ — 24 cze 2014, o 23:18

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T22:53:58+01:00

masz pan

[syntax=c]char p;[/syntax]

więc co to jest p ??? ... to jest zmienna o wielkości jakiej ? .... jednego bajta tak ?

no to teraz co robisz w pętli?

[syntax=c]for(k=0;k<8;k++) {
p=gSensorIDs[0][k]; // <--- PIERWSZY czujnik na 1wire

}[/syntax]

w pierwszym obiegu pętli twoja zmienna p (nie wiedzieć czemu typ char) ... równa się wartości pierwszej liczby z ośmiu, całego numeru czujnika .... np 0xAF

drugi obieg pętli, i znowu p= .... i teraz w zmiennej p zamazujesz 0xAF drugą liczbą z ośmiu całego numeru czujnika czyli np 0x2C

trzeci obieg pętli ..

czwarty ...

....

ostatni obieg pętli i twoje p = ostatniej wartości niech to będzie np 0xd2

więc po za kończeniu pętli masz pan zmienną jednobajtową z wartością ostatniego członu numeru czujnika np 0xd2 ... i co ci z tego przyjdzie - skoro numer zapisany jest aż na OŚMIU bajtach w tablicy sensors ???

ale to nie wszystko .... stworzyłeś sobie jakieś 3 zmienne p i q ... i co z tego ? gdzie później z nich korzystasz w kodzie ? nie zastanowiło cię to ? nigdzie - więc kompilator tak w ogóle to wyżyna ci w pień te dwie pętle z kodu i w ogóle się nie kompilują bo są niepotrzebne - uznaje je za śmieci

więc nie ma co zgadywać czy p powinna być tablicą czy nie wiadomo czym bo przede wszystkim to trzeba jakoś to przekazać do bufora ramki UDP

a skoro mamy bufor ramki udp to po co w ogóle tworzyć jakieś dodatkowe byty w postaci innych tablic skoro można je przekazać wprost z tablicy sensors ?

Statystyki: Napisane przez mirekk36 — 24 cze 2014, o 22:53

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T22:06:01+01:00

Już poprawione u góry. Sorry za nie dopatrzenie.

edit: @Mirek36: zmienna 'p' powinna być tablicą ??

Statystyki: Napisane przez Lex_ — 24 cze 2014, o 22:06

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T22:04:30+01:00

Jaglarz napisał(a):

Na razie proszę się zastosować do wymagań: topic7402.html

Lex_ przeczytaj topic zaproponowany przez Prosiaczka i popraw swój pierwszy post. Bo bez tych kolorków w ogóle nie widzę co program robi, a wrodzone lenistwo nie pozwala mi przekopiowac tego do notepada++ czy innego ustrojstwa koloryzującego składnie. Jak już pozmieniasz to może ktoś będzie Ci w stanie bardziej pomóc.

Statystyki: Napisane przez WoodPaker — 24 cze 2014, o 22:04

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T22:01:06+01:00

Wg mego mniemania puszczam w ruch pętlę, która szuka 8 czujników linii.
Może to niepotrzebne wyprzedzenie, bo mam tylko dwa, ale w przyszłości miałbym większe możliwości.

No chyba że się mylę

edit: sorry - zapomniałem o tym syntax:

[syntax=c]//operacje w warunku wykonują się po otwarciu w przeglądarce karty 'imie termometru'
if (strncmp("/imie_termometru",(char *)&(buf[dat_p+4]),15)==0){
dat_p=http200ok();

//operacje na linii 1wire - dane czujnika
uint8_t k;
char p; //surowy odczyt rom_code
char q;

/* wskaźniki, które wskazują na nazwy czujników a potem są przekazywane
dla funkcji wysyłającej te dane */
char *r;
char *s;

r = &p;
s = &q;
for(k=0;k<8;k++) {
p=gSensorIDs[0][k]; // <--- PIERWSZY czujnik na 1wire

}

for(k=0;k<8;k++) {
q=gSensorIDs[1][k]; // <--- DRUGI czujnik na 1wire

}

dat_p=fill_tcp_data(buf,dat_p,r); // r - przekazywanie wskaźnika jako argument funkcji

dat_p=fill_tcp_data_p(buf,dat_p,PSTR("

"));

dat_p=fill_tcp_data(buf,dat_p,s); // s - przekazywanie wskaźnika jako argument funkcji

goto SENDTCP;

}[/syntax]

[syntax=c]else if (strncmp("/temperatura",(char *)&(buf[dat_p+4]),7)==0){

DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* czekamy 750ms na dokonanie konwersji przez podłączone czujniki */
_delay_ms(750);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) )
{
itoa(cel,bufor,16);

dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("temperatura: "));
dat_p=fill_tcp_data(buf,dat_p,bufor);;
}
else {
dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("BRAK CZUJNIKA "));

}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) )
{
itoa(cel,bufor,16); // konwersja liczby stałej na łańcuch znaków

dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("temperatura czujnika nr 2: "));
dat_p=fill_tcp_data(buf,dat_p,bufor);;

goto SENDTCP;

}
else {
dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("BRAK CZUJNIKA "));

goto SENDTCP;
}
}[/syntax]

Statystyki: Napisane przez Lex_ — 24 cze 2014, o 22:01

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T21:59:12+01:00

Na razie proszę się zastosować do wymagań: topic7402.html

Statystyki: Napisane przez Jaglarz — 24 cze 2014, o 21:59

Re: Czujnik DS18B20 - przekazanie parametru

2014-06-24T21:56:26+01:00

spróbuj sam przeanalizować co ty tutaj wyprawiasz

[syntax=c]for(k=0;k<8;k++) {
p=gSensorIDs[0][k]; // <--- PIERWSZY czujnik na 1wire

}[/syntax]

i opisz to - ale dokładnie krok po kroku

Statystyki: Napisane przez mirekk36 — 24 cze 2014, o 21:56

Czujnik DS18B20 - przekazanie parametru

2014-06-24T22:15:09+01:00

Witam Mistrzów

Siedzę przy programie, który oparty jest na atmedze 32 i magistrali 1Wire.
Chciałbym za pomocą znanych już dwóch czujników temperatury DS18B20 mierzyć temperaturę i wysłać ją oraz ich ROM_CODE na stronę internetową.
Napisałem kod, wgrałem w mikro kontroler i niestety wartość temperatury na stronie się nie pokazuje i ROM_CODE też. Być może pomyliłem się w typach używanych zmiennych i z stąd zonk. Nie proszę o gotowe rozwiązanie tylko o wskazówki co poprawić bo kod kompiluje się bez błędów i warning'ów.
W skrócie zrobiłem to tak (używam biblioteki pana Mirka dla DS18B20):

[syntax=c]//operacje w warunku wykonują się po otwarciu w przeglądarce karty 'imie termometru'
if (strncmp("/imie_termometru",(char *)&(buf[dat_p+4]),15)==0){
dat_p=http200ok();

//operacje na linii 1wire - dane czujnika
uint8_t k;
char p; //surowy odczyt rom_code
char q;

/* wskaźniki, które wskazują na nazwy czujników a potem są przekazywane
dla funkcji wysyłającej te dane */
char *r;
char *s;

r = &p;
s = &q;
for(k=0;k<8;k++) {
p=gSensorIDs[0][k]; // <--- PIERWSZY czujnik na 1wire

}

for(k=0;k<8;k++) {
q=gSensorIDs[1][k]; // <--- DRUGI czujnik na 1wire

}

dat_p=fill_tcp_data(buf,dat_p,r); // r - przekazywanie wskaźnika jako argument funkcji

dat_p=fill_tcp_data_p(buf,dat_p,PSTR("

"));

dat_p=fill_tcp_data(buf,dat_p,s); // s - przekazywanie wskaźnika jako argument funkcji

goto SENDTCP;

}[/syntax]

Tutaj listing ze sprawdzeniem poprawności odczytu temperatury dla czujnika 1, 2 i jej wysłanie:
[syntax=c]else if (strncmp("/temperatura",(char *)&(buf[dat_p+4]),7)==0){

DS18X20_start_meas( DS18X20_POWER_EXTERN, NULL );

/* czekamy 750ms na dokonanie konwersji przez podłączone czujniki */
_delay_ms(750);

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[0], &subzero, &cel, &cel_fract_bits) )
{
itoa(cel,bufor,16);

dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("temperatura: "));
dat_p=fill_tcp_data(buf,dat_p,bufor);;
}
else {
dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("BRAK CZUJNIKA "));

}

if( DS18X20_OK == DS18X20_read_meas(gSensorIDs[1], &subzero, &cel, &cel_fract_bits) )
{
itoa(cel,bufor,16); // konwersja liczby stałej na łańcuch znaków

dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("temperatura czujnika nr 2: "));
dat_p=fill_tcp_data(buf,dat_p,bufor);;

goto SENDTCP;

}
else {
dat_p=http200ok();
dat_p=fill_tcp_data_p(buf,dat_p,PSTR("BRAK CZUJNIKA "));

goto SENDTCP;
}
}[/syntax]

Za każdą pomoc będę bardzo wdzięczny.

Statystyki: Napisane przez Lex_ — 24 cze 2014, o 21:48