Atmega128 - Odbiór danych z MCP3204-CI/P i wyświetlenie na GLCD

Mam problem z odczytem danych z ADC po SPI. Nie jestem pewien czy napisany program jest poprawny dlatego prosiłbym o ewentualne wyszukanie błędu i jakieś naprowadzenie jak go poprawić. Na CH1 układu MCP3204 podaje napięcie z potencjometru. Dane chce wyświetlić w postaci liczby oddawanej przez ADC. GLCD działa i zmienne, które miały być wyświetlone działają, tylko w miejscu danych z ADC widać "0".


main.c


spi.c


spi.h


Jeszcze dodam, że kompilator wywala mi takie warningi.

A jak Twój program ma się do obrazka 6-1 z dokumentacji układu (sterowanie linii CS)?

Dlaczego nie poprawiłeś programu tak, żeby nie generował ostrzeżeń przy kompilacji?

Jest progres Pozbyłem się warningów i ustawiania pinu CS w funkcji WysIOdb tak jak sugerowałeś @BlueDraco. Także CS jest ustawiony na 0 podczas odbioru 3 bajtów ciągiem. W odpowiednim miejscu na GLCD pojawiły się w końcu liczby. Tylko, że nie ma reakcji na zmianę napięcia. Co jeszcze może być nie tak ?

Spójrz na rysunki: FIGURE 6-1, FIGURE 6-2 w dokumentacji do tego układu.

W twoim kodzie zmieniasz stan pinu CS co bajt, a wg noty powinien on być w stanie niskim dla całej transmisji. Po za tym, wydaje mi się, że w złej kolejności wysyłasz bity konfiguracyjne, które mają zawierać się w pierwszych dwóch bajtach.

Wynik zawiera się w dwóch ostatnich bajtach, więc nie wiem, dlaczego składasz go z trzech bajtów, bo raczej powinno być:

Po analizie figure 6-1 zmieniłem zapis na taki



Rozpisałem swój tok myślenia, który może być nie do końca poprawny jako, że jestem zielony. Niestety ta zmiana nie pomogła. Po starcie programu na wyświetlaczu na chwilę pojawia się liczba "65535" po czym zmienia się na "0" i już tak zotaje. Dalej żadnej reakcji na zmianę napięcia. Czy problem może leżeć w typie zmiennych?

Dodam jeszcze, że F_CPU = 8000000UL i preskaler ustawiony SPR1 = 1, SPR0 = 1 co daje /128.

Pokaż definicję funkcji: WysIOdb(), bo w tej dotychczasowej brakuje Ci średnika po pętli while(...).

W pętlach oczekujących na spełnienie warunku polecam stosować zapis:

Typy zmiennych generalnie są ok, chociaż w przypadku mikrokontrolerów (Zwłaszcza tych 8 bit), preferuje się używanie typów: uint8_t, int8_t, uint16_t, int16_t, itd., w których masz pewność co do wielkości. Ponieważ typ int, short int, itd. na różnych platformach może mieć różny rozmiar.

Faktycznie @Sparrowhawk miałeś racje co do braku średnika. Teraz definicja funkcji wygląda tak.



Niestety jest jak było i nie działa.

Sparrowhawk napisał:

W pętlach oczekujących na spełnienie warunku polecam stosować zapis:

Kod: C / C++

Zaloguj się, aby zobaczyć kod

W jakim celu?

@dondu Po to aby zasugerować, że ktoś naprawdę miał intencję napisania pustej pętli (tzn samo sprawdzanie warunku). Jest to nadmiarowe ale w niczym nie przeszkadza.

@dondu: Aby uniknąć tego typu błędów, jaki przytrafił się koledze. Po za tym, w ten sposób wyraźnie zaznaczasz swoje intencje. Oczywiście można też zastosować:
Uważam, że zapis z continue jest najbardziej 'bezpieczny', a z drugiej strony taki zapis:
choć poprawny, potrafi sprawić kłopoty .

@Jablonaack: Nie mam teraz dostępu do noty układu 3204, ale możesz zwrócić uwagę, na odpowiednie ustawienia Data order, Clock Polarity, oraz Clock Phase.

Możesz też zewrzeć MISO z MOSI, spróbować wysłać 3 bajty, i wyświetlić to co odebrałeś. Być może pomocny byłby tu oscyloskop (Ale pewnie nie masz do niego dostępu), albo analizator stanów logicznych, możesz pokazać też schemat urządzenia, oraz jak obecnie wygląda pętla główna twojego programu.

Tutaj masz też notę aplikacyjną korzystania z interfejsu SPI w mikrokontrolerach AVR: SPI Application note AVR151

Sparrowhawk napisał:

@dondu: Aby uniknąć tego typu błędów, jaki przytrafił się koledze. Po za tym, w ten sposób wyraźnie zaznaczasz swoje intencje. Oczywiście można też zastosować:

Kod: C / C++

Zaloguj się, aby zobaczyć kod

Uważam, że zapis z continue jest najbardziej 'bezpieczny', a z drugiej strony taki zapis:

Kod: C / C++

Zaloguj się, aby zobaczyć kod

choć poprawny, potrafi sprawić kłopoty .

Zadałem to pytanie po to byś to wytłumaczył, bo suche wskazanie rozwiązania dla osób które czytają to teraz i tych którzy będą czytali w przyszłości może być niezrozumiałe, bo przecież w dokumentacji mikrokontrolera jest inaczej.

Podsumowując więc dla tych co czytają ten wątek, wszystkie wskazane wyżej rozwiązania:


wygenerują dokładnie ten sam kod:

dondu napisał:

Podsumowując więc dla tych co czytają ten wątek, wszystkie wskazane wyżej rozwiązania:

Kod: C / C++

Zaloguj się, aby zobaczyć kod

wygenerują dokładnie ten sam kod:

Kod: text

Zaloguj się, aby zobaczyć kod

I to mi się podoba .
Pisząc w ASM napiszę tylko raz

I wiem że ma czekać ...
Natomiast w C są dziwadła z nawiasami itp.

Wystarczy:

while (~SPSR & 1 << SPIF);

.. i obejdzie się bez nawiasów.

Jeszcze raz przejrzałem program i okazało się, że zdefiniowany pin CS był w komentarzu i do tego brakowało zakończenia nawiasu i średnika przy ustawianiu bitów dla SPI (nie wiem czemu tego kompilator nie pokazał) . Mój błąd ale bez waszych podpowiedzi bym nie dał rady, więc bardzo dziękuje wszystkim udzielającym się. Wrzucam działający program, może komuś się przyda.

spi.c


spi.h



main.c