¿Cómo usar AVR ADC con multiplicador?

Estoy aprendiendo sobre la programación de microcontroladores AVR y actualmente tengo una luz intermitente LED simple configurada en un ATtiny26.

Un potenciómetro de 10k está conectado a ADC7 (pin 7) como en el ejemplo de la parte 1 de este tutorial . Los pines de salida del puerto B están conectados a los LED a través de transistores en una matriz multiplexada. (El circuito funciona bien sin el ADC involucrado).

Para probar/aprender, estoy parpadeando un solo LED y esperando una cantidad de milisegundos igual a lo que se lee del ADC. Tengo un problema cuando trato de multiplicar el valor para implementar demoras más largas.

Pensé que el problema era que la variable en la que almaceno el ADCH era una intpero necesita ser otra cosa porque este es un microcontrolador de 8 bits y intsolo puede ser 0-255. Sin embargo, sustituir longno funcionó.

Aquí está el código que funciona:

#define F_CPU 1000000UL
#define ADC_VREF_TYPE = 0x40
int SD = 1;
int main()
{
    // Set port A to output (1)
    DDRA = 0b11111111;
    // Set port B, pin 3, to input (0)
    DDRB &= ~(1 << PB3);
    // Set control and status register
    ADCSR = 0b11100111;
    // Set ADC multiplexer selection register
    ADMUX  = 0b00100111;

    while (1)
    {
        // Set SD (step delay) to value from ADC
        SD = ADCH;
        // Turn on LED (it is a multiplexer, so two pins are used)
        PORTA = 0b00010001;
        // Wait for SD milliseconds
        milli_delay(SD);
        // Turn off LED
        PORTA = 0b00000000;
        // Wait for SD milliseconds
        milli_delay(SD);
    }
}

El problema ocurre cuando trato de usar un multiplicador como este:

SD = ADCH * 10;

Al hacer esto, el LED parpadeará rápidamente en un extremo del barrido del potenciómetro, pero se iluminará fijo y permanecerá así en algún punto mientras gira el potenciómetro a velocidades más lentas (valores ADCH más grandes); esencialmente "bloqueándose" y requiriendo un reinicio mientras el potenciómetro está nuevamente en la posición original. Supongo que estoy tratando de almacenar un valor mayor que el que contiene el tipo numérico, pero parece que no puedo superar esto cambiando el tipo de variable. Lo he intentado longy uint32_trealmente no entiendo qué tienen los tipos numéricos en este micro.

P: ¿Qué debo hacer para poder implementar demoras que van desde 0 hasta, por ejemplo, 2000 milisegundos?

FYI tengo las siguientes bibliotecas incluidas:

#include <inttypes.h>
#include <avr/io.h>
#include <util/delay.h>
#include <stdlib.h>
#include <math.h>

Editar:

Lo había colocado _delay_ms(x)directamente en mi ejemplo para evitar publicar las funciones que realmente estoy usando y que llaman _delay_ms(1)dentro de un bucle. Pensé que sería más conciso, pero como señaló una respuesta, en realidad parecía ser un error.