[Buildroot] [PATCH 3/3] weston: add weston-imx variant when using imx-gpu-viv
Gary Bisson
gary.bisson at boundarydevices.com
Mon Jun 4 09:52:35 UTC 2018
Hi Arnout
On Thu, May 17, 2018 at 11:19:18PM +0200, Arnout Vandecappelle wrote:
>
>
> On 17-05-18 15:08, Gary Bisson wrote:
> > Hi Arnout,
> >
> > On Wed, May 16, 2018 at 11:24:17PM +0200, Arnout Vandecappelle wrote:
> >>
> >>
> >> On 16-05-18 17:52, Gary Bisson wrote:
> [snip]
> >>> +ifeq ($(BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_WL),y)
> >>> +ifeq ($(BR2_PACKAGE_IMX_GPU_G2D),y)
> >>
> >> WESTON_CONF_OPTS += --enable-imxg2d
> >
> > Ok, I forgot you wanted that --enable-xxx options to be explicit
> > although it ends up in the 'unrecognized options'.
>
> So I took a look at the configure.ac, and of course they got it wrong;
> --enable-imxg2d will disable it. So your patch was correct.
>
> Reading documentation is apparently difficult :-)
Haha, thanks for checking.
> Since I'm not entirely convinced of the config option myself anyway:
>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
Thomas, I've seen you applied the rest of the series, do you have any
objection about that patch?
Regards,
Gary
More information about the buildroot
mailing list