From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH] gpio: acpi: Skip _AEI entries without a handler rather then aborting the scan Date: Tue, 27 Jun 2017 12:12:09 +0300 Message-ID: <1498554729.22624.178.camel@linux.intel.com> References: <20170623072613.3768-1-hdegoede@redhat.com> <20170626160814.GO629@lahna.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga02.intel.com ([134.134.136.20]:63022 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751513AbdF0JMO (ORCPT ); Tue, 27 Jun 2017 05:12:14 -0400 In-Reply-To: <20170626160814.GO629@lahna.fi.intel.com> Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Mika Westerberg , Hans de Goede Cc: Heikki Krogerus , Linus Walleij , linux-gpio@vger.kernel.org On Mon, 2017-06-26 at 19:08 +0300, Mika Westerberg wrote: > On Fri, Jun 23, 2017 at 09:26:13AM +0200, Hans de Goede wrote: > > acpi_walk_resources will stop as soon as the callback passed in > > returns > > an error status. On a x86 tablet I have the first GpioInt in the > > _AEI > > resource list has no handler defined in the DSDT, causing > > acpi_walk_resources to abort scanning the rest of the resource list, > > which does define valid ACPI GPIO events. > > > > This commit changes the return for not finding a handler from > > AE_BAD_PARAMETER to AE_OK so that the rest of the resource list will > > get scanned normally in case of missing event handlers. > > > > Signed-off-by: Hans de Goede > > Acked-by: Mika Westerberg Acked-by: Andy Shevchenko (based on private discussion with Mika) -- Andy Shevchenko Intel Finland Oy