[PATCHv2 0/1] meta-intel misc pull request

[PATCHv2 0/1] meta-intel misc pull request

[nitin.a.kamble]

From: Nitin A Kamble <nitin.a.kamble@intel.com>

Updated the commit log as per Darren's recommendations.

Thanks,
Nitin

The following changes since commit bc6f631c1eaa7154e667325da8fd636a05628f87:

  jasperforest: uprev v3.4 kernel commit ids (2012-10-04 14:21:55 -0500)

are available in the git repository at:
  git://git.yoctoproject.org/meta-intel-contrib nitin/misc
  http://git.yoctoproject.org/cgit.cgi/meta-intel-contrib/log/?h=nitin/misc

Nitin A Kamble (1):
  crownbay.conf: add kernel parameters for EMGD video acceleration

 meta-crownbay/conf/machine/crownbay.conf |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
1.7.3.4

[PATCHv2 1/1] crownbay.conf: add kernel parameters for EMGD video acceleration

[nitin.a.kamble]

From: Nitin A Kamble <nitin.a.kamble@intel.com>

This is recommended in the EMGD User Guide.

My understanding is the emgd kernel driver need to allocate memory
dynamically, and the "vmalloc=256MB" parameter ensures enough will
be available for the driver.

Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
---
 meta-crownbay/conf/machine/crownbay.conf |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta-crownbay/conf/machine/crownbay.conf b/meta-crownbay/conf/machine/crownbay.conf
index 40dbd1d..f02615e 100644
--- a/meta-crownbay/conf/machine/crownbay.conf
+++ b/meta-crownbay/conf/machine/crownbay.conf
@@ -21,7 +21,7 @@ PREFERRED_VERSION_xserver-xorg ?= "1.9.3"
 PREFERRED_VERSION_mesa-dri ?= "7.11"
 PREFERRED_VERSION_xf86-input-evdev ?= "2.6.0"
 
-APPEND += "video=vesafb vga=0x318"
+APPEND += "vga=current vmalloc=256MB"
 
 VA_FEATURES = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", \
 	    "commercial", "gst-va-intel va-intel", "va-intel", d)}"
-- 
1.7.3.4

Re: [PATCHv2 1/1] crownbay.conf: add kernel parameters for EMGD video acceleration

[Darren Hart]

On 10/04/2012 03:23 PM, nitin.a.kamble@intel.com wrote:
> From: Nitin A Kamble <nitin.a.kamble@intel.com>
> 
> This is recommended in the EMGD User Guide.
> 
> My understanding is the emgd kernel driver need to allocate memory
> dynamically, and the "vmalloc=256MB" parameter ensures enough will
> be available for the driver.

OK, this explains the change to vmalloc.

Why the change of vga=current and the drop of video=vesafb ?

--
Darren

> 
> Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
> ---
>  meta-crownbay/conf/machine/crownbay.conf |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta-crownbay/conf/machine/crownbay.conf b/meta-crownbay/conf/machine/crownbay.conf
> index 40dbd1d..f02615e 100644
> --- a/meta-crownbay/conf/machine/crownbay.conf
> +++ b/meta-crownbay/conf/machine/crownbay.conf
> @@ -21,7 +21,7 @@ PREFERRED_VERSION_xserver-xorg ?= "1.9.3"
>  PREFERRED_VERSION_mesa-dri ?= "7.11"
>  PREFERRED_VERSION_xf86-input-evdev ?= "2.6.0"
>  
> -APPEND += "video=vesafb vga=0x318"
> +APPEND += "vga=current vmalloc=256MB"
>  
>  VA_FEATURES = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", \
>  	    "commercial", "gst-va-intel va-intel", "va-intel", d)}"
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Re: [PATCHv2 1/1] crownbay.conf: add kernel parameters for EMGD video acceleration

[Kamble, Nitin A]

> -----Original Message-----
> From: Hart, Darren
> Sent: Thursday, October 04, 2012 3:34 PM
> To: Kamble, Nitin A
> Cc: Zanussi, Tom; yocto@yoctoproject.org
> Subject: Re: [PATCHv2 1/1] crownbay.conf: add kernel parameters for EMGD
> video acceleration
> 
> On 10/04/2012 03:23 PM, nitin.a.kamble@intel.com wrote:
> > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> >
> > This is recommended in the EMGD User Guide.
> >
> > My understanding is the emgd kernel driver need to allocate memory
> > dynamically, and the "vmalloc=256MB" parameter ensures enough will be
> > available for the driver.
> 
> OK, this explains the change to vmalloc.
> 
> Why the change of vga=current and the drop of video=vesafb ?

Darren,
   Good catch. I changed this for cleanup, but now I see that it affects the splash screen. So I will resend the commit by dropping these extra changes.

Thanks,
Nitin


> 
> --
> Darren
> 
> >
> > Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
> > ---
> >  meta-crownbay/conf/machine/crownbay.conf |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta-crownbay/conf/machine/crownbay.conf
> > b/meta-crownbay/conf/machine/crownbay.conf
> > index 40dbd1d..f02615e 100644
> > --- a/meta-crownbay/conf/machine/crownbay.conf
> > +++ b/meta-crownbay/conf/machine/crownbay.conf
> > @@ -21,7 +21,7 @@ PREFERRED_VERSION_xserver-xorg ?= "1.9.3"
> >  PREFERRED_VERSION_mesa-dri ?= "7.11"
> >  PREFERRED_VERSION_xf86-input-evdev ?= "2.6.0"
> >
> > -APPEND += "video=vesafb vga=0x318"
> > +APPEND += "vga=current vmalloc=256MB"
> >
> >  VA_FEATURES = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", \
> >  	    "commercial", "gst-va-intel va-intel", "va-intel", d)}"
> >
> 
> --
> Darren Hart
> Intel Open Source Technology Center
> Yocto Project - Linux Kernel