Serial.begin(9600);
The stone age is over. Pick up the speed.
mapvalue = map(Reading, 0, 1023, 0, 255); //mapping 0-1023 value to 0-255
That's a pretty expensive way to divide by 4.
delay(10);
WTF? I thought you said speed was important.
delay(10);
delay(10);
I guess not. Never mind. Forget I posted anything.