From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm2-f13.google.com (mail-wm2-f13.google.com [74.125.225.141]) (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 38D8149BD91 for ; Mon, 14 Sep 2026 18:08:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.141 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789409323; cv=none; b=gbqeI1vN4NB5x6I0OPiXLaPBO4CP+nDPaOIhYOtO3lkY+378a/cew3hUUrh6VPjb022Ga6GjqkG6Lr2KofHepzpXWeqIk7DFnwOpGzEl2wJb3oGDYW/nkAf444zd63A7A9iUc9AtLKKydCvarXJwFVWlWd/4tr74KWfWMFxaOFU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789409323; c=relaxed/simple; bh=MZeHia5Sjdq+jl2QWpcNU33wRrn06cJxwy3XHGpOMVA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=jWpfBMbPtAKTgnEudU2zrxKm9ScVofMPutHUFaDN539OER9GMtDaGBnyVK/K6QC16Mlwwsc2JaUbKs1FuF/jEVhNhx/ywY41OE0dE1YYqKlMjj1+ixp+ryhgG9Hd6YZYxda9qPdPvNd6ZAN5v3aR2GgV75bIdvtGOkpQZa+JYpA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DBSErzxi; arc=none smtp.client-ip=74.125.225.141 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DBSErzxi" Received: by mail-wm2-f13.google.com with SMTP id 5b1f17b1804b1-49e6b885ef8so10892335e9.1 for ; Mon, 14 Sep 2026 11:08:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789409305; x=1790014105; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=npapGo+2obRZMka7BZe1zrgegdn0mSsQ3mrLs63xQeY=; b=DBSErzxiOSzf2OnzrO7JHkpREcpm2ksOTkgYrGrJ3Tj6qFFgBjyoKMx5qzbLO3fjTu rE+/EBXeWpOfJ1kx+x6m7/f+B4HezJ4DBkyv0EKxogtaa3fhKLpGwjgWji3w+IEab9xI 0jX0p31wqHYxot/wp/s5+A3kiz2Fn5WSm6RVcFQMWivd+3RyJH3UyZY3AIs69gqlBN2k XZQjtG8zyCuVkf3H8o5LXUunO/rLs9oKOct5Ms6G9lCctXAfgWzSyJI//5dkdIlC7DdI apba8DkWoeqtldJRnrMxzVwDdqElhlBDWpomnhdulwFg7op+yR1rOONgOFT2vNT8A7a1 abvg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789409305; x=1790014105; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=npapGo+2obRZMka7BZe1zrgegdn0mSsQ3mrLs63xQeY=; b=mSMU8NkZSlG/ILHTvzeTKTtXtimC6ONQH3IFR2x9XVU1PG6SanOlHpgfSaKZfJPEr4 oHh6xBA4jygBWEELyOYapvGPz0VeG9xjA1MsOrk5WreMxgO5+xoIghWCyeTlDKR9f8LO 6hXajKoClMQQ9Wvgd/hJ3lgwh4Xs+VFChO9ATqSRGmzPCxAnIOUQTpTJ0xR9fsYQYecu 41RPn+9ohyW7yVrevj+UyUwE+DEEBg/eZiGcGlTT92Z+erKskxgWyyH59ibfo3w4ILST FwDhI/Soj50rm5HMljXRFkIl0cPq02L2Efla48bG4DSR2cxCRbytibMmbBUF6Lc27y04 ohUQ== X-Forwarded-Encrypted: i=1; AKwUvBwYYq1O7y6pd2Urft5iWZ5aOjKL7fxcBec5zQs4RhVL4axSQEprK3JWeZArJw3YAhD9VVNA7UEH6XnL2g==@vger.kernel.org X-Gm-Message-State: AFuF++lBz0oMu/9Et8U14kHrxTww9iZwb0/UW1j6ZH3anuf4SmEn9Fan jjkrBIPB9o/MeVc+QlNWEIk71lnOrjREX39CZATbzpTBEZLUKQg2rPxz X-Gm-Gg: AYBFou1WyxBxJjOFCnFgMo6hFvV1tmhMYBzzJ7lrqXI95LgHkYkJ2AGS5li6JgKLNf6 3NLvMiTe9TrIfBM5oA8NbIRygXCfDX5XqxOgS0l5gKdhy+QkwYX2Js23uN2ymYCisvQ8mEHpxJq Xb+y/29vwlHNQevU8DHrBy+yiDvOAkLlGF3+eOzdFi/Zi6pANQjhMtlzjXZTIq0elCSR/xTHu6V lpEfvcYU6rkG0rdMmv3ZLYH3ApUF5Ceml8T3LvR7CxWQbRqnoJ1/OV8jpCKaeh6T2vAxn4EtJHa s4CQUb8srJwiLEXouTAGuIQovjU2xtEhB5/3i3nfD/MUmUVZPYGu/ck4rSjLkenCNOTvAidoKpF uSDOM2NtbxMXS/ZqZb9y2U6g5h2ojLNMjqREo3pnIjEQwovI5C6tSLz7hEpSjG987qU/5fxTjhC WIQR+tKvMxTZl79B/JMtotyhmhPFiQQ9n3M3gIj/VN+/u7nMXNdrdX4Om9Rnvnw9Fu19Yf3oTqC n9KKzg376RYNZ2LoAasUmXoLt+q2eQC X-Received: by 2002:a05:600c:3484:b0:49c:fa21:e742 with SMTP id 5b1f17b1804b1-49e7a68c148mr52389725e9.24.1789409304507; Mon, 14 Sep 2026 11:08:24 -0700 (PDT) Received: from raviolimobile.tail5f26fd.ts.net ([84.65.89.105]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49e7d2717a6sm7069495e9.2.2026.09.14.11.08.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Sep 2026 11:08:24 -0700 (PDT) From: Fernando Rimoli To: "D. Manresa" Cc: Fernando Rimoli , Sakari Ailus , Daniel Scally , Hans de Goede , Jakob Berg Jespersen , Fil Dunsky , Kengo Oki , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 0/7] media: Enable the OV5693 front camera on IPU6 Surface devices Date: Mon, 14 Sep 2026 19:06:56 +0100 Message-ID: <20260914180807.150971-1-fernandorimoli11@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260906073932.24090-1-dmanresa@gmail.com> References: <20260831181858.325109-1-fernandorimoli11@gmail.com> <20260902142322.73523-1-fernandorimoli11@gmail.com> <20260906073932.24090-1-dmanresa@gmail.com> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Hi Manresa, On Sun, Sep 06, 2026 at 09:39:32AM +0200, D. Manresa wrote: > - The sensor's 2x2 binned 1296x972 readout (a downstream mode, not in > mainline) streams through the IPU6 hardware ISP (PSYS) at ~28.7 fps Thank you, and sorry for the slow reply. Your tag is the only one that covers the binned readout and the PSYS path, which is how the Pro 7+ front camera is actually used, so it filled a real gap in the coverage. The point about the CSI-2 error signature being identical to the 0x2d build over the same five capture cycles is also the cleanest confirmation so far that bits 0, 2 and 3 were doing nothing. The series was applied to media-committers next on 2026-09-11, and your tag did not make it into the commits. I have pointed Sakari at your message in the follow-up fix, in case that commit can still be amended. You were also right about the property name, but I was slow and the literal went in as 8e3def7bf410 with the bug intact. I have sent the fix as a follow-up, with you as Reported-by: media: ipu-bridge: Keep the clock-noncontinuous property name out of rodata It routes the name through ipu_property_names exactly as you suggested, so it matches what you already had applied locally when you ran the rebind test. One detail: the dangling pointer does not depend on your series at all. The nodes are already never unregistered on a successful init, so the literal dangles in the tree as it stands today. Your 1/2 is still the fix for the remaining two, the link-frequencies values and the lens-focus name, both of which are still literals in next. Thanks again for the review and the testing. Fernando