Re: [PATCH v7 7/8] media: imagination: Round to closest multiple for cropping region
From: Andy Shevchenko
Date: Fri May 10 2024 - 11:11:10 EST
On Fri, May 10, 2024 at 12:10:01AM +0530, Devarsh Thakkar wrote:
> If neither of the flags to round down (V4L2_SEL_FLAG_LE) or round up
> (V4L2_SEL_FLAG_GE) are specified by the user, then round to nearest
> multiple of requested value while updating the crop rectangle coordinates.
>
> Use the rounding macro which gives preference to rounding down in case two
> nearest values (high and low) are possible to raise the probability of
> cropping rectangle falling inside the bound region.
This is arguable. How do we know that the bigger range is supported?
The safest side is to go smaller than bigger.
--
With Best Regards,
Andy Shevchenko