On Tue, Oct 10, 2017 at 05:06:42PM +0200, Marcin Wojtas wrote: > 2017-10-10 17:03 GMT+02:00 Leif Lindholm <leif.lindh...@linaro.org>: > > On Tue, Oct 10, 2017 at 04:45:10PM +0200, Marcin Wojtas wrote: > >> Hi Leif, > >> > >> 2017-10-10 16:37 GMT+02:00 Leif Lindholm <leif.lindh...@linaro.org>: > >> > On Mon, Oct 09, 2017 at 07:00:50PM +0200, Marcin Wojtas wrote: > >> >> In order to enable modification of dynamic PCD's for the libraries > >> >> and DXE drivers, this patch introduces new driver. It is > >> >> executed prior to other drivers. Mpp, ComPhy and Utmi libraries > >> >> initialization were moved from PrePi stage to DXE. > >> >> > >> >> To force the correct driver dispatch sequence, introduce a protocol GUID > >> >> and install the protocol as a NULL protocol when PlatInitDxe executes. > >> >> > >> >> Contributed-under: TianoCore Contribution Agreement 1.1 > >> >> Signed-off-by: Marcin Wojtas <m...@semihalf.com> > >> >> Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> > >> > > >> > What does Ard's Signed-off-by signify here? > >> > (I know the authorship on some of these is a bit blurred, since you've > >> > been working together, but I'd like to be clear.) > >> > >> These were the lines, introducing/installing protocol GUID stuff. It > >> was in a small separate patch, but I squashed it into bigger one. > > > > Personally, I would in this instance do: > > <me> > > Ard > > <me> > > > > It's verbose, but reasonably clear. > > > > How about: > > In order to enable modification of dynamic PCD's for the libraries > and DXE drivers, this patch introduces new driver. It is > executed prior to other drivers. Mpp, ComPhy and Utmi libraries > initialization were moved from PrePi stage to DXE. > > Signed-off-by: Marcin Wojtas <m...@semihalf.com> > > To force the correct driver dispatch sequence, introduce a protocol GUID > and install the protocol as a NULL protocol when PlatInitDxe executes. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> > Signed-off-by: Marcin Wojtas <m...@semihalf.com> > > ? > > Was that, what you meant?
I think Contibuted-under: still needs to come first. I don't think we have an explicit policy for how to deal with multi-contributor patches. The ones we do see tend to just keep a single commit message and list the contributors. In Linux. it would be something like Signed-off-by: Marcin Wojtas <m...@semihalf.com> [Introduce protocol GUID to force correct driver dispatch order] Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> Signed-off-by: Marcin Wojtas <m...@semihalf.com> I would be quite happy to use the same format here. / Leif _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel