LCD 16x1 4bit

Hallo
Habe ein Problem ich will mein Lcd ansteuern aber es funktioniert nicht 
so richtig. Hier mal der Code:

//Pin description
/*
P1.3 to P1.6 is data bus
P1.0 is RS
P1.1 is E
*/

void writedata(char t)
{
   RS = 1;             // This is data

   P1 &= 0xE1;       // Make P1.3 to P1.6 zero
   P1 |= (t&0x1E);     // Write Upper nibble of data

   E  = 1;             // => E = 1
   delay(150);
   E  = 0;             // => E = 0
   delay(150);

   P1 &= 0xE1;       // Make P1.3 to P1.6 zero
   P1 |= ((t<<4)&0x1E);// Write Lower nibble of data

   E  = 1;             // => E = 1
   delay(150);
   E  = 0;             // => E = 0
   delay(150);
}

Kann man das mit den Null setzen so machen?
und kann ich die Pinbelegung so lassen?