Welcome to our site! Electro Tech is an online community (with over 170,000 members) who enjoy talking about and building electronic circuits, projects and gadgets. To participate you need to register. Registration is free. Click here to register now.
The pcb looks ok.
How about some text on the screen to tell you who's playing, 'p1 playing' 'p2 playing' or something like that.
asm is a good thing to know, higher level languages such as C are quicker to develop, but slower run time.
Indeed a nice looking project.
Eventually though, you should start looking at microcontrollers which incorporate the memory and peripherals within the chip itself.
Plus, there will be support for higher level languages that will make coding so much easier, that you'll be able to incorporate new features easily.