From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 515F4C433EF for ; Thu, 28 Oct 2021 17:59:19 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B91B1610D2 for ; Thu, 28 Oct 2021 17:59:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org B91B1610D2 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 77EC9832C6; Thu, 28 Oct 2021 19:59:16 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=konsulko.com header.i=@konsulko.com header.b="DGSE237C"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id AE78682F33; Thu, 28 Oct 2021 19:59:14 +0200 (CEST) Received: from mail-qv1-xf35.google.com (mail-qv1-xf35.google.com [IPv6:2607:f8b0:4864:20::f35]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 23B74832C6 for ; Thu, 28 Oct 2021 19:59:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-qv1-xf35.google.com with SMTP id d6so4621491qvb.3 for ; Thu, 28 Oct 2021 10:59:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=Ua4NATgaS7Odb+JRTX+P8FLAEWMcNkpnkjRo9bHQ730=; b=DGSE237CUYuz9XcP/9MVjMQgThCtDLmKi0RXexyEuS4S7suaGGY6pXQT/GEHf61ES6 lmsmDwWr6toxLxUFEPklI+TL1WiTyVF5Uj9NSM4QxuqhgD5yqQITlAGSRogJRWSlrXj5 KdJjqZd/D0AyU14Fyfv7ZKq/9Ntk2qJJ78UBA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=Ua4NATgaS7Odb+JRTX+P8FLAEWMcNkpnkjRo9bHQ730=; b=kYYnpVM6zGaPDbMFnHqeXPANR0XLQjN88pYZXCpRAHRMsIA8Uh+9V5M/macEL34jR7 3LVhaEUk5LxZ6b1O0uCgX9DRJMv+8XBvOEqeXI05S3k1X1O1DfDqDSMSZUfxCTKhwdmU B70H5pv5HKkK8AktdwboAyhOkcKJWxGomJ7uSOjY6BicEnYaFnfKOXsaKDL4CpQ3A3vp 3B9qLiUUONy5eWQMQbMv8SBYkQiLQ9DUttbepiHDUghqT1WfarVAUbumYs2yReB+aEtx JdGpN6D2BEHhaP3CxatKbu0ksYM/nrS8XOL8Yuhw+lV026NfiYbFFtvysbfbajoeSboA dc4A== X-Gm-Message-State: AOAM5308kSGbz6nC159vpAlp5wSA7fr+0Bt47cmBve+gmZ8VvvcgJA0Y Z3DeU6nCJ0Tf+kYxql6Fga5jjg== X-Google-Smtp-Source: ABdhPJwuL0MgqjyWnmS3IGmOxwwsd39Y6oQgJRLvAlK3XB5abi0d2GZQ9B1MtA9okPyz+dIynGafZA== X-Received: by 2002:a0c:ed26:: with SMTP id u6mr5649930qvq.58.1635443945742; Thu, 28 Oct 2021 10:59:05 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b01-cbda-015b-1704-43dd-8832.res6.spectrum.com. [2603:6081:7b01:cbda:15b:1704:43dd:8832]) by smtp.gmail.com with ESMTPSA id j15sm2746829qtx.67.2021.10.28.10.59.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Oct 2021 10:59:05 -0700 (PDT) Date: Thu, 28 Oct 2021 13:59:02 -0400 From: Tom Rini To: Peter Robinson Cc: Simon Glass , Heinrich Schuchardt , Michal Simek , Ilias Apalodimas , Daniel Schwierzeck , Steffen Jaeckel , Marek =?iso-8859-1?Q?Beh=FAn?= , Lukas Auer , Dennis Gilmore , Jaehoon Chung , Marek Vasut , Masahiro Yamada , Pavel Herrmann , Peng Fan , Stephen Warren , Stephen Warren , =?iso-8859-1?Q?Fran=E7ois?= Ozog , Heinrich Schuchardt , U-Boot Mailing List Subject: Re: [PATCH v2 00/41] Initial implementation of standard boot Message-ID: <20211028175902.GG8284@bill-the-cat> References: <20211023232635.9195-1-sjg@chromium.org> <3fde4b98-e7c0-71c3-d7c4-22c6f43eae31@canonical.com> <20211028174721.GE8284@bill-the-cat> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="4FfB8yLUWtffWRXD" Content-Disposition: inline In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean --4FfB8yLUWtffWRXD Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 28, 2021 at 06:50:02PM +0100, Peter Robinson wrote: > On Thu, Oct 28, 2021 at 6:47 PM Tom Rini wrote: > > > > On Thu, Oct 28, 2021 at 06:37:42PM +0100, Peter Robinson wrote: > > > On Wed, Oct 27, 2021 at 3:11 PM Simon Glass wrote: > > > > > > > > Hi Heinrich, > > > > > > > > On Wed, 27 Oct 2021 at 05:38, Heinrich Schuchardt > > > > wrote: > > > > > > > > > > > > > > > > > > > > On 10/24/21 01:25, Simon Glass wrote: > > > > > > > > > > > > The bootflow feature provide a built-in way for U-Boot to autom= atically > > > > > > boot an Operating System without custom scripting and other cus= tomisation. > > > > > > This is called 'standard boot' since it provides a standard way= for > > > > > > U-Boot to boot a distro, without scripting. > > > > > > > > > > > > It introduces the following concepts: > > > > > > > > > > > > - bootdev - a device which can hold a distro > > > > > > - bootmeth - a method to scan a bootdev to find bootflows (= owned by > > > > > > U-Boot) > > > > > > - bootflow - a description of how to boot (owned by the dis= tro) > > > > > > > > > > Please, describe why you are suggesting this change. > > > > > > > > > > Replacing a script by a devicetree chunk is just decreasing flexi= bility > > > > > and increasing complexity. Where is the benefit? > > > > > > > > > > I am missing a description here of where and how a boot flow shal= l be > > > > > defined. Describing some device-tree binding in patch 40/41 does = not > > > > > replace describing the development and usage workflow. Who will p= rovide > > > > > which bootflow information when? > > > > > > > > > > You still have an open discussion with Linaro about the source of > > > > > devicetrees. This discussion needs to be finalized before conside= ring > > > > > this series. > > > > > > > > > > In my view bootflows cannot be defined in the devicetree because = prior > > > > > firmware providing a devicetree is completely independent of any = distro > > > > > and therefore cannot provide a distro specific bootflow. > > > > > > > > There is actually no need to use devicetree here. I think you might > > > > have the wrong end of the stick. It is certainly possible to add no= des > > > > to configure things, if needed, but it works find without any chang= es > > > > to the devicetree, as you can see from the rpi_3 patch. > > > > > > > > There are several aims with this effort: > > > > > > > > - Provide a standard way to boot anything on U-Boot, that everyone = can > > > > plug into (distros, board vendors, people using a custom flow) > > > > > > I as a distro maintainer don't want this, we already get the "standard > > > way to boot" from UEFI, this just feels like another unnecessary > > > abstraction to that. > > > > Right. But part of the problem is "How do I find UEFI". Something > > somewhere needs to be configurable to say where to look, yes? >=20 > Is this question from the board PoV, the developer of U-Boot or the > distro trying to boot? >=20 > If you mean from a boot flow PoV for UEFI to find the HW that contains > the OS to boot I thought that was handled elsewhere in the series. What I mean is that today looking at Pi we have: #define BOOT_TARGET_DEVICES(func) \ BOOT_TARGET_MMC(func) \ BOOT_TARGET_USB(func) \ BOOT_TARGET_PXE(func) \ BOOT_TARGET_DHCP(func) As the board maintainer set that as the list of places to start looking for the next payload (say the GRUB EFI binary). Simon's series replaces that with I think he said "bootflow scan -b". And then the above env portion is replaced with, well, what's documented in patch #40 if you don't just want to rely on device probe order. Because we need to have _something_ that says where to look for what, yes? Or should that be replaced entirely with efi vars and BootOrder/BootXXX and "bootefi bootmgr $fdt_addr_r" ? --=20 Tom --4FfB8yLUWtffWRXD Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmF65OYACgkQFHw5/5Y0 tyxNkAv7B8qOUn+OoVyOse3iKdi14n2Kj/yioNjx8nhiZzDNKjVvjiKP1pxpEkg3 TFF72WwBd527pYhAQRlcjA7BKaANiuOIOz6QaQoeqSnb970O+HUFEzClOlk6S9pX 0tAKc1ceMB7B6GSJuLKT+mpPvGFQ9oXRZhiOP4QYm1W50eLXtGFfQvKMIWuii1xD C9l6YC+3NFP7VYArdFn/KoFkS04UI2KXRZn1wqZJ9hlwiXJi3+6jS95s0/bxwovi KZnfbKQ3psxL2rgT9o2Hv1M9uK+dbHf8zkPbsZWalspRhMDMsg3KZhsQxdm/d+BT tXADGTzLIGkNf/HlVGEWSGucaCoE6LZ1ixlvjcPi9Xbvuh0uwrzXObwrcjgKVx8N c07oKy0luO5NHCx6kerEZJonHCi5icQyJYYXrWkyRwiUm4yyEZmDdlj+Mr5HlmWB kWplDeFMKDmoaJ2PiZZY8CTigHIeWLzRIP00YkPug+MCgt9ZGmORCgqzQgxsmJ94 h9GbOUDr =D3xp -----END PGP SIGNATURE----- --4FfB8yLUWtffWRXD--