On Thu, 2012-07-05 at 07:24 +0200, Gwenole Beauchesne wrote: 
> 2012/7/5 Xiang, Haihao <haihao.xi...@intel.com>:
> >> +/** \brief Base DRM state. */
> >> +struct drm_state {
> >> +    /** \brief DRM connection descriptor. */
> >> +    int         fd;
> >> +    /** \brief DRM connection type. */
> >> +    int         type;
> >> +};
> >
> > It doesn't make sense to me. Does it mean DRM connection also is based
> > on DRI1/DRI2 protocols ?
> 
> No, but a VA_DRM type could be added, or to replace the "dummy" value.

It confused me a bit. This patch adds the following *DRM* connection
types:

+/** \brief DRM connection types. */
+enum {
+    VA_NONE     = 0,
+    VA_DRI1     = 1,
+    VA_DRI2     = 2,
+    VA_DUMMY    = 3,
+};

so I think VA_DRI1 and VA_DRI2 all are one of *DRM* connect type.


_______________________________________________
Libva mailing list
Libva@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to