pandocs icon indicating copy to clipboard operation
pandocs copied to clipboard

Instruction timings are listed in "clock cycles" — should they be named T-states?

Open aaaaaa123456789 opened this issue 2 years ago • 2 comments

The instruction table's header names the column listing the execution time for each instruction "clock cycles". This is an ambiguous term, as it could mean M or T cycles. While there is a description at the top, the term remains confusing, particularly for someone looking at the documentation quickly; it is also at odds with most community usage, where "cycles" means M-cycles.

Would it be better to label the column "T-states" explicitly instead? Another option would be to quarter the values, but I assume this wasn't done for some reason I'm unaware of.

aaaaaa123456789 avatar Aug 07 '23 14:08 aaaaaa123456789

I'd vote for "clocks", since it seems rather well understood that "clocks" are 4 MiHz, but "cycles" are 1 MiHz. Of course, restating both equivalences is a must.

ISSOtm avatar Aug 08 '23 15:08 ISSOtm

While it's not the first time I hear of that convention, I've never heard it from anyone but you. Are you sure it's as well understood as you say?

aaaaaa123456789 avatar Aug 08 '23 15:08 aaaaaa123456789