I think in general it would be nice to understand the long term picture of a 
change, esp. since there is already FDT support in EDK2 in various forms (with 
libraries and drivers depending on the existing FdtLib). So it would really of 
confusing to see another FDT library in MdePkg, without a clear reasoning for 
the work (this isn't reflected in the BZ) and a clear action plan to end up 
with just one FDT library in MdePkg in some identified time frame.

I do think FDT lib *does* belong in MdePkg, but it seems the shortest path to 
get there is to simply move the existing EmbeddedPkg one (and update all 
users). Subsequent cleanup can be incremental. And regardless, every existing 
FdtLib user ought to be updated to use the new one, so there need to be more 
patches (we're not just throwing the code over the wall, right?)

A

> -----Original Message-----
> From: Pedro Falcato <pedro.falc...@gmail.com>
> Sent: Friday, April 7, 2023 8:24 AM
> To: devel@edk2.groups.io; Tan, Lean Sheng <sheng....@9elements.com>
> Cc: Warkentin, Andrei <andrei.warken...@intel.com>; Lin, Benny
> <benny....@intel.com>; Kinney, Michael D <michael.d.kin...@intel.com>;
> Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang
> <zhiguang....@intel.com>; Sean Brogan <sean.bro...@microsoft.com>;
> Michael Kubacki <mikub...@linux.microsoft.com>
> Subject: Re: [edk2-devel] [PATCH 0/2] Support FDT library.
> 
> On Thu, Apr 6, 2023 at 5:34 PM Sheng Lean Tan
> <sheng....@9elements.com> wrote:
> >
> > Thanks for the nice feedback Pedro, Gerd and Andrei! Yeah it seems like a
> valid concern here as Mik mentioned on edk2-libc, and it seems to fits edk2
> long term interest on this.
> > Can we file this as an issue in Bugzilla for tracking or something? Since 
> > this
> will take some time to work on this as it involves a bigger discussion,
> personally I think we could get this FDT patch in first meanwhile, and also
> remove the FDT from Embedded Pkg as next step, per discussion with Leif?
> > What do you think?
> 
> I'm all for not merging this without a proper solution in that regard (I even
> presented a quick RFC solution which wasn't tested by anyone involved in
> this patch, yet).
> 
> But if there really is an urgent need for this lib, I'm O-K with merging this
> given that all my concerns are addressed (minus libc duplication).
> 
> --
> Pedro


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#102737): https://edk2.groups.io/g/devel/message/102737
Mute This Topic: https://groups.io/mt/97955736/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to