ATmega8 ADC PWM - proszę sprawdzić kod

Witam, mógłby mi ktoś sprawdzić kod, szczególnie ustawianie adc? Zmiana rezystancji na potencjometrze ma powodować ruch serwa o kat proporcjonalny do zmiany rezystancji. Wcześniej zrobiłem dla jednego kanału adc i było ok ale po zmianach na dwa kanały serwa tylko drgają.

Najpierw starutjesz konwersję, potem po jej zakończeniu zmieniasz wejście ADC. Poza tym, jeśli odczytujesz rejestr ADC, to odczytujesz 16 bitów, przypisanie wyniku do zmiennej 8-bitowej, automatycznie nie wybiera ośmiu najbardziej znaczących bitów wyniku, lecz 8 najmniej znaczących bitów wyniku.

Zmieniłem z ADC na ADCW, ustawiam wejście przed zakończeniem konwersacji ale serwa dalej nie poruszają się tak jak powinny, drgają jeszcze bardziej(o większy kąt nie szybciej).

Do ADMUX wpisuj po prostu odpowiednie wartości, bez tych | i &. Kolejna sprawa - ADCW to 16-bitowy wynik z ADC, ty zapewne chcesz ADCH, przy czym musisz ustawić wyrównanie wyniku ADC do lewej, inaczej w ADCH będziesz miał tylko 2 najbardziej znaczące bity wyniku.

Ustawiłem bit ADLAR na 1, usunąłem | i &. Serwa przestały drgać, ale dalej sie nie ruszają. Jestem dosyć świeży więc proszę o wyrozumiałość

Mam wrażenie, że chaotycznie coś próbujesz zrobić, zupełnie bez zastanawiania się nad sensem zmian. Napierw masz ADMUX = (1<<REFS1) | (1<<REFS0)| (1<<ADLAR), potem ADMUX = _BV(0), co kasuje ci ustawienia napięcia referencyjnego i bit ADLAR, po czym znowu używasz tych magicznych | i &. Nie możesz po prostu zmieniając ADMUX robić normalne przypisanie, łącznie z konfiguracją referencji i sposobu ustawienia bitów wyniku? Poza tym w Atmel Studio masz coś takiego jak symulator (jeśli nie dypsonujesz sprzętowym debugerem). Wystarczy zasymulować ten kod linia po linii i obserować konfigurację ADC, a w mig wszystko stanie się jasne.

Program już działa, problem tkwił właśnie w złym ustawieniu rejestru ADMUX.