[PATCH v3] ddr: marvell: a38x: Add support for DDR4 from Marvell mv-ddr-marvell repository
Tony Dinh
mibodhi at gmail.com
Fri Jan 27 05:58:10 CET 2023
Hi Pali,
On Thu, Jan 26, 2023 at 1:26 AM Stefan Roese <sr at denx.de> wrote:
>
> On 1/19/23 04:03, Tony Dinh wrote:
> > This syncs drivers/ddr/marvell/a38x/ with the master branch of repository
> > https://github.com/MarvellEmbeddedProcessors/mv-ddr-marvell.git
> >
> > up to the commit "mv_ddr: a3700: Use the right size for memset to not overflow"
> > d5acc10c287e40cc2feeb28710b92e45c93c702c
> >
> > This patch was created by following steps:
> >
> > 1. Replace all a38x files in U-Boot tree by files from upstream github
> > Marvell mv-ddr-marvell repository.
> >
> > 2. Run following command to omit portions not relevant for a38x, ddr3, and ddr4:
> >
> > files=drivers/ddr/marvell/a38x/*
> > unifdef -m -UMV_DDR -UMV_DDR_ATF -UCONFIG_APN806 \
> > -UCONFIG_MC_STATIC -UCONFIG_MC_STATIC_PRINT -UCONFIG_PHY_STATIC \
> > -UCONFIG_PHY_STATIC_PRINT -UCONFIG_CUSTOMER_BOARD_SUPPORT \
> > -UCONFIG_A3700 -UA3900 -UA80X0 -UA70X0 -DCONFIG_ARMADA_38X -UCONFIG_ARMADA_39X \
> > -UCONFIG_64BIT $files
> >
> > 3. Manually change license to SPDX-License-Identifier
> > (upstream license in upstream github repository contains long license
> > texts and U-Boot is using just SPDX-License-Identifier.
> >
> > After applying this patch, a38x, ddr3, and ddr4 code in upstream Marvell github
> > repository and in U-Boot would be fully identical. So in future applying
> > above steps could be used to sync code again.
> >
> > The only change in this patch are:
> > 1. Some fixes with include files.
> > 2. Some function return and basic type defines changes in
> > mv_ddr_plat.c (to correct Marvell bug).
> > 3. Remove of dead code in newly copied files (as a result of the
> > filter script stripping out everything other than a38x, dd3, and ddr4).
> >
> > Reference:
> > "ddr: marvell: a38x: Sync code with Marvell mv-ddr-marvell repository"
> > https://source.denx.de/u-boot/u-boot/-/commit/107c3391b95bcc2ba09a876da4fa0c31b6c1e460
> >
> > Signed-off-by: Tony Dinh <mibodhi at gmail.com>
>
> Applied to u-boot-marvell/master
>
> Thanks,
> Stefan
Looking at the history of the work we've done to sync u-boot code back
to the mv-ddr-marvell repo, I think I should follow the same approach.
So I've pushed 2 commits to my GitHub repo.
https://github.com/MarvellEmbeddedProcessors/mv-ddr-marvell/compare/master...mibodhi:mv-ddr-marvell:u-boot-ddr4-2023.01-fix
Please review these 2 commits and create a pull request to Marvell,
like you did for others last year during DDR3 code sync.
Thanks,
Tony
More information about the U-Boot
mailing list