Está bien si estuviéramos en 2010. Como ya te dijeron, HTML5 con etiquetas semánticas. Si quieres venderlo utiliza ids y clases en inglés. No deberías meter código PHP en una template, solo limitarías la plantilla, si alguien quiere tomarla y dividirla con includes será su trabajo. El menú del footer debería ser una lista (o un nav en HTML5). Jamás uses
align="center"
, para eso está css lo mismo para las propiedades del body que pusiste. El contenido debería estar en párrafos no directamente en el div.
Saludos