ascent12 / drm_info

Moved to https://gitlab.freedesktop.org/emersion/drm_info
MIT License
63 stars 13 forks source link

Add support for drmModeGetFB2 #50

Closed emersion closed 4 years ago

emersion commented 4 years ago

This relies on 1.

emersion commented 4 years ago

Hmm. Since this depends on a new libdrm function, we can't use the subproject trick, we'll need to wait for Debian to get a recent enough libdrm. Maybe that's a case where an #ifdef is in order?

ascent12 commented 4 years ago

Yeah, I think an #ifdef is appropriate here.

emersion commented 4 years ago

This is now ready for review. Tested with libdrm-git and drm-tip (linux 5.6).

progandy commented 4 years ago

Is there a reason the commit message mentions drmModeAddFB2 when the code calls drmModeGetFB2?

emersion commented 4 years ago

Good catch, fixed.

emersion commented 4 years ago

Linux 5.7 is now in Arch repos and has the new GETFB2 ioctl.

ascent12 commented 4 years ago

Thanks