Apologies, I was refactoring a git tree and switched off the world :) Well, being a bit late to the party ... my view is it would be preferable if we put the byteswapping in the library.
The DT format is *defined* to be big-endian, 32-bit cells. So frankly, I have no problems with pushing responsibility onto individual platforms if they have decided to be "clever" by ignoring that - if that gets rid of a bunch of boilerplate for sensibly behaving platforms. Maybe another excuse for getting https://github.com/tianocore/edk2-platforms/blob/master/Silicon/NXP/Library/IoAccessLib/IoAccessLib.c into MdePkg? / Leif On Thu, Apr 20, 2023 at 16:44:03 +0000, Michael D Kinney wrote: > Hi Leif, > > What is your opinion on FDT being big endian. > > Do we want the lib to help with that aspect, or leave it up to the consumer > of the lib to > convert as needed? Seems like an area that could be very error prone if we > are not > careful. > > Mike > > > -----Original Message----- > > From: Lin, Benny <benny....@intel.com> > > Sent: Wednesday, April 19, 2023 9:25 PM > > To: Kinney, Michael D <michael.d.kin...@intel.com>; devel@edk2.groups.io > > Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang > > <zhiguang....@intel.com>; Pedro Falcato <pedro.falc...@gmail.com> > > Subject: RE: [PATCH v3 2/3] MdePkg: Support FDT library. > > > > > > > > >-----Original Message----- > > >From: Kinney, Michael D <michael.d.kin...@intel.com> > > >Sent: Thursday, April 20, 2023 5:04 AM > > >To: Lin, Benny <benny....@intel.com>; devel@edk2.groups.io > > >Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang > > ><zhiguang....@intel.com>; Pedro Falcato <pedro.falc...@gmail.com>; Kinney, > > Michael D <michael.d.kin...@intel.com> > > >Subject: RE: [PATCH v3 2/3] MdePkg: Support FDT library. > > > > > >Responses below > > > > > >Mike > > > > > >> -----Original Message----- > > >> From: Lin, Benny <benny....@intel.com> > > >> Sent: Wednesday, April 19, 2023 10:12 AM > > >> To: Kinney, Michael D <michael.d.kin...@intel.com>; > > >> devel@edk2.groups.io > > >> Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang > > >> <zhiguang....@intel.com>; Pedro Falcato <pedro.falc...@gmail.com> > > >> Subject: RE: [PATCH v3 2/3] MdePkg: Support FDT library. > > >> > > >> Please find my feedback below. > > >> We can remove FDT_RESERVE_ENTRY but keep FDT_NODE_HEADER, what do you > > >> think? > > >> > > >> QQQ > > >> Benny > > >> > > >> >-----Original Message----- > > >> >From: Kinney, Michael D <michael.d.kin...@intel.com> > > >> >Sent: Wednesday, April 19, 2023 11:54 PM > > >> >To: Lin, Benny <benny....@intel.com>; devel@edk2.groups.io > > >> >Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang > > >> ><zhiguang....@intel.com>; Pedro Falcato <pedro.falc...@gmail.com>; > > >> >Kinney, > > >> Michael D <michael.d.kin...@intel.com> > > >> >Subject: RE: [PATCH v3 2/3] MdePkg: Support FDT library. > > >> > > > >> >A few comments below. > > >> > > > >> >Mike > > >> > > > >> >> -----Original Message----- > > >> >> From: Lin, Benny <benny....@intel.com> > > >> >> Sent: Sunday, April 16, 2023 10:35 PM > > >> >> To: devel@edk2.groups.io > > >> >> Cc: Lin, Benny <benny....@intel.com>; Kinney, Michael D > > >> >> <michael.d.kin...@intel.com>; Gao, Liming > > >> >> <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; > > >> >> Pedro Falcato <pedro.falc...@gmail.com> > > >> >> Subject: [PATCH v3 2/3] MdePkg: Support FDT library. > > >> >> > > >> >> From: Benny Lin <benny....@intel.com> > > >> >> > > >> >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4392 > > >> >> Add FDT support in EDK2 by submodule 3rd party libfdt > > >> >> (https://github.com/devicetree-org/pylibfdt/tree/main/libfdt) > > >> >> and refer to LibcLib implementation by Pedro. > > >> >> > > >> >> Cc: Michael D Kinney <michael.d.kin...@intel.com> > > >> >> Cc: Liming Gao <gaolim...@byosoft.com.cn> > > >> >> Cc: Zhiguang Liu <zhiguang....@intel.com> > > >> >> Acked-by: Pedro Falcato <pedro.falc...@gmail.com> > > >> >> Signed-off-by: Benny Lin <benny....@intel.com> > > >> >> --- > > >> >> MdePkg/Include/Library/FdtLib.h | 314 ++++++++++++++++++++ > > >> >> MdePkg/Library/BaseFdtLib/BaseFdtLib.inf | 62 ++++ > > >> >> MdePkg/Library/BaseFdtLib/BaseFdtLib.uni | 14 + > > >> >> MdePkg/Library/BaseFdtLib/FdtLib.c | 301 +++++++++++++++++++ > > >> >> MdePkg/Library/BaseFdtLib/LibFdtSupport.h | 99 ++++++ > > >> >> MdePkg/Library/BaseFdtLib/LibFdtWrapper.c | 173 +++++++++++ > > >> >> MdePkg/Library/BaseFdtLib/limits.h | 10 + > > >> >> MdePkg/Library/BaseFdtLib/stdbool.h | 10 + > > >> >> MdePkg/Library/BaseFdtLib/stddef.h | 10 + > > >> >> MdePkg/Library/BaseFdtLib/stdint.h | 10 + > > >> >> MdePkg/Library/BaseFdtLib/stdlib.h | 10 + > > >> >> MdePkg/Library/BaseFdtLib/string.h | 10 + > > >> >> MdePkg/MdePkg.ci.yaml | 17 +- > > >> >> MdePkg/MdePkg.dec | 4 + > > >> >> MdePkg/MdePkg.dsc | 1 + > > >> >> 15 files changed, 1043 insertions(+), 2 deletions(-) > > >> >> > > >> >> diff --git a/MdePkg/Include/Library/FdtLib.h > > >> >> b/MdePkg/Include/Library/FdtLib.h new file mode 100644 index > > >> >> 0000000000..d59b749c25 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Include/Library/FdtLib.h > > >> >> @@ -0,0 +1,314 @@ > > >> >> +/** @file > > >> >> > > >> >> + Flattened Device Tree Library. > > >> >> > > >> >> + > > >> >> > > >> >> + Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> + SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#ifndef FDT_LIB_H_ > > >> >> > > >> >> +#define FDT_LIB_H_ > > >> >> > > >> >> + > > >> >> > > >> >> +/// > > >> >> > > >> >> +/// Flattened Device Tree definition > > >> >> > > >> >> +/// > > >> >> > > >> >> +typedef struct { > > >> >> > > >> >> + UINT32 Magic; /* magic word FDT_MAGIC */ > > >> >> > > >> >> + UINT32 TotalSize; /* total size of DT block */ > > >> >> > > >> >> + UINT32 OffsetDtStruct; /* offset to structure */ > > >> >> > > >> >> + UINT32 OffsetDtStrings; /* offset to strings */ > > >> >> > > >> >> + UINT32 OffsetMemRsvmap; /* offset to memory reserve map */ > > >> >> > > >> >> + UINT32 Version; /* format version */ > > >> >> > > >> >> + UINT32 LastCompVersion; /* last compatible version */ > > >> >> > > >> >> + > > >> >> > > >> >> + /* version 2 fields below */ > > >> >> > > >> >> + UINT32 BootCpuidPhys; /* Which physical CPU id we're > > >> >> > > >> >> + booting on */ > > >> >> > > >> >> + /* version 3 fields below */ > > >> >> > > >> >> + UINT32 SizeDtStrings; /* size of the strings block */ > > >> >> > > >> >> + > > >> >> > > >> >> + /* version 17 fields below */ > > >> >> > > >> >> + UINT32 SizeDtStruct; /* size of the structure block */ > > >> >> > > >> >> +} FDT_HEADER; > > >> > > > >> >I see the macros below use SwapBytes32() on all the fields from this > > >> >structure. Does this mean this structure is big >endian? If > > >> that is the case, then the description of this structure and all > > >> individual fields must make that very clear >so any consumer of the > > >> structure knows it does not follow the default endianness of UEFI/PI and > > >> that consumers >must either use the macros or consumers > > must use SwapBytes32() on their own when using this structure directly. > > >> > > >> Yes, it's big endian. I will complete more descriptions in patch v4. > > > > > >Would it be simpler if we have a lib service that converts the entire > > >header from big endian to little endian. > > >That way, the consumer can use the FDT_HEADER directly without any of the > > >macros? > > > > > >Also, is the rest of the FDT blob also big endian? > > > > FDT blob format is binary encoding of FDT data with big endian. In > > addition, the consumer will set or get FDT data with FDT blob. We > > can complete more descriptions to warn consumers to be careful if just use > > header struct for mapping and get/set FDT data with FDT > > blob. > > > > For provide lib services that converts between big endian and little > > endian, we can provide a get header function to convert between > > big endian and little endian. However, we also need to consider the > > property value for setting and getting then we can convert in > > wrapper functions. > > > > For example, > > > > INT32 > > EFIAPI > > FdtSetProp ( > > IN VOID *Fdt, > > IN INT32 NodeOffset, > > IN CONST CHAR8 *Name, > > IN CONST VOID *Value, > > IN UINT32 Length > > ) > > { > > return fdt_setprop (Fdt, NodeOffset, Name, Value, (int)Length); > > } > > > > More functions for various byte width. > > INT32 > > EFIAPI > > FdtSetProp16 ( > > IN VOID *Fdt, > > IN INT32 NodeOffset, > > IN CONST CHAR8 *Name, > > IN CONST UINT16 *Value, > > IN UINT32 Length > > ) > > { > > return fdt_setprop (Fdt, NodeOffset, Name, SwapBytes16(Value), > > (int)Length); > > } > > > > INT32 > > EFIAPI > > FdtSetProp32 ( > > IN VOID *Fdt, > > IN INT32 NodeOffset, > > IN CONST CHAR8 *Name, > > IN CONST UINT32 *Value, > > IN UINT32 Length > > ) > > { > > return fdt_setprop (Fdt, NodeOffset, Name, SwapBytes32(Value), > > (int)Length); > > } > > > > >> > > >> >> > > >> >> + > > >> >> > > >> >> +typedef struct { > > >> >> > > >> >> + UINT64 Address; > > >> >> > > >> >> + UINT64 Size; > > >> >> > > >> >> +} FDT_RESERVE_ENTRY; > > >> > > > >> > > > >> >I do not see this type used anywhere in the lib class or the lib > > >> >instance. > > >> >Can it be removed? > > >> > > >> I think we can remove it because it might be not used for further usage > > >> of FDT support. > > >> > > >> > > > >> >> > > >> >> + > > >> >> > > >> >> +typedef struct { > > >> >> > > >> >> + UINT32 Tag; > > >> >> > > >> >> + CHAR8 Name[]; > > >> >> > > >> >> +} FDT_NODE_HEADER; > > >> > > > >> > > > >> >I do not see this type used anywhere in the lib class or the lib > > >> >instance. > > >> >Can it be removed? > > >> > > >> When we need retrieve a node from a FDT blob, we will get a node > > >> offset then we could use the struct to map the node content. It's very > > >> useful. > > > > > >Please add a comment header for this structure and describe the usage as > > >it related to the rest of the FdtLib >services. > > > > I will add descriptions for the usage and remove FDT_RESERVE_ENTRY in patch > > v4. > > > > > > > >> > > >> > > > >> >> > > >> >> + > > >> >> > > >> >> +typedef struct { > > >> >> > > >> >> + UINT32 Tag; > > >> >> > > >> >> + UINT32 Length; > > >> >> > > >> >> + UINT32 NameOffset; > > >> >> > > >> >> + CHAR8 Data[]; > > >> >> > > >> >> +} FDT_PROPERTY; > > >> >> > > >> >> + > > >> >> > > >> >> +#define FDT_GET_HEADER(Fdt, Field) SwapBytes32(((CONST FDT_HEADER > > >> >> +*)(Fdt))->Field) > > >> >> > > >> >> + > > >> >> > > >> >> +#define FDT_MAGIC(Fdt) (FDT_GET_HEADER(Fdt, Magic)) > > >> >> > > >> >> +#define FDT_TOTAL_SIZE(Fdt) (FDT_GET_HEADER(Fdt, TotalSize)) > > >> >> > > >> >> +#define FDT_OFFSET_DT_STRUCT(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> OffsetDtStruct)) > > >> >> > > >> >> +#define FDT_OFFSET_DT_STRINGS(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> +OffsetDtStrings)) > > >> >> > > >> >> +#define FDT_OFFSET_MEM_RSVMAP(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> +OffsetMemRsvmap)) > > >> >> > > >> >> +#define FDT_VERSION(Fdt) (FDT_GET_HEADER(Fdt, Version)) > > >> >> > > >> >> +#define FDT_LAST_COMP_VERSION(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> +LastCompVersion)) > > >> >> > > >> >> +#define FDT_BOOT_CPUID_PHYS(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> BootCpuidPhys)) > > >> >> > > >> >> +#define FDT_SIZE_DT_STRINGS(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> SizeDtStrings)) > > >> >> > > >> >> +#define FDT_SIZE_DT_STRUCT(Fdt) (FDT_GET_HEADER(Fdt, > > >> >> SizeDtStruct)) > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Verify the header of the Flattened Device Tree > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtCheckHeader ( > > >> >> > > >> >> + IN CONST VOID *Fdt > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Create a empty Flattened Device Tree. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Buffer The pointer to allocate a pool for FDT > > >> >> blob. > > >> >> > > >> >> + @param[in] BufferSize The BufferSize to the pool size. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +RETURN_STATUS > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtCreateEmptyTree ( > > >> >> > > >> >> + IN VOID *Buffer, > > >> >> > > >> >> + IN UINTN BufferSize > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next node from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + @param[in] Depth The depth to the level of tree hierarchy. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextNode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset, > > >> >> > > >> >> + IN INT32 *Depth > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node under the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtFirstSubnode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next node from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextSubnode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node which includes the given name. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] ParentOffset The offset to the node which start find > > >> >> under. > > >> >> > > >> >> + @param[in] Name The name to search the node with the > > >> >> name. > > >> >> > > >> >> + @param[in] NameLength The length of the name to check only. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to node offset with given node name. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtSubnodeOffsetNameLen ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 ParentOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN INT32 NameLength > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node which includes the given property > > >> >> name and value. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] StartOffset The offset to the starting node to find. > > >> >> > > >> >> + @param[in] PropertyName The property name to search the node > > >> >> including the named property. > > >> >> > > >> >> + @param[in] PropertyValue The property value to check the same > > >> >> property value. > > >> >> > > >> >> + @param[in] PropertyLength The length of the value in PropertValue. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to node offset with given property. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNodeOffsetByPropValue ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 StartOffset, > > >> >> > > >> >> + IN CONST CHAR8 *PropertyName, > > >> >> > > >> >> + IN CONST VOID *PropertyValue, > > >> >> > > >> >> + IN INT32 PropertyLength > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a property with the given name from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] NodeOffset The offset to the given node. > > >> >> > > >> >> + @param[in] Name The name to the property which need be > > >> >> searched > > >> >> > > >> >> + @param[in] Length The length to the size of the property > > >> >> found. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The property to the structure of the found property. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST FDT_PROPERTY * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetProperty ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN INT32 *Length > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first property in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to the node which need be > > >> >> searched. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to first property offset in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtFirstPropertyOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next property from the given property. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous property. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next property offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextPropertyOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a property from the given offset of the property. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to the given offset of the > > >> >> property. > > >> >> > > >> >> + @param[in] Length The length to the size of the property > > >> >> found. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The property to the structure of the given property offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST FDT_PROPERTY * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetPropertyByOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset, > > >> >> > > >> >> + IN INT32 *Length > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a string by the given string offset. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] StrOffset The offset to the location in the > > >> >> strings block of FDT. > > >> >> > > >> >> + @param[in] Length The length to the size of string which > > >> >> need be retrieved. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The string to the given string offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST CHAR8 * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetString ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 StrOffset, > > >> >> > > >> >> + IN INT32 *Length OPTIONAL > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Add a new node to the FDT. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] ParentOffset The offset to the node offset which want > > >> >> to add in. > > >> >> > > >> >> + @param[in] Name The name to name the node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to the new node. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtAddSubnode ( > > >> >> > > >> >> + IN VOID *Fdt, > > >> >> > > >> >> + IN INT32 ParentOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Add or modify a property in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] NodeOffset The offset to the node offset which want > > >> >> to add in. > > >> >> > > >> >> + @param[in] Name The name to name the property. > > >> >> > > >> >> + @param[in] Value The value to the property value. > > >> >> > > >> >> + @param[in] Length The length to the size of the property. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtSetProp ( > > >> >> > > >> >> + IN VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN CONST VOID *Value, > > >> >> > > >> >> + IN UINT32 Length > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +#endif /* FDT_LIB_H_ */ > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/BaseFdtLib.inf > > >> >> b/MdePkg/Library/BaseFdtLib/BaseFdtLib.inf > > >> >> new file mode 100644 > > >> >> index 0000000000..730e568ff6 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/BaseFdtLib.inf > > >> >> @@ -0,0 +1,62 @@ > > >> >> +## @file > > >> >> > > >> >> +# Flattened Device Tree Library. > > >> >> > > >> >> +# > > >> >> > > >> >> +# Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +# > > >> >> > > >> >> +# SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> +# > > >> >> > > >> >> +# > > >> >> > > >> >> +## > > >> >> > > >> >> + > > >> >> > > >> >> +[Defines] > > >> >> > > >> >> + INF_VERSION = 0x0001001B > > >> >> > > >> >> + BASE_NAME = BaseFdtLib > > >> >> > > >> >> + MODULE_UNI_FILE = BaseFdtLib.uni > > >> >> > > >> >> + FILE_GUID = > > >> >> C64DCB01-B037-4FF6-9CF3-E8CEC206DE04 > > >> >> > > >> >> + MODULE_TYPE = BASE > > >> >> > > >> >> + VERSION_STRING = 1.0 > > >> >> > > >> >> + LIBRARY_CLASS = FdtLib > > >> >> > > >> >> + > > >> >> > > >> >> + DEFINE FDT_LIB_PATH = libfdt/libfdt > > >> >> > > >> >> + > > >> >> > > >> >> +# > > >> >> > > >> >> +# VALID_ARCHITECTURES = IA32 X64 > > >> >> > > >> >> +# > > >> >> > > >> >> + > > >> >> > > >> >> +[Sources] > > >> >> > > >> >> + FdtLib.c > > >> >> > > >> >> + LibFdtWrapper.c > > >> >> > > >> >> + # header Wrapper files > > >> >> > > >> >> + limits.h > > >> >> > > >> >> + stdbool.h > > >> >> > > >> >> + stddef.h > > >> >> > > >> >> + stdint.h > > >> >> > > >> >> + stdlib.h > > >> >> > > >> >> + string.h > > >> >> > > >> >> + > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt.h > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_addresses.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_check.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_empty_tree.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_overlay.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_ro.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_rw.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_strerror.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_sw.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/fdt_wip.c > > >> >> > > >> >> + $(FDT_LIB_PATH)/libfdt.h > > >> >> > > >> >> + $(FDT_LIB_PATH)/libfdt_env.h > > >> >> > > >> >> + $(FDT_LIB_PATH)/libfdt_internal.h > > >> >> > > >> >> + > > >> >> > > >> >> +[Packages] > > >> >> > > >> >> + MdePkg/MdePkg.dec > > >> >> > > >> >> + > > >> >> > > >> >> +[LibraryClasses] > > >> >> > > >> >> + BaseLib > > >> >> > > >> >> + BaseMemoryLib > > >> >> > > >> >> + > > >> >> > > >> >> +[BuildOptions] > > >> >> > > >> >> + MSFT:*_*_IA32_CC_FLAGS = /wd4146 /wd4245 > > >> >> > > >> >> + MSFT:*_*_X64_CC_FLAGS = /wd4146 /wd4244 /wd4245 /wd4267 > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/BaseFdtLib.uni > > >> >> b/MdePkg/Library/BaseFdtLib/BaseFdtLib.uni > > >> >> new file mode 100644 > > >> >> index 0000000000..3f7e45ea6f > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/BaseFdtLib.uni > > >> >> @@ -0,0 +1,14 @@ > > >> >> +// /** @file > > >> >> > > >> >> +// Flattened Device Tree Library. > > >> >> > > >> >> +// > > >> >> > > >> >> +// Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +// > > >> >> > > >> >> +// SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> +// > > >> >> > > >> >> +// **/ > > >> >> > > >> >> + > > >> >> > > >> >> + > > >> >> > > >> >> +#string STR_MODULE_ABSTRACT #language en-US "Instance of > > >> >> FDT Library" > > >> >> > > >> >> + > > >> >> > > >> >> +#string STR_MODULE_DESCRIPTION #language en-US "This module > > >> >> provides FDT Library implementation." > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/FdtLib.c > > >> >> b/MdePkg/Library/BaseFdtLib/FdtLib.c > > >> >> new file mode 100644 > > >> >> index 0000000000..200a496c7e > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/FdtLib.c > > >> >> @@ -0,0 +1,301 @@ > > >> >> +/** @file > > >> >> > > >> >> + Flattened Device Tree Library. > > >> >> > > >> >> + > > >> >> > > >> >> + Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> + SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <libfdt/libfdt/libfdt.h> > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Verify the header of the Flattened Device Tree > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtCheckHeader ( > > >> >> > > >> >> + IN CONST VOID *Fdt > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_check_header(Fdt); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Create a empty Flattened Device Tree. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Buffer The pointer to allocate a pool for FDT > > >> >> blob. > > >> >> > > >> >> + @param[in] BufferSize The BufferSize to the pool size. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtCreateEmptyTree ( > > >> >> > > >> >> + IN VOID *Buffer, > > >> >> > > >> >> + IN UINT32 BufferSize > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_create_empty_tree (Buffer, (int)BufferSize); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next node from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + @param[in] Depth The depth to the level of tree hierarchy. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextNode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset, > > >> >> > > >> >> + IN INT32 *Depth > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_next_node (Fdt, Offset, Depth); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node under the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtFirstSubnode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_first_subnode (Fdt, Offset); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next node from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next node offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextSubnode ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_next_subnode (Fdt, Offset); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node which includes the given name. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] ParentOffset The offset to the node which start find > > >> >> under. > > >> >> > > >> >> + @param[in] Name The name to search the node with the > > >> >> name. > > >> >> > > >> >> + @param[in] NameLength The length of the name to check only. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to node offset with given node name. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtSubnodeOffsetNameLen ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 ParentOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN INT32 NameLength > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_subnode_offset_namelen (Fdt, ParentOffset, Name, > > >> >> + NameLength); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first node which includes the given property > > >> >> name and value. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] StartOffset The offset to the starting node to find. > > >> >> > > >> >> + @param[in] PropertyName The property name to search the node > > >> >> including the named property. > > >> >> > > >> >> + @param[in] PropertyValue The property value to check the same > > >> >> property value. > > >> >> > > >> >> + @param[in] PropertyLength The length of the value in PropertValue. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to node offset with given property. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNodeOffsetByPropValue ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 StartOffset, > > >> >> > > >> >> + IN CONST CHAR8 *PropertyName, > > >> >> > > >> >> + IN CONST VOID *PropertyValue, > > >> >> > > >> >> + IN INT32 PropertyLength > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_node_offset_by_prop_value (Fdt, StartOffset, > > >> >> + PropertyName, PropertyValue, PropertyLength); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a property with the given name from the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] NodeOffset The offset to the given node. > > >> >> > > >> >> + @param[in] Name The name to the property which need be > > >> >> searched > > >> >> > > >> >> + @param[in] Length The length to the size of the property > > >> >> found. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The property to the structure of the found property. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST struct fdt_property * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetProperty ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN INT32 *Length > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_get_property (Fdt, NodeOffset, Name, Length); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of first property in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] NodeOffset The offset to the node which need be > > >> >> searched. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to first property offset in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtFirstPropertyOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_first_property_offset (Fdt, NodeOffset); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a offset of next property from the given property. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to previous property. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to next property offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtNextPropertyOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_next_property_offset (Fdt, Offset); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a property from the given offset of the property. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] Offset The offset to the given offset of the > > >> >> property. > > >> >> > > >> >> + @param[in] Length The length to the size of the property > > >> >> found. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The property to the structure of the given property offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST struct fdt_property * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetPropertyByOffset ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 Offset, > > >> >> > > >> >> + IN INT32 *Length > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_get_property_by_offset (Fdt, Offset, Length); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Returns a string by the given string offset. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] StrOffset The offset to the location in the > > >> >> strings block of FDT. > > >> >> > > >> >> + @param[in] Length The length to the size of string which > > >> >> need be retrieved. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The string to the given string offset. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +CONST CHAR8 * > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtGetString ( > > >> >> > > >> >> + IN CONST VOID *Fdt, > > >> >> > > >> >> + IN INT32 StrOffset, > > >> >> > > >> >> + IN INT32 *Length OPTIONAL > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_get_string (Fdt, StrOffset, Length); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Add a new node to the FDT. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] ParentOffset The offset to the node offset which want > > >> >> to add in. > > >> >> > > >> >> + @param[in] Name The name to name the node. > > >> >> > > >> >> + > > >> >> > > >> >> + @return The offset to the new node. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtAddSubnode ( > > >> >> > > >> >> + IN VOID *Fdt, > > >> >> > > >> >> + IN INT32 ParentOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_add_subnode (Fdt, ParentOffset, Name); > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +/** > > >> >> > > >> >> + Add or modify a property in the given node. > > >> >> > > >> >> + > > >> >> > > >> >> + @param[in] Fdt The pointer to FDT blob. > > >> >> > > >> >> + @param[in] NodeOffset The offset to the node offset which want > > >> >> to add in. > > >> >> > > >> >> + @param[in] Name The name to name the property. > > >> >> > > >> >> + @param[in] Value The value to the property value. > > >> >> > > >> >> + @param[in] Length The length to the size of the property. > > >> >> > > >> >> + > > >> >> > > >> >> + @return Zero for successfully, otherwise failed. > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> +INT32 > > >> >> > > >> >> +EFIAPI > > >> >> > > >> >> +FdtSetProp ( > > >> >> > > >> >> + IN VOID *Fdt, > > >> >> > > >> >> + IN INT32 NodeOffset, > > >> >> > > >> >> + IN CONST CHAR8 *Name, > > >> >> > > >> >> + IN CONST VOID *Value, > > >> >> > > >> >> + IN UINT32 Length > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + return fdt_setprop (Fdt, NodeOffset, Name, Value, (int)Length); > > >> >> > > >> >> +} > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/LibFdtSupport.h > > >> >> b/MdePkg/Library/BaseFdtLib/LibFdtSupport.h > > >> >> new file mode 100644 > > >> >> index 0000000000..e738f4ded4 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/LibFdtSupport.h > > >> >> @@ -0,0 +1,99 @@ > > >> >> +/** @file > > >> >> > > >> >> + Root include file of C runtime library to support building the > > >> >> + third-party > > >> >> > > >> >> + libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> + Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> + SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#ifndef FDT_LIB_SUPPORT_H_ > > >> >> > > >> >> +#define FDT_LIB_SUPPORT_H_ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <Base.h> > > >> >> > > >> >> +#include <Library/BaseLib.h> > > >> >> > > >> >> +#include <Library/BaseMemoryLib.h> > > >> >> > > >> >> + > > >> >> > > >> >> +typedef UINT8 uint8_t; > > >> >> > > >> >> +typedef UINT16 uint16_t; > > >> >> > > >> >> +typedef INT32 int32_t; > > >> >> > > >> >> +typedef UINT32 uint32_t; > > >> >> > > >> >> +typedef UINT64 uint64_t; > > >> >> > > >> >> +typedef UINTN uintptr_t; > > >> >> > > >> >> +typedef UINTN size_t; > > >> >> > > >> >> +typedef BOOLEAN bool; > > >> >> > > >> >> + > > >> >> > > >> >> +#define true (1 == 1) > > >> >> > > >> >> +#define false (1 == 0) > > >> >> > > >> >> + > > >> >> > > >> >> +// > > >> >> > > >> >> +// Definitions for global constants used by libfdt library > > >> >> +routines > > >> >> > > >> >> +// > > >> >> > > >> >> +#define INT_MAX 0x7FFFFFFF /* Maximum (signed) int > > >> >> value */ > > >> >> > > >> >> +#define INT32_MAX 0x7FFFFFFF /* Maximum (signed) int32 > > >> >> value */ > > >> >> > > >> >> +#define UINT32_MAX 0xFFFFFFFF /* Maximum unsigned int32 > > >> >> value */ > > >> >> > > >> >> + > > >> >> > > >> >> +// > > >> >> > > >> >> +// Function prototypes of libfdt Library routines > > >> >> > > >> >> +// > > >> >> > > >> >> +void * > > >> >> > > >> >> +memset ( > > >> >> > > >> >> + void *, > > >> >> > > >> >> + int, > > >> >> > > >> >> + size_t > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +int > > >> >> > > >> >> +memcmp ( > > >> >> > > >> >> + const void *, > > >> >> > > >> >> + const void *, > > >> >> > > >> >> + size_t > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +int > > >> >> > > >> >> +strcmp ( > > >> >> > > >> >> + const char *, > > >> >> > > >> >> + const char * > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +char * > > >> >> > > >> >> +strchr ( > > >> >> > > >> >> + const char *, > > >> >> > > >> >> + int > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +char * > > >> >> > > >> >> +strrchr ( > > >> >> > > >> >> + const char *, > > >> >> > > >> >> + int > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +unsigned long > > >> >> > > >> >> +strtoul ( > > >> >> > > >> >> + const char *, > > >> >> > > >> >> + char **, > > >> >> > > >> >> + int > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +char * > > >> >> > > >> >> +strcpy ( > > >> >> > > >> >> + char *strDest, > > >> >> > > >> >> + const char *strSource > > >> >> > > >> >> + ); > > >> >> > > >> >> + > > >> >> > > >> >> +// > > >> >> > > >> >> +// Macros that directly map functions to BaseLib, BaseMemoryLib, > > >> >> +and DebugLib functions > > >> >> > > >> >> +// > > >> >> > > >> >> +#define memcpy(dest, source, count) CopyMem(dest,source, > > >> >> (UINTN)(count)) > > >> >> > > >> >> +#define memset(dest, ch, count) SetMem(dest, > > >> >> (UINTN)(count),(UINT8)(ch)) > > >> >> > > >> >> +#define memchr(buf, ch, count) ScanMem8(buf, > > >> >> (UINTN)(count),(UINT8)ch) > > >> >> > > >> >> +#define memcmp(buf1, buf2, count) (int)(CompareMem(buf1, > > >> >> buf2, (UINTN)(count))) > > >> >> > > >> >> +#define memmove(dest, source, count) CopyMem(dest, source, > > >> >> (UINTN)(count)) > > >> >> > > >> >> +#define strlen(str) > > >> >> (size_t)(AsciiStrLen(str)) > > >> >> > > >> >> +#define strnlen(str, count) > > >> >> (size_t)(AsciiStrnLenS(str, count)) > > >> >> > > >> >> +#define strncpy(strDest, strSource, count) AsciiStrnCpyS(strDest, > > >> >> +MAX_STRING_SIZE, strSource, (UINTN)count) > > >> >> > > >> >> +#define strcat(strDest, strSource) AsciiStrCatS(strDest, > > >> >> MAX_STRING_SIZE, strSource) > > >> >> > > >> >> +#define strcmp(string1, string2, count) > > >> >> (int)(AsciiStrCmp(string1, string2)) > > >> >> > > >> >> +#define strncmp(string1, string2, count) > > >> >> (int)(AsciiStrnCmp(string1, string2, (UINTN)(count))) > > >> >> > > >> >> + > > >> >> > > >> >> +#endif /* FDT_LIB_SUPPORT_H_ */ > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/LibFdtWrapper.c > > >> >> b/MdePkg/Library/BaseFdtLib/LibFdtWrapper.c > > >> >> new file mode 100644 > > >> >> index 0000000000..772bba4839 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/LibFdtWrapper.c > > >> >> @@ -0,0 +1,173 @@ > > >> >> +/** @file > > >> >> > > >> >> + ISO C implementations of strchr, strrchr and strtoul. > > >> >> > > >> >> + > > >> >> > > >> >> + Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> + Copyright (c) 2023 Pedro Falcato All rights reserved. > > >> >> > > >> >> + SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <Base.h> > > >> >> > > >> >> + > > >> >> > > >> >> +#define ULONG_MAX 0xFFFFFFFF /* Maximum unsigned long > > >> >> value */ > > >> >> > > >> >> + > > >> >> > > >> >> +// Very quick notes: > > >> >> > > >> >> +// We only go through the string once for both functions > > >> >> > > >> >> +// They are minimal implementations (not speed optimized) of ISO C > > >> >> +semantics > > >> >> > > >> >> +// strchr and strrchr also include the null terminator as part of > > >> >> +the string > > >> >> > > >> >> +// so the code gets a bit clunky to handle that case specifically. > > >> >> > > >> >> + > > >> >> > > >> >> +char * > > >> >> > > >> >> +strchr ( > > >> >> > > >> >> + const char *Str, > > >> >> > > >> >> + int Char > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + char *S; > > >> >> > > >> >> + > > >> >> > > >> >> + S = (char *)Str; > > >> >> > > >> >> + > > >> >> > > >> >> + for ( ; ; S++) { > > >> >> > > >> >> + if (*S == Char) { > > >> >> > > >> >> + return S; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + if (*S == '\0') { > > >> >> > > >> >> + return NULL; > > >> >> > > >> >> + } > > >> >> > > >> >> + } > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +char * > > >> >> > > >> >> +strrchr ( > > >> >> > > >> >> + const char *Str, > > >> >> > > >> >> + int Char > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + char *S, *last; > > >> >> > > >> >> + > > >> >> > > >> >> + S = (char *)Str; > > >> >> > > >> >> + last = NULL; > > >> >> > > >> >> + > > >> >> > > >> >> + for ( ; ; S++) { > > >> >> > > >> >> + if (*S == Char) { > > >> >> > > >> >> + last = S; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + if (*S == '\0') { > > >> >> > > >> >> + return last; > > >> >> > > >> >> + } > > >> >> > > >> >> + } > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +STATIC > > >> >> > > >> >> +int > > >> >> > > >> >> +__isspace ( > > >> >> > > >> >> + int ch > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + // basic ASCII ctype.h:isspace(). Not efficient > > >> >> > > >> >> + return ch == '\r' || ch == '\n' || ch == ' ' || ch == '\t' || ch > > >> >> + == '\v' || ch == '\f'; > > >> >> > > >> >> +} > > >> >> > > >> >> + > > >> >> > > >> >> +unsigned long > > >> >> > > >> >> +strtoul ( > > >> >> > > >> >> + const char *Nptr, > > >> >> > > >> >> + char **EndPtr, > > >> >> > > >> >> + int Base > > >> >> > > >> >> + ) > > >> >> > > >> >> +{ > > >> >> > > >> >> + BOOLEAN Negate; > > >> >> > > >> >> + BOOLEAN Overflow; > > >> >> > > >> >> + unsigned long Val; > > >> >> > > >> >> + > > >> >> > > >> >> + Negate = FALSE; > > >> >> > > >> >> + Overflow = FALSE; > > >> >> > > >> >> + Val = 0; > > >> >> > > >> >> + > > >> >> > > >> >> + // Reject bad numeric bases > > >> >> > > >> >> + if ((Base < 0) || (Base == 1) || (Base > 36)) { > > >> >> > > >> >> + return 0; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + // Skip whitespace > > >> >> > > >> >> + while (__isspace (*Nptr)) { > > >> >> > > >> >> + Nptr++; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + // Check for + or - prefixes > > >> >> > > >> >> + if (*Nptr == '-') { > > >> >> > > >> >> + Negate = TRUE; > > >> >> > > >> >> + Nptr++; > > >> >> > > >> >> + } else if (*Nptr == '+') { > > >> >> > > >> >> + Nptr++; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + // Consume the start, autodetecting base if needed > > >> >> > > >> >> + if ((Nptr[0] == '0') && ((Nptr[1] == 'x') || (Nptr[1] == 'X')) > > >> >> + && ((Base == 0) || (Base == 16))) { > > >> >> > > >> >> + // Hex > > >> >> > > >> >> + Nptr += 2; > > >> >> > > >> >> + Base = 16; > > >> >> > > >> >> + } else if ((Nptr[0] == '0') && ((Nptr[1] == 'b') || (Nptr[1] == > > >> >> + 'B')) && ((Base == 0) || (Base == 2))) { > > >> >> > > >> >> + // Binary (standard pending C23) > > >> >> > > >> >> + Nptr += 2; > > >> >> > > >> >> + Base = 2; > > >> >> > > >> >> + } else if ((Nptr[0] == '0') && ((Base == 0) || (Base == 8))) { > > >> >> > > >> >> + // Octal > > >> >> > > >> >> + Nptr++; > > >> >> > > >> >> + Base = 8; > > >> >> > > >> >> + } else { > > >> >> > > >> >> + if (Base == 0) { > > >> >> > > >> >> + // Assume decimal > > >> >> > > >> >> + Base = 10; > > >> >> > > >> >> + } > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + while (TRUE) { > > >> >> > > >> >> + int Digit; > > >> >> > > >> >> + char C; > > >> >> > > >> >> + unsigned long NewVal; > > >> >> > > >> >> + > > >> >> > > >> >> + C = *Nptr; > > >> >> > > >> >> + Digit = -1; > > >> >> > > >> >> + > > >> >> > > >> >> + if ((C >= '0') && (C <= '9')) { > > >> >> > > >> >> + Digit = C - '0'; > > >> >> > > >> >> + } else if ((C >= 'a') && (C <= 'z')) { > > >> >> > > >> >> + Digit = C - 'a' + 10; > > >> >> > > >> >> + } else if ((C >= 'A') && (C <= 'Z')) { > > >> >> > > >> >> + Digit = C - 'A' + 10; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + if ((Digit == -1) || (Digit >= Base)) { > > >> >> > > >> >> + // Note that this case also handles the \0 > > >> >> > > >> >> + if (EndPtr) { > > >> >> > > >> >> + *EndPtr = (char *)Nptr; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + break; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + NewVal = Val * Base + Digit; > > >> >> > > >> >> + > > >> >> > > >> >> + if (NewVal < Val) { > > >> >> > > >> >> + // Overflow > > >> >> > > >> >> + Overflow = TRUE; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + Val = NewVal; > > >> >> > > >> >> + > > >> >> > > >> >> + Nptr++; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + if (Negate) { > > >> >> > > >> >> + Val = -Val; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + if (Overflow) { > > >> >> > > >> >> + Val = ULONG_MAX; > > >> >> > > >> >> + } > > >> >> > > >> >> + > > >> >> > > >> >> + // TODO: We're lacking errno here. > > >> >> > > >> >> + return Val; > > >> >> > > >> >> +} > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/limits.h > > >> >> b/MdePkg/Library/BaseFdtLib/limits.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/limits.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/stdbool.h > > >> >> b/MdePkg/Library/BaseFdtLib/stdbool.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/stdbool.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/stddef.h > > >> >> b/MdePkg/Library/BaseFdtLib/stddef.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/stddef.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/stdint.h > > >> >> b/MdePkg/Library/BaseFdtLib/stdint.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/stdint.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/stdlib.h > > >> >> b/MdePkg/Library/BaseFdtLib/stdlib.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/stdlib.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/Library/BaseFdtLib/string.h > > >> >> b/MdePkg/Library/BaseFdtLib/string.h > > >> >> new file mode 100644 > > >> >> index 0000000000..f6cf8d5702 > > >> >> --- /dev/null > > >> >> +++ b/MdePkg/Library/BaseFdtLib/string.h > > >> >> @@ -0,0 +1,10 @@ > > >> >> +/** @file > > >> >> > > >> >> + Include file to support building the third-party libfdt library. > > >> >> > > >> >> + > > >> >> > > >> >> +Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> + > > >> >> > > >> >> +**/ > > >> >> > > >> >> + > > >> >> > > >> >> +#include <LibFdtSupport.h> > > >> >> > > >> >> + > > >> >> > > >> >> diff --git a/MdePkg/MdePkg.ci.yaml b/MdePkg/MdePkg.ci.yaml index > > >> >> 6ba85ebe61..f039daccdb 100644 > > >> >> --- a/MdePkg/MdePkg.ci.yaml > > >> >> +++ b/MdePkg/MdePkg.ci.yaml > > >> >> @@ -2,7 +2,7 @@ > > >> >> # CI configuration for MdePkg > > >> >> > > >> >> # > > >> >> > > >> >> # Copyright (c) Microsoft Corporation > > >> >> > > >> >> -# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> +# Copyright (c) 2023, Intel Corporation. All rights reserved.<BR> > > >> >> > > >> >> # Copyright (c) 2021, Arm Limited. All rights reserved.<BR> > > >> >> > > >> >> # Copyright (c) 2023, Loongson Technology Corporation Limited. All > > >> >> rights reserved.<BR> > > >> >> > > >> >> # SPDX-License-Identifier: BSD-2-Clause-Patent > > >> >> > > >> >> @@ -67,7 +67,8 @@ > > >> >> "Include/Library/PcdLib.h", > > >> >> > > >> >> "Include/Library/SafeIntLib.h", > > >> >> > > >> >> "Include/Protocol/DebugSupport.h", > > >> >> > > >> >> - > > >> >> "Test/UnitTest/Library/BaseSafeIntLib/TestBaseSafeIntLib.c" > > >> >> > > >> >> + > > >> >> + "Test/UnitTest/Library/BaseSafeIntLib/TestBaseSafeIntLib.c", > > >> >> > > >> >> + "Library/BaseFdtLib" > > >> >> > > >> >> ] > > >> >> > > >> >> }, > > >> >> > > >> >> ## options defined ci/Plugin/CompilerPlugin > > >> >> > > >> >> @@ -164,5 +165,17 @@ > > >> >> "ExtendWords": [], # words to extend to the > > >> >> dictionary for this package > > >> >> > > >> >> "IgnoreStandardPaths": [], # Standard Plugin defined paths > > >> >> that should be ignore > > >> >> > > >> >> "AdditionalIncludePaths": [] # Additional paths to spell > > >> >> check (wildcards supported) > > >> >> > > >> >> + }, > > >> >> > > >> >> + # options defined in .pytool/Plugin/UncrustifyCheck > > >> >> > > >> >> + "UncrustifyCheck": { > > >> >> > > >> >> + "IgnoreFiles": [ > > >> >> > > >> >> + "Library/BaseFdtLib/libfdt", > > >> >> > > >> >> + "Library/BaseFdtLib/limits.h", > > >> >> > > >> >> + "Library/BaseFdtLib/stdbool.h", > > >> >> > > >> >> + "Library/BaseFdtLib/stddef.h", > > >> >> > > >> >> + "Library/BaseFdtLib/stdint.h", > > >> >> > > >> >> + "Library/BaseFdtLib/stdlib.h", > > >> >> > > >> >> + "Library/BaseFdtLib/string.h" > > >> >> > > >> >> + ] > > >> >> > > >> >> } > > >> >> > > >> >> } > > >> >> > > >> >> diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index > > >> >> e70182bf7f..9804a7a6bd 100644 > > >> >> --- a/MdePkg/MdePkg.dec > > >> >> +++ b/MdePkg/MdePkg.dec > > >> >> @@ -289,6 +289,10 @@ > > >> >> # > > >> >> > > >> >> ArmTrngLib|Include/Library/ArmTrngLib.h > > >> >> > > >> >> > > >> >> > > >> >> + ## @libraryclass Provides APIs for third-party library libfdt. > > >> >> > > >> >> + # > > >> >> > > >> >> + FdtLib|Include/Library/FdtLib.h > > >> >> > > >> >> + > > >> >> > > >> >> [LibraryClasses.IA32, LibraryClasses.X64, LibraryClasses.AARCH64] > > >> >> > > >> >> ## @libraryclass Provides services to generate random number. > > >> >> > > >> >> # > > >> >> > > >> >> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc index > > >> >> 152c02991a..b38c863812 100644 > > >> >> --- a/MdePkg/MdePkg.dsc > > >> >> +++ b/MdePkg/MdePkg.dsc > > >> >> @@ -137,6 +137,7 @@ > > >> >> > > >> >> MdePkg/Library/SmmCpuRendezvousLibNull/SmmCpuRendezvousLibNull.inf > > >> >> > > >> >> > > >> >> > > >> >> MdePkg/Library/JedecJep106Lib/JedecJep106Lib.inf > > >> >> > > >> >> + MdePkg/Library/BaseFdtLib/BaseFdtLib.inf > > >> >> > > >> >> > > >> >> > > >> >> [Components.IA32, Components.X64, Components.ARM, > > >> >> Components.AARCH64] > > >> >> > > >> >> # > > >> >> > > >> >> -- > > >> >> 2.39.1.windows.1 > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103490): https://edk2.groups.io/g/devel/message/103490 Mute This Topic: https://groups.io/mt/98313203/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
