Покритикуйте разводку (и схему тоже, и код тоже)

void __vector_adc() SIGNAL;
void __vector_tim0_sompa() SIGNAL;
void servo_entry() TASK;
 
static void
adc_enable()
{
DIDR0 = (1<<ADC5D) | (1<<ADC4D) | (1<<ADC3D)
| (1<<ADC2D) | (1<<ADC1D) | (1<<ADC0D);
 
ADMUX = (1<<REFS0);
ADCSRA = (1<<ADIM) | (1<<ADSC) | (1<<ADPS2);
 
while (ADCSRA & (1<<ADSC)) ;
 
ADCSRB = (1<<ADTS1) | (1<<ADTS0);
ADCSRA = (1<<ADIM) | (1<<ADATE) | (1<<ADIF)
| (1<<ADIE) | (1<<ADPS2);
}
 
static void
timer_enable()
{
TCNT0 = 0;
OCR0A = (CONFIG_MCU_CLOCK / 256UL / 1000UL) - 1UL;
TCCR0A = (1<<WGM01);
TCCR0B = (1<<CS02);
 
TIMSK0 |= (1<<OCIE0A);
}
 
void __vector_adc()
{
#if 0
sr.w_adc_23 = ADC;
 
//TIFR0 |= (1<<OCF0A);
 
ADMUX = (1<<REFS0) | (1<<MUX2) | (1<<MUX0);
ADCSRA = (1<<ADIM) | (1<<ADSC) | (1<<ADATE)
| (1<<ADIE) | (1<<ADPS2);
 
while (ADCSRA & (1<<ADSC)) ;
 
sr.w_adc_28 = ADC;
 
//pwm_dc();
 
ADMUX = (1<<REFS0);
ADCSRA = (1<<ADIM) | (1<<ADATE) | (1<<ADIF)
| (1<<ADIE) | (1<<ADPS2);
#endif
sei();
 
debug_task();
timer_task();
 
/*servo_task();
rs485_task();*/
}
 
void __vector_tim0_sompa()
{
 
}
 
static void
servo_board_enable()
{
ACSR = (1<<ACD);
 
DDRD |= (1<<PD5);
}
 
void servo_entry()
{
servo_board_enable();
debug_enable();
adc_enable();
//pwm_enable();
timer_enable();
 
sei();
 
puts("JK" EOL);
 
//ADCSRA |= (1<<ADSC);
 
do {
cli_task();
relax();
}
while (1);
}