public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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 --]

      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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox