Hola el ejercicio está bastante bien. Te señalo las cosas a mejorar:
<div style="border: 1px solid black";">
le sobran unas comillas, las tienes duplicadas
el ; iría después de black y antes de las comillas, así: <div style="border: 1px solid black;">
<hr style="2px" blackground-color="black"" color="black">
No es blackground sino background y a un elemento hr no le puedes aplicar background-color
Has escrito "black"" y ahí te sobran unas comillas
En <p> Aprender a programar es un objeto que se plantea mucha gente y que no todos alcanzan.
<br/>
No debes poner elementos <br/> dentro de <p>. Si quieres crear un salto de línea, cierra con </p> y vuelve a crear otro párrafo.
En los elementos img conviene poner alt y title
La estructura html está bastante bien, para mejorar la presentación (css):
http://aprenderaprogramar.com/index.php?option=com_content&view=category&id=75&Itemid=203Salu2!