[u-boot][master][PATCH 1/3] pico-imx7d: add support for 2GB memory SoMs

[Szőke Kálmán Benjamin]

Original code was a nested if statement for checking 1g() and 2g().https://github.com/TechNexion/u-boot-tn-imx/blob/tn-imx_v2021.04_5.10.72_2.2.0-stable/board/technexion/pico-imx7d/pico-imx7d_spl.c#L106Do you have any information about the PCB layout of the different i.MX7 pico SoM revisions?Question is, what is the electrical states of GPIO1_12 and GPIO1_13 in real for these variants? My fear is that maybe one of them is not connected to any VDD (high) or GND (low). Question can be, what is the default logical state for it high or low if one of them is not connected neither to GND or VDD? I totaly do not understand why you need to completly refactoring the code if you have no posibbilites to test it with all revision of SoMs. I rather prefere to keep the original if statement code from Technexion, because that should works 100% and tested. I have no any 2GB varians SoMs yet, also i can not test it yet.So i am prefering more to apply my last (copy pasted from Technexion) patch, to support 2GB SoMs in u-boot mainline.https://lists.denx.de/pipermail/u-boot/2022-December/502465.html-------- Eredeti levél --------Feladó: Fabio Estevam <festevam at gmail.com>Dátum: 2022 december 31 17:31:01Tárgy: Re: [u-boot][master][PATCH 1/3] pico-imx7d: add support for 2GB memory SoMsCímzett: egyszeregy at freemail.huHi Benjamin, On Sun, Dec 18, 2022 at 9:52 AM <egyszeregy at freemail.hu> wrote: > > From: Benjamin Szőke <egyszeregy at freemail.hu> > > Take over codes from Techenxion to support SoMs with 2GB DDR3. > > Signed-off-by: Benjamin Szőke <egyszeregy at freemail.hu> > --- >  board/technexion/pico-imx7d/Makefile          |  2 +- >  .../pico-imx7d/{spl.c => pico-imx7d_spl.c}    | 30 +++++++++++++++++-- >  2 files changed, 28 insertions(+), 4 deletions(-) >  rename board/technexion/pico-imx7d/{spl.c => pico-imx7d_spl.c} (83%) Could you please test whether the attached patch works for you? I don't have a 2GB board variant here to test it myself. You also need to apply the following patch to fix the boot against U-Boot top of tree: https://lore.kernel.org/u-boot/20221231162514.334303-1-festevam@denx.de/T/#u