Does the trace look any different if you arrange your code like this
SPI.beginTransaction(SPISettings(2000000, MSBFIRST, SPI_MODE1));
digitalWrite(slaveSelectPin, LOW); // take the SS pin low to select the chip:
stat = SPI.transfer(0x00)...
I may be wrong, but I've always thought that the reason for cutting the trace was to protect the PC, not the teensy. I imagine some PCs might not be too happy with >5V being delivered at USB Out
I think you should change:
if (channel == MIDIchannel
&& control == servo1)
{
value = map(value, 0, 127, 0, 179);
value = pos1;
myservo.write(pos1); // sets the servo position according to the scaled value...