Just quickly looked it over, and noticed a few things:
Your pull ups for the I2C bus seems pretty low, I think you could get away with something higher unless you really have a lot of devices on the bus.
You LEDS are also really brightly lit with 10mA, If they have to be seen from far away this might be intended. But if it's just for debugging/development I think they will be to bright, might become annoying.
Your buck converter uses 1uH make sure this gives you a low enough ripple for your calculations.
Lastly why not use the split termination available on your CAN transceivers?