Am Freitag, 7. Juni 2019, 01:34:32 CEST schrieb Doug Anderson: > Hi, > > On Thu, Jun 6, 2019 at 10:56 AM Matthias Kaehlcke <mka@xxxxxxxxxxxx> wrote: > > > > On Thu, Jun 06, 2019 at 12:46:03PM +0200, Heiko Stuebner wrote: > > > Am Mittwoch, 5. Juni 2019, 23:52:00 CEST schrieb Heiko Stübner: > > > > Am Mittwoch, 5. Juni 2019, 23:24:27 CEST schrieb Matthias Kaehlcke: > > > > > On Wed, Jun 05, 2019 at 11:11:12PM +0200, Heiko Stübner wrote: > > > > > > Am Mittwoch, 5. Juni 2019, 22:43:20 CEST schrieb Matthias Kaehlcke: > > > > > > > This enables wake up on Bluetooth activity when the device is > > > > > > > suspended. The BT_HOST_WAKE signal is only connected on devices > > > > > > > with BT module that are connected through UART. > > > > > > > > > > > > > > Signed-off-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > > > > > > > Signed-off-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx> > > > > > > > > > > > > Housekeeping question, with the two Signed-off-by lines, is Doug the > > > > > > original author, or was this Co-developer-by? > > > > > > > > > > Good question, it's derived from Doug's patch for CrOS 3.14 and > > > > > https://crrev.com/c/1575556 also from Doug. Let's say I did the > > > > > porting to upstream, but I'm pretty sure Doug spent more time on it. > > > > > > > > > > Maybe I should resend it with Doug as author and include the original > > > > > commit message, which has more information. > > > > > > > > It's just that the first Signed-off should be from the original author. > > > > (And the sender the second) > > > > In the co-developed-by case (see Kernel documentation) the order > > > > doesn't matter. > > > > > > Holding off on this patch till we could clarify the authorship. > > > > I'd say let's attribute the authorship to Doug. FTR, the original > > downstream Chrome OS patch is https://crrev.com/c/278190. > > > > Not sure if the information in the commit message of the original > > patch is relevant for the upstream version, in the end it seems LPM > > was never implemented, so it should be sufficient to say what is > > actually done today. > > > > In summary, I propose to take the patch with the current commit > > message, with Doug as the author. Heiko, can you change the authorship > > or should I send a new version? > > > > Doug if you have objections or want updates in the commit message > > (yours tend to be more verbose ;-), holler. > > It's fine with me. while I did just want to apply this one with the relevant adaptions, I didn't get to it early enough and Matthias was faster with his v2 fixed patch, so I've applied that one instead ;-) Heiko