ATNEL tech-forum

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-19T23:38:22+01:00

Statystyki: Napisane przez mirekk36 — 19 lis 2014, o 23:38

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-19T23:28:00+01:00

Kurde aż przekląłem... A i uśmiałem się też. Gdybyś był obok, nie zdziwiłbym się gdybyś mnie niebieską i zieloną książką zdzielił przez łeb. Pozostaje mi tylko zwalić winę na zmęczenie. Po oczywistej poprawce, wszystko śmiga. Ja już szukałem podobnych tematów i na bliźniaczej stronie wyszło, że ktoś miał skopaną atmegę. Już wymieniłem ją na nową i to samo. Nie wiem skąd wziąłem ten zapis, ale na pewno bezmyślnie skopiowałem i wkleiłem. Aż sam jestem ciekaw skąd. Wielkie dzięki Zapis preskalera też poprawiłem.

Statystyki: Napisane przez Jarosław J — 19 lis 2014, o 23:28

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-19T23:06:37+01:00

Jarek - mówię ci przemyśl ten zapis:

[syntax=c]TIMSK |= (TOIE2<<1);[/syntax]

oj przemyśl - jak zobaczysz o co chodzi to się sam uśmiejesz

------------------------ [ Dodano po: 1 minucie ]

o jo jooooj - i nie tylko ten zapis - ..... robisz babole trudne do wyłapania bo to jak dysleksja wygląda nie obraź się ale strasznie czeskie błędy popełniasz - więc skup się na tym co jest nie tak w tym zapisie - dojdź do tego sam - to ZROZUMIESZ a jak podpowiem to ZAPOMNISZ i po czasie znowu będziesz to robił

Statystyki: Napisane przez mirekk36 — 19 lis 2014, o 23:06

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-19T22:01:11+01:00

Hej Poradziłem sobie z problemem poprzez wyłączenie Timera2 Nie wiecie może dla czego nie mogę go uruchomić? Może komunikacja TWI go wykorzystuje choć przejrzałem bibliotekę i nie widzę. Na razie wyłączam, ale przydałby mi się bardzo, wiec jeśli wiecie co to za konflikt to dajcie znać.

Dodam tylko, że ustawienie preskalera nic nie zmienia dopiero jak zezwolę na przerwania od przepełnienia to takie rzeczy się dzieją.

[syntax=c]TIMSK |= (TOIE2<<1);//Timer/Counter2, Overflow Interrupt Enable[/syntax]

Statystyki: Napisane przez Jarosław J — 19 lis 2014, o 22:01

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-18T19:38:48+01:00

tak w stopce mojej jest napisane.

Statystyki: Napisane przez Jarosław J — 18 lis 2014, o 19:38

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-18T19:30:03+01:00

Jeszcze jedno pytanie (bo nie pamiętam czy już pytałem wcześniej).
Czy masz niebieską książkę Mirka?

Statystyki: Napisane przez MirkoT — 18 lis 2014, o 19:30

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-18T19:11:44+01:00

Atmega 32 taktowana zewnętrznym rezonatorem 16 MHz

Wywaliłem wszystko co zbędne - bez żadnych zmian.

Statystyki: Napisane przez Jarosław J — 18 lis 2014, o 19:11

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-18T19:07:07+01:00

Możesz jeszcze przypomnieć na jakim to jest procesorze?

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

A, wyrzuć do testów wszystkie includy których w tych początkowych testach
nie potrzebujesz. np 1wire, eeprom, ir... itd.
Zostaw tylko to co niezbędne na tym etapie

Statystyki: Napisane przez MirkoT — 18 lis 2014, o 19:07

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-18T18:35:01+01:00

No i wracam. Skończyłem dziś systematyzowanie kodu Teraz plik main wygląda tak:

[syntax=c]#include
#include
#include
#include
#include
#include
#include "IR_UNI/ir_config.h"
#include "IR_UNI/ir_rc5.h"
#include "display_led/display_led.h"
#include "I2C_TWI/i2c_twi.h"
#include "1Wire/crc8.h"
#include "1Wire/ds18x20.h"
#include "1Wire/onewire.h"
#include "ds3231/ds3231.h"

int main (void)
{
display_led_init();
i2cSetBitrate(200);
DS3231_init();
sei();
//DS3231_set_time( 22,5,0 );
//DS3231_set_date( 20,14, 11, 14, 5 );
while (1)

{

pobierz_z_ds3231();
przepisz();
rozbij_i_wyswietl ((GODZINY*100)+MINUTY,SEKUNDY,0,3);

}
}[/syntax]

natomiast wyczyszczony plik do obsługi LCD wygląda tak:

[syntax=c]#include
#include
#include
#include "display_led.h"

uint16_t cyfry[25] PROGMEM =
{
(SEG_A|SEG_B|SEG_C|SEG_D|SEG_E|SEG_F),//cyfra0
(SEG_B|SEG_C),//cyfra1
(SEG_A|SEG_B|SEG_D|SEG_E|SEG_G),//cyfra2
(SEG_A|SEG_B|SEG_C|SEG_D|SEG_G),//cyfra3
(SEG_B|SEG_C|SEG_F|SEG_G),//cyfra4
(SEG_A|SEG_C|SEG_D|SEG_F|SEG_G),//cyfra5
(SEG_A|SEG_C|SEG_D|SEG_E|SEG_F|SEG_G),//cyfra6
(SEG_A|SEG_B|SEG_C|SEG_F),//cyfra7
(SEG_A|SEG_B|SEG_C|SEG_D|SEG_E|SEG_F|SEG_G),//cyfra8
(SEG_A|SEG_B|SEG_C|SEG_D|SEG_F|SEG_G),//cyfra9
~(SEG_A|SEG_B|SEG_C|SEG_D|SEG_E|SEG_F|SEG_G|SEG_P), //puste 10
(SEG_G),//minus 11
(SEG_A|SEG_D|SEG_E|SEG_F|SEG_G),//litera E 12
(SEG_E|SEG_G),//litera r 13
(SEG_A|SEG_B|SEG_E|SEG_F|SEG_G),//litera P 14
(SEG_D),//podkr 15
(SEG_C|SEG_D|SEG_E),//w16
(SEG_A|SEG_D|SEG_E|SEG_F),//c17
(SEG_G|SEG_C|SEG_E),//n18
(SEG_D|SEG_E|SEG_F|SEG_G),//t19
(SEG_A|SEG_B|SEG_C|SEG_E|SEG_F|SEG_G),//a20
(SEG_C|SEG_D|SEG_E|SEG_F|SEG_G),//b21
(SEG_A|SEG_B|SEG_D|SEG_F|SEG_G),//on22
(SEG_A|SEG_C|SEG_D|SEG_E|SEG_G),//off23
(SEG_D|SEG_E|SEG_F),//l24
};

void display_led_init(void)
{
LED_DATA_DIR = 0b11111111;// wszystkie katody jako wyjscia
LED_DATA = 0b00000000;// wygaszenie wszystkich katod segmentów stanem niskim
ANODY_DIR |= 0b00111111;// 6 pinów portu wyswietlaczy jako wyjscia
ANODY_PORT |= DA1 | DA2 | DA3 | DA4| DA5| DA6;// wygaszenie wszystkich wyswietlaczy anody
TCCR0 |=(1<TIMSK |= (1<TCCR2 |=(1<TCCR2 |=(1<OCR2=255;
TIMSK |=(1<}

void rozbij_i_wyswietl(uint16_t liczba_duzego_wyswietlacza, uint8_t liczba_malego_wyswietlacza, uint8_t znak, uint8_t tryb_wyswietlania)
{
cyfra1=liczba_duzego_wyswietlacza/1000;
liczba_duzego_wyswietlacza=liczba_duzego_wyswietlacza%1000;
cyfra2=liczba_duzego_wyswietlacza/100;
liczba_duzego_wyswietlacza=liczba_duzego_wyswietlacza%100;
cyfra3=liczba_duzego_wyswietlacza/10;
cyfra4=liczba_duzego_wyswietlacza%10;
cyfra5=liczba_malego_wyswietlacza/10;
cyfra6=liczba_malego_wyswietlacza%10;
}
//----------------------------------------------------------------------------------------------------------------------------------------------------------------
ISR (TIMER0_OVF_vect)
{
PORTB &= ~(0b11111111);// wyłączam cały port katod
PORTA &= ~(0b00111111);// wyłączam cały port anod
static uint8_t licznik=0; //zmienna do przełączania anod wyswietlacza

if (licznik==0){}
else if (licznik==1)
{
LED_DATA =pgm_read_byte ( &cyfry[cyfra1]);
if (kropka==1) LED_DATA |= 0b00001000;
//if ((cursor==1)&&(przerwa==1))cyfra1=10;
ANODY_PORT |=0b00000001;
}
else if (licznik ==2)
{
LED_DATA =pgm_read_byte (&cyfry[cyfra2]);
if (kropka==2) LED_DATA |= 0b00001000;
//if ((cursor==2)&&(przerwa==1))cyfra2=10;
ANODY_PORT |=0b00000010;
}
else if (licznik ==3)
{
LED_DATA =pgm_read_byte (&cyfry[cyfra3]);
if (kropka==3) LED_DATA |= 0b00001000;
//if ((cursor==3)&&(przerwa==1))cyfra3=10;
ANODY_PORT |=0b00000100;
}
else if (licznik ==4)
{
LED_DATA =pgm_read_byte (&cyfry[cyfra4]);
if (kropka==4) LED_DATA |= 0b00001000;
//if ((cursor==4)&&(przerwa==1))cyfra4=10;
ANODY_PORT |=0b00001000;
}
else if (licznik ==5)
{
LED_DATA =pgm_read_byte (&cyfry[cyfra5]);
if (kropka==5) LED_DATA |= 0b00001000;
//if ((cursor==4)&&(przerwa==1))cyfra4=10;
ANODY_PORT |=0b00010000;
}
else if (licznik ==6)
{
LED_DATA =pgm_read_byte (&cyfry[cyfra6]);
if (kropka==6) LED_DATA |= 0b00001000;
//if ((cursor==4)&&(przerwa==1))cyfra4=10;
ANODY_PORT |=0b00100000;
}
++licznik;
if (licznik >6) licznik = 0;
}[/syntax]

Trochę jednak rozjechało formatowanie. Niestety nic - dosłownie nic nie zmieniło się w działaniu programu Nadal wyświetla godzinę na tle słabiej żarzących się zer.

Zauważam nadal jeszcze takie zjawisko:

Jeżeli w pętli głównej do funkcji wyświetlającej wpiszę na przykład takie wyrażenie:
[syntax=c]rozbij_i_wyswietl (1234,56,0,3);[/syntax]
wszystko wyświetla się pięknie.
Tak samo wszystko działa, jeśli wyświetlę zmienną jako argument, której nadam wcześniej jakąś wartość.

[syntax=c]uint16_t licznik;

int main (void)
{
display_led_init();
i2cSetBitrate(200);
DS3231_init();
sei();
//DS3231_set_time( 22,5,0 );
//DS3231_set_date( 20,14, 11, 14, 5 );
while (1)

{

licznik = 8745;
rozbij_i_wyswietl (licznik,0,0,3);

}
}[/syntax]

Kiedy poszedłem o krok dalej i przy pomocy takiego kodu w pętli głównej :

[syntax=c]while (1)

{

licznik ++;
rozbij_i_wyswietl (licznik,0,0,3);
_delay_ms (100);

}[/syntax]

Chciałem tym osiągnąć to, żeby względnie wolno, tak aby dało się zauważyć jak zmienia się wartość licznika, widoczne były kolejne cyfry na wyświetlaczu.
Niestety to co zobaczyłem to tylko cyfrę 0001 Tak jakby utknęło na delayu. Ale nic. Usunąłem delaya Pomyślałem, że najwyżej będę widział tylko najwolniej zmieniającą się pierwszą cyfrę na wyświetlaczu.
Zobaczyłem jednak liczbę 0088 ze słabiej świecącym się segmentem "E" Tak więc wyciągam wniosek, że liczenie odbywa się. Jednak wiem, że uint_16_t potrafi zliczyć więcej niż do 99 (co sugeruje wynik) wiec ponownie wprowadziłem opóźnienie tym razem _delay_ms (10); Tym razem wyświetliły mi się trzy zera od lewej i na miejscu jedności cyfra 2 nałożona z cyfrą 1 Taka litera d z segmentem a. Wrodzona dociekliwość zmusiła mnie do zwiększenia opóźnienia, jednak większej wartości _delay_ms_ (50 dał mi stały nie zmieniający się wynik 0001.

Choć funkcja delay_ms() jest raczej niezawodna, postanowiłem za radą kolegów powołać przerwanie od przepełnienia licznika. Korzystając z mkAVRCalculator, i z zakładki AVR Timer Calculator postanowiłem skorzystać z przerwania od timera 1 w trybie normal jak wynika z tablicy z Wówczas będę miał przerwanie co 10 ms co przy ciągłym dodawaniu będzie dało mi możliwość zaobserwowania zmian nawet tej najszybciej zmieniającej się cyfry.

Więc w main'ie zrobiłem to tak:

[syntax=c]volatile uint16_t licznik;

int main (void)
{
display_led_init();
i2cSetBitrate(200);
DS3231_init();

TCCR1B |= (CS11<<1)|(CS10<<1);//prescaler na 64
TIMSK |= (TOIE1<<1);//Timer/Counter1, Overflow Interrupt Enable

sei();
//DS3231_set_time( 22,5,0 );
//DS3231_set_date( 20,14, 11, 14, 5 );
while (1)

{
rozbij_i_wyswietl (licznik,00,0,3);
}
}

ISR (TIMER1_OVF_vect)
{
licznik++;
}[/syntax]

Wynikiem jest, zmieniająca się z częstotliwością 100 hz cyfra "0" na przemian z "1"

Nie wiem co jest powodem takiego zachowania

Wyświetla mi prawidłowo zmienne stałe więc program do LCD (musi być) raczej jest OK.
Nie zwiększa mi prawidłowo zmiennych w pętli ani przerwaniem ani delay'em

Jakie wnioski z tego wyciągać. Co mi radzicie jeszcze sprawdzić
Jeśli to obsługa wyświetlacza LED, to o czym zapomniałem. Nie mam zupełnie pomysłów
i brakuje już mi nerwów. Programy nie są obszerne, napisałem je wydaje mi się że przejrzyście więc
bardzo proszę rzućcie na nie okiem.

Statystyki: Napisane przez Jarosław J — 18 lis 2014, o 18:35

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:39:15+01:00

A my wolelibyśmy, żebyś wrócił!
I swoją wiedzą dzielił się z innymi i starał się pomagać
mimo, że tak jak np. ja i inni Koledzy na forum.
Choć ja to akurat wiedzę mam maluteńką...

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 21:39

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:35:58+01:00

Przeglądnę sobie to jeszcze raz. Przekonaliście mnie Opiszę jak należy.
Funkcja ob_i_wy znaczy oblicz i wyświetl Kiedyś walczyłem z tym i musiałem tę zmienną liczba1 zdefiniować żeby móc przeprowadzić jakieś działanie dzielenia w dalszej części kodu. Postaram się ułożyć to wszystko, wrócić i Was zaskoczyć (choć wolałbym nie wracać )

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 21:35

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:27:52+01:00

Jarosław J napisał(a):

Oto zawartość pliku d_led.c
[syntax=c]void ob_i_wy(uint16_t liczba_dw, uint8_t liczba_mw, uint8_t znak, uint8_t tryb)
{
uint16_t liczba1;

if (znak==0) //znak rowny 0 temperatura dodatnia
{
if ((liczba_dw <=999) && (tryb!=3)){cyfra1=10;} //pusty wyświetlacz jeśli 0
else{cyfra1=liczba_dw/1000;}
}
else //znak rozny od zera temperatura ujemna
{cyfra1=11;} //minus
liczba1=liczba_dw;
liczba1 %=1000;

if ((liczba_dw<=99)&&(tryb==0)){cyfra2=10;}//pusty wyświetlacz jeśli 0 temperatura a 0 jeli godziny
else{cyfra2=liczba1/100;}

liczba1%= 100;
cyfra3=liczba1/10;
cyfra4=liczba1%10;
cyfra5=liczba_mw/10;
cyfra6=liczba_mw%10;
}[/syntax]

Nie bardzo wiem po co robisz takie kombinacje alpejskie no ale może jest w tym jakiś sens.
No to zobaczmy. Przypuśćmy że jest 17:21:35
Wtedy do swojej funkcji przekazujesz 1721, 35, 0, 3
[syntax=c]void ob_i_wy(1721, 35, 0, 3);[/syntax]
co dzieje się w funkcji
pierwszy if się wykona bo znak == 0
a w tym ifie else bo liczba_dw>999
otrzymamy cyfra1=1
Następny if się nie wykona bo liczba_dw>999
za to wykona się else i co otrzymamy?
cyfra2=liczba1/100
tylko pytanie ile wynosi liczba1?
Bo jakoś nie widzę do tej pory żebyś przypisywał jej jakąś wartość.
A ponieważ to zmienna lokalna to mamy jakąś wartość losową.
To chyba tu już jakiś babol się wkrada?

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 21:27

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:23:25+01:00

Jarosław J napisał(a):

i zacznę od pewnego momentu od nowa.

Tutaj na tym etapie sprawdzania przyjrzyj się przykładowi z BB multipleksowania ale też koniecznie rzuć proszę okiem na rozdział "Wstęp do systemów czasu rzeczywistego" żeby zobaczyć co to znaczy odpalać timerki programowe i robić kilka rzeczy naraz w pętli głównej - każda w swoim czasie i na pewno nie milion razy na sekundę bo to cię właśnie gubi wbrew pozorom - to uproszczenie, jak je nazwałeś.

Statystyki: Napisane przez mirekk36 — 16 lis 2014, o 21:23

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:19:29+01:00

Dzięki Mirku! Poświecę dzisiejszy wieczór na ustalenie tego wszystkiego, cofnę się jak mi doradziłeś i zacznę od pewnego momentu od nowa.

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 21:19

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T21:13:41+01:00

Jarosław J napisał(a):

Zrobiłem to w pętli głównej bo tak jest przejrzyściej według mnie niż na te osiem czy dziesięć linijek miałbym tworzyć funkcję.

Powtórzę tylko za kolegą MirkoT ..... niestety takie podejście, (uwierz mi nie pisze tego żeby ci dogryźć bo widzę że się uczysz i szukasz drogi) ... ale niestety to podejście to MASKARA nad masakrami ...

jak tylko dojdziesz do kolejnego etapu programowania to zobaczysz że FUNKCJE to PODSTAWA PODSTAW w C, dlatego od razu słuchaj tych rad które podaje MirkoT bo dobrze prawi

tak samo jak odpalanie tego MILION razy na sekundę - toż samo to stwarza ci już dziesiątki problemów bo nie panujesz nad tym co robisz. Określaj sobie CO I ILE MA TRWAĆ lub CO ILE MA SIĘ WYKONYWAĆ

jak tworzysz funkcję to nadawaj jej nazwę, która coś mówi nie oszczędzaj słów typu

ob_i_wo() .... co to jest ? równie dobrze można by napisać krócej: o_i_w() ... albo jeszcze krócej oiw() albo jeszcze o() .... jak myślisz do czego to prowadzi ? nie dość że sam się w tym gubisz to pokazujesz kod podczas zadawania pytania który jest KOMPLETNIE nieczytelny. Po to pokazuję w Bluegooku podejście gdy tworzę funkcje czy do uarta czy do LCD itp - pisz więcej np:

odbierz_i_wyslij()

nie wiem czy akurat to ma robić twoja funkcja ale podaję ci przykład jak podejść do nadawania nazw - czy to nie byłoby czytelniejsze ? jak myślisz ? szczególnie dla osób którym zadajesz pytanie ....

Jarosław J napisał(a):

Przykłąd MirkoT nie zadziała ponieważ jak już napisałem zmienne mam tylko przez krótką chwilę Jakiekolwiek opóźnienie np delayem, od razu je zeruje i na wyświetlacz trafia 0000.

Gwarantuję ci że zadziała - tylko ty najpierw zrobiłeś błąd jeśli chodzi o sam sposób wyświetlania - a teraz próbujesz robić coś dalej i tak brniesz w kolejne dziesiątki błędów ? po co ? po co .... po co pisać w ogóle dalej kod jeśli nie przetestowałeś sobie POD KAŻDYM względem wyświetlania - a widać że kompletnie tego nie zrobiłeś jak na razie ...

więc moja porada - odłóż na chwilę ten kod na bok

załóż nowy projekt testowy

i stwórz w pętli głównej (Uwaga!) Timer programowy i za jego pomocą zmieniaj dane na wyświetlaczu co sekundę i sprawdź jak się zachowuje wyświetlanie - jeśli źle - to schodzisz o poziom niżej do swojego piekła i naprawiasz wyświetlanie rozumiesz ?

to KOMPLETNIE nie ma NIC wspólnego z tym czy ty umiesz jakieś tam dane przepisać z unii czy nie ... zrozum kompletnie nic ...

więc trzeba zrobić w tył zwrot i taką drogą iść dalej ...

obejrzeć też poradnik z bloga o pułapkach programowych:
http://mirekk36.blogspot.com/2014/04/pu ... ednej.html

Statystyki: Napisane przez mirekk36 — 16 lis 2014, o 21:13

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:43:54+01:00

Quote:

A według mnie to właśnie w ob_i_wy lub w przerwaniu masz coś skopane. Stawiam bardziej na funkcję

Funkcję ob_i_wy(...) zamieściłem w całości powyżej. Uważam że też jest prosta, i nie powinna stwarzać takich problemów Używałem jej w tej formie w moim wcześniejszym projekcie przy odczycie z PCFa nie stwarzała żadnych problemów, jednak tam odczytywałem PCFa bezpośrednio To znaczy na odczyt i konwersję każdej wartości pamięci miałem właśnie oddzielną funkcję. Gdybyś mógł na nią spojrzeć. Może zauważysz coś czego sam nie dostrzegam.

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 20:43

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:32:33+01:00

Jarosław J napisał(a):

Tym czasem kiedy nie mogę poprawnie odczytać unii jak mi się wydaje,

no właśnie wydaje Ci się. Gdybyś nie mógł odczytać te cyfry by Ci nie "przebijały"
Powtórzę - to nie tu jest problem

Jarosław J napisał(a):

wszystko tkwi w pętli głównej, i tak jest przejrzyściej.

Nie zdecydowanie nie jest przejrzyściej! Szczególnie gdy dajesz ten kod do czytania innym osobom
i prosisz je o pomoc

Jarosław J napisał(a):

Wcześniej również napisałem że jeśli przypiszę na sztywno zmienne wysyłane do funkcji ob_i_wy sama funkcja działa doskonale. Stąd właśnie mój wniosek, że to z przepisaniami lub pobieraniem danych z unii musi być problem.

A według mnie to właśnie w ob_i_wy lub w przerwaniu masz coś skopane. Stawiam bardziej na funkcję

------------------------ [ Dodano po: 1 minucie ]

Jarosław J napisał(a):

No nie obrażaj się. Sam niewiele wiem i dla tego tu jestem. Po prostu NA RAZIE dla mnie jest tak przejrzyściej Nie zmienia to nic w działaniu programu. (albo w jego nie działaniu)

Nie obrażam się. Po prostu przyznaję się do swojej niewiedzy
a tą odrobiną którą mam staram się podzielić

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 20:32

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:30:22+01:00

No nie obrażaj się. Sam niewiele wiem i dla tego tu jestem. Po prostu NA RAZIE dla mnie jest tak przejrzyściej Nie zmienia to nic w działaniu programu. (albo w jego nie działaniu)

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 20:30

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:26:11+01:00

Jarosław J napisał(a):

Jest taka książka "Czysty kod" i tam mądrzy ludzie, profesjonaliści a nie tacy laicy jak ja napisali, że dobra funkcja to właśnie taka
która ma te 8 czy 10 linijek kodu Ale co ja tam wiem. Jestem całkowicie początkujący.

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 20:26

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:22:44+01:00

To nie jest jeszcze projekt tylko jego zarys. Podstawowa funkcjonalność to odczyt godziny i tego uprościć się nie da. Zrobiłem to w pętli głównej bo tak jest przejrzyściej według mnie niż na te osiem czy dziesięć linijek miałbym tworzyć funkcję. Po uporaniu się z tym problemem na pewno rozbiję to na bloki Tym czasem kiedy nie mogę poprawnie odczytać unii jak mi się wydaje, wszystko tkwi w pętli głównej, i tak jest przejrzyściej.
Jak już pisałem próbowałem odczytywać to i przepisywać z różną częstotliwością wykorzystywałem opóźnienia delay'em i przerwaniem od timera wewnętrznego. Wszystko sprowadzało się do tego że po wprowadzeniu opóźnienia powyżej 100ms na wyświetlaczu są same zera Im opóźnienie mniejsze tym godzina wyświetla się wyraźniej. Wybacz ale jestem przekonany, ze jeśli dam opóźnienie od przerwania zewnętrznego efekt będzie dokładnie ten sam. Dla tego przynajmniej na razie nie widzę sensu ingerowania w sam układ. Jeśli jesteś mi w stanie to bardziej uzasadnić nie będę się upierał, i zrezygnuję z jakiejś funkcjonalności i dołączę ten sygnał. Jak już napisałem wcześniej, wszystkie nóżki mam już rozdysponowane.

Wcześniej również napisałem że jeśli przypiszę na sztywno zmienne wysyłane do funkcji ob_i_wy sama funkcja działa doskonale. Stąd właśnie mój wniosek, że to z przepisaniami lub pobieraniem danych z unii musi być problem.

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 20:22

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:12:02+01:00

Jarosławie, a ja Ci radzę... zrób jak mówię
Ale to Twój projekt i zrobisz jak zechcesz.
Przepraszam ale Twój kod czyta się nienajlepiej
więc lepiej go uprościć. W pętli głównej zostawić
tylko to co być musi.
Powtórzę jeszcze raz.
Po co miliony razy na sekundę odczytujesz DS'a?
Po co miliony razy na sekundę ustawiasz zmienne
które się w tym czasie nie zmieniają?
Ale rób jak uważasz.

------------------------ [ Dodano po: 6 minutach ]

Szukasz rozwiązania nie tam gdzie jest problem.
Skoro tak się dzieje to szukaj w:
procedurze ob_i_wy()
obsłudze multipleksowania czyli w przerwaniu

------------------------ [ Dodano po: 8 minutach ]

Jarosław J napisał(a):

Spróbuję jeszcze raz opisać sedno.

Przykłąd MirkoT nie zadziała ponieważ jak już napisałem zmienne mam tylko przez krótką chwilę Jakiekolwiek opóźnienie np delayem, od razu je zeruje i na wyświetlacz trafia 0000. Wydaje mi się że winne jest samo pobieranie zmiennych z unii lub ich przepisywanie do zmiennych globalnych.

Ten przykład >musi< zadziałać bo tak właśnie się to robi

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 20:12

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T20:01:43+01:00

Spróbuję jeszcze raz opisać sedno.

W pętli głównej programu umieszczam tylko te polecenia

[syntax=c]while (1)

{
DS3231_get_datetime( &datetime );
DS3231_get_temp( &temperature );
sekundy = datetime.ss;
minuty = datetime.mm;
godziny = datetime.hh;
dzien_tygodnia = datetime.dayofwek;
dzien = datetime.day;
miesiac = datetime.month;
rok = datetime.year;
wiek = datetime.century;

ob_i_wy (((godziny*100)+minuty),sekundy,0,3);
}[/syntax]
czyli pobieram zmienne z unii i przepisuję je do zmiennych globalnych
Zaraz potem w funkcji ob_i_wy() wrzucam je na wyświetlacz

Jeśli robię to raz za razem jak jest w tym przykładzie to wyświetla mi poprawną godzinę, a na wyświetlaczu delikatnie prześwitują mi zera na każdej z pozycji.

Jeśli wprowadzę jakiekolwiek opóźnienie między przepisaniem zmiennych a ich wyświetleniem to na wyświetlaczach mam wyłącznie 0000

Jeśli w pętli przypiszę bezpośrednio
[syntax=c]sekundy = 30;
minuty =20;
godziny = 10;[/syntax]

Wyświetlacz bez najmniejszego problemu wyświetla to w prawidłowy sposób. Bez żadnego przebicia zer.

Stąd mój wniosek ze zmienne pobrane z unii pobierane są tylko na chwilę a potem coś je zeruje. Im szybciej je wyświetlę tym wyraźniej je widać

Nie potrafię wstawić jakiejś pułapki żeby wyczaić miejsce gdzie się te zmienne zerują ponieważ program jest zbyt porosty. Nie potrafię znaleźć takiego miejsca gdzie mógłbym to zerowanie sprawdzić

Dodam jeszcze , ze jak ustawię bitrate na 50 to wyświetlane są praktycznie same zera delikatnie mocniej świecą segmenty wyświetlanej godziny
Najwyraźniej godzinę widać przy bitrate powyżej 200.

Nie znam jeszcze na tyle dobrze zasad przepisywania tych zmiennych z unii żeby móc wywnioskować cokolwiek, dla tego proszę o pomoc.

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

Przykłąd MirkoT nie zadziała ponieważ jak już napisałem zmienne mam tylko przez krótką chwilę Jakiekolwiek opóźnienie np delayem, od razu je zeruje i na wyświetlacz trafia 0000. Wydaje mi się że winne jest samo pobieranie zmiennych z unii lub ich przepisywanie do zmiennych globalnych.

Statystyki: Napisane przez Jarosław J — 16 lis 2014, o 20:01

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T19:56:54+01:00

A ja mam takie pytanko. Po co setki, tysiące razy robisz to przypisanie w pętli głównej?
Może zrobiłbyś tak.
Ten DS na pewno może wywoływać przerwanie co 1 sek.
To dlaczego z tego nie skorzystasz?
Podłącz nóżkę DS to któregokolwiek pinu procesora który może obsłużyć przerwanie zewnętrzne.
W procedurze obsługi przerwania ustaw flagę np. "sekunda"
w pętli głównej wpisz taki kod:
[syntax=c]if (sekunda) {
odczytaj_czas();
ustaw_zmienne_do_wyswietlania_czasu();
sekunda=0
}[/syntax]
I to wszystko co potrzebujesz mieć w pętli głównej
Oczywiście odnośnie odczytu czasu i jego wyświetlania.
Zobacz jaka oszczędność czasu procesora i jak upraszcza się pętla główna

Statystyki: Napisane przez MirkoT — 16 lis 2014, o 19:56

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T19:37:30+01:00

Ja z kolei nie rozumiem problemu żeby pomóc ... musiałbyś precyzyjniej opisać i podać fragment kodu o który chodzi - bo z tego opisu 3 posty wyżej to żeby dojść co tam jest nie tak musiałbym chyba mieć taki sam układ na biurku żeby móc go testować i zobaczyć w czym rzecz ...

trzeba zawsze postarać nauczyć się wyizolować problem i małymi kawałkami rozwiązywać kłopoty w kodzie - zamiast pytać co jest nie tak w całym kodzie bo coś tam się źle wyświetla ... przynajmniej ja nie umiem inaczej podejść ....

Statystyki: Napisane przez mirekk36 — 16 lis 2014, o 19:37

Re: DS3231 i odczyt Prosta sprawa, ale kłopotliwa

2014-11-16T19:33:43+01:00

Sluchaj bardzo chcialbym Ci pomoc tylko ,ze jako poczatkujacy moge nie miec tyle dosw

Ale probujac znalezc glowny problem. To na chwile wyelminuj sobie unie/struktury i zobacz czy bedziesz nadal miec ten sam problem. Wtedy bedziemy wiedziec gdzie dalej szukac problemu

Statystyki: Napisane przez RafPe — 16 lis 2014, o 19:33