Servo myservo ;
Servo myservo2 ;
When you are counting servos, do you count "Yes, that is a servo, 2, 3..." or do you count "1, 2, 3..."?
if(!digitalRead(12))
txdata.buttonstate = HIGH;
else
txdata.buttonstate = LOW;
The digitalRead() function does not return a boolean value. You should not write code whose intent is not clear. There should be no reason to think about whether HIGH or LOW is true, and the effect of negating that. Use
if(digitalRead(12) == HIGH)
{
}
else
{
}
and put the correct actions in the correct blocks.
What happens when you execute that code?