void value not ignored as it ought to be

Here are two more illegal conversions from byte pointer to uint8_t

void fadeLedIn(int pinNumber){ 
  for(int j = 0; j <= 255; j+=10){
    analogWrite(pinNumber,j);
    delay(10);
  }  
}

void fadeLedOut(int pinNumber){ 
  for(int j = 255; j >= 0; j-=10){
    analogWrite(pinNumber,j);
    delay(10);
  }  
  digitalWrite(pinNumber, LOW); 
}

Go through your code and fix all these. If you are going to use the array you'll need to put the index with it. Get all that fixed, and post us new code to look at.