xdisp = (xmath-658);
The brackets aren't necessary here and make it look like you left out a function call by mistake.
#define trigPin 12
#define echoPin 13
int sleepPin = 4;
int xpin = A0;
int ypin = A1;
int zpin = A2;
xpin and ypin etc. don't change, right? So make them const. And make up your mind if you are going to use defines or ints.
eg. better would be:
const byte trigPin = 12;
const byte echoPin = 13;
const byte sleepPin = 4;
const byte xpin = A0;
const byte ypin = A1;
const byte zpin = A2;
Wire.requestFrom(deviceAddress, 1);
while(!Wire.available()) {
}
v = Wire.read();
Wire.requestFrom doesn't return until the requested data is there, or something is wrong. So the call to Wire.available() is redundant. In fact, it will make it hang if the device isn't there. Better would be:
if (Wire.requestFrom(deviceAddress, 1) != 1)
{
// handle error
return -1;
}
// we must have data now
v = Wire.read();