Newbie would like his DIY Arduino schematic checked

X1 and X2 look like the same connector. Make them different. Otherwise you are going to plug the wrong cable into one of them at some stage. At least make one male and one female.