lkml.org 
[lkml]   [2016]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v3 2/3] x86/platform/p2sb: New Primary to Sideband bridge support driver for Intel SOC's
Date


> -----Original Message-----
> From: Mika Westerberg [mailto:mika.westerberg@linux.intel.com]
> Sent: Tuesday, June 21, 2016 3:24 PM
> To: Tan, Jui Nee <jui.nee.tan@intel.com>
> Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>;
> heikki.krogerus@linux.intel.com; tglx@linutronix.de; mingo@redhat.com;
> hpa@zytor.com; x86@kernel.org; ptyser@xes-inc.com;
> lee.jones@linaro.org; linux-gpio@vger.kernel.org; linux-
> kernel@vger.kernel.org; Linus Walleij <linus.walleij@linaro.org>; Yong,
> Jonathan <jonathan.yong@intel.com>; Yu, Ong Hock
> <ong.hock.yu@intel.com>; Voon, Weifeng <weifeng.voon@intel.com>; Wan
> Mohamad, Wan Ahmad Zainie
> <wan.ahmad.zainie.wan.mohamad@intel.com>
> Subject: Re: [PATCH v3 2/3] x86/platform/p2sb: New Primary to Sideband
> bridge support driver for Intel SOC's
>
> On Tue, Jun 21, 2016 at 05:03:20AM +0000, Tan, Jui Nee wrote:
> > > Maybe, or make it modular and use MODULE_DEVICE_TABLE(x86cpu, ...)
> > > to match the corresponding CPUs.
> >
> > We need CONFIG_X86_INTEL_NON_ACPI Kconfig option to select
> CONFIG_PINCTRL.
> > This is to solve kbuidbot complaint about kernel configuration, i.e.
> > CONFIG_PINCTRL=n. Appreciate if you could advise something on this.
>
> Good point.
>
> Then I guess you might add similar Kconfig option but can you call it
> something else than CONFIG_X86_INTEL_NON_ACPI. Perhaps something
> that relates to the actual product so distro people can then decide whether
> they want to support it or not. Along the lines of CONFIG_X86_INTEL_CE and
> so on.
Thanks for your suggestion. CONFIG_X86_INTEL_NON_ACPI will be renamed
to CONFIG_X86_INTEL_APL in next patch-set.

\
 
 \ /
  Last update: 2016-06-28 10:21    [W:0.070 / U:0.200 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site