[U-Boot] [PATCH 2/2] libavb: Fix build warnings after updating the lib
igor.opaniuk at gmail.com
Mon Aug 19 09:50:28 UTC 2019
On Mon, Aug 19, 2019 at 12:40 PM Igor Opaniuk <igor.opaniuk at gmail.com> wrote:
> Hi Eugeniu,
> On Mon, Aug 19, 2019 at 11:28 AM Eugeniu Rosca <erosca at de.adit-jv.com> wrote:
> > Hi all,
> > On Fri, Aug 16, 2019 at 11:46:03AM -0400, David Zeuthen wrote:
> > > Hi,
> > > As for upstreaming libavb patches, I'd be interested in landing them
> > > upstream... makes it easier for anyone.
> > > Our upstream is AOSP and we use gerrit for
> > > code-review: https://android-review.googlesource.com/q/project:platform%252Fexternal%252Favb
> > > Here's a guide to
> > > contributing: https://source.android.com/setup/contribute/submit-patches ...
> > > hope it's not too painful to use the AOSP process!
> > > Thanks,
> > > David
> > David, thanks for being open about discussing/accepting the fixes.
> > U-Boot folks,
> > What's your opinion w.r.t. the right/best order between accepting
> > the libavb fixes locally in U-Boot and pushing them to upstream?
> > Which should come first? I suppose they can be first applied internally
> > and replaced later on if they get reworked/optimized during upstreaming.
> > --
> > Best Regards,
> > Eugeniu.
> IMHO, the best approach here is to upstream fixes to AOSP libavb first,
> only then port stuff to U-boot.
> Nevertheless, taking into account the amount of time needed
> (sometimes it can take months, or even years, Sam has good
> stories about that) to get something applied to AOSP, I would suggest
> to proceed internally with commits like this ASAP, but somehow (extended
> commit message for each "libavb sync" patch with the list of patches out of
> AOSP tree; or introduce "doc/android/libavb-porting.txt", where we can
> provide all details about list of commits from AOSP + out-of-tree commits,
> how-to do porting etc; other ideas?) keep track of changes that are not
> up-streamed to AOSP yet.
> Ideally we have to achieve 1:1 mapping (with the only difference - SPDX tags),
And avb_sysdeps_posix.c (implementation of avb_abort() and list of includes).
You see, I'm already missing some of introduced changes on top of
> of  in lib/libavb/, otherwise we will fight with constant code
> divergence (obviously).
>  https://android.googlesource.com/platform/external/avb/+/<revision>/libavb
> Best regards - Freundliche Grüsse - Meilleures salutations
> Igor Opaniuk
> mailto: igor.opaniuk at gmail.com
> skype: igor.opanyuk
> +380 (93) 836 40 67
Best regards - Freundliche Grüsse - Meilleures salutations
mailto: igor.opaniuk at gmail.com
+380 (93) 836 40 67
More information about the U-Boot