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