sdram_control is now (for the most part) in Opencores coding standards. I can't say I like all of it (the _I and _O seems excessive) but it does clarify things a little. Of course the top level port map needs to be fixed, so the design won't compile at the moment. Check it out if you want to see what the guidelines change Jeff