[PATCH 1/2] drm: Add infrastructure for CRTC background color property

Emil Velikov emil.l.velikov at gmail.com
Wed Nov 18 15:09:45 PST 2015


Hi Matt,

On 23 October 2015 at 01:25, Matt Roper <matthew.d.roper at intel.com> wrote:

> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h

> +typedef struct {
> +       uint64_t v;
> +} drm_rgba_t;
> +
Humble request - please don't add typedefs. The drm subsystem (barring
legacy core and certain drivers) is relatively clean of them. The
extra "struct" is not that much to type :-)

Thanks
Emil


More information about the dri-devel mailing list