[linux-yocto] [PATCH 1/1] meta: remove CONFIG_BYT_LPSS_BRD from valleyisland cfg file

[Chang, Rebecca Swee Fun]

Yes, ACPI driver is able to setup themselves. PCI mode drivers rely on the clock framework in the board file.
Therefore we need a board file in PCI driver setup.

> -----Original Message-----
> From: Darren Hart [mailto:dvhart at linux.intel.com]
> Sent: 12 February, 2014 10:59 AM
> To: Chang, Rebecca Swee Fun; linux-yocto at yoctoproject.org
> Subject: Re: [linux-yocto] [PATCH 1/1] meta: remove
> CONFIG_BYT_LPSS_BRD from valleyisland cfg file
> 
> 
> On 2/11/14, 18:53, "Chang, Rebecca Swee Fun"
> <rebecca.swee.fun.chang at intel.com> wrote:
> 
> >No, it is only needed in PCI mode device enumeration.
> 
> Huh... I guess I need to go review what all was in that board file. I thought
> there was some hardware description in there ACPI wasn't yet capable of
> describing. So the ACPI driver is able to setup the SPI bus, for example?
> 
> Thanks Rebecca,
> 
> Darren
> 
> >> -----Original Message-----
> >> From: Darren Hart [mailto:dvhart at linux.intel.com]
> >> Sent: 12 February, 2014 10:32 AM
> >> To: Chang, Rebecca Swee Fun; linux-yocto at yoctoproject.org
> >> Subject: Re: [linux-yocto] [PATCH 1/1] meta: remove
> >> CONFIG_BYT_LPSS_BRD from valleyisland cfg file
> >>
> >> On 2/11/14, 9:31, "rebecca.swee.fun.chang at intel.com"
> >> <rebecca.swee.fun.chang at intel.com> wrote:
> >>
> >> >From: "Chang, Rebecca Swee Fun" <rebecca.swee.fun.chang at intel.com>
> >> >
> >> >Remove CONFIG_BYT_LPSS_BRD from valleyisland.cfg file.
> >> >CONFIG_BYT_LPSS_BRD is a duplicated config. It has been included in
> >> >features/valleyisland-io/valleyisland-io-pci.
> >>
> >> Is it only relevant for the PCI enumeration? Seems to me it would be
> >>needed  for both PCI and ACPI until such time as ACPI Device
> >>Properties lands in the  ACPI 6.0 specification... Yes/no?
> >>
> >> --
> >> Darren
> >>
> >> >
> >> >Signed-off-by: Chang, Rebecca Swee Fun
> >> ><rebecca.swee.fun.chang at intel.com>
> >> >---
> >> > meta/cfg/kernel-cache/bsp/valleyisland/valleyisland.cfg |    1 -
> >> > 1 file changed, 1 deletion(-)
> >> >
> >> >diff --git a/meta/cfg/kernel-cache/bsp/valleyisland/valleyisland.cfg
> >> >b/meta/cfg/kernel-cache/bsp/valleyisland/valleyisland.cfg
> >> >index 0e3a863..140a006 100644
> >> >--- a/meta/cfg/kernel-cache/bsp/valleyisland/valleyisland.cfg
> >> >+++ b/meta/cfg/kernel-cache/bsp/valleyisland/valleyisland.cfg
> >> >@@ -1,6 +1,5 @@
> >> > CONFIG_MCORE2=y
> >> > CONFIG_X86_INTEL_LPSS=y
> >> >-CONFIG_BYT_LPSS_BRD=y
> >> > CONFIG_PRINTK=y
> >> > CONFIG_PRINTK_TIME=y
> >> >
> >> >--
> >> >1.7.10.4
> >> >
> >> >_______________________________________________
> >> >linux-yocto mailing list
> >> >linux-yocto at yoctoproject.org
> >> >https://lists.yoctoproject.org/listinfo/linux-yocto
> >> >
> >>
> >>
> >
> >
> 
>