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

2017-01-12 Thread Maarten Lankhorst
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. >> >

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

2017-01-12 Thread 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 sam

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

2017-01-12 Thread Chris Wilson
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 sam

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

2017-01-12 Thread Maarten Lankhorst
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. Signed-off