The comment you're referring to is actually in the testbench[1], where these values are overidden because it would be pretty boring to have to wait for 4800 cycles of simulation time before the memory controller comes out of its initialisation phase. The default values in the RTL[2] are correct for the MT48LC8M16A2 SDRAM.
Whether this memory-controller is appropriate for you depends largely on your needs. I actually wrote this memory-controller originally as part of the UMDKv2 project[3], for which the main constraint was latency, not throughput. So for this reason it exposes signals allowing manual control over the scheduling of refresh cycles, but it does only one 16-bit read or write at a time, taking four 48MHz cycles for each, giving a raw throughput of a little less than 24MB/s (one read/write takes four clocks, but transfers two bytes, so the throughput is half the 48MHz clock speed).
If that throughput is acceptable to you, then this controller should work fine for you. If you need more throughput, you could easily modify my controller to do longer (but still fixed-length) bursts. I'd be happy to do that kind of work for you under contract, if you'd rather not to do it yourself.
Chris