From nobody Sun Dec 14 21:34:11 2025 Received: from mail-vk1-f171.google.com (mail-vk1-f171.google.com [209.85.221.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9723F2063E1 for ; Mon, 3 Feb 2025 11:56:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738583772; cv=none; b=TwBVVFZCHxnzi2x26xukpfj5Kaa1YX7I5H4UJBa3Y3lN0mm4ysCEnUhUQX62lqyI8fnAZGDmxM51o9Ty3ZoQWQOqljSd6/bi9XnRe+Co14zr8G+TYzhH770OnZkfgD/Nn7LK3nfr3ePH7I0FIBFRJ3kebzmz72p08zb8nku/t2M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738583772; c=relaxed/simple; bh=XDJx0T33scoarv55Ada/16pLsRQJjIA/CdDzzum4e2E=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=mDBhnXq1k38m4A50oW1j2KBP/lojWaIYC7ryKzsWxMjuU4IDWsF+qjGH23wUSkCmEwjDiCy+rPcOBVgGdbJHcCes3QPrC0IDJbL/Uj5ZtzJq9rZOIk0ejjt2qqmByEBy4hKwzN4U1T420F/ogw1a7ZRkiCfgZXvMgvsN8GYas5M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=EL7wCEHR; arc=none smtp.client-ip=209.85.221.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="EL7wCEHR" Received: by mail-vk1-f171.google.com with SMTP id 71dfb90a1353d-51eb181331bso1257899e0c.0 for ; Mon, 03 Feb 2025 03:56:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1738583769; x=1739188569; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=eCSuaKj/Nx5vcwnNqpNliqFOdiZqLoZIJ7Krw5nQYZo=; b=EL7wCEHRDdMtvAMMZN9/cRSoajjzY4EEht2dtVQPMKfmPfGCJ8j27/8nu4G4IMG5Yt K/Vuln37L++o7WnbTpihMzxvP+cGzIaYU5Bm6KK/Xt8HYwpdrGreIjrCUGEhqdK2P+rr +ZwTAI9B1mcfWL+kciN/u/JjQFHDCjtXRdysY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1738583769; x=1739188569; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=eCSuaKj/Nx5vcwnNqpNliqFOdiZqLoZIJ7Krw5nQYZo=; b=qPPchnJmfJUvt27uNcaPpA4AyW8reR4egdbfgAp6pVDu9z+24zmdniYYP6h4xM9ZDP C9JiQ6KK07AfsFiE7FlMB4cay11jeOKH6z98AwfSzyN61Kvj97pKa+iTTIfJmYGhF2JR m0aJy8zRrFtACSujukd8zHortpevmyg8hcJlNphxMZOq05rggSnYiI0rkB8oDRIH6uKq M+H7iC5edXuRLnWidivdGTyIgsoHlKv3uz3eaSs8uoDFg92v6uNsl1vKfKxws2qFXcl0 dz0pQQci6e/mvieId0k/1raYf68+pzPbS7EseVu1s4U68rzacuTi92p+/YPDSss+Vdit bdsA== X-Forwarded-Encrypted: i=1; AJvYcCV9DTbn941DHMiMZ8QtsuEXmvKQDXrj8ny61EiZQ0F4hipNfeIUcrnIAp/nNitg6aVDHTjdkpm0a0ij40o=@vger.kernel.org X-Gm-Message-State: AOJu0Yxq27oAe6XwdD9pM5cTofoIANodw5ZkcwCt5XHw2VMHsSlW827h iCbyiu5TnemQ5mPCizqYLmWO9V0OTLOFNQYdxF7Rfd5T85FTSsGg7s8XEN3JleDzPqDyxqQjZbn Dzg== X-Gm-Gg: ASbGncvsUpl43d8Fk2VtFPCIWy5eWDZQ77a8Ejx0N7QHDrPnSFZskQ4Mf/YwE7vmEvQ T6xxX4ogF3QI4Q9drEphByjuBQMVyGuajyHSrRQl9ZeI4loyNuZH9A4xpZTw4Ukge2lUmmAJDu9 uToYzuQ3o/8iMheykQ1m/jISCOSvp4Nq7TbWiDPmz1vlbfq1y4zkQ7533+rUti72SKw3igjVWRv Y+udhkg287h19MKSQPzvvRaRIO38KsZZNmKL0Sw6HpNKhbqxa7R2g15vgG7orfB7ajsnYkHvrWg ZVL9GW54X3mzCTMttUUPIYVVoNgLthletQIw44rpVqA+6W1aOSAqJ/UNrywYTMXR6A== X-Google-Smtp-Source: AGHT+IGiKPkKkCdl8AzBK7lyBxS49xfhaKnn9hD0wU8c+BzhADHjxPcS/3Jvh90K0/wtUXUuHzQKJQ== X-Received: by 2002:a05:6102:2926:b0:4b6:35c4:6fcd with SMTP id ada2fe7eead31-4b9a4f1d458mr15145086137.7.1738583769361; Mon, 03 Feb 2025 03:56:09 -0800 (PST) Received: from denia.c.googlers.com (5.236.236.35.bc.googleusercontent.com. [35.236.236.5]) by smtp.gmail.com with ESMTPSA id ada2fe7eead31-4b9baa9aefcsm1592770137.12.2025.02.03.03.56.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Feb 2025 03:56:08 -0800 (PST) From: Ricardo Ribalda Date: Mon, 03 Feb 2025 11:55:49 +0000 Subject: [PATCH v17 13/17] media: uvcvideo: let v4l2_query_v4l2_ctrl() work with v4l2_query_ext_ctrl Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250203-uvc-roi-v17-13-5900a9fed613@chromium.org> References: <20250203-uvc-roi-v17-0-5900a9fed613@chromium.org> In-Reply-To: <20250203-uvc-roi-v17-0-5900a9fed613@chromium.org> To: Laurent Pinchart , Mauro Carvalho Chehab , Hans de Goede , Sakari Ailus , Hans Verkuil Cc: Yunke Cao , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Ricardo Ribalda , Yunke Cao X-Mailer: b4 0.13.0 v4l2_query_ext_ctrl contains information that is missing in v4l2_queryctrl, like elem_size and elems. With this change we can handle all the element_size information inside uvc_ctrl.c. Now that we are at it, remove the memset of the reserved fields, the v4l2 ioctl handler should do that for us. There is no functional change expected from this change. Reviewed-by: Yunke Cao Tested-by: Yunke Cao Reviewed-by: Hans de Goede Signed-off-by: Ricardo Ribalda --- drivers/media/usb/uvc/uvc_ctrl.c | 24 ++++++++++++++---------- drivers/media/usb/uvc/uvc_v4l2.c | 35 +++++++++++++++-------------------- drivers/media/usb/uvc/uvcvideo.h | 2 +- 3 files changed, 30 insertions(+), 31 deletions(-) diff --git a/drivers/media/usb/uvc/uvc_ctrl.c b/drivers/media/usb/uvc/uvc_c= trl.c index 4e29db334b2d..d50137d19561 100644 --- a/drivers/media/usb/uvc/uvc_ctrl.c +++ b/drivers/media/usb/uvc/uvc_ctrl.c @@ -1259,7 +1259,8 @@ static int __uvc_query_v4l2_class(struct uvc_video_ch= ain *chain, u32 req_id, } =20 static int uvc_query_v4l2_class(struct uvc_video_chain *chain, u32 req_id, - u32 found_id, struct v4l2_queryctrl *v4l2_ctrl) + u32 found_id, + struct v4l2_query_ext_ctrl *v4l2_ctrl) { int idx; =20 @@ -1407,7 +1408,7 @@ static u32 uvc_get_ctrl_bitmap(struct uvc_control *ct= rl, static int __uvc_queryctrl_boundaries(struct uvc_video_chain *chain, struct uvc_control *ctrl, struct uvc_control_mapping *mapping, - struct v4l2_queryctrl *v4l2_ctrl) + struct v4l2_query_ext_ctrl *v4l2_ctrl) { if (!ctrl->cached) { int ret =3D uvc_ctrl_populate_cache(chain, ctrl); @@ -1473,7 +1474,7 @@ static int __uvc_queryctrl_boundaries(struct uvc_vide= o_chain *chain, static int __uvc_query_v4l2_ctrl(struct uvc_video_chain *chain, struct uvc_control *ctrl, struct uvc_control_mapping *mapping, - struct v4l2_queryctrl *v4l2_ctrl) + struct v4l2_query_ext_ctrl *v4l2_ctrl) { struct uvc_control_mapping *master_map =3D NULL; struct uvc_control *master_ctrl =3D NULL; @@ -1511,6 +1512,9 @@ static int __uvc_query_v4l2_ctrl(struct uvc_video_cha= in *chain, v4l2_ctrl->flags |=3D V4L2_CTRL_FLAG_INACTIVE; } =20 + v4l2_ctrl->elem_size =3D sizeof(s32); + v4l2_ctrl->elems =3D 1; + if (v4l2_ctrl->type >=3D V4L2_CTRL_COMPOUND_TYPES) { v4l2_ctrl->flags |=3D V4L2_CTRL_FLAG_HAS_PAYLOAD; v4l2_ctrl->default_value =3D 0; @@ -1524,7 +1528,7 @@ static int __uvc_query_v4l2_ctrl(struct uvc_video_cha= in *chain, } =20 int uvc_query_v4l2_ctrl(struct uvc_video_chain *chain, - struct v4l2_queryctrl *v4l2_ctrl) + struct v4l2_query_ext_ctrl *v4l2_ctrl) { struct uvc_control *ctrl; struct uvc_control_mapping *mapping; @@ -1650,7 +1654,7 @@ static void uvc_ctrl_fill_event(struct uvc_video_chai= n *chain, struct uvc_control_mapping *mapping, s32 value, u32 changes) { - struct v4l2_queryctrl v4l2_ctrl; + struct v4l2_query_ext_ctrl v4l2_ctrl; =20 __uvc_query_v4l2_ctrl(chain, ctrl, mapping, &v4l2_ctrl); =20 @@ -2175,7 +2179,7 @@ static int uvc_mapping_get_xctrl_std(struct uvc_video= _chain *chain, struct uvc_control_mapping *mapping, u32 which, struct v4l2_ext_control *xctrl) { - struct v4l2_queryctrl qc; + struct v4l2_query_ext_ctrl qec; int ret; =20 switch (which) { @@ -2189,19 +2193,19 @@ static int uvc_mapping_get_xctrl_std(struct uvc_vid= eo_chain *chain, return -EINVAL; } =20 - ret =3D __uvc_queryctrl_boundaries(chain, ctrl, mapping, &qc); + ret =3D __uvc_queryctrl_boundaries(chain, ctrl, mapping, &qec); if (ret < 0) return ret; =20 switch (which) { case V4L2_CTRL_WHICH_DEF_VAL: - xctrl->value =3D qc.default_value; + xctrl->value =3D qec.default_value; break; case V4L2_CTRL_WHICH_MIN_VAL: - xctrl->value =3D qc.minimum; + xctrl->value =3D qec.minimum; break; case V4L2_CTRL_WHICH_MAX_VAL: - xctrl->value =3D qc.maximum; + xctrl->value =3D qec.maximum; break; } =20 diff --git a/drivers/media/usb/uvc/uvc_v4l2.c b/drivers/media/usb/uvc/uvc_v= 4l2.c index ff7d0fd9e458..41430720ed47 100644 --- a/drivers/media/usb/uvc/uvc_v4l2.c +++ b/drivers/media/usb/uvc/uvc_v4l2.c @@ -963,40 +963,35 @@ static int uvc_ioctl_s_input(struct file *file, void = *fh, unsigned int input) return ret; } =20 -static int uvc_ioctl_queryctrl(struct file *file, void *fh, - struct v4l2_queryctrl *qc) +static int uvc_ioctl_query_ext_ctrl(struct file *file, void *fh, + struct v4l2_query_ext_ctrl *qec) { struct uvc_fh *handle =3D fh; struct uvc_video_chain *chain =3D handle->chain; =20 - return uvc_query_v4l2_ctrl(chain, qc); + return uvc_query_v4l2_ctrl(chain, qec); } =20 -static int uvc_ioctl_query_ext_ctrl(struct file *file, void *fh, - struct v4l2_query_ext_ctrl *qec) +static int uvc_ioctl_queryctrl(struct file *file, void *fh, + struct v4l2_queryctrl *qc) { struct uvc_fh *handle =3D fh; struct uvc_video_chain *chain =3D handle->chain; - struct v4l2_queryctrl qc =3D { qec->id }; + struct v4l2_query_ext_ctrl qec =3D { qc->id }; int ret; =20 - ret =3D uvc_query_v4l2_ctrl(chain, &qc); + ret =3D uvc_query_v4l2_ctrl(chain, &qec); if (ret) return ret; =20 - qec->id =3D qc.id; - qec->type =3D qc.type; - strscpy(qec->name, qc.name, sizeof(qec->name)); - qec->minimum =3D qc.minimum; - qec->maximum =3D qc.maximum; - qec->step =3D qc.step; - qec->default_value =3D qc.default_value; - qec->flags =3D qc.flags; - qec->elem_size =3D 4; - qec->elems =3D 1; - qec->nr_of_dims =3D 0; - memset(qec->dims, 0, sizeof(qec->dims)); - memset(qec->reserved, 0, sizeof(qec->reserved)); + qc->id =3D qec.id; + qc->type =3D qec.type; + strscpy(qc->name, qec.name, sizeof(qc->name)); + qc->minimum =3D qec.minimum; + qc->maximum =3D qec.maximum; + qc->step =3D qec.step; + qc->default_value =3D qec.default_value; + qc->flags =3D qec.flags; =20 return 0; } diff --git a/drivers/media/usb/uvc/uvcvideo.h b/drivers/media/usb/uvc/uvcvi= deo.h index 2f9a1bf8b48a..6fc1cb9e99d1 100644 --- a/drivers/media/usb/uvc/uvcvideo.h +++ b/drivers/media/usb/uvc/uvcvideo.h @@ -771,7 +771,7 @@ void uvc_status_put(struct uvc_device *dev); extern const struct v4l2_subscribed_event_ops uvc_ctrl_sub_ev_ops; =20 int uvc_query_v4l2_ctrl(struct uvc_video_chain *chain, - struct v4l2_queryctrl *v4l2_ctrl); + struct v4l2_query_ext_ctrl *v4l2_ctrl); int uvc_query_v4l2_menu(struct uvc_video_chain *chain, struct v4l2_querymenu *query_menu); =20 --=20 2.48.1.362.g079036d154-goog