[Intel-gfx] [PATCH] drm/i915: Remove useless casts to intel_plane_state

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Thu Jan 12 14:52:16 UTC 2017

Op 12-01-17 om 10:58 schreef Ville Syrjälä:
> On Thu, Jan 12, 2017 at 10:43:45AM +0100, Maarten Lankhorst wrote:
>> The visible member used to be in intel_plane_state->visible,
>> but has been moved to drm_plane_state->visible. In the conversion
>> some casts were left in that are now useless.
>> to_intel_plane_state(x)->base.visible is the same as x->visible,
>> so use the latter to clear up the code a little.
> I would generally prefer switching over to intel_foo_state all over.
> But patch looks sane anyway
> Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
Applied, thanks for reviewing. :)

More information about the Intel-gfx mailing list