Погледнах го съвсем отгоре, както точно не ти работи. Начина по който е направнео приемането е доста баламски, не знам от какво му пращаш данни но е възможно да изтърваш единицата ако се предава и още нещо.

Значе сега ти гледам майн-а само, преимайки че другото работи :
Първото което ми прави впечатление е че имаш
if(ReadUSART()=='1') // If the received data is equal to the 'A'
В коментара си написал ако е А но това работи ако си изпрати 1, и говорим за ASCI 1, или това е 0x31 , та не знам дали подаваш А или 1.
След това имаш
while(!DataRdyUSART());
къде го чистиш тоя флаг който проверяваш, очевидно тази функция връща дали ти е превършило четенто, но никъде в майна не го обработваш, т.е. това като се вдигне кое ще го свали ?
и след това имаш
PORTB == data; // The receive data is send to PORTB

това не е оператор за присвояване а сравнение, следва да бъде
PORTB = data; // The receive data is send to PORTB

Да не говорим че отварящите и затварящите скоби след while не са много на място, какво отварят и затварят?
Същото и за If имаш го, но отварящи и затварящи не са на място, би следвало да е нещо такова :
if(ReadUSART()=='1'){ // If the received data is equal to the 'A'
LED1_on(); // LED blinking
Delay10KTCYx(25); // Dealay 250K cycles
LED1_off();}