diff options
| author | Imre Deak <imre.deak@intel.com> | 2021-10-27 01:50:59 +0300 |
|---|---|---|
| committer | Imre Deak <imre.deak@intel.com> | 2021-11-02 18:07:07 +0200 |
| commit | 2ee5ef9c934ad26376c9282171e731e6c0339815 (patch) | |
| tree | cd5bbea7d350399fe9e46bb27482f6ec1201999a /drivers/gpu/drm/i915/display/intel_fb.c | |
| parent | 112a87c48e83cf226325735009712b515f48c495 (diff) | |
| download | linux-2ee5ef9c934ad26376c9282171e731e6c0339815.tar.gz linux-2ee5ef9c934ad26376c9282171e731e6c0339815.tar.bz2 linux-2ee5ef9c934ad26376c9282171e731e6c0339815.zip | |
drm/i915/fb: Fix rounding error in subsampled plane size calculation
For NV12 FBs with odd main surface tile-row height the CCS surface
height was incorrectly calculated 1 less than the actual value. Fix this
by rounding up the result of divison. For consistency do the same for
the CCS surface width calculation.
Fixes: b3e57bccd68a ("drm/i915/tgl: Gen-12 render decompression")
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211026225105.2783797-2-imre.deak@intel.com
Diffstat (limited to 'drivers/gpu/drm/i915/display/intel_fb.c')
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_fb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_fb.c b/drivers/gpu/drm/i915/display/intel_fb.c index 117e32fb3648..221486dd42e4 100644 --- a/drivers/gpu/drm/i915/display/intel_fb.c +++ b/drivers/gpu/drm/i915/display/intel_fb.c @@ -822,8 +822,8 @@ static void intel_fb_plane_dims(const struct intel_framebuffer *fb, int color_pl intel_fb_plane_get_subsampling(&main_hsub, &main_vsub, &fb->base, main_plane); intel_fb_plane_get_subsampling(&hsub, &vsub, &fb->base, color_plane); - *w = main_width / main_hsub / hsub; - *h = main_height / main_vsub / vsub; + *w = DIV_ROUND_UP(main_width, main_hsub * hsub); + *h = DIV_ROUND_UP(main_height, main_vsub * vsub); } static u32 intel_adjust_tile_offset(int *x, int *y, |