[meta-freescale] [meta-fsl-arm][PATCH] xf86-dri-vivante: Fix build with latest X server base

[Gary Thomas]

On 2013-03-19 07:33, Otavio Salvador wrote:
> On Tue, Mar 19, 2013 at 10:19 AM, Gary Thomas <samoht.yrag at gmail.com> wrote:
>> mibstore functions are no longer supported.  This patch
>> removes them from this driver.
>>
>> Signed-off-by: Gary Thomas <gary at mlbassoc.com>
>> ---
>>   .../xf86-dri-vivante/remove-mibstore.patch         |   24
>> ++++++++++++++++++++
>>   .../xorg-driver/xf86-dri-vivante_12.09.01.bb       |    4 +++-
>>   2 files changed, 27 insertions(+), 1 deletion(-)
>>   create mode 100644
>> recipes-graphics/xorg-driver/xf86-dri-vivante/remove-mibstore.patch
>>
>> diff --git
>> a/recipes-graphics/xorg-driver/xf86-dri-vivante/remove-mibstore.patch
>> b/recipes-graphics/xorg-driver/xf86-dri-vivante/remove-mibstore.patch
>> new file mode 100644
>> index 0000000..1c48563
>> --- /dev/null
>> +++ b/recipes-graphics/xorg-driver/xf86-dri-vivante/remove-mibstore.patch
>> @@ -0,0 +1,24 @@
>> +Index:
>> xserver-xorg-video-imx-viv-12.09.01/src/vivante_util/vivante_common.h
>> +===================================================================
>> +---
>> xserver-xorg-video-imx-viv-12.09.01.orig/src/vivante_util/vivante_common.h
>> ++++ xserver-xorg-video-imx-viv-12.09.01/src/vivante_util/vivante_common.h
>> +@@ -48,7 +48,6 @@ extern "C" {
>> + + + #include "mipointer.h"
>> +-#include "mibstore.h"
>> + #include "micmap.h"
>> + #include "mipointrst.h"
>> + #include "inputstr.h"
>> +Index:
>> xserver-xorg-video-imx-viv-12.09.01/src/vivante_fbdev/vivante_fbdev_driver.c
>> +===================================================================
>> +---
>> xserver-xorg-video-imx-viv-12.09.01.orig/src/vivante_fbdev/vivante_fbdev_driver.c
>> ++++
>> xserver-xorg-video-imx-viv-12.09.01/src/vivante_fbdev/vivante_fbdev_driver.c
>> +@@ -746,7 +746,6 @@ VivScreenInit(SCREEN_INIT_ARGS_DECL) {
>> + + +     xf86SetBlackWhitePixels(pScreen);
>> +-    miInitializeBackingStore(pScreen);
>> +     xf86SetBackingStore(pScreen);
>> + +     pScrn->vtSema = TRUE;
>
> What the side effects of this change?

Unknown, but it seems to work fine.  I found this same change in other
drivers in OE, e.g.
   http://patches.openembedded.org/patch/46133/

This checkin shows when/where that file was deleted and mentions the reasons.
   http://lists.x.org/archives/xorg-devel/2012-September/033575.html

>
>> diff --git a/recipes-graphics/xorg-driver/xf86-dri-vivante_12.09.01.bb
>> b/recipes-graphics/xorg-driver/xf86-dri-vivante_12.09.01.bb
>> index acc5479..9f63fb8 100644
>> --- a/recipes-graphics/xorg-driver/xf86-dri-vivante_12.09.01.bb
>> +++ b/recipes-graphics/xorg-driver/xf86-dri-vivante_12.09.01.bb
>> @@ -6,7 +6,9 @@ SECTION = "x11/base"
>>   DEPENDS = "virtual/libx11 util-macros xf86-video-imxfb-vivante"
>>   LIC_FILES_CHKSUM =
>> "file://src/dri.h;enline=27;md5=79a9064e49ae41adca4827cd2281a014"
>>   -SRC_URI = "${FSL_MIRROR}/dri-xorg-graphic-imx-viv-${PV}.bin;fsl-eula=true"
>> +SRC_URI = "${FSL_MIRROR}/dri-xorg-graphic-imx-viv-${PV}.bin;fsl-eula=true \
>> +           file://remove-mibstore.patch \
>> +"
>>   SRC_URI[md5sum] = "8c90045cd5f4dba81095856634ba5136"
>>   SRC_URI[sha256sum] =
>> "c844dc180e43901359bbdb4f797ab178b3821fbf63bdee9577e5a0afe5d7f6ad"
>>   -- 1.7.9.5