On (22/05/26 14:07), Yunke Cao wrote: > @@ -888,6 +899,12 @@ static int std_validate_compound(const struct v4l2_ctrl *ctrl, u32 idx, > return -EINVAL; > break; > > + case V4L2_CTRL_TYPE_RECT: > + rect = p; > + if (!rect->width || !rect->height) > + return -EINVAL; > + break; Should we allow (0,0,0,0) rectangles or not? From UVC point of view, I assume that anything that is within GET_MIN/GET_MAX is OK. Is GET_MIN always guaranteed to be at least (0,0,1,1) or do some firmwares permit and use (0,0,0,0)? As a "disable ROI" type of thing, for instance.