Hi Koop,

2015-08-09 14:58 GMT+02:00 Koop Mast <k...@freebsd.org>:
> +#define DRM_DEV_GID    44      /* "video" group */

I think it looks a bit weird that DRM has these definitions locally.
<sys/conf.h> already has definitions for various users and groups
(UID_* and GID_*). Would it make sense to add GID_VIDEO to
<sys/conf.h> and change the definitions in drmP.h to the following?

#define DRM_DEV_UID UID_ROOT
#define DRM_DEV_GID GID_VIDEO

-- 
Ed Schouten <e...@nuxi.nl>
Nuxi, 's-Hertogenbosch, the Netherlands
KvK-nr.: 62051717
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to