> -----Original Message-----
> From: Mika Westerberg [mailto:[email protected]]
> Sent: Tuesday, June 28, 2016 4:19 PM
> To: Tan, Jui Nee <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; linux-
> [email protected]; [email protected]; Yong, Jonathan
> <[email protected]>; Yu, Ong Hock <[email protected]>; Voon,
> Weifeng <[email protected]>; Wan Mohamad, Wan Ahmad Zainie
> <[email protected]>
> Subject: Re: [PATCH v5 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake
> GPIO pinctrl in non-ACPI system
> 
> 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"
Thanks for your comments. The changes will be applied in next patch-set.

Reply via email to