On Mon, 16 Nov 2015, Maarten Lankhorst <[email protected]> 
wrote:
> Op 13-11-15 om 18:16 schreef [email protected]:
>> From: Ville Syrjälä <[email protected]>
>>
>> Let's set crtc_y to 0 instead of setting src_y twice.
>>
>> Multiple assignments in one statement is a good way to hide bugs.
>> Please don't do that.
>>
>> Cc: Maarten Lankhorst <[email protected]>
>> Fixes: be5651f2d581 ("drm/i915: Update missing properties in 
>> find_initial_plane_obj")
>> Signed-off-by: Ville Syrjälä <[email protected]>
>>
> Oops.
>
> R-b

Pushed to drm-intel-fixes, thanks for the patch and review.

Maarten, please give Damien and our tooling a chance and spell out the
R-b...

BR,
Jani.



> _______________________________________________
> Intel-gfx mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to