Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

[media] media/v4l2-ctrls: Code cleanout validate_new()

We can simplify the code removing the if().

v4l2_ctr_new sets ctrls->elems to 1 when !ctrl->is_ptr.

Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

authored by

Ricardo Ribalda and committed by
Mauro Carvalho Chehab
4366dfef 4a99362d

-15
-15
drivers/media/v4l2-core/v4l2-ctrls.c
··· 1678 1678 unsigned idx; 1679 1679 int err = 0; 1680 1680 1681 - if (!ctrl->is_ptr) { 1682 - switch (ctrl->type) { 1683 - case V4L2_CTRL_TYPE_INTEGER: 1684 - case V4L2_CTRL_TYPE_INTEGER_MENU: 1685 - case V4L2_CTRL_TYPE_MENU: 1686 - case V4L2_CTRL_TYPE_BITMASK: 1687 - case V4L2_CTRL_TYPE_BOOLEAN: 1688 - case V4L2_CTRL_TYPE_BUTTON: 1689 - case V4L2_CTRL_TYPE_CTRL_CLASS: 1690 - case V4L2_CTRL_TYPE_INTEGER64: 1691 - return ctrl->type_ops->validate(ctrl, 0, p_new); 1692 - default: 1693 - break; 1694 - } 1695 - } 1696 1681 for (idx = 0; !err && idx < ctrl->elems; idx++) 1697 1682 err = ctrl->type_ops->validate(ctrl, idx, p_new); 1698 1683 return err;