From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932123AbeEWIys (ORCPT ); Wed, 23 May 2018 04:54:48 -0400 Received: from mga09.intel.com ([134.134.136.24]:15903 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754473AbeEWIyq (ORCPT ); Wed, 23 May 2018 04:54:46 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,432,1520924400"; d="scan'208";a="60994321" Message-ID: Subject: Re: [PATCH v2] pinctrl: cherryview: limit Strago DMI workarounds to version 1.0 From: Andy Shevchenko To: Dmitry Torokhov , Mika Westerberg , Linus Walleij Cc: Heikki Krogerus , Guenter Roeck , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org Date: Wed, 23 May 2018 11:54:43 +0300 In-Reply-To: <20180522204753.GA260493@dtor-ws> References: <20180522204753.GA260493@dtor-ws> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.28.1-2 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2018-05-22 at 13:47 -0700, Dmitry Torokhov wrote: > As Google/Intel will fix the BIOS/Coreboot issues with hardcoding > virtual interrupt numbers for keyboard/touchpad/touchscreen > controllers > in ACPI tables, they will also update BOARD version number from 1.0 > to 1.1. Let's limit the DMI quirks that try to preserve virtual IRQ > numbers on Strago boards to those that still carry older BIOSes. > > Note that ideally not BOARD but BIOS version should have been updated. > However the BIOS version used by Chrome devices has format of > Google_BUILD.BRANCH.PATCH which is not well suited for DMI matching as > we do not have "less than" match mode for DMI data. > Acked-by: Andy Shevchenko > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=197953 > Signed-off-by: Dmitry Torokhov > --- > > v1->v2: > > - switch from matching on DMI_BIOS_VERSION to DMI_BOARD_VERSION > > > drivers/pinctrl/intel/pinctrl-cherryview.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/pinctrl/intel/pinctrl-cherryview.c > b/drivers/pinctrl/intel/pinctrl-cherryview.c > index b1ae1618fefea..44133e6846303 100644 > --- a/drivers/pinctrl/intel/pinctrl-cherryview.c > +++ b/drivers/pinctrl/intel/pinctrl-cherryview.c > @@ -1527,6 +1527,7 @@ static const struct dmi_system_id > chv_no_valid_mask[] = { > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "GOOGLE"), > DMI_MATCH(DMI_PRODUCT_FAMILY, > "Intel_Strago"), > + DMI_MATCH(DMI_BOARD_VERSION, "1.0"), > }, > }, > { > @@ -1534,6 +1535,7 @@ static const struct dmi_system_id > chv_no_valid_mask[] = { > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "HP"), > DMI_MATCH(DMI_PRODUCT_NAME, "Setzer"), > + DMI_MATCH(DMI_BOARD_VERSION, "1.0"), > }, > }, > { > @@ -1541,6 +1543,7 @@ static const struct dmi_system_id > chv_no_valid_mask[] = { > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "GOOGLE"), > DMI_MATCH(DMI_PRODUCT_NAME, "Cyan"), > + DMI_MATCH(DMI_BOARD_VERSION, "1.0"), > }, > }, > { > @@ -1548,6 +1551,7 @@ static const struct dmi_system_id > chv_no_valid_mask[] = { > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "GOOGLE"), > DMI_MATCH(DMI_PRODUCT_NAME, "Celes"), > + DMI_MATCH(DMI_BOARD_VERSION, "1.0"), > }, > }, > {} > -- > 2.17.0.441.gb46fe60e1d-goog > > -- Andy Shevchenko Intel Finland Oy