Hi Ricardo, Thanks for the review. On Wed, 2024-06-19 at 08:15 +0200, Ricardo Ribalda wrote: > > nit: The following would probably be more correct but less clear: > > if (is_relative_ptz_ctrl(xctrl->id)) > min = -max; > else > min = mapping->get(mapping, UVC_GET_MIN,...) > > So up to you what do you/Laurent what is better ;) > > I like this better. I'll send a v4. Best regards, Gergo Koteles