From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 3723F20D516 for ; Fri, 25 Oct 2024 19:39:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729885183; cv=none; b=DbWuNjTjyxJYKefkb8M84GBRF0odbwXsVSKxJTFVnY8wvH8sEM+uTsr+weIMC+5kcycxcMAaNobtLSHubfXvFA7q3bfK9LMge/jUCgeE6eUKQJvvrpvt4Agu+SeNUA8JQoaxeqQzn4MKSJi+EZryNreUsjVUTEiB6kpHhT5HWVo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729885183; c=relaxed/simple; bh=h3swVIaCm3gH7Ok3aF1n02u5px+9KfKP7NCgLtPvcuw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=YRZ0Dk2EeKgV/M82lLFJRmoIseIcX2BwOVmPDZMyhrPZy5EMlVGAYK1SDselOKuFaqEXJKIDNQUJ754YESgwkUHkPGxAw4d6Ii1tEjwL/B823CgqKX+7kRegYXNYy3/5zRqj+Oxqd7PgIHO0AMVm8mSyeAzA0Y+SB6nYPsqjiO8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=mkZWIwbt; arc=none smtp.client-ip=209.85.128.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="mkZWIwbt" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-431616c23b5so17154895e9.0 for ; Fri, 25 Oct 2024 12:39:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1729885179; x=1730489979; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=rNudOakrgi09S9+yizP0qtRalV2gTsXxi5NDtk4FIIo=; b=mkZWIwbtA5nFwj4Fyz7/LyY7WhvPrzRWWnqvZbfvWehopHlcyAyMfa62/4wc8K6k6X PipJAX3qdP8wunX0ULERO7Z7gKpu1X/t9DdCE9fqOZnLRYubMpGdHNjRbWQu+na1fgxF N0HGSKMFycBU/+H7PYu/IaGNrUngqMcnMWxS9F1LlZqefJs7wUbevmZknBT2f0Msb47b C8RNyrLMvOblGeqoujZg5m4T4lfecntejg53pMBb+OklFXwxZRthK+wyWFvE0sr+pyuq BE13AmU0tKuUkb4YgX3oObaF7y69MhzXmfY7Wf1T3of2tm0ZtfaIq0vDJ588zyGgAwdF lv1A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1729885179; x=1730489979; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=rNudOakrgi09S9+yizP0qtRalV2gTsXxi5NDtk4FIIo=; b=UFkEBgosEYReAxmUUQWeQH56AUvMdhmzbBpSCI/od17IBj4e6tttSb/58cdzAzPjRP SJ2CwoIt9glyGJMWdZM2NjyFF+QXBRgzOTaYmT9XZuiSKCBGd69tl/ieoRBs5ekEkcdh icQ60XvKXVucanoyzblvofffilahWxhgbp+Jq+scbffiNoRqPyHiPaE77QF8LcY0M69H eEn/nngrhCsAmWkBHR/7FHVetrhkTJEaeTgMvBQV0GsuYPfCxjANi4W5eEVZXvqVAD12 OYdnfdZxwn3KsbaS6WV/wxlHTepzzjrSJTZXelzQhxYMfvlOWCmXhQhcCFd9ClhHYg0U o/YQ== X-Forwarded-Encrypted: i=1; AJvYcCV+M5z+IYSyp2Ce36C9siqrv+BJvJDSbwxgRa9NCWAS5EwJJKt+NMmm5U4R8Pq5SW5HMofjzfiHXecR@vger.kernel.org X-Gm-Message-State: AOJu0Yw6pMf/6A5w0pKg+PK7NtCJdmoRq+/N08+ySB3MfICwKPOhenHv Tl8gFxbcKmKj0/nGpBtIylgmsSHJ2dRJGZd5jc6YvVSUAdXydAfb7lEBogSj6g4= X-Google-Smtp-Source: AGHT+IFAsGfICqjFEVag/1dSMIzHpMXghtsoTxIwbYv1J4KwUX5ZFmRRVF/tsXOYk/1h2HaqEmKmmQ== X-Received: by 2002:a05:600c:45c3:b0:431:55af:a220 with SMTP id 5b1f17b1804b1-4319a5bbffemr5287305e9.12.1729885177852; Fri, 25 Oct 2024 12:39:37 -0700 (PDT) Received: from [192.168.0.40] ([176.61.106.227]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43193573cd4sm26695675e9.5.2024.10.25.12.39.36 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 25 Oct 2024 12:39:37 -0700 (PDT) Message-ID: <1bd45da1-0b14-441c-8ad8-1837e31eb449@linaro.org> Date: Fri, 25 Oct 2024 20:39:36 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 3/6] arm64: dts: qcom: sc8280xp-x13s: Drop redundant clock-lanes from camera@10 To: Konrad Dybcio , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Depeng Shao , Vikram Sharma , Vladimir Zapolskiy , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20241025-b4-linux-next-24-10-25-camss-dts-fixups-v1-0-cdff2f1a5792@linaro.org> <20241025-b4-linux-next-24-10-25-camss-dts-fixups-v1-3-cdff2f1a5792@linaro.org> <7620b929-dbf0-47f1-bedc-1ce87bf6e988@oss.qualcomm.com> Content-Language: en-US From: Bryan O'Donoghue In-Reply-To: <7620b929-dbf0-47f1-bedc-1ce87bf6e988@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 25/10/2024 18:23, Konrad Dybcio wrote: > On 25.10.2024 5:43 PM, Bryan O'Donoghue wrote: >> clock-lanes does nothing here - the sensor doesn't care about this >> property, remove it. >> >> Tested-by: Bryan O'Donoghue # x13s >> Signed-off-by: Bryan O'Donoghue >> --- > > drivers/media/platform/qcom/camss/camss.c : camss_of_parse_endpoint_node() > > seems to reference it and pass it on to the PHY drivers > > Konrad Yep but that parses the clock-lane indicator in camss ports &camss { vdda-phy-supply = <&vreg_l6d>; vdda-pll-supply = <&vreg_l4d>; status = "okay"; ports { port@0 { csiphy0_lanes01_ep: endpoint@0 { reg = <0>; // HERE clock-lanes = <7>; data-lanes = <0 1>; remote-endpoint = <&ov5675_ep>; }; }; }; }; this patch and its sister patch remove clock-lanes form the sensor &cci2_i2c1 { camera@10 { port { ov5675_ep: endpoint { clock-lanes = <0>;//HERE data-lanes = <1 2>; }; }; }; }; This is removing the camera@10{} clock-lanes not the camss{} clock lanes which you're right is referenced. --- bod