From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.4 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C28DDC3F2C6 for ; Sat, 29 Feb 2020 20:58:01 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 94C1D21556 for ; Sat, 29 Feb 2020 20:58:01 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="W05fmqHU" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727305AbgB2U6B (ORCPT ); Sat, 29 Feb 2020 15:58:01 -0500 Received: from us-smtp-2.mimecast.com ([205.139.110.61]:43203 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726663AbgB2U6B (ORCPT ); Sat, 29 Feb 2020 15:58:01 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1583009880; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=grkL4cK36yhQ8pKdn5AWpPmByRCbwaCyrviGJG1XD54=; b=W05fmqHUF/jAPQwWZ+WLm88PJ633GQrzxNFs1uGNotu/d6u5pNX+67x4BhHQvnmfPTBncd CYNIGgc1U1/tzF84tz1LAoTvcsCBWA08WiF4abjec1MWB+8CqpkpuO8CT6vkAvjZqj1H/1 GV0wR0IsU18wLrj4Z4pff7IEHhdkOwQ= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-386-JNmYBEpmOiS_OJ4qpwEfIQ-1; Sat, 29 Feb 2020 15:57:55 -0500 X-MC-Unique: JNmYBEpmOiS_OJ4qpwEfIQ-1 Received: by mail-wr1-f71.google.com with SMTP id f10so3334941wrv.1 for ; Sat, 29 Feb 2020 12:57:55 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=grkL4cK36yhQ8pKdn5AWpPmByRCbwaCyrviGJG1XD54=; b=PwunKhLoqsKUKkmwgChuEcUCZH1O1RXCNAg5OqJn3/e8vuEICuN/mb2loQTThfBGXs zCvriKuCJm2Wsj/uKNiNwwaan3ZuuzPR01HCVAAYRaX3EOMP/FHnxEPgO1PVNG3UrVV4 /eQ5rEvvgBVmEPoR4r+T1xxKViapgMJoo7w/KmKcjvY9q4uAjrKtU5AVi0Q9ny8FwJ/m Ps5q0oAp5ptrDOteEY/ZF4emphVR+pkWmDl0pAP2BF8/BwoWnY+mViOmtFWlsNWZesGX ZBpdhj1MRbDFXGrrVUvYvSsfqCQ1Ikq/48vtafVGDZIU3/sgocjq0R9cDmBcPpS8KilV 0yxw== X-Gm-Message-State: APjAAAXZaHML9ML7E1oF+G6wojWZdFjPYLR87V1QLruWEFpRdd3sFcdD T9ZN31FOEU2KWr0AX4ok6PTFC6zH13ey1wn5gvrpx8obzf7o2Q4uEmnLQd4+3JOS7P8OEVQW3Tk F/w6Wl70ifqo+4GWBpl7RSA== X-Received: by 2002:adf:fcc4:: with SMTP id f4mr12645581wrs.247.1583009874238; Sat, 29 Feb 2020 12:57:54 -0800 (PST) X-Google-Smtp-Source: APXvYqxPvtTPPm3BpDSKVd+9jIKsf1LtljBDmV1DoLIVXvSaB5EIwbizX9zavy0NQwR82A2rfmIapA== X-Received: by 2002:adf:fcc4:: with SMTP id f4mr12645569wrs.247.1583009874059; Sat, 29 Feb 2020 12:57:54 -0800 (PST) Received: from x1.localdomain (2001-1c00-0c0c-fe00-fc7e-fd47-85c1-1ab3.cable.dynamic.v6.ziggo.nl. [2001:1c00:c0c:fe00:fc7e:fd47:85c1:1ab3]) by smtp.gmail.com with ESMTPSA id s1sm19071282wro.66.2020.02.29.12.57.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 29 Feb 2020 12:57:53 -0800 (PST) Subject: Re: [PATCH resend 2/3] gpiolib: acpi: Rename honor_wakeup option to ignore_wake, add extra quirk From: Hans de Goede To: Andy Shevchenko Cc: Mika Westerberg , Bartosz Golaszewski , Linus Walleij , Marc Lehmann , linux-gpio@vger.kernel.org, linux-acpi@vger.kernel.org References: <20200225102753.8351-1-hdegoede@redhat.com> <20200225102753.8351-3-hdegoede@redhat.com> <20200225105437.GG10400@smile.fi.intel.com> <20200225123425.GK10400@smile.fi.intel.com> <20200225125700.GL10400@smile.fi.intel.com> <44cc5510-571c-21c4-1855-f3141f72eaa3@redhat.com> Message-ID: Date: Sat, 29 Feb 2020 21:57:52 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 MIME-Version: 1.0 In-Reply-To: <44cc5510-571c-21c4-1855-f3141f72eaa3@redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org Hi, On 2/28/20 12:22 PM, Hans de Goede wrote: > Hi, > > On 2/25/20 1:57 PM, Andy Shevchenko wrote: >> On Tue, Feb 25, 2020 at 02:34:25PM +0200, Andy Shevchenko wrote: >>> On Tue, Feb 25, 2020 at 12:26:04PM +0100, Hans de Goede wrote: >> >>> Let's do it as a list of pairs, but in slightly different format (I see some >>> potential to derive a generic parser, based on users described in >>> Documentation/admin-guide/kernel-parameters.txt), i.e. >>> >>>     ignore_wake=pin:controller[,pin:controller[,...]] >> >> Another possible format >> >>     ignore_wake=controller@pin[;controller@pin[;...]] > > I like this one, the other one with the pin first feels wrong, the pin is > part of the controller, not the other way around. > > I will rework the patch series to use the ignore_wake=controller@pin format. Just a quick note. I've changed the separator from ; to , for some reason grub, at least as used in Fedora with Fedora's grub2 BLS (boot loader spec) implementation does not like it when there is a ; in the kernel commandline. I will also send an email about this to Fedora grub maintainer, but for now it is easiest to just avoid the problem. Regards, Hans