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=-0.7 required=3.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS 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 C67A9C48BD1 for ; Thu, 10 Jun 2021 14:25:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B2195613D9 for ; Thu, 10 Jun 2021 14:25:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231210AbhFJO10 (ORCPT ); Thu, 10 Jun 2021 10:27:26 -0400 Received: from mga09.intel.com ([134.134.136.24]:11363 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230153AbhFJO1Y (ORCPT ); Thu, 10 Jun 2021 10:27:24 -0400 IronPort-SDR: m3ke9AlduWM3gybvvkZdqqWTy4ZCOrwBBPUYA3bwgMYf0y/+kbJI9vWQXuZr6EuLC45AfzukQz BOazx+N7ts+w== X-IronPort-AV: E=McAfee;i="6200,9189,10011"; a="205267272" X-IronPort-AV: E=Sophos;i="5.83,263,1616482800"; d="scan'208";a="205267272" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jun 2021 07:25:08 -0700 IronPort-SDR: hCAbLj1fQwzqoL6BHxgCVeU2vNdluBm4WylVZkxZ976mLCh4TDFm/a1VI41LmrHJ6ULT+gAS0z qfq5yc61ZPCQ== X-IronPort-AV: E=Sophos;i="5.83,263,1616482800"; d="scan'208";a="402876746" Received: from smile.fi.intel.com (HELO smile) ([10.237.68.40]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jun 2021 07:25:06 -0700 Received: from andy by smile with local (Exim 4.94) (envelope-from ) id 1lrLcG-001AbZ-6U; Thu, 10 Jun 2021 17:25:04 +0300 Date: Thu, 10 Jun 2021 17:25:04 +0300 From: Andy Shevchenko To: Henning Schild Cc: Mika Westerberg , Linux Kernel Mailing List , "open list:GPIO SUBSYSTEM" , Andy Shevchenko , Linus Walleij Subject: Re: [PATCH] pinctrl: intel: fix NULL pointer deref Message-ID: References: <20210609062722.9132-1-henning.schild@siemens.com> <20210609130816.3631f0aa@md1za8fc.ad001.siemens.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210609130816.3631f0aa@md1za8fc.ad001.siemens.net> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 09, 2021 at 01:08:16PM +0200, Henning Schild wrote: > Am Wed, 9 Jun 2021 13:33:34 +0300 > schrieb Andy Shevchenko : ... > In order to use GPIO from the drivers i need to make sure > "broxton-pinctrl" comes up even if p2sb is hidden. > > Long story short, i thought the patch was simple enough to merge even > taken out of my special context. > > Currently intel_pinctl only works if "ps2b is not hidden by BIOS" or > "ACPI tables are correct", lifting the ban on the hidden p2sb seems > like a useful thing in general (i.e. sysfs gpio interface). And i was > hoping Andy would take the lead on that. It is something my Siemens > drivers would depend on, but really a generic thing as far as i > understand it. >From p2sb series discussion it appears that this patch is not needed. The case is when BIOS already provides an ACPI device. So, the initial bug is in that series that needs to check if the ACPI device is exposed and forbid platform device instantiation in that case. -- With Best Regards, Andy Shevchenko