On Mon, Aug 09, 2021 at 10:14:05AM +0200, Cédric Le Goater wrote: > On 8/9/21 10:06 AM, Philippe Mathieu-Daudé wrote: > > On 8/9/21 9:55 AM, Cédric Le Goater wrote: > >> Hello Phil, > >> > >> On 8/9/21 9:06 AM, Philippe Mathieu-Daudé wrote: > >>> Hi Cédric, > >>> > >>> On 8/6/21 8:00 PM, Cédric Le Goater wrote: > >>>> It includes support for the POWER10 processor and the QEMU platform. > >>> > >>> 1/ Can you include the output of 'git shortlog v6.4..820d43c0' here? > >> > >> OK. See attachement. > > > > By "here" I meant in the commit description ;) > > yeah I know but David queued the patch already.
I can replace it with a new version. Including the shortlog is
probably worth it.
>
> >
> >>>>
> >>>> Built from submodule.
> >>>
> >>> 2/ Could we have a CI job building this, during 6.2 cycle?
> >>> (See .gitlab-ci.d/edk2.yml and .gitlab-ci.d/opensbi.yml)
> >>
> >> Sure. It doesn't look too complex.
> >>
> >> I plan to add acceptance tests for the QEMU powernv machines also
> >> once the OpenPOWER files (zImage.epapr and rootfs.cpio.xz) are
> >> published on GH.
> >>
> >>>>
> >>>> Signed-off-by: Cédric Le Goater <c...@kaod.org>
> >>>> ---
> >>>> pc-bios/skiboot.lid | Bin 1667280 -> 2528128 bytes
> >>>> roms/skiboot | 2 +-
> >>>> 2 files changed, 1 insertion(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/pc-bios/skiboot.lid b/pc-bios/skiboot.lid
> >>>> index
> >>>> 504b95e8b6611aff3a934ff10f789934680591f9..8a3c278512a428a034ed5b1ddbed017ae8c0a9d0
> >>>> 100644
> >>>> GIT binary patch
> >>>> literal 2528128
> >>>
> >>> Consider using 'git-format-patch --no-binary' and a reference
> >>> to your repository to fetch a such big binary patch.
> >>
> >> David would pull from my tree then ? So that's like doing a PR.
> >> We can do that next time I send an update if David is OK with
> >> that. I should send an update for v7.0 tag.
> >
> > As you wish. Big patches gave us troubles, i.e. they make crash
> > the 'patches' instance. 2.5MiB is probably borderline and I'm
> > being nit-picky.
> >
>
> If we can do a PR next time, all should be fine.
>
> Thanks,
>
> C.
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
