From: Jochen Sprickerhof <jspricke@debian.org>
To: Vagrant Cascadian <vagrant@debian.org>
Cc: 1012269@bugs.debian.org, u-boot@lists.denx.de
Subject: Re: Bug#1012269: u-boot-exynos on Odroid XU4 searches for exynos5422-smdk5420xu4.dtb instead of exynos5422-odroidxu4.dtb
Date: Thu, 29 Dec 2022 08:06:54 +0100 [thread overview]
Message-ID: <Y608jpEXWezefpmF@vis.fritz.box> (raw)
In-Reply-To: <87h6xfhsto.fsf@yucca>
[-- Attachment #1: Type: text/plain, Size: 1418 bytes --]
* Vagrant Cascadian <vagrant@debian.org> [2022-12-28 18:00]:
>Hrm. This suggests the fix might not have been effective, which should
>have been included in 2022.07... CCing upstream...
>
>commit e744bf3a4ba442a0e9ee1c509c70e1452e3a15d0
>Author: Tom Rini <trini@konsulko.com>
>Date: Wed Jun 8 14:30:14 2022 -0400
>
> odroid_xu3: Fix board environment variable
>
> When migrating CONFIG_CONS_INDEX to Kconfig, on this platform we changed
> what "board" evaluated to in the environment. This in turn meant that
> we would no longer try and find the correct fdtfile via the normal
> distro boot logic. Fix this by overriding board in the default
> environment, as done on other platforms where CONFIG_SYS_BOARD is not
> what we want to be in the board environment variable.
>
> Fixes: f76750d11133 ("Convert CONFIG_CONS_INDEX et al to Kconfig")
> Reported-by: Gabriel Hojda <ghojda@yo2urs.ro>
> Tested-by: Gabriel Hojda <ghojda@yo2urs.ro>
> Signed-off-by: Tom Rini <trini@konsulko.com>
>
>
>Alternately... did you at any point run saveenv? This might have stored
>the incorrect value... and then it might not probe for the correct
>value.
I'm sure I never did that. Out of interest, where would it save the
environment? Afaik there is no persistent memory on the Odroid XU4
besids the emmc/sd card and my system uses a simple extlinux.conf
generated by u-boot-menu.
Cheers Jochen
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2022-12-29 12:57 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <165418392817.1809.3917910993952958001.reportbug@vis.fritz.box>
[not found] ` <87o7tmct4t.fsf@contorta>
[not found] ` <Y6iajMdDH7YC3SNE@vis.fritz.box>
2022-12-29 2:00 ` Bug#1012269: u-boot-exynos on Odroid XU4 searches for exynos5422-smdk5420xu4.dtb instead of exynos5422-odroidxu4.dtb Vagrant Cascadian
2022-12-29 7:06 ` Jochen Sprickerhof [this message]
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=Y608jpEXWezefpmF@vis.fritz.box \
--to=jspricke@debian.org \
--cc=1012269@bugs.debian.org \
--cc=u-boot@lists.denx.de \
--cc=vagrant@debian.org \
/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.