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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Matt,

On 23 October 2015 at 01:25, Matt Roper <matthew.d.roper@xxxxxxxxx> 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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux