That all looks correct as far as I can tell. The only thing I cant honestly recall is the SBUS input from the X8R. It has been a good year or so since I have looked at a skyline32 FC, so cant remember if you need to do anything speciail with SBUS on this board (as some of the older FC you did need to).
But worst case if you cant get SBUS working, just use CPPM from your X8R and it will all be perfect
Also what did you use to make your wiring diagram as it is very nice and neat