[Atraktor]

http://pastebin.com/tVxVg1ZH

Nie mam pojęcia co tu się #!$%@?, ale ten kawałek kodu resetuje mi w kółko mikrokontroler. Wywoluje tu funkcje na pobieranie temperatury z czujnika ds18b20 przy użyciu załączonej biblioteki, a następnie zwracam prędkość dźwięku w tej temp.
Dodałem dupa-printy do środka i wychodzi, że całość działa, ale się wywala przy returnie, bo print "dupa1-1" pojawia się, ale "dupa2" juz nie ლ(ಠ_ಠ ლ)

#arduino #avr #atmega #mikrokontrolery

[WasylWasaty]

@Atraktor: Szczerze nie widzę nic co mogło by popsuć program. Jeżeli chodzi o typy zmiennych skoro dupa1-1 jest zwracane to dziwne żeby dalej się wywołało. Nie wiem jak to jest z bibliotekami arduino czy watchdog nie jest wlaczony? I wywala się na delay?

[Atraktor]

@WasylWasaty: delay nie ma tu wplywu, dodalem je tylko zeby zdazyc zobaczyc na wyswietlaczu co się dzieje. Jak wywale wszystkie delaye i printy to tylko widze, że Atmega w kolko mi inicjalizuje wyswietlacz. Wyglada jakby pamięć się wyjebywała przy returnie, ale czy to w ogóle możliwe? Jakiś overflow?

[WasylWasaty]

@Atraktor: Czy wywala się przy pierwszej iteracji?

[Atraktor]

@WasylWasaty: tak

[radekrad]

@Atraktor: może funkcje getAddress i requestTemperaturesByAddress wymagają innych argumentów i pisząc po 8 bitowym t_a nadpisują adres zwrotu?

[A.....k]

@Atraktor: Pokaż cały kod. A unsigned int to uint8_t czy uint16_t? Bo jesli to pierwsze to masz możliwość nadpisać dane ze stosu.

[WasylWasaty]

@Analityk: a która metoda zapisuje coś pod adresem t_a?

[Atraktor]

@WasylWasaty: @radekrad: @Analityk: metodą wywalania kodu wychodzi, że winna jest funkcja

sensor.getAddress(&ta,0);

jeśli zamiast wywolywać ją, wpiszę coś do zmiennej ta to nie ma crasha przy returnie

[A.....k]

@Atraktor: To spróbuj ustawić typ ta na uint32_t.

[Atraktor]

@Analityk: nie mogę, funkcja z biblioteki przyjmuje tylko uint8_t jako argument

[wik5]

@Atraktor: możliwe że ta zmiana maskuje problem a w rzeczywistości problemem jest zbyt mały zakres wynik. Spróbuj zrobić uint32_t wynik

[A.....k]

@Atraktor: Wyłącz watchdoga. Konwersja temp. zajmuje 1 sekundę, a wdt może restartować proca dużo razy w tym czasie.

[Atraktor]

@wik5: dalej to samo
@Analityk: tez to samo

[wik5]

@Atraktor: a typ zwracany GetSpeedOfSound i zmiennej speedofsound też zmieniłeś?

[wik5]

@Atraktor: i czy zrobiłeś wcześniej sensor.begin() ?

[Atraktor]

@wik5: tak, zresztą zakres zmiennej na pewno nie jest tu winny, bo przerabiałem też problemy z nimi. Na wyniku tu będzie liczba +/- 35000, gdzie unsigned int ma zakres do 65535. No i w przypadku ew przekroczenia po prostu dostałbym błędny wynik, a nie reset

[kao3991]

@Atraktor: 1. na 100% wykluczyłeś problem sprzętowy, typu wiszący pin resetu? (chyba że w arduino nie ma takiego czegoś) 2. Próbowałeś wyrzucić z funkcji sam pomiar i podstawić jakąś liczbę na sztywno? 3. jak dasz kolejną dupę za ostatnim delayem to ją zobaczysz czy nie?

[Atraktor]

@kao3991: tak,tak,tak. Wystarczy taki kod i już resetuje mikro. Jeśli wykomentuję wywolanie sensor.getAddress a zamiast tego wpiszę do zmiennej ta np 40 to wszystko działa.

unsigned int GetSpeedOfSound(void)

{

 uint8_t t_a;

sensor.getAddress(&t_a,0);

 return 34321;

}

[radekrad]

@Atraktor: sprawdź jednak argument getAddress - to nie ma być przypadkiem wskaźnik na tablicę a nie na zmienną? Zmień na
uint8t ta[20];
sensor.getAddress(t_a,0);