From nobody Mon Feb 9 20:55:00 2026 Received: from mail-qv1-f48.google.com (mail-qv1-f48.google.com [209.85.219.48]) (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 432A119D088 for ; Thu, 14 Nov 2024 19:10:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731611448; cv=none; b=tHbrK7TOESI9cM0tk3qYRcqxsiEmuL3wRkChEf5vxCPpsjA0p9BdFMNs+BKKZj9KbaNyhtXpDs3nZ3B9CyZMr3OreZjVVAP13/+xzjcEg9H0+nldQdfaMuTufHpOppW1mtBirxAX8TqIe48cbWILAvq9s0P10W99DEZ9GgDcebo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731611448; c=relaxed/simple; bh=SC8L6uuUf+/qQ8Ex1Sbi6Yirs91l4EQxu1/mHdLrT3c=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=d+cE13oBm4zlt9h7Kzg+QJtv/s+814ad0jYyMJ67eGM0Q+i4eNBS3Hqj899nzAkx+upgOnhLrWdbvLxZ0T8PV7smtMvMRIUsyyLDZ4gE7NWvoqbKELd1y9aAIHacS1UA0tv95OPdMVO/KcfDGBxPb8UqDWr3jnlyr3u1NeSR8pk= 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=FNTxAIjg; arc=none smtp.client-ip=209.85.219.48 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="FNTxAIjg" Received: by mail-qv1-f48.google.com with SMTP id 6a1803df08f44-6cd7a2ed34bso5340196d6.2 for ; Thu, 14 Nov 2024 11:10:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1731611446; x=1732216246; 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=g3mKyz8dYUWXDVhEiyfGTODSac7INNAwm9mfRIbHOA8=; b=FNTxAIjg+AKMOAkmRibmn0/OK/f/baVvp9f4agC+dyHYJbjbXK5zv1OzAJ+fq1Qd6N mo//d+fWUNdUaRr5iXKX0vRUCTGeca4QEZekiE9OVxvqjsKSuqy4pWDCarrmAHZoCBZr TfsVzdewDr3Q37NLZ7qFRuesouMnk/cOmxJG4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1731611446; x=1732216246; 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=g3mKyz8dYUWXDVhEiyfGTODSac7INNAwm9mfRIbHOA8=; b=e+7Dy57QgjhUNOLFJxG+Su2ydwTyJ4FT9iSoQDZ2N1S7T78VthBw+VYVPZDGA+Xg8J +pFn46dQ0wyls4hXr1Hx+J9GriIjR1lcV0eWK+OuH5kp6MFzDsBPRVj3FDSTzAh1XHZS XDx1uwzLsRbGwUuJXpNAJgiJtxTdP7JWawdt2Bl5cOg19CG6/BozCHv5AZZmlrh+fKf6 6X3SEl5S4ohgNpjtCVCV0ICvWuBHlA/GUOfYa4JKNZHtyCnKuYabji575jczWltGsSqq o4Gvmtk7FEAihD0+0rgcS/x3yO7zCDXMHd5+aKcuVshJCCwamF9SLERU07PFxqWNG7Cq h3HQ== X-Forwarded-Encrypted: i=1; AJvYcCUlLOc60o67rEqmnB4CV1zw8Aa4zaCbdJ5gugcqYeQQkAT9bNHU2eGcw5PHHL8ttsIPDsfU0eZz61aDKqo=@vger.kernel.org X-Gm-Message-State: AOJu0Yzx6zd4inmS14xbGOXQ0QDt9ucELtCF/hWqt0jA7yBm3x7h7Xuc Hwo4hYw49hKpEqeLE+2Esxeq5jHN8x7Y2j/mfaW5x494ndZe5xDN/nfa316IXg== X-Google-Smtp-Source: AGHT+IFboiersPw9AuVt765dOS55ZVKctVly2CX95Gh3rA/va9J1s5reVsHiqKvLb+PaK3c6MG2r2A== X-Received: by 2002:a0c:fb09:0:b0:6d3:f904:5359 with SMTP id 6a1803df08f44-6d3f90454f5mr10161666d6.33.1731611446042; Thu, 14 Nov 2024 11:10:46 -0800 (PST) Received: from denia.c.googlers.com (189.216.85.34.bc.googleusercontent.com. [34.85.216.189]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-6d3ee7cc7e1sm8857766d6.53.2024.11.14.11.10.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 14 Nov 2024 11:10:45 -0800 (PST) From: Ricardo Ribalda Date: Thu, 14 Nov 2024 19:10:30 +0000 Subject: [PATCH v15 01/19] media: uvcvideo: Fix event flags in uvc_ctrl_send_events 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: <20241114-uvc-roi-v15-1-64cfeb56b6f8@chromium.org> References: <20241114-uvc-roi-v15-0-64cfeb56b6f8@chromium.org> In-Reply-To: <20241114-uvc-roi-v15-0-64cfeb56b6f8@chromium.org> To: Laurent Pinchart , Mauro Carvalho Chehab , Hans de Goede , Ricardo Ribalda , Sakari Ailus , Hans Verkuil Cc: Yunke Cao , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Ricardo Ribalda , stable@vger.kernel.org X-Mailer: b4 0.13.0 If there is an event that needs the V4L2_EVENT_CTRL_CH_FLAGS flag, all the following events will have that flag, regardless if they need it or not. This is because we keep using the same variable all the time and we do not reset its original value. Cc: stable@vger.kernel.org Fixes: 805e9b4a06bf ("[media] uvcvideo: Send control change events for slav= e ctrls when the master changes") Signed-off-by: Ricardo Ribalda --- drivers/media/usb/uvc/uvc_ctrl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/uvc/uvc_ctrl.c b/drivers/media/usb/uvc/uvc_c= trl.c index 4fe26e82e3d1..bab9fdac98e6 100644 --- a/drivers/media/usb/uvc/uvc_ctrl.c +++ b/drivers/media/usb/uvc/uvc_ctrl.c @@ -1673,13 +1673,13 @@ static void uvc_ctrl_send_events(struct uvc_fh *han= dle, { struct uvc_control_mapping *mapping; struct uvc_control *ctrl; - u32 changes =3D V4L2_EVENT_CTRL_CH_VALUE; unsigned int i; unsigned int j; =20 for (i =3D 0; i < xctrls_count; ++i) { - ctrl =3D uvc_find_control(handle->chain, xctrls[i].id, &mapping); + u32 changes =3D V4L2_EVENT_CTRL_CH_VALUE; =20 + ctrl =3D uvc_find_control(handle->chain, xctrls[i].id, &mapping); if (ctrl->info.flags & UVC_CTRL_FLAG_ASYNCHRONOUS) /* Notification will be sent from an Interrupt event. */ continue; --=20 2.47.0.338.g60cca15819-goog