From: Benjamin Hahn <B.Hahn@phytec.de>
To: Lukasz Majewski <lukma@denx.de>, Fabio Estevam <festevam@gmail.com>
Cc: Teresa Remmet <T.Remmet@phytec.de>,
Stefano Babic <sbabic@denx.de>,
"uboot-imx@nxp.com" <uboot-imx@nxp.com>,
Cem Tenruh <C.Tenruh@phytec.de>, "Tom Rini" <trini@konsulko.com>,
"u-boot@lists.denx.de" <u-boot@lists.denx.de>,
Yannic Moog <Y.Moog@phytec.de>
Subject: Re: [PATCH v2] config: imx: Define CFG_SYS_UBOOT_BASE for Phytec's imx8mm SoM QSPI boot
Date: Fri, 30 Aug 2024 13:23:42 +0000 [thread overview]
Message-ID: <dbf9bf5e-b96b-4e34-816c-215e68289526@phytec.de> (raw)
In-Reply-To: <20240828095520.76f904bc@wsk>
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. 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
next prev parent reply other threads:[~2024-08-30 13:23 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-20 14:59 [PATCH v2] config: imx: Define CFG_SYS_UBOOT_BASE for Phytec's imx8mm SoM QSPI boot Lukasz Majewski
2024-08-27 18:49 ` Fabio Estevam
2024-08-28 7:55 ` Lukasz Majewski
2024-08-30 13:23 ` Benjamin Hahn [this message]
2024-08-30 15:19 ` Lukasz Majewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=dbf9bf5e-b96b-4e34-816c-215e68289526@phytec.de \
--to=b.hahn@phytec.de \
--cc=C.Tenruh@phytec.de \
--cc=T.Remmet@phytec.de \
--cc=Y.Moog@phytec.de \
--cc=festevam@gmail.com \
--cc=lukma@denx.de \
--cc=sbabic@denx.de \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=uboot-imx@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.