c51单片机串口控制led灯的亮灭

发送1-8数字，分别控制八个流水灯的亮灭。
#include <REG52.H>
#include <INTRINS.H>
unsigned char t,flag;

void delay(unsigned int n){

unsigned int a,b;
for(a = n; a > 0; a--)
for(b = 114; b > 0; b--);

}

void UART_Init(){

ET1=0;
TMOD=0x10;
TH1=0xfd;
TL1=0xfd;
TR1=1;
SM0=0;
REN=1;
EA=1;
ES=1;

}

void show(t){
unsigned char x;
x=0xfe;
switch(t){
case 8:P2=x;delay(1000);break;
case 7:x=crol(x,1);P2=x;delay(1000);break;
case 6:x=crol(x,2);P2=x;delay(1000);break;
case 5:x=crol(x,3);P2=x;delay(1000);break;
case 4:x=crol(x,4);P2=x;delay(1000);break;
case 3:x=crol(x,5);P2=x;delay(1000);break;
case 2:x=crol(x,6);P2=x;delay(1000);break;
case 1:x=crol(x,7);P2=x;delay(1000);break;
}

}

void main(){
UART_Init();
while(1){
if(flag==1){ //等待接收完成
ES=0;
SBUF=t; //放入发送缓冲区，通过串口发往PC

        while(!TI);    //发送完毕
        TI=0;
        ES=1;
        flag=0;
    
    }
}

}

void set() interrupt 4{ //串口中断服务程序

RI=0;
t=SBUF;        //接收数据，接收PC端发来的信息
flag=1;
show(t);

}
这个代码哪里出了问题，为什么不管输入的数据是多少，都默认只在2，6，8led灯亮

SCON还有PCON没有加上去，没有设置波特率和输入方式

你好好看看把

   P2 = 0xFE; // 1111 1110 点亮D1LED
        delay_10us(50000);    
        P2 = 0xFC; // 1111 1100 点亮D2LED
        delay_10us(50000);    
        P2 = 0xF8; // 1111 1000 点亮D3LED
        delay_10us(50000);    
        P2 = 0xF0; // 1111 0000 点亮D1~D4
        delay_10us(50000);    


        P2 = 0xE0; // 1110 0000 点亮D5LED
        delay_10us(50000);
        P2 = 0xC0; // 1100 0000 点亮D6LED
        delay_10us(50000);
        P2 = 0x80; // 1000 0000 点亮D7LED
        delay_10us(50000);
        P2 = 0; // 0000 0000 点亮D5~D8
        delay_10us(50000);

看不出什么大问题，要不是试试在中断里先读SBUF再清空RI,而且读串口前应该先判断RI,这个中断不一定是接收触发的。