From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f169.google.com (mail-lj1-f169.google.com [209.85.208.169]) (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 B1C143D9DC0 for ; Thu, 19 Mar 2026 14:56:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773932185; cv=none; b=mWOWFYc7I3s+YvIywMUk8EryQl0IlO17GOFhUeOTXrGMpPrsh/t6sX+gCk4+5+PrmCuDX/gq//1XaRHiHrmkHLSyOmW8TJxk2M0M/K9xtArejmQnCnpyjWUg4RTWJHrBz+tE/Ig242BpcspjcQugN1G1q4f0sfzmvNsJAezU35M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773932185; c=relaxed/simple; bh=gd7SCs27uyPu79nVL6/LCI+yIXMU2kzESBFAMZUgfLI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=lTH8QT/upIwwBdvVUzrJuWDS9ZZPcp6TWgCwkrNkfGJqlnSBhg5Wf/odzpjwjP41+Rk/4xpawt5g0HUI3WzaqGzyITc2tMhOn8hrfn8D1F9RmcCsQzTbK+3hd0Ru8pRH3mxiJloLFNYWbnp96Na0bTBLbdz/OVB6nSVi4W1c25o= 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=OU+QOCS2; arc=none smtp.client-ip=209.85.208.169 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="OU+QOCS2" Received: by mail-lj1-f169.google.com with SMTP id 38308e7fff4ca-386c441ba1dso950541fa.2 for ; Thu, 19 Mar 2026 07:56:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1773932182; x=1774536982; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to:subject :user-agent:mime-version:date:message-id:from:to:cc:subject:date :message-id:reply-to; bh=nFE8LLUUKG9iTuvDaB93WvqlPoGXC9p6rn4PhnM6x18=; b=OU+QOCS22gmyJtPUxAd2+gZZf8fRVP81YY929OBTR1/7TmKG+QnvpvK5vx0f7hA9hv GOcXrJ3H02NqKoDfh6e6xKYCrvW75SSP6QfRJLtH62ZPwgDyzw2eoDg8ClOK4YW1P0Az hR3gm/65Q4xp0NPKN8HhshunAgoC6d9xzd/3sCjCCEs49A3MzX3kDsHYnn8dbq6UltDs HUTyZXPg2zMmLSGEM1+Esf4BaW89RTp6GcPgxM+GPSGAwVH9DkoOgUE5c5IunV75l0sr mTJWFuE5jfd0+j+YJmQ+zIjztR7WK2TbLLa+CTPWMhpv2sDAvfpn4GycDJfs7j6kQvzQ x/Vg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773932182; x=1774536982; h=content-transfer-encoding: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; bh=nFE8LLUUKG9iTuvDaB93WvqlPoGXC9p6rn4PhnM6x18=; b=nCbPV0DCnguve/9kB3iDktKQLqAuECFpC5/hDAgMXnGIGaXQ9jrs35G/phdDstgAVE /bqOY3cFuxIj3enT0Mex1Hk4RdKGivucPC6vI04FxmaBkHGRxwqmFQeWyft03EEILYL/ esqGkt0poNmK3e/N9Ao0ZWfrnplHv6mLj4F5dVHnDaojjxhtvDZq/t1Jz5zZg+i1GPJS fYhC2hn9ZYMbksHtRkOup2YIrov87VRdawqtIKCDSMranSBVQE/nWlDQWWpZjMLF3cjU xitiKcl3iNEDAR7DzncbS/8MrD4I70KEylpj+PoyEYp3CjDcwipnf2K8FRcUUuO06eLn M3ng== X-Forwarded-Encrypted: i=1; AJvYcCV7Xk2XoW+C3eiz/gSsLfwk3rSb6z2VWfUcutAA8Kf7YhabLT6xRoFE4nmb2bB2ouqzr2DgdKuHY24qy1k=@vger.kernel.org X-Gm-Message-State: AOJu0Ywlv9jMoMt2tZmGXxfwnqYB2nsjUWhiGSEl2tHYnDRHYCS619ZU wbhOwVgUeYqLGsi2FAOedGUVJiWiyojlYuw9vVelzh3WbFLwM0vAHl4S1QQFQ1XisuU= X-Gm-Gg: ATEYQzwpiSsUGh0nYTQZXJH4TEUWhLou5bTznuQOrIDnjra8m85guU2J6CvlSa0qtWM jiVUbI+vvhXOXcz49s+QNZgsY0EXhj0hisLa0iOHcJRgDVi+ceoOedOf0tyOucFnRK78vi4i2Sm u3cseRibOJz0ZKPEtzAghDFcETHrZgopDWAlGvQ98gzcelJTID0y4C4J8LzqDYfGoDNCkhURbqC ltQ6eVupATPPQjwS5gHRcs9eTEtHLdo6uMBoXZRewizEp95tJNIFIUCppxaRP0Aayum+NpX1M3I Qe/XsRuAXmxCOJOFTurBAFOucZrhP6VFO8J0sXYEBDDamhF/X2+xqEjWvNjT9H7bl6c/bgYghFM FU+QAAElU3jzNARxQcleBCw4HPlsJIObdjkqfNh/FRUycNRQ9cZgjqNAtSNmDiyZVMEnFbWeAoh n1F2N37tWn4DdcRd+pY3jBdsPddPWc/rV2unaFcFbGfMl8qgEHKFbW00HM4JIUF/IXZtoC7xo2d IouxA== X-Received: by 2002:a05:6512:3b82:b0:5a2:7c19:414f with SMTP id 2adb3069b0e04-5a27c1941bfmr1196514e87.3.1773932181770; Thu, 19 Mar 2026 07:56:21 -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-5a279c2be56sm1230577e87.19.2026.03.19.07.56.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 19 Mar 2026 07:56:21 -0700 (PDT) Message-ID: Date: Thu, 19 Mar 2026 16:56:20 +0200 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 v4 2/2] phy: qcom-mipi-csi2: Add a CSI2 MIPI DPHY driver To: Bryan O'Donoghue , Dmitry Baryshkov , Neil Armstrong Cc: Bryan O'Donoghue , Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , 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 References: <20260315-x1e-csi2-phy-v4-0-90c09203888d@linaro.org> <20260315-x1e-csi2-phy-v4-2-90c09203888d@linaro.org> <3f11de22-b729-4d06-b6c8-18e649e1979c@linaro.org> <80ddc2b4-d6f8-4e8d-a45e-69c05d100aa2@linaro.org> <16b10f17-ecd3-4cdd-ac3f-f64127d60ace@linaro.org> <26XTdUyQTB41Oc4D5HnMtSm_QpZRjlkljQRJVw-u1Zp3Ltn9s4LVU-LQkP6drdl3Z3GGssLCCbsVYPFEqssHcQ==@protonmail.internalid> <65e06b2e-eeb9-45af-97ac-4ae60f652361@linaro.org> <9578400d-30ac-4d8c-9295-ee4ec8af3b2c@kernel.org> From: Vladimir Zapolskiy In-Reply-To: <9578400d-30ac-4d8c-9295-ee4ec8af3b2c@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 3/19/26 15:17, Bryan O'Donoghue wrote: > On 19/03/2026 13:08, Vladimir Zapolskiy wrote: >>> Why do you want a media driver? Isn't PHY driver enough? >>> >> As for today CAMSS CSIPHY are already media devices, and a user applies media >> specific properties to them, for instance media bus format, resolution etc. >> Technically this might be removed from CAMSS, but if so, then it should be >> done before this new PHY driver model is applied. >> >> -- >> Best wishes, > > There's no reason to remove that from CAMSS - it would be an ABI break > in user-space anyway. If technically CAMSS CSIPHY could be excluded from the list of CAMSS media subdevices, then for the sake of simplification it should be done for all supported platforms in advance, such a change will be independent from this particular phy series, and vice versa, this CAMSS only driver change will prepare a ground for media-less CAMSS CSIPHY device drivers, hence it shall precede this particular CAMSS CSIPHY series. For backward compatibility with userspace a noop stub will be good enough, it's not an issue at all. > The media entity in CAMSS msm_csiphyX handles format negotiation and > pipeline routing. The PHY driver handles electrical configuration. They > don't conflict and there multiple cited examples of this upstream already. > -- Best wishes, Vladimir