This series makes minor clean-up changes to the i.MX DDR init code.

Unfortunately, I don't have access to any NXP hardware. I ran into LTO
issues with my Fedora toolchain building imx8mm_evk_defconfig. And
binman issues because of missing binary blobs with some imx9*
defconfigs. But compilation was successful.

To support an upcoming ADI SoC that also uses LPDDR4 and Synpsys IP I
would like to leverage the i.MX implementation. Therefore this clean-up
can also serve as a kind of pre-RFC. I would love to know if the
maintainers are open to transitioning to a more generic implementation.
If so, I'll submit a proper RFC. I'll also be at Embedded World next
week if anyone would like to discuss in person.

Some additional context and a comparison of DDR init implementations:

https://philipmolloy.com/ddr-init.html

I reviewed the NXP i.MX downstream repo on Github.[1] The changes in
this series are not fixed downstream. The only major difference with
downtream for the PHY driver appears to be "Quickboot" support, which is
specific to the i.MX 95.

[1]: https://github.com/nxp-imx/uboot-imx

Philip Molloy (4):
  ddr: imx: Remove duplicate includes
  ddr: imx: Remove undefined declarations
  ddr: imx: Remove unused macros
  ddr: imx: Remove unnecessary macro argument

 arch/arm/include/asm/arch-imx8m/ddr.h | 10 ++++------
 arch/arm/include/asm/arch-imx9/ddr.h  |  8 ++------
 drivers/ddr/imx/imx8m/ddr_init.c      |  8 ++++----
 drivers/ddr/imx/phy/ddrphy_utils.c    | 15 +++++++--------
 drivers/ddr/imx/phy/helper.c          |  4 +---
 5 files changed, 18 insertions(+), 27 deletions(-)

-- 
2.53.0

base-commit: 7995bf8dea2d5b3eb7fcb836636f4773924ec35d
branch: ddr-imx-clean-up

Reply via email to