Re: [PATCH v3 2/3] arm: dts: am335x-sancloud-bbe-lite: UEFI SPI export

From: Heinrich Schuchardt <xypron.glpk@gmx.de>
To: Tom Rini <trini@konsulko.com>, Paul Barker <paul.barker@sancloud.com>
Cc: u-boot@lists.denx.de,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Marek Vasut <marek.vasut@gmail.com>, Marek Vasut <marex@denx.de>
Subject: Re: [PATCH v3 2/3] arm: dts: am335x-sancloud-bbe-lite: UEFI SPI export
Date: Mon, 26 Sep 2022 15:33:44 +0200	[thread overview]
Message-ID: <1a71de0a-ff6f-4013-109c-ec4a30695785@gmx.de> (raw)
In-Reply-To: <20220926131349.GX3044094@bill-the-cat>

On 9/26/22 15:13, Tom Rini wrote:
> On Mon, Sep 26, 2022 at 02:33:23PM +0300, Ilias Apalodimas wrote:
>> Hi Tom
>>
>> On Wed, 21 Sept 2022 at 19:15, Tom Rini <trini@konsulko.com> wrote:
>>>
>>> On Wed, Sep 21, 2022 at 05:06:27PM +0100, Paul Barker wrote:
>>>> Add properties to the Authenta SPI flash device node to enable access by
>>>> a UEFI application using a fixed GUID. Also specify that this device is
>>>> JEDEC compatible so that it is correctly initialized when running
>>>> `sf probe`.
>>>>
>>>> Signed-off-by: Paul Barker <paul.barker@sancloud.com>
>>>> ---
>>>>   arch/arm/dts/am335x-sancloud-bbe-lite.dts | 10 ++++++++--
>>>>   1 file changed, 8 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/arch/arm/dts/am335x-sancloud-bbe-lite.dts b/arch/arm/dts/am335x-sancloud-bbe-lite.dts
>>>> index d6ef19311a91..f1ff9d6024cb 100644
>>>> --- a/arch/arm/dts/am335x-sancloud-bbe-lite.dts
>>>> +++ b/arch/arm/dts/am335x-sancloud-bbe-lite.dts
>>>> @@ -37,14 +37,20 @@
>>>>        pinctrl-names = "default";
>>>>        pinctrl-0 = <&bb_spi0_pins>;
>>>>
>>>> -     channel@0 {
>>>> +     authenta-flash@0 {
>>>>                #address-cells = <1>;
>>>>                #size-cells = <0>;
>>>>
>>>> -             compatible = "micron,spi-authenta";
>>>> +             compatible = "micron,spi-authenta", "jedec,spi-nor";
>>>>
>>>>                reg = <0>;
>>>>                spi-max-frequency = <16000000>;
>>>>                spi-cpha;
>>>> +
>>>> +             uefi-spi-vendor = "micron";
>>>> +             uefi-spi-part-number = "mt25ql128abb";
>>>> +             /* GUID in UEFI format: 77126730-a4ca-4386-b341-881fe18e7f7d */
>>>> +             uefi-spi-io-guid = [30 67 12 77 ca a4 86 43
>>>> +                                 b3 41 88 1f e1 8e 7f 7d];
>>>>        };
>>>>   };
>>>
>>> Are we far enough along with part one of this series to talk about
>>> getting these properties in the upstream binding document now?
>>
>> You mean those bindings being part of the DT spec?
>
> Yes, getting these added to
> Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml in the kernel.
>

There is a related patch
[PATCH] dt-bindings: spi: Add YAML DT binding document for trivial devices
https://lore.kernel.org/all/20220407194936.223041-1-marex@denx.de/
that didn't make it into the kernel yet.

Best regards

Heinrich