Re: [PATCH v2] config: imx: Define CFG_SYS_UBOOT_BASE for Phytec's imx8mm SoM QSPI boot

[Lukasz Majewski <lukma@denx.de>]

[-- Attachment #1: Type: text/plain, Size: 2736 bytes --]

Hi Benjamin,

> Hi Lukasz,
> 
> On 28.08.24 09:55, Lukasz Majewski wrote:
> > Hi Fabio,
> >
> >> Hi Lukasz,
> >>
> >> On Tue, Aug 20, 2024 at 12:00 PM Lukasz Majewski <lukma@denx.de>
> >> wrote:
> >>> The image offset when booting from SPI-NOR (QSPI, FSPI driver) is
> >>> different than one for SD card / eMMC as extra space (0x1000) is
> >>> consumed by FSPI configuration header (CONFIG_FSPI_CONF_HEADER).
> >>>
> >>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> >>> ---
> >>> Changes for v2:
> >>> - Avoid line breaks with defines
> >>  From the other thread, my understanding is that QSPI boot is
> >> still not working with this patch applied.
> >>
> > But this is an orthogonal problem - the problem is with binman image
> > generation.
> >
> > This patch follows the code already present in e.g. imx8mm_beacon*
> > boards.
> >
> >> Please resend this patch when the QSPI boot problem is resolved.
> >>
> > As I've written above - problem is solely with binman code
> > generation - after reverting 3 last commits from imx8mm-u-boot.dtsi
> > the image is correctly generated.
> 
> I can also see this. When I apply all your patches and revert those 3 
> commits, SDP boot via uuu and flashing SPI NOR flash with the qspi
> Image works. And then also SPI boot works.
> When I don't revert the 3 commit, it does not work. (SDP boot via uuu 
> does not work and the Image does not boot when flashed into SPI NOR
> flash).
> 
> I haven't investigated what the problem is or how it can be fixed 
> though. 

The "Hackish" solution is to add:

@@ -85,6 +85,10 @@ void ddr_load_train_firmware(enum fw_type type)
                }
        }
 
+       imem_start -= 0x1000;
+       dmem_start -= 0x1000;
+
        pr_from32 = imem_start;
        pr_to32 = IMEM_OFFSET_ADDR;
        for (i = 0x0; i < imem_len; ) {

(The -= 0x1000 is the size of QSPI header added to the beginning of the
image.)

Simon, has recently prepared set of binman related patches, so maybe
they provided offset to fix it.

> So I guess further investigation is needed here.
> 
> Benjamin
> 
> >
> >> Thanks
> >
> >
> >
> > Best regards,
> >
> > Lukasz Majewski
> >
> > --
> >
> > DENX Software Engineering GmbH,      Managing Director: Erika Unter
> > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> > lukma@denx.de
> 
> 




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]