[Libva] [PATCH] [G45] Fix check for H.264 decoding support (v3)

[Xiang, Haihao]

Hi, 
The following BSD check should be moved to intel_batchbuffer_init(). Others are ok for me.
+    if (intel->has_bsd) {
+        intel->batch_bcs = calloc(1, sizeof(*(intel->batch_bcs)));
+        assert(intel->batch_bcs);
+        intel->batch_bcs->intel = intel;
+        intel->batch_bcs->flag = I915_EXEC_BSD;
+        intel->batch_bcs->run = drm_intel_bo_mrb_exec;
+        intel_batchbuffer_reset(intel->batch_bcs);
+    }


Thanks
Haihao


-----Original Message-----
From: libva-bounces+haihao.xiang=intel.com at lists.freedesktop.org [mailto:libva-bounces+haihao.xiang=intel.com at lists.freedesktop.org] On Behalf Of gbeauchesne at splitted-desktop.com
Sent: Wednesday, January 26, 2011 1:24 AM
To: libva at lists.freedesktop.org
Subject: Re: [Libva] [PATCH] [G45] Fix check for H.264 decoding support (v3)

Hi,

> Similarly to the VC-1 patch, this one makes sure the underlying HW 
> does support H.264 decoding and is actually enabled. e.g. new enough 
> libdrm and kernel module.

Here is an updated patch that defines HAS_H264() on Sandy Bridge or Ironlake only.

Regards,
Gwenole.