drm/rockchip: Declare framebuffer width/height bounds

The VOP2 has limitations on its input and output sizes. The clipped
display region must be at least 4px in each dimension for both
framebuffer source and plane destination, and the clipped source region
must be no greater than a per-version limit.

It is never valid for VOP2 to have a framebuffer which is less than four
pixels in either dimension, so declare that as our min width/height,
enforced by AddFB failing if the user tries. It can theoretically be
valid to have a single large framebuffer of which only certain clipped
regions are shown, but this is a very uncommon case. Declaring to
userspace that the framebuffer's maximum width and height is the maximum
source clip helps it make better decisions as to which mode to use,
instead of trying unsupported sizes and having to fall back.

Signed-off-by: Daniel Stone <daniels@collabora.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://lore.kernel.org/r/20251015110042.41273-3-daniels@collabora.com
This commit is contained in:
Daniel Stone
2025-10-15 12:00:31 +01:00
committed by Heiko Stuebner
parent f233921d98
commit 4bfaa85bb5

View File

@@ -2647,6 +2647,12 @@ static int vop2_bind(struct device *dev, struct device *master, void *data)
if (IS_ERR(vop2->map))
return PTR_ERR(vop2->map);
/* Set the bounds for framebuffer creation */
drm->mode_config.min_width = 4;
drm->mode_config.min_height = 4;
drm->mode_config.max_width = vop2_data->max_input.width;
drm->mode_config.max_height = vop2_data->max_input.height;
ret = vop2_win_init(vop2);
if (ret)
return ret;