I think what CJ was implying is you spent to much $$$ on the ram. Slower, and hence cheaper ram would have saved you money. There is little reason to buy DDR3 ram with a C2D/C2Q setup, the FSB is simply to slow. Cheaper DDR2 could have done what your doing.
Overall it looks ok to me, other then you spent to much.