Commit 912bbf7e authored by Steve Longerbeam's avatar Steve Longerbeam Committed by Philipp Zabel

gpu: ipu-v3: image-convert: Fix image downsize coefficients

The output of the IC downsizer unit in both dimensions must be <= 1024
before being passed to the IC resizer unit. This was causing corrupted
images when:

input_dim > 1024, and
input_dim / 2 < output_dim < input_dim

Some broken examples were 1920x1080 -> 1024x768 and 1920x1080 ->
1280x1080.

Fixes: 70b9b6b3 ("gpu: ipu-v3: image-convert: calculate per-tile
resize coefficients")
Signed-off-by: default avatarSteve Longerbeam <slongerbeam@gmail.com>
Signed-off-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
parent bca4d70c
...@@ -400,12 +400,14 @@ static int calc_image_resize_coefficients(struct ipu_image_convert_ctx *ctx, ...@@ -400,12 +400,14 @@ static int calc_image_resize_coefficients(struct ipu_image_convert_ctx *ctx,
if (WARN_ON(resized_width == 0 || resized_height == 0)) if (WARN_ON(resized_width == 0 || resized_height == 0))
return -EINVAL; return -EINVAL;
while (downsized_width >= resized_width * 2) { while (downsized_width > 1024 ||
downsized_width >= resized_width * 2) {
downsized_width >>= 1; downsized_width >>= 1;
downsize_coeff_h++; downsize_coeff_h++;
} }
while (downsized_height >= resized_height * 2) { while (downsized_height > 1024 ||
downsized_height >= resized_height * 2) {
downsized_height >>= 1; downsized_height >>= 1;
downsize_coeff_v++; downsize_coeff_v++;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment