On Mon, Feb 09, 2015 at 03:56:20PM +0200, Ville Syrjälä wrote: > > @@ -2544,8 +2557,9 @@ skl_allocate_pipe_ddb(struct drm_crtc *crtc, > > * promote the expression to 64 bits to avoid overflowing, the > > * result is < available as data_rate / total_data_rate < 1 > > */ > > - plane_blocks = div_u64((uint64_t)alloc_size * data_rate, > > - total_data_rate); > > + plane_blocks = minimum[plane]; > > + plane_blocks += div_u64((uint64_t)alloc_size * data_rate, > > + total_data_rate); > > The minimum[] array seems pointless. The value here is always going to > be 8. Not in the future with new features appearing. -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx