Как правильно написать функцию

void auto_motion_down(int m)
{
 int x; 
 if (A>83)
  {
    x=D;
    PORTB |= (1<< PB0);
  }
     if (D<=x-m)
        {
         PORTB &= ~ (1 << PB0);
        }
}

#define  START()   ( PORTB |= (1<<PB0) )
#define  STOP()    ( PORTB &= ~(1<<PB0) )
 
void auto_motion_down(unsigned int m){
   unsigned int x;// может x как-то все-таки инициализировать?
   if (A > 83){
   x = D;
   START();
        if (D<=x-m){ // это вложенный цикл, тк иначе вне зависимости от A>83 у вас будет вычисляться это условие
         STOP();
       }
    }
}

float x = 1000000.0;// почему у вас x объявлена как int? m тоже как float объявляйте
// или наоборот приводите все к int - x = (int)D;

int x = (int) (D*100.0);// если 2 знака после запятой значащие
if ( (int)(A*100.) > 8300) {START();};
if ( (int)(D*100.) >= (x - m) ) {STOP();};// m адаптируйте под нужный масштаб

void auto_motion_down(int m)
{   
        static float x;
    if(A>83)
     {  
         x=D-m;  
         START_DOWN();
     }
     if(D<=x)
     {
         STOP_DOWN();
     }  
}

int STATE;
int main(){
}