"Change the variable x to unsigned char. The way it is now, P3 is probably assigned the MSB of x, which is 0x00"

Yes, I didn't notice that.

Also, your inner loop will never execute because you used a signed integer. 0xffff == -1.