search

MarvellEmbeddedProcessors / mv-ddr-marvell

20 stars 23 forks source link

a3700: change Channel 0 PHY Control 2 value for DDR4 1CS #44

Closed bschnei closed 5 months ago

bschnei commented 8 months ago

I have an ESPRESSObin Ultra (A3720 DDR4 1CS 1GB) that crashes reliably when using a bootloader built from the current master branch of this repo. This is reproducible across all versions of TF-A and U-Boot.

Changing this setting back to the original value it had prior to being modified by this commit results in a stable system. There was no commit message associated with that change so the motivation for changing it is unclear. The distributor of the ESPRESSObin Ultra (Globalscale) also overrides this value in their repos.

I have been running the patched bootloader on two different devices used as home office routers without any issues for over a week. I'm submitting this change to help other users who also try to update their bootloader using TF-A directions.

bschnei commented 6 months ago

@jiam-marvell would you be willing to review and/or comment on this? I'm still using the patched bootloader on two devices and they remain stable.