All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Zanussi <tom.zanussi@intel.com>
To: Darren Hart <dvhart@linux.intel.com>
Cc: poky@yoctoproject.org
Subject: Re: [PATCH 1/1] n450: fix wrong video driver
Date: Thu, 22 Mar 2012 23:32:54 -0500	[thread overview]
Message-ID: <1332477174.2310.3.camel@elmorro> (raw)
In-Reply-To: <4F6BF3B8.20502@linux.intel.com>

On Thu, 2012-03-22 at 20:53 -0700, Darren Hart wrote:
> 
> On 03/22/2012 06:36 PM, edwin.zhai@intel.com wrote:
> > From: Zhai Edwin <edwin.zhai@intel.com>
> > 
> > n450 should use i915 rather than i965.
> > 
> > [YOCTO #1840] fixed
> > 
> > Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
> > ---
> >  conf/machine/include/ia32-base.inc       |    4 ++++
> >  meta-sugarbay/conf/machine/sugarbay.conf |    2 +-
> >  2 files changed, 5 insertions(+), 1 deletions(-)
> > 
> > diff --git a/conf/machine/include/ia32-base.inc b/conf/machine/include/ia32-base.inc
> > index 7da53f6..9b2ef5a 100644
> > --- a/conf/machine/include/ia32-base.inc
> > +++ b/conf/machine/include/ia32-base.inc
> > @@ -49,6 +49,10 @@ XSERVER_IA32_EXT = "xserver-xorg-extension-dri \
> >             "
> >  
> >  XSERVER_IA32_I915 = "xf86-video-intel \
> > +           mesa-dri-driver-i915 \
> > +           "
> > +
> > +XSERVER_IA32_I965 = "xf86-video-intel \
> >             mesa-dri-driver-i965 \
> >             "
> 
> Looks good.
> 
> > diff --git a/meta-sugarbay/conf/machine/sugarbay.conf b/meta-sugarbay/conf/machine/sugarbay.conf
> > index 776089e..a44f918 100644
> > --- a/meta-sugarbay/conf/machine/sugarbay.conf
> > +++ b/meta-sugarbay/conf/machine/sugarbay.conf
> > @@ -11,5 +11,5 @@ require conf/machine/include/ia32-base.inc
> >  
> >  XSERVER ?= "${XSERVER_IA32_BASE} \
> >             ${XSERVER_IA32_EXT} \
> > -           ${XSERVER_IA32_I915} \
> > +           ${XSERVER_IA32_I965} \
> >             "
> 
> Tom, this one is your call - I presume this is right? Will leave to you
> to pull in, I'm good with the common i915 definition.
> 

Yeah, there was a typo in this cleanup.

/me again learns not to touch anyone else's BSPs.  ;-(

Tom

> Thanks,
> 




  parent reply	other threads:[~2012-03-23  4:32 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-23  1:36 [PATCH 0/1] choose right video driver for n450 edwin.zhai
2012-03-23  1:36 ` [PATCH 1/1] n450: fix wrong video driver edwin.zhai
2012-03-23  3:53   ` Darren Hart
2012-03-23  4:15     ` Zhai, Edwin
2012-03-23  4:32     ` Tom Zanussi [this message]
2012-03-23  3:49 ` [PATCH 0/1] choose right video driver for n450 Darren Hart
2012-03-23  4:08   ` Zhai, Edwin
2012-03-23  6:05     ` Darren Hart
2012-03-23  4:31 ` Tom Zanussi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1332477174.2310.3.camel@elmorro \
    --to=tom.zanussi@intel.com \
    --cc=dvhart@linux.intel.com \
    --cc=poky@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.