As I have defined it, e = y-r, but note that u = -Ke => u = K(r-y) (which is what you did before). This sort of convention (and confusion) is common in control and you have to be careful to get the sign of the controller right. (Only one of these gives a stable closed loop control.)