From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933067AbdEVK5F (ORCPT ); Mon, 22 May 2017 06:57:05 -0400 Received: from mail-wr0-f182.google.com ([209.85.128.182]:34874 "EHLO mail-wr0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752693AbdEVK5E (ORCPT ); Mon, 22 May 2017 06:57:04 -0400 Date: Mon, 22 May 2017 11:57:00 +0100 From: Lee Jones To: Hans de Goede Cc: linux-kernel@vger.kernel.org, Andy Shevchenko Subject: Re: [PATCH v4 1/2] mfd: intel_soc_pmic: Select designware i2c-bus driver Message-ID: <20170522105700.dyw7ade37z366nby@dell> References: <20170515182322.7583-1-hdegoede@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170515182322.7583-1-hdegoede@redhat.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 15 May 2017, Hans de Goede wrote: > The Crystal Cove PMIC provides an ACPI OPRegion handler, which must be > available before other drivers using it are loaded, which is why > INTEL_SOC_PMIC is a bool. > > Just having the driver is not enough, the driver for the i2c-bus must > also be built in, to ensure this, this patch adds a select for it. > > This fixes errors like these during boot: > > mmc0: SDHCI controller on ACPI [80860F14:00] using ADMA > ACPI Error: No handler for Region [REGS] (ffff93543b0cc3a8) [UserDefinedRegion] (20170119/evregion-166) > ACPI Error: Region UserDefinedRegion (ID=143) has no handler (20170119/exfldio-299) > ACPI Error: Method parse/execution failed [\_SB.PCI0.I2C7.PMI5.GET] (Node ffff93543b0cde10), AE_NOT_EXIST (20170119/psparse-543) > ACPI Error: Method parse/execution failed [\_SB.PCI0.SHC1._PS0] (Node ffff93543b0b5cd0), AE_NOT_EXIST (20170119/psparse-543) > acpi 80860F14:02: Failed to change power state to D0 > > While at it this patch also changes the human readable name of the Kconfig > option to make clear the INTEL_SOC_PMIC option selects support for the > Intel Crystal Cove PMIC and documents why this is a bool. > > Cc: Andy Shevchenko > Signed-off-by: Hans de Goede Would be good to get an Ack by the users/maintainers of this driver. > --- > Note this patch will partially conflicts with (contains the same changes as) > a patch in Andy Shevchenko's tree. Has this been sorted now? > --- > Changes in v2: > -Fix Kconfig depends and selects to fix warning reported by kbuild test robot > -Improve commit msg (add example of ACPI errors this avoids) > Changes in v3: > -No changes > Changes in v4: > -Add explanation why this is a bool and why it selects i2c-designwaree > to the help text rather then as comments in the Kconfig > --- > drivers/mfd/Kconfig | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 1b6a83a4a114..61ebda81e134 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -448,17 +448,24 @@ config LPC_SCH > > config INTEL_SOC_PMIC > bool "Support for Crystal Cove PMIC" > - depends on GPIOLIB > - depends on I2C=y > + depends on HAS_IOMEM > + select GPIOLIB > + select I2C > select MFD_CORE > select REGMAP_I2C > select REGMAP_IRQ > + select COMMON_CLK > + select I2C_DESIGNWARE_PLATFORM > help > Select this option to enable support for Crystal Cove PMIC > on some Intel SoC systems. The PMIC provides ADC, GPIO, > thermal, charger and related power management functions > on these systems. > > + This option is a bool as it provides an ACPI Opregion which must be Nit: s/Opregion/OpRegion/ > + available before any devices using it are probed. This option also > + causes the designware-i2c driver to be builtin for the same reason. > + > config INTEL_SOC_PMIC_BXTWC > tristate "Support for Intel Broxton Whiskey Cove PMIC" > depends on INTEL_PMC_IPC -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog