On Mon, 16 Nov 2015, Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> wrote: > Op 13-11-15 om 18:16 schreef ville.syrjala@xxxxxxxxxxxxxxx: >> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> >> >> 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 <maarten.lankhorst@xxxxxxxxxxxxxxx> >> Fixes: be5651f2d581 ("drm/i915: Update missing properties in find_initial_plane_obj") >> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> >> > 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 > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx