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

[Otavio Salvador]

On Tue, Mar 19, 2013 at 10:41 AM, Gary Thomas <samoht.yrag at gmail.com> wrote:
> 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

So please put this information in the commit log.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br