If you've decided what crystal you are using, you can delete 2 of Q1, Q2, Q3.
Check the pinouts on J1 - looks like its set up to accept an FTDI Basic or similar - make sure Tx on whatever that device is goes to Rx on the '328.
The '328 you have shown is the leadless part. You will need to bring its Reset, SCK, MOSI, MISO, +5, GND out to a 6 pin header so you can download a bootloader (or a sketch with no bootloaded) into it.
You will not be using a crystal on the '328? I think you will have better results with high speed communications (such as SPI) if you use a 16 MHz crystal/22 pF caps with it. You can always not install it later, but it will be hard to add one if you don't have room for it at the start. You can use a small surface mount part if you want:http://www.digikey.com/product-detail/en/406C35D16M00000/CTX856TR-ND/2292780
I think there's an existing library symbol for this part.
You have several signals that end in LED - where these intended to drive LEDs that show status or something?
When you think you're all set, run the eagle Design check (bottom left button on the left hand menu bar) to help identify things that look connected but may not be.