Serial.flush();
Right after opening the serial port, there is absolutely no excuse for doing this.
if (val == 110) // n = 110 in dec N = OPEN
Then, why the heck not make it easy?
if(val == 'n')
Getting rid of excess { and } would certainly make your code easier to read, as would getting rid of excess blank lines, and using Tools + Auto format.
As it is, your code looks like:
if (Serial.available())
{
val = Serial.read();
Serial.print(val);
if (val == 110) // n = 110 in dec N = OPEN
{
servo.write(0);
servo2.write(180);
delay(25);
}
}
else if (val == 109) //109 = m in dec M = CLOSED
{
servo2.write(0);
servo.write(180);
delay(25);
}
from which it is clearly obvious that the else if is in the wrong place.