From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f52.google.com (mail-lf1-f52.google.com [209.85.167.52]) (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 39116449ED6 for ; Tue, 21 Jul 2026 09:56:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784627818; cv=none; b=mv6ckCHeotGXpikb88qOHezcCxW/gq/SAELGtCPdg3qBa5yLQsLmaAk6wW0MK7GqJI/b8Z1esifrzzc98nS6enZtv1lduU8satxaTy7IqTHLOmeZ0PeHredltO5OgSgG+ZtDkS24sQ2gq1euOCQj4q63f396qb9ao9E4TQdnT3w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784627818; c=relaxed/simple; bh=Gcg+IT6+eQ9C30qzQD969Z8vRx7pDNkFThBtXNgQK9g=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=oz973PvnZi03Vrq39bWAe5ePqD4AEA9iVHGI8dVFD3krfPuq7v5TDwX94DY8aZTU3u3Hivfhf2lnSC9q7sYp/BDVv7uR9HjlAiQc4cTsO9ycjAQ3yh2uULXNze6/TOTEbD7O2lYNHlbreTaNyNTb95VRsAI6R9g9Xq7ql3dLp10= 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=Cmmf0O1e; arc=none smtp.client-ip=209.85.167.52 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="Cmmf0O1e" Received: by mail-lf1-f52.google.com with SMTP id 2adb3069b0e04-5aebe49b227so2697414e87.2 for ; Tue, 21 Jul 2026 02:56:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1784627815; x=1785232615; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:from:to:cc :subject:date:message-id:reply-to:content-type; bh=PlDMh90wfY4dRTCIsx8e18xxobLQnLayOa3bHMPPBVg=; b=Cmmf0O1eSNhDl/dE9SY4jDmIDqiPYr7R+tS5sgvK/g1aSqN/26uT3AEIbZzkEGpjQA F6sN4R53AGiZqpKnAw4tLbqMxkN2NIBacTVMhWh8YU4JhPK5J9h8XIpYq5jbM4xDKzh1 AjNjR927d7QxAqkwoXILDOejvM/WnL2VWRcaRmxH2R/+ZFikP/L2xERSZpFqWbwuUtvM TgTspazMSFtakWNTK8sjXnFNmj+yd2PP+30h5EuZdHRNnxzAweOUO5t9ovBHABkWU/8Z j9VDgmH3BktogVsmW9VJs6mbFLitWb3XnyHh8bYDGkIyPNiPa+nXO1xEd1n4YU2tmXdJ lrhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784627815; x=1785232615; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=PlDMh90wfY4dRTCIsx8e18xxobLQnLayOa3bHMPPBVg=; b=C1t6WYcwdbXiH2OuK+NU0j26+3lUInMFRWitCNMqh5RXUr1qwZwEmjoy+G6NGOMj9C UhHInKu912/rTt/xxjX3r2ImCZqLSpogdd7QipcDdPVvlK5XD9zE1axKR32MkydbdIxA jQ9BNKN5rcRUJwSz9wh9+kqnX3Dt8Pct/ctwf/xrhui6KT72djx022+ecBj1Cb+qWsPC vV8lzVFUL3QTw1UM3IcgIEyD4KG+JvJ8OtQBfNn2VWlFPZmcRpDfuiFWms6ath6WMOD7 Dlp0h/+NBNGKulegpR5o3z7Nm7pZ3oBGaC0RDA9vJ8XowQ3WpfDmiMdgSiHokr6XFCSN GLhQ== X-Forwarded-Encrypted: i=1; AHgh+Ro9pTJnA3mbLcbzb4YwR9tWewIsMD0LLGtycFAHTvyhytjfulKm74Ohp7PZRUcApIZLmWU4sMm+jygMb/w=@vger.kernel.org X-Gm-Message-State: AOJu0Yw9CAdzE1X+xH5B5voJ8q+ZOUJfTAf2nb6K7VL7vebmJry0ngc+ xqvqtNP1nh+rYnkaospCkzsR6YImyJx5DKrdsDAsuNy9ADO6TbRhM2z8urEzqAQr0qA= X-Gm-Gg: AR+sD12xR5bakd82fHrp8XEsgAXdx9689RB/C7PgyV88l8nluvWxU3QeaaVJ69cxVLE 5HF4OCyyBit+SkIhV15/z2YYHuRnFFlGCLWNHqi2E0ezrW+EvTTFdRWEogHXXmkms2YX9tKm2Qs bBav1rthvkDQDC+OHrwmUTvlOWDTqmt33XBD2eAxPkSKsdCSHAqnWRC5AzNQIPUwumj7yq2PFZ2 QQMzI4ez2NwdM+tETd9CdFwcLGwLNioPbQ+Wk82kZ1UoY4eXrUyXJTRW+a9wW8wldHFejr5fniH CgoXY9kMmhKNYK6m/NzfEB7YmcBfQtUYXVAThHmlXX3VZXmtvedy77AKtXeuXfr+25KsrQGjtrN 5WCYOnh0/fYvtJgFcCNiDF5UU88cpKEvphDMFGXwPBXLYOZDpMCd63t9O6cj67mdeMwh2qoLl7/ 4Tld1/H9nGvQKvXWAYXDIRAmpPjuflPCC82lZdKjjAiEeXWkaJONMJvNAA X-Received: by 2002:a05:6512:3b98:b0:5ae:c5b2:9cbd with SMTP id 2adb3069b0e04-5b2a4a7e164mr328522e87.4.1784627814976; Tue, 21 Jul 2026 02:56:54 -0700 (PDT) Received: from [192.168.1.100] (91-159-24-186.elisa-laajakaista.fi. [91.159.24.186]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b291e18811sm2797516e87.60.2026.07.21.02.56.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 21 Jul 2026 02:56:54 -0700 (PDT) Message-ID: <345ca98e-09f6-4965-87bb-e64dac096498@linaro.org> Date: Tue, 21 Jul 2026 12:56:53 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v13 1/2] dt-bindings: phy: qcom: Add CSI2 C-PHY/DPHY schema To: Bryan O'Donoghue , Bryan O'Donoghue , Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Neil Armstrong Cc: linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski References: <20260720-x1e-csi2-phy-v13-0-160c31958863@linaro.org> <20260720-x1e-csi2-phy-v13-1-160c31958863@linaro.org> <0da6d314-bfa9-493a-bcc6-8c3b75751d0c@linaro.org> <88ae6662-cb85-4e3a-bbb5-1b536348df34@kernel.org> From: Vladimir Zapolskiy In-Reply-To: <88ae6662-cb85-4e3a-bbb5-1b536348df34@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 7/21/26 12:39, Bryan O'Donoghue wrote: > On 21/07/2026 09:17, Vladimir Zapolskiy wrote: >>> + "#phy-cells": >>> + const: 1 >>> + description: >>> + The single cell specifies the PHY operating mode. >> This has been reported before, #phy-cells shall be 0, the operation >> mode is selected by bus-type of endpoints as it is described in >> video-interfaces.yaml >> >> Two competing values are redundant and it opens a door to a wrong >> hardware description, the technical discussion has not been started yet. > > So as I said before, I'm following Rob Herring's input on this and > Krzysztof has given RB for this binding. At this stage, I don't see > scope to change it. > > Adding endpoints was already a compromise to address your feedback, > which I hoped would bring you on board with the design. That's my comment dated March 27, 2026, and it says moving phy type from phy cell to the endpoint property excludes the need to set any phy cells: https://lore.kernel.org/linux-arm-msm/e37ce438-12c7-462d-b2bc-1351be62b806@linaro.org/ The technical comment is left unresponded so far since then. Since there is no expected phy cells to get on consumer's side, and since there is another link between CSIPHY and CSID described in video-interfaces.yaml, any kind of dependency of this IP on phy can be just removed, which serves the purpose of hardware description simplification. > > There are several SoC bindings as well as new CPHY features which are > gated on these changes. > > Vinod, Kishon I'd request to merge the changes as-is. > -- Best wishes, Vladimir