svn commit: r286524 - in head: . etc sys/dev/drm sys/dev/drm2

Ed Schouten ed at nuxi.nl
Tue Aug 11 14:01:56 UTC 2015


Hi Koop,

2015-08-09 14:58 GMT+02:00 Koop Mast <kwm at 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 <ed at nuxi.nl>
Nuxi, 's-Hertogenbosch, the Netherlands
KvK-nr.: 62051717


More information about the svn-src-head mailing list