Timings - am I overthinking this?

So, that means that reading using "(1) set address, clock high, (2) read data, clock low, (1) set address, etc" is OK? (unless I get told otherwise)

(It's only a 50MHz clock running this cycle so well within the internal RAM's capability)

I was hoping that was the case, and it certainly makes things a lot simpler :-)

(For writing to the memory, I tend to do (1)set address/data, clock low, (2) clock high, (1) set address/data, clock low, so setup/hold weren't worrying me there)