From: Tom Rini <trini@konsulko.com>
To: Paul Barker <paul.barker@sancloud.com>
Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>,
u-boot@lists.denx.de, Heinrich Schuchardt <xypron.glpk@gmx.de>
Subject: Re: [PATCH v4 3/6] efi_loader: Add SPI I/O protocol support
Date: Tue, 25 Oct 2022 13:26:03 -0400 [thread overview]
Message-ID: <20221025172603.GP2020586@bill-the-cat> (raw)
In-Reply-To: <6becf071-b6da-8487-f2ec-329ceb19568a@sancloud.com>
[-- Attachment #1: Type: text/plain, Size: 654 bytes --]
On Tue, Oct 25, 2022 at 05:39:48PM +0100, Paul Barker wrote:
> Hi Ilias,
>
> On 24/10/2022 12:54, Ilias Apalodimas wrote:
> > Hi Paul,
> >
> > I think the series overall is in a good state, but we are trying to figure
> > out if we can avoid defining SPI uuid's in the DT. OTOH U-Boot uses the DT
> > for it's config so that might be okay...
>
> What may make sense is to prefix the DT property names with `u-boot,`.
> i.e. they'd become:
>
> * u-boot,uefi-spi-vendor
> * u-boot,uefi-spi-part-number
> * u-boot,uefi-spi-io-guid
Note that this still means updating the upstream yaml file that
describes the binding.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2022-10-25 17:26 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-05 12:18 [PATCH v4 0/6] Support UEFI SPI I/O protocol Paul Barker
2022-10-05 12:18 ` [PATCH v4 1/6] spi: Implement spi_set_speed Paul Barker
2022-10-06 3:15 ` Heinrich Schuchardt
2022-10-05 12:18 ` [PATCH v4 2/6] efi: Add string conversion helper Paul Barker
2022-10-06 3:26 ` Heinrich Schuchardt
2022-10-06 6:19 ` Ilias Apalodimas
2022-10-05 12:18 ` [PATCH v4 3/6] efi_loader: Add SPI I/O protocol support Paul Barker
2022-10-24 11:54 ` Ilias Apalodimas
2022-10-25 16:39 ` Paul Barker
2022-10-25 17:26 ` Tom Rini [this message]
2022-10-05 12:18 ` [PATCH v4 4/6] efi_selftest: Add tests for SPI " Paul Barker
2022-10-05 12:18 ` [PATCH v4 5/6] arm: dts: am335x-sancloud-bbe-lite: UEFI SPI export Paul Barker
2022-10-06 4:12 ` [U-Boot PATCH " Heinrich Schuchardt
2022-10-05 12:18 ` [PATCH v4 6/6] am335x_evm_defconfig: Enable Micron SPI flash support Paul Barker
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=20221025172603.GP2020586@bill-the-cat \
--to=trini@konsulko.com \
--cc=ilias.apalodimas@linaro.org \
--cc=paul.barker@sancloud.com \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.de \
/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.