From: "Eric Bénard" <eric@eukrea.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-arm][danny][PATCH 2/2] xf86-video-imxfb-vivante: fix missing headers
Date: Sat, 16 Feb 2013 19:46:36 +0100 [thread overview]
Message-ID: <20130216194636.2bf7dd7b@e6520eb> (raw)
In-Reply-To: <CAP9ODKq5WSj06HP_R3-gkf8QneHPMPYBAW3ycae-rCLqeEbWDw@mail.gmail.com>
Le Sat, 16 Feb 2013 16:24:42 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :
> On Sat, Feb 16, 2013 at 4:15 PM, Eric Bénard <eric@eukrea.com> wrote:
> > Le Sat, 16 Feb 2013 15:57:36 -0200,
> > Otavio Salvador <otavio@ossystems.com.br> a écrit :
> >
> >> On Sat, Feb 16, 2013 at 2:21 PM, Eric Bénard <eric@eukrea.com> wrote:
> >> > - dri.h and sarea.h are missing during compilation (reproduced on several
> >> > build from scratch using oe-core + meta-oe) leading to a failure to build
> >> > fsl-image-gui (for example)
> >> > - These headers are provided by xf86-dri-vivante which depends on
> >> > xf86-video-imxfb-vivante.
> >> > - So we provide the headers here to fix the compilation and get
> >> > danny build again for imx6qsabrelite.
> >> > - Tested on Fedora 17 and Ubuntu 12.10 hosts, for imx6qsabrelite
> >> > target building fsl-image-gui from scratch.
> >> >
> >> > Signed-off-by: Eric Bénard <eric@eukrea.com>
> >>
> >> Did you try to reverse the dependency? make video driver to depends on
> >> the dri? I think I did it in upcoming 1.1.0 patches.
> >>
> > I tried that and reversing the dependency doesn't work because
> > xf86-dri-vivante needs vivante_priv.h which is provided by
> > xf86-video-imxfb-vivante.
>
> Ah I see; I workarounded it in 1.1.0 upgrade.
>
I didn't saw such a workaround in 1.1.0 and xf86-dri-vivante depends
on xf86-video-imxfb-vivante also in 1.1.0.
Maybe patch 2/2 can be pushed as it fix the problem in 12.09.01 at
least in danny ?
Eric
next prev parent reply other threads:[~2013-02-16 18:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-16 16:21 [meta-fsl-arm][danny][PATCH 1/2] xf86-dri-vivante: fix licence chekcsum Eric Bénard
2013-02-16 16:21 ` [meta-fsl-arm][danny][PATCH 2/2] xf86-video-imxfb-vivante: fix missing headers Eric Bénard
2013-02-16 17:57 ` Otavio Salvador
2013-02-16 18:15 ` Eric Bénard
2013-02-16 18:24 ` Otavio Salvador
2013-02-16 18:44 ` Otavio Salvador
2013-02-16 18:46 ` Eric Bénard [this message]
2013-02-16 18:50 ` Otavio Salvador
2013-02-16 18:00 ` [meta-fsl-arm][danny][PATCH 1/2] xf86-dri-vivante: fix licence chekcsum Otavio Salvador
2013-02-16 18:01 ` Otavio Salvador
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=20130216194636.2bf7dd7b@e6520eb \
--to=eric@eukrea.com \
--cc=meta-freescale@yoctoproject.org \
--cc=otavio@ossystems.com.br \
/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.