Re: [PATCH v2 6/8] board: tqma7: add code for u-boot with spl

[Tom Rini <trini@konsulko.com>]

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

On Wed, Mar 25, 2026 at 06:50:59PM +0100, Feilke, Alexander wrote:

> Hi,
> 
> Please wait with applying the series. I just stumbled over a weird issue
> related to the environment.
> 
> The default rootpath is set to `rootpath="/srv/nfs"` (with quotes).
> This is wrong. Running `env default rootpath` will reset it correctly
> to rootpath=/srv/nfs
> However, `env default -fa` resets it back to `rootpath="/srv/nfs"`
> 
> Independently of that, the variables inside load_nfs are not being replaced
> correctly. They appear to be empty, as I get `Filename '/'.` in the run log
> when it should be something like `Filename '/srv/nfs/boot/imx7d-mba7.dtb'`
> 
> We will fix this before the patch should be accepted. The first issue seems
> pretty weird tho. Any idea what might be causing this?

Using plain text environment can be funny about quotes, especially when
dealing with values that come from another CONFIG option and so come
from Kconfig. One thing to look at, at least while debugging, is that
"make u-boot-initial-env" will give you the resulting environment in
u-boot-initial-env so you can check the output quicker than flashing
cycles. If there's a quoting issue, commit b3087d21f022 ("env: Provide
another work-around for unquoting fdtfile") may get you pointed in the
right direction for a solution at least.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]