From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) (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 A39673A7597 for ; Mon, 10 Aug 2026 08:35:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786350928; cv=none; b=NPQ1tV/t89Dl7ORIvGxbkfgOziYUnvwgFqXSAbIbTJeaPXHyQYHbGIB4T5kd7dh0pQq4Xgmd8ecPvK+ocRxwdndIpuyBxbRB617IwmGg3JaFg9Nr1LjtSz6tarHCWNxIbU/lLdxDp9jbVz3w3R4BBDYNep8u20Y/daDT1ek5bso= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786350928; c=relaxed/simple; bh=UHHCbcurusZy4Y523Q7T4669vLhobI+gpJ2has1/PiY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Gs7GGDnIEdWbSTKg4cznmxxnCUMAzUiODZPIsoaAQMkdfSN7Ja3wD8Bf42MK1KEHDdWX5H7PvZlHlRvSj1Bq8KuZniYGfypL1qTQxbGfNK6DaharBCoNu+7SvSdXWxBGR+DC3W/u3eg6UUrgS/AqTYMINE8X6DKTpnTBGxU64W4= 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=PBXDJ4wJ; arc=none smtp.client-ip=209.85.216.45 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="PBXDJ4wJ" Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-3900e39d935so1583499a91.0 for ; Mon, 10 Aug 2026 01:35:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786350927; x=1786955727; 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=qGBM92czm9YEwobDEuNTbqIYrxD+/Bct4RxHXWVwj0s=; b=PBXDJ4wJXkQAPO7cw9JbV3jJUlHzZFVQM8XaxO3ffNNtWF4ndbpZoxVelRKRMV7p5X Cm48Vaj17iqnngOzcGhqyudoupRWOKpQt9ir8qiN1xCSxTh/g4jOwHB/jA6WFXeWQrP5 cP3RIxhtnn1V6c+kzyuL3Nptq1AFjRweo9qUHtfp1f0AOK+vtcgjBuoLNb6FDoieK4s1 AuYvaKt30QE1rOtSOcvAM+VJCy4EMzpwGuh3PSs/jx9tDfNaLmp/WCEfLjRT18p5ImPL syJe2mPohSwUqeKtrDU2N4uIzqkCs4XQOkyxpGPyKCjmGOHjFzo6fH1w/N5SEuX0FgM4 787g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786350927; x=1786955727; 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=qGBM92czm9YEwobDEuNTbqIYrxD+/Bct4RxHXWVwj0s=; b=E7hc2hP2qn9dfE1Rx06uF+t9WO2WHY7sF6ma5oCQ/XBZIsPu6ipnmQ+ZEc6tTu3A22 QAVkE+Ys3G/qG6G0awrTbkmnAtYAESIFguRbGQYT9gerPleXJjypBoEnyAjpstjN9LGa sDEhicsOENcdhYPMZ/U8fx2wIPD93X6Zg+VmOY64cuyBEDi09qyS246cchTtKuDBvsff aQ6E6hAX1Us7KMU3G+2qeg2I728tDD2RJGf+bWgAEpF0Xl46DVFUM+yzq737x0+yteci z/Vwa7zv2DTIQoE2UVK3ZaEVLtq9tziQQobEUMENJC65D7dj6XAv6IxK/mZloZNvgjpD Roow== X-Forwarded-Encrypted: i=1; AHgh+RqOsqq/MSE6CIMrnBQqIXMwh+Oco7j3jjowS00aE4iN6ObmUqbv4UlUkzGyAOIbIUHJ62I36YmGevdFdn0=@vger.kernel.org X-Gm-Message-State: AOJu0Yxm2ZG+rWsVYsyIGJsSbsD6/YqpO4J0abz1GARnpjN/ZznCMJ5l K+y3LFoSSf/U3U3gdI/UiRWeVchyoqwSgaktKG9hmCoRJ8KKioSFH5ELWYNsvKcHEEM= X-Gm-Gg: AR+sD11DuCXZU4YBc7VCMJ9L3CsiVs0x5Ofr1DkT4EAjLQM1K9w/TDCgIsUTqaHfqJ7 MU0DPDXAma0guHKB3UZpvfZqUESzDIvqhbxddu6XkrSlsYVzn71HvU8EACnSnquHireYCjT/51L C5XF52G6uLS2xIr9lwkoGOhv82wKXuy7i2O/PgGnR0yV7igMdXJQJgNXontuzKf7WjJgsP/82Zh IAd2R/515QrBV2y5AMYgs3ylxXT2brVWysDqQyJuTcca9UykdYj9vr24IAK32ncxFcOuwaWbMSY Hnz6Psipl6bDDZVYYgD/pvVVRszAXEMAr7r9nC2AN10808SR81OKUJ2audwzcUavs2NNsIXsVCH 2YZWo6BnUNvUfdLVv7o7Y9J48RuQbByWfMZVA+DnlD6Co3cm2/nm6u25pKaLg+JT/olP9nHT52V RbA5xecPOeROltcBHZ1I3hGYQcUtRiGApYn9FzZvVC4Yz4orAZvqbXTTkYh8PBC0F1MIzvnnU2E jRabDJef5ikpgbQmq4x X-Received: by 2002:a17:90b:538f:b0:38f:9674:ef61 with SMTP id 98e67ed59e1d1-3903c584291mr41497369a91.13.1786350926920; Mon, 10 Aug 2026 01:35:26 -0700 (PDT) Received: from sahan-i3-12100 (121-45-161-118.tpgi.com.au. [121.45.161.118]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-390b30d465csm6156520a91.2.2026.08.10.01.35.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Aug 2026 01:35:26 -0700 (PDT) From: Sahan Nissanka To: platform-driver-x86@vger.kernel.org, linux-media@vger.kernel.org Cc: Sahan Nissanka , dan.scally@ideasonboard.com, sakari.ailus@linux.intel.com, hansg@kernel.org, ilpo.jarvinen@linux.intel.com, mchehab@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] platform/x86: int3472: Add TPS68470 board data for Dell 7320 Detachable Date: Mon, 10 Aug 2026 18:32:54 +1000 Message-ID: <20260810083425.9797-1-adee.sahan@gmail.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260809042540.15849-2-adee.sahan@gmail.com> References: <20260809042540.15849-1-adee.sahan@gmail.com> <20260809042540.15849-2-adee.sahan@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Please do not apply this patch. The GPIO mapping in it is wrong, and I would rather say so than have it land. Charles Drolet, who has the same machine, reported against my out-of-tree repository [1] that the front sensor probes with no pin assignment at all. That is why a wrong mapping is invisible here: the real reset line sits released by default, so the camera comes up either way. Two specifics. There is no powerdown pin. This part needs no hardware to confirm - ov5675.c requests only "reset", via devm_gpiod_get_optional(dev, "reset", ...), and no "powerdown" con_id exists anywhere in that driver. So GPIO_LOOKUP("tps68470-gpio", 4, "powerdown", GPIO_ACTIVE_LOW), is never looked up. It is dead code, and my belief that the pin was needed was unfounded. The reset is on tps68470-gpio 5, not 3. Charles held the line with gpioset against a build with no pin assignment, so nothing else had claimed it, and reloaded the module: held low the sensor fails to identify with -EIO, released it identifies correctly. That is consistent with an active-low reset on line 5. I have not reproduced this myself yet. I should also correct how I arrived at 3 and 4. I took them from the Dell 7212 and 5290 board data as prior art, and treated "the camera works" as confirmation. It is not. It confirms the sensor runs; it says nothing about whether the mapping describes this board. The rail mapping may need revisiting too. Charles has avdd on AUX1 at 2.8 V where this patch has it on VSIO at 1.8 V, and the part wants roughly 2.8 V analogue. Both configurations work, which suggests these rails may serve as enables for regulators further down the line rather than as the supplies themselves. I would rather resolve that than guess again. The machine this work was done on was a loan and has gone back. A second 7320 Detachable arrives in the next few days, and I will verify the mapping on it before sending a v2 - including checking whether the board data generalises across units or merely described the one I had. Patches 2/3 and 3/3 are not affected by any of this. Charles independently arrived at the same ipu-bridge entry and the same ACPI id addition. My thanks to Charles for finding this and for testing it properly. [1] https://github.com/githomeserver/latitude-7320-camera/issues/1 -- Sahan Nissanka