U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Stephan Gerhold <stephan.gerhold@linaro.org>
To: Linus Walleij <linusw@kernel.org>
Cc: u-boot@lists.u-boot-project.org, Tom Rini <trini@konsulko.com>,
	Stefan Hansson <newbyte@postmarketos.org>
Subject: Re: [PATCH v2 7/8] arm: u8500: Switch Stemmy to upstream Janice device tree
Date: Tue, 18 Aug 2026 12:39:16 +0200	[thread overview]
Message-ID: <aoQ2VMH_unoQaQxO@linaro.org> (raw)
In-Reply-To: <CAD++jLnTaaN9ektt9h5S_jYZ27sezSfuymmjTGiL1_j8NSw6bw@mail.gmail.com>

On Mon, Aug 17, 2026 at 10:54:51PM +0200, Linus Walleij wrote:
> On Mon, Aug 17, 2026 at 6:45 PM Stephan Gerhold
> <stephan.gerhold@linaro.org> wrote:
> 
> > Hm okay so you're basically dropping support for the internal eMMC in
> > this patch? Can we support both? BOOTSTD should be able to scan multiple
> > storages for boot targets.
> 
> Yeah just laziness on my part, I'll fix it up.
> 
> > I'm also not really sure about selecting Janice as "default" device tree
> > here. Do you expect users to change this setting when they want to build
> > stemmy for another board?
> 
> I have a script that loops over the Samsung device trees,
> and recompiles U-Boot for each of them.
> https://dflund.se/~triad/krad/makefiles/ux500-uboot.mak
> 
> > I wonder if we could continue using a single generic binary for all
> > boards using CONFIG_MULTI_DTB_FIT. We should have "fw_mach" in
> > board/ste/stemmy/stemmy.c that should tell us which device we are
> > running on, which seems to be mostly(?) unique:
> >
> > https://github.com/stericsson-mainline/android_kernel/blob/9508895090960012f916561b2ede5eaf8124fabd/arch/arm/tools/mach-types#L3956-L3965
> >
> > I'm not sure about codina-tmo, you would need to check which number the
> > bootloader passes.
> >
> > If these numbers are unique, it should be easy to implement
> > board_fit_config_name_match().
> 
> I see the appeal of the ambition to have a "one size fits (pun!) all"
> U-Boot.
> 
> But as it is, some of the DTS attributes on the external SD card are
> different between the platforms, so we would need to modify the
> U-Boot internal Stemmy device tree for those, copying over yet
> properties from the device trees and into the boardfile.
> 
> For example Janice and Codina uses a regulator for
> vqmmc-supply, and those using the "sd_level_translator"
> for vqmmc-supply actually use different GPIO pins on say
> Gavini compared to Codina or Golden.
> 
> The pin direction settings on the MMC host vary. (Some of
> these were really tricky to get right.)
> 
> So I think no, if we want external SD card boot we need
> per-device compiled U-Boot binaries with the unique device
> trees.
> 

I wasn't suggesting to use the same DT for all devices. What I suggested
is to bundle all the upstream DTBs for the individual devices into a
single U-Boot binary (CONFIG_MULTI_DTB_FIT) and pick the correct one at
runtime. :)

That should be equivalent to your setup, just more convenient for the
user.

The only open question for me is if we can reliably detect codina-tmo
based on the mach-type ID, I can't find that in the kernel sources I
have.

Thanks,
Stephan

  reply	other threads:[~2026-08-18 10:39 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-17  9:31 [PATCH v2 0/8] arm: u8500: Enable upstream DT based SD card boot Linus Walleij
2026-08-17  9:31 ` [PATCH v2 1/8] pinctrl: Add compact Nomadik pin controller Linus Walleij
2026-08-17 15:57   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 2/8] mmc: arm_pl180: Configure Ux500 signal direction Linus Walleij
2026-08-17 15:59   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 3/8] mmc: arm_pl180: Set initial supply voltages Linus Walleij
2026-08-17 16:15   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 4/8] power: regulator: Add AB8500 AUX3 support Linus Walleij
2026-08-17 16:21   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 5/8] configs: stemmy: Enable SD card regulators Linus Walleij
2026-08-17 16:24   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 6/8] configs: stemmy: Boot EFI from external SD card Linus Walleij
2026-08-17 16:29   ` Stephan Gerhold
2026-08-17  9:31 ` [PATCH v2 7/8] arm: u8500: Switch Stemmy to upstream Janice device tree Linus Walleij
2026-08-17 16:45   ` Stephan Gerhold
2026-08-17 20:54     ` Linus Walleij
2026-08-18 10:39       ` Stephan Gerhold [this message]
2026-08-18 13:31         ` Linus Walleij
2026-08-19  8:53         ` Linus Walleij
2026-08-17  9:31 ` [PATCH v2 8/8] arm: u8500: Give Golden panel regulators unique names Linus Walleij
2026-08-17 16:47   ` Stephan Gerhold
2026-08-17 20:36     ` Linus Walleij
2026-08-17 21:14       ` Tom Rini

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=aoQ2VMH_unoQaQxO@linaro.org \
    --to=stephan.gerhold@linaro.org \
    --cc=linusw@kernel.org \
    --cc=newbyte@postmarketos.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.u-boot-project.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