From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752470AbcF1IT2 (ORCPT ); Tue, 28 Jun 2016 04:19:28 -0400 Received: from mga02.intel.com ([134.134.136.20]:24288 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752437AbcF1ITY (ORCPT ); Tue, 28 Jun 2016 04:19:24 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,540,1459839600"; d="scan'208";a="726112922" Date: Tue, 28 Jun 2016 11:19:17 +0300 From: Mika Westerberg To: Tan Jui Nee Cc: heikki.krogerus@linux.intel.com, andriy.shevchenko@linux.intel.com, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org, ptyser@xes-inc.com, lee.jones@linaro.org, linus.walleij@linaro.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, jonathan.yong@intel.com, ong.hock.yu@intel.com, weifeng.voon@intel.com, wan.ahmad.zainie.wan.mohamad@intel.com Subject: Re: [PATCH v5 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake GPIO pinctrl in non-ACPI system Message-ID: <20160628081917.GF1711@lahna.fi.intel.com> References: <1467100608-11549-1-git-send-email-jui.nee.tan@intel.com> <1467100608-11549-4-git-send-email-jui.nee.tan@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1467100608-11549-4-git-send-email-jui.nee.tan@intel.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.6.1 (2016-04-27) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 28, 2016 at 03:56:48PM +0800, Tan Jui Nee wrote: > +config X86_INTEL_APL > + bool "Non-ACPI support for Intel Apollo Lake platforms" > + select PINCTRL > + ---help--- > + Select this option to enable MMIO BAR access over the P2SB for > + non-ACPI Intel Apollo Lake SoC platforms. This driver uses the P2SB > + hide/unhide mechanism cooperatively to pass the PCI BAR address to > + the platform driver, currently GPIO. > + Sorry but IMHO this still does not explain the option clearly enough. You are using this in some automotive IVI system with specialized bootloader, correct? Why not use that here instead? Something along the lines of config X86_INTEL_IVI bool "Intel in-vehicle infotainment (IVI) systems used in cars"