Re: [Intel-gfx] [PATCH 1/5] drm/i915: don't recheck for invalid pipe bpp

2012-09-24 Thread Rodrigo Vivi
Feel free to use: Reviewed-by: Rodrigo Vivi On Thu, Sep 20, 2012 at 6:36 PM, Paulo Zanoni wrote: > From: Paulo Zanoni > > As noticed by Daniel Vetter, intel_pipe_choose_bpp_dither should > already check for invalid bpp values and set a valid value, so remove > the recheck inside ironlake_crtc_m

[Intel-gfx] [PATCH 1/5] drm/i915: don't recheck for invalid pipe bpp

2012-09-20 Thread Paulo Zanoni
From: Paulo Zanoni As noticed by Daniel Vetter, intel_pipe_choose_bpp_dither should already check for invalid bpp values and set a valid value, so remove the recheck inside ironlake_crtc_mode_set and also replace a "default" switch case inside ironlake_set_pipeconf with a BUG(). Signed-off-by: P