On Wed, Sep 25, 2019 at 05:10:58PM +0300, Jarkko Sakkinen wrote: > On Wed, Sep 25, 2019 at 05:09:03PM +0300, Jarkko Sakkinen wrote: > > > > [1] Intel SDM: 38.1.4 Intel SGX Launch Control Configuration > > > > > > > > Signed-off-by: Sean Christopherson <sean.j.christopher...@intel.com> > > > > Co-developed-by: Haim Cohen <haim.co...@intel.com> > > > > Signed-off-by: Haim Cohen <haim.co...@intel.com> > > > > Signed-off-by: Jarkko Sakkinen <jarkko.sakki...@linux.intel.com> > > > > > > This time checkpatch is right: > > > > > > WARNING: Missing Signed-off-by: line by nominal patch author 'Kai Huang > > > <kai.hu...@linux.intel.com>' > > > > > > And looking at the SOB chain, sounds like people need to make up their > > > mind about authorship... > > > > I'll make myself the sole author for this one as 98% of the effort in > > this patch is really the commit message, which I rewrote for v22, and 2% > > are the code changes (mechanical, peek at SDM). This patch was squashed > > from three patches, all like one line changes, and Kai was author of one > > of them. > > > > The next version will thus have only my SOB and author information will > > be changed. I doubt anyone will complain if I do that. > > I'll take the same action also for "x86/cpufeatures: x86/msr: Add Intel > SGX hardware bits"
I put to both: Signed-off-by: Jarkko Sakkinen <jarkko.sakki...@linux.intel.com> Co-developed-by: Sean Christopherson <sean.j.christopher...@intel.com> Signed-off-by: Sean Christopherson <sean.j.christopher...@intel.com> And changed author to me. /Jarkko
