Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754495AbaB0Api (ORCPT ); Wed, 26 Feb 2014 19:45:38 -0500 Received: from mail-vc0-f181.google.com ([209.85.220.181]:42129 "EHLO mail-vc0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754452AbaB0Apg (ORCPT ); Wed, 26 Feb 2014 19:45:36 -0500 MIME-Version: 1.0 In-Reply-To: <1670100.tBHUbYFGo0@vostro.rjw.lan> References: <1393429360-4344-1-git-send-email-clancy.kieran@gmail.com> <1670100.tBHUbYFGo0@vostro.rjw.lan> From: Kieran Clancy Date: Thu, 27 Feb 2014 11:15:05 +1030 Message-ID: Subject: Re: [PATCH] ACPI / EC: Clear stale EC events on Samsung systems To: "Rafael J. Wysocki" Cc: Len Brown , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Lan Tianyu , Juan Manuel Cabo , Dennis Jansen Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 27, 2014 at 10:15 AM, Rafael J. Wysocki wrote: > On Thursday, February 27, 2014 02:12:40 AM Kieran Clancy wrote: >> Signed-off-by: Kieran Clancy >> Signed-off-by: Lan Tianyu >> Signed-off-by: Juan Manuel Cabo >> Signed-off-by: Dennis Jansen > > There are too many sign-offs under this patch. I suppose some of them > should be Acked-by or Reviewed-by. > > Are you the author? Sorry about that, I wasn't sure of the best way to acknowledge the people involved. I am the primary author, but I based the loop which calls acpi_ec_query_unlocked() on a patch by Lan Tianyu. Juan provided the initial idea (userspace workaround), and Dennis and Lan Tianyu reviewed and suggested some changes to the code. What would you usually do this kind of situation? I suppose I could put something such as: Signed-off-by: Kieran Clancy Signed-off-by: Lan Tianyu Suggested-by: Juan Manuel Cabo Reviewed-by: Lan Tianyu Reviewed-by: Dennis Jansen Would this be preferable? Thanks, Kieran Clancy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/