ho visto dopo che val era dichiarata UINT, e bastava, però ci sono PIC dove l'ADC è a 12bit

...
Forse è meglio che lo rileggi il K&R, in C le operazioni matematiche "+ - * /" sono eseguite da sinistra verso destra, da destra verso sinistra sono eseguite le assegnazioni con operazione come "+=".
Dato che non ha specificato il modello di PIC è possibile che ha un ADC a 12 bit, però 4096 * 5 = 20480 che entra senza problemi anche in un signed int, quindi non c'è nessun motivo per usare i long.
Giusto per togliere ogni dubbio questo è uno sketch per Arduino che fa la stessa cosa e funziona perfettamente senza usare le parentesi, che comunque è sempre bene mettere ai fini della leggibilità del programma e per eliminare eventuali ambiguità, oltre ad accendere il led sul pin 13, per 100 ms, se il valore è maggiore di 3 stampa sul monitor seriale i valori letti che variano tra 0 e 4 come previsto.
Da notare che con il calcolo proposto da Cece il valore massimo è 4 e non 5 come probabilmente pensa lui perché l'out del ADC è al massimo 1023 e 1023 * 5 / 1024 = 4.995 che viene troncato a 4.
int SV;
void setup() {
pinMode(13, OUTPUT);
Serial.begin(19200);
}
void loop() {
SV = analogRead(A0) * 5 / 1024;
if(SV>3) digitalWrite(13,HIGH);
Serial.println(SV,DEC);
delay(100);
digitalWrite(13, LOW);
}