From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751773AbdI0RVy (ORCPT ); Wed, 27 Sep 2017 13:21:54 -0400 Received: from bombadil.infradead.org ([65.50.211.133]:40260 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751016AbdI0RVw (ORCPT ); Wed, 27 Sep 2017 13:21:52 -0400 Date: Wed, 27 Sep 2017 10:21:50 -0700 From: Darren Hart To: Mario Limonciello Cc: Andy Shevchenko , LKML , platform-driver-x86@vger.kernel.org, quasisec@google.com, pali.rohar@gmail.com Subject: Re: [PATCH v2 03/14] platform/x86: dell-wmi: Don't match on descriptor GUID modalias Message-ID: <20170927172150.GH23572@fury> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.0 (2017-02-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 26, 2017 at 01:50:01PM -0500, Mario Limonciello wrote: > The descriptor GUID is not used to indicate that WMI notifications > in the dell-wmi driver work properly. As such a modalias should > not be present that causes this driver to load on systems with this > GUID. > > Reviewed-by: Pali Pohar > Signed-off-by: Mario Limonciello I can do this manually, but for future reference: Nit, in general, the author's sign off should be first, followed by reviewers and testers. The maintainer will add their sign off at the end - this keeps things clear regarding the development and delivery path. In general, assume chronological order and you'll have the right idea. Informational tags, like Reported, Suggested, Tested, and Fixes can come before the Author sign off, which is consistent with the chronological order. -- Darren Hart VMware Open Source Technology Center