From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f53.google.com (mail-ej1-f53.google.com [209.85.218.53]) (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 45CCB381EB3 for ; Tue, 21 Jul 2026 14:06:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784642795; cv=none; b=VHrCP9ZheyfR7DF7PgEKyF75ijxYyZronSALf0GdcUh/bT+FGPEf8FZaPVGQvXEtHMJJC5wsX8WqNSZcmmc0TDscgrz6CsT5M1psOeeTI5ljH8klSH08zwqey/gsgMSBA9Z/H9p/vC6ha8xqiTW8p3iDCqdldEBrl7UOZLb88Yk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784642795; c=relaxed/simple; bh=QVCAARSUbw9D5YSPa8BhULoKdPaGPLDjhQh8GG3iwEo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=hB8slKqCMRbJEeTmn8FaDleBIZDFW4syYube59KfVusOaujnwLAfyDy9PNmE07bdMvTLrF9UUpwRxDz7EJFLcxPLI94lGASB3gf+5p/RbEus+a6+YUVjq9ltKHxTcW+xkye6JrN0C2o8oEcHMM8Al38+azd8UeOJvpQ96HD6jPk= 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=uV3p9Nsu; arc=none smtp.client-ip=209.85.218.53 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="uV3p9Nsu" Received: by mail-ej1-f53.google.com with SMTP id a640c23a62f3a-c15ba3a2b4bso1403972366b.1 for ; Tue, 21 Jul 2026 07:06:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1784642792; x=1785247592; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to:content-type; bh=sizHsiXLEVRMUTlHJ17CUIbyBUb9Anbko+cLuXMaFUs=; b=uV3p9Nsusezmnh18SVl+vlJoNU0H+Qdoc15wVF6gGjm2AUBx58q+tuiC0L4tMvDcRi eMLU4LMUrzkktTzfCNCmi+oTzZaB7wOHbtPxm9sZx6rc8iEyAYWVIEMharOLymvVzTta dexIQXvUVm7tkaD8TUnTMf/8CTWefea866nXPi9Wck0Tm96w3dRI8oMk5nwCFsQEcdCv mSyPb0zXWNEV1j2dt5F+VhT98LqGrSaeGrRpC0gVKbkcFdwyz7LPxQlCA/5f2tX/Krgq LyTbdWfFcAhRuCjPTk00bXHgOokIXNAHbS5dG7VGztNd5IZV7+94djuJwrz2oHUx2JaH WvVQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784642792; x=1785247592; h=content-transfer-encoding:content-type:in-reply-to:content-language :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=sizHsiXLEVRMUTlHJ17CUIbyBUb9Anbko+cLuXMaFUs=; b=HfHzR1A+4yqnouoVtMth3uqnpxGdHnDaRIQezQvZF+KowzluZEw5+wvm4xgFhoZknV kjl46/xyVtjG6BqawIncdXCtG6Jbph22rtfmx6Yh314HNVsW3KM44ldZoQkF6ziE5mWg 2c78cqnjwvrIQgptHZQmBiSRzabARKAHW+iUoIdnLjfycw3y+Ay7BH0kN61muUUfO3CK qlATPTfmd3iY5teL/Cfkd4eJYBts3uiA0etcybVJP1aVSNJo6PWleNWu94Q1l7H+JHXv xhPmL1HZVi1nryMhpNuJMXk1w7O9DPBhV84S8Etb9D5SFclNVoF1/cfK7jP2DjykDhAz wsEA== X-Forwarded-Encrypted: i=1; AHgh+RogaQ/LNMeL+gkkSl/cZP8b3g1V03nLYSAFmT4dnImtnmvecqZxqPrvJ1FkC3J4kA+ISyTKZIOFv0WvOP8=@vger.kernel.org X-Gm-Message-State: AOJu0YyzcemJsgxymyJIuwEn+UtiRENIrldmnehbzmGt2g0ErCbxeyVR FbqxUlWRFNXaEv5MWen0KbnWmyYIwfOx2e2iMkMsi70YduvXIxXrlMApOjInvvxMw/s= X-Gm-Gg: AR+sD10U7lrPlrhhDCWwvd2xYrbKWEJ0O038NSmuLwiU9CFBc43+xXzyUSHO/OshoOJ LD3tjC7MfmLZp4ESvJWKskQSwU7uIJHiQr7EB7oG8R6+HYMNU6XPW9d7wkATmX6YCFwIqZ6NQ+P KGoihSjZtHK5eUVLkVtWWh/lx9+ozR0lZLcTFkUIRA3XYK7HpbMfwjveWw5P+pDBQk9GLL9CjqX IKHRhTbfEsAZY3kek4MWyHGiUjDP36HJ/48FXGjBwIwVbAPmtD/lP9Ia8kVEPFqgNvAdzPZkwWK iNdAcYF5Xw2vxZnRA3N4CK19XRDUeoyfZlaGKsZds8YAoGZhtNnFnQ2aQabPw1klacuzTepJk+o zHpNXOCjM8Cu1aWwd0g0by+EjQq6TnR2fGGHfvB8zNprRJPRvibfAwFn/x7vbaclfxWOpEdIav+ w5EiG/+qBA8DobkmNkUZsrW5COYg== X-Received: by 2002:a17:907:9627:b0:c1c:2c22:3aa4 with SMTP id a640c23a62f3a-c1c2c224b1fmr47399366b.56.1784642792212; Tue, 21 Jul 2026 07:06:32 -0700 (PDT) Received: from [192.168.0.101] ([109.76.186.235]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c1740bf5df5sm589883666b.61.2026.07.21.07.06.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 21 Jul 2026 07:06:31 -0700 (PDT) Message-ID: <33b60b3f-c256-4c48-9608-5bd23f18b1e5@linaro.org> Date: Tue, 21 Jul 2026 15:06:28 +0100 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: Vladimir Zapolskiy , 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> <_t1jiMTX3iIXhUkTDz0Cbh_uZhdUK_Y8vlIwxt1VNRbuKuRnpVCWabal-b4AuFrVaKrDnQSFOWtDn6-VCbS3DA==@protonmail.internalid> <345ca98e-09f6-4965-87bb-e64dac096498@linaro.org> <1f128571-ed46-4a60-9594-74fdd5f440d0@kernel.org> <864c13a0-a52c-43f4-9d94-646bf66ba31a@linaro.org> From: Bryan O'Donoghue Content-Language: en-GB In-Reply-To: <864c13a0-a52c-43f4-9d94-646bf66ba31a@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 21/07/2026 14:35, Vladimir Zapolskiy wrote: >> >> Had a brief discussion with Vlad, Neil and Vinod >> >> Resolution: >> >> - phys = <> retained > > It should suffice as an optional property, moreover any kind of > phy interface usage is not supposed to be done from CSID driver. > >> - phy-cells = <0> >> - bus-type from the media-controller >>     Already specified with camss still aligns with Rob's declaration >>     "the consumer defines the mode" so I'm comfortable with this >>     compromise position. >>     This will still facilitate CPHY work/mode-selection via >>     phy_ops->configure() with how to define the CPHY part TBD. > > As it was discussed there should be no phy specific interfaces > between this phy driver and CAMSS driver. I'm probably not getting this point, sorry about that. For clarity though CAMSS will use phys = <> and the PHY API phy_get(), phy_power_on(), or phy_configure() as there's no other possible interface to this driver. > >> - data-lanes = <0 1 2 3> valid > > Would be nice to get an ack from linux-media maintainers on it. >> - MAX_LANES 8 instead of MAX_DATA_LANES 7 > > Since 'data-lanes' value follows some register bits and not > the actual PCB lanes as in video-intefaces.yaml, this shall be > explicitly described in the CSIPHY dt binding documentation. Yes we said that. >> - v14 with these changes shortly. >> --- bod