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 B8373C433F5 for ; Wed, 27 Oct 2021 14:09:25 +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 1165660EC0 for ; Wed, 27 Oct 2021 14:09:24 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 1165660EC0 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=chromium.org 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 E8C52835BA; Wed, 27 Oct 2021 16:09:22 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=chromium.org 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=chromium.org header.i=@chromium.org header.b="bNd0R2Wm"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id F14BB83542; Wed, 27 Oct 2021 16:09:20 +0200 (CEST) Received: from mail-ua1-x92f.google.com (mail-ua1-x92f.google.com [IPv6:2607:f8b0:4864:20::92f]) (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 4C832835C6 for ; Wed, 27 Oct 2021 16:09:17 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=sjg@google.com Received: by mail-ua1-x92f.google.com with SMTP id z22so2812853uaq.12 for ; Wed, 27 Oct 2021 07:09:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=aIWAdKCdxxxXu89ZSt0Wh1/MxeiD2ItogMHkW8sxyN8=; b=bNd0R2WmUmP9TI0VVikoQOHEXGQPl5KGeSGdXn+lL9cO1Lz9tWlDi9uxiHDkHWzOwo X7+Oqppum0zOQ4Qx4nvmGuqPK3S8xmqe7T5c3klXP1z4ob5G2uxY48+3/Fiosd6tfRgY DRNEKFCI9Oz5Z3nk7VSg7oOetiN+Abct9u/hY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=aIWAdKCdxxxXu89ZSt0Wh1/MxeiD2ItogMHkW8sxyN8=; b=yBKoOjxy6Mdlt9C2tzo7Q2rh6+L7eO2i2XbWAjXzt768gr099jRNUaE5hJeewdDDJN hOojLOcACzLIoVYPlLgkay7628bF2/NhPVW6uBQRaE8EgTx1rfC72cdro6GXNy8WTOn7 H7mPCJTSqLPKcg8kecNvDm0sejUWK7T43m8WGOfM0UWlMKfBR5Ne1ZLD3jj4X/JKfutC gmjGtQV9y28H5SglON/w+GRZ73oVy4tU78vBcVPsFbj3FPNmHuTu3hji5U0cQAjfAXpi lCegHKr1Kosq6IlgNm7AgFh/DJJ49TOMeUI2MzFw6OCqahRCfmnvSQ1sddKDalF/G0+4 G1/Q== X-Gm-Message-State: AOAM5300QpU9xiZfttyOm4blbq8C/sikkXnik8/TdfhNfjqa+/krwX1F Yk9cva3PyT6un3i0LrORlaBH4H92nviE5NyfmnUjPg== X-Google-Smtp-Source: ABdhPJzakt7/Z1VGHcLTr/UrdLDXwNYFWgUHP5fW4t/36tUsK1pZa4aB9lnHLsJdPxY7aYizZGxgR9X9L9h2tAgQ6U0= X-Received: by 2002:a67:d51c:: with SMTP id l28mr19812313vsj.58.1635343755860; Wed, 27 Oct 2021 07:09:15 -0700 (PDT) MIME-Version: 1.0 References: <20211023232635.9195-1-sjg@chromium.org> <20211023232635.9195-22-sjg@chromium.org> In-Reply-To: From: Simon Glass Date: Wed, 27 Oct 2021 08:09:04 -0600 Message-ID: Subject: Re: [PATCH v2 31/41] bootstd: Add an implementation of EFI boot To: Ilias Apalodimas Cc: U-Boot Mailing List , Michal Simek , Heinrich Schuchardt , Tom Rini , Daniel Schwierzeck , Steffen Jaeckel , =?UTF-8?B?TWFyZWsgQmVow7pu?= , Lukas Auer , Dennis Gilmore Content-Type: text/plain; charset="UTF-8" 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 Hi Ilias, On Wed, 27 Oct 2021 at 02:36, Ilias Apalodimas wrote: > > Hi Simon, > > On Sun, 24 Oct 2021 at 02:27, Simon Glass wrote: > > > > Add a bootmeth driver which handles EFI boot, using EFI_LOADER. > > > > In effect, this provides the same functionality as the 'bootefi' command > > and shares the same code. But the interface into it is via a bootmeth, > > so it does not require any special scripts, etc. > > > > For now this requires the 'bootefi' command be enabled. Future work may > > tidy this up so that it can be used without CONFIG_CMDLINE being enabled. > > I'll leave this up to Heinrich, but personally I wouldn't include this > patch at all. EFI has it's bootmgr which can handle booting just fine. > I don't see why we should duplicate the functionality. The new boot > method can just have an entry called 'EFI' and then let the existing > EFI code to decide. This is needed so that EFI boot is actually invoked. If bootmgr starts being used then it can still be invoked from standard boot. The point is that there is a standard way of booting that supports EFI and other things. This series is about replacing the scripts we currently have with a proper C implementation that uses driver model. > Regards > /Ilias > [..] Regards, Simon