Skip to content

driver: ddramc: fix JEDEC timing#199

Open
sonicpp wants to merge 1 commit into
linux4sam:masterfrom
sonicpp:jedec-fixes
Open

driver: ddramc: fix JEDEC timing#199
sonicpp wants to merge 1 commit into
linux4sam:masterfrom
sonicpp:jedec-fixes

Conversation

@sonicpp
Copy link
Copy Markdown

@sonicpp sonicpp commented May 29, 2026

tWTR is defined in ns unit, so it has to be converted to tCK, while tXSRD, on the other hand, is already in tCK.

This can be seen for example when the DDR timing is defined as

CONFIG_DDR_SET_BY_DEVICE=y
CONFIG_DDR_W971GG6SB=y

then the values are correct:
tWTR=2
tXSRD=200

But when setting following option

CONFIG_DDR_SET_BY_JEDEC=y

then the values are wrong:
tWTR=8
tXSRD=40
while these values should be same as above as per JEDEC specs.

tWTR is defined in ns unit, so it has to be converted to tCK.
tXSRD, on the other hand, is already in tCK.

Signed-off-by: Jan Havran <jan.havran@advantech.cz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant