This is very well written and the code looks very clean! It's a bit verbose, but mostly because he makes everything parametrized so you can easily customize it

This is very well written and the code looks very clean! It's a bit verbose, but mostly because he makes everything parametrized so you can easily customize it for different targets. There are only two things I don't like: he is calling some registers latches and he is mixing std_logic_vector and unsigned in the interface.