Whats wrong with my faces
If you just select the whole mesh and press F, Blender won't be able to know what's supposed to be holes and what's not, and it will fill everything. So create some additional vertices and fill the faces one by one with some F, except what's supposed to be holes: