[meta-intel] [PATCH 2/4 v3] meta-haswell-wc: provide machine configuration for haswell-wc

[Ong, Boon Leong]

> -----Original Message-----
> From: Zanussi, Tom
> Sent: Saturday, September 28, 2013 6:33 AM
> To: Ong, Boon Leong
> Cc: meta-intel at yoctoproject.org
> Subject: Re: [meta-intel] [PATCH 2/4 v3] meta-haswell-wc: provide machine
> configuration for haswell-wc
> 
> On Sat, 2013-09-28 at 00:26 +0800, boon.leong.ong at intel.com wrote:
> > From: Ong Boon Leong <boon.leong.ong at intel.com>
> >
> > Create a new machine named 'haswell-wc'.
> >
> > Signed-off-by: Ong Boon Leong <boon.leong.ong at intel.com>
> > ---
> >  meta-haswell-wc/conf/machine/haswell-wc.conf |   24
> ++++++++++++++++++++++++
> >  1 file changed, 24 insertions(+)
> >  create mode 100644 meta-haswell-wc/conf/machine/haswell-wc.conf
> >
> > diff --git a/meta-haswell-wc/conf/machine/haswell-wc.conf
> > b/meta-haswell-wc/conf/machine/haswell-wc.conf
> > new file mode 100644
> > index 0000000..a8982e8
> > --- /dev/null
> > +++ b/meta-haswell-wc/conf/machine/haswell-wc.conf
> > @@ -0,0 +1,24 @@
> > +#@TYPE: Machine
> > +#@NAME: haswell-wc
> > +
> > +#@WEBTITLE: Intel 4th Generation Core Platforms: Core i3, i5, i7
> > +(Haswell)
> > +
> > +#@DESCRIPTION: Machine configuration for Shark Bay systems # i.e.
> > +Haswell + Lynx Point
> > +
> > +PREFERRED_VERSION_linux-yocto ?= "3.8%"
> > +MACHINE_FEATURES += "pcbios efi va-impl-intel"
> > +
> > +require conf/machine/include/tune-x86_64.inc
> > +require conf/machine/include/ia32-base.inc
> > +
> > +MACHINE_HWCODECS ?= "va-intel gst-va-intel"
> > +
> > +XSERVER ?= "${XSERVER_IA32_BASE} \
> > +           ${XSERVER_IA32_EXT} \
> > +	   ${XSERVER_IA32_I965} \
> > +           "
> > +
> > +# This Kernel Option is required to relax ACPI resource conflict
> > +complaints # happens when loading SMBus driver APPEND +=
> > +"acpi_enforce_resources=lax"
> 
> Could you explain why 'acpi_enforce_resources=lax' is needed?  I see that
> crystalforest-server and crystalforest-gladden say they need it for 'Intel
> Quick Assist Technology Software' (why does Quick Assist need it?), but why
> is it needed here (I see it's causes complaints - what happens, just a
> message in the kernel log or does the system not boot?
> 
[Ong, Boon Leong] 
	"acpi_enforce_resources=lax" was ported in from baytrail platform for haswell platform because of the "acpi_check_resource_conflict()" error in i801_probe() that happened in baytrail platform.
	I have just double-confirmed that on haswell platform with updated BIOS, there is no need for this any more. I will resubmit another revision to this patch soon. 
	Thanks for the good question.

> Thanks,
> 
> Tom
> 
>