From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A7B153F4130; Thu, 9 Jul 2026 10:31:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783593115; cv=none; b=t5o2XS/rh/Dsxo/9TfdtqwYGimxTss6r0seqAyMFQW9xr4ep3g/y2xdLL8tlwwwBGeR1DA79ZmogtqK+/Rsj311CDgoIfUiaX/8SlPrLFAf2/9zRm3734lUNpDZ9BJAFmFKvHyxuFGEim90zNHXddtj3Vcc882+EJh3MZktVPEI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783593115; c=relaxed/simple; bh=4kBh0293uRNcjEIm8JcDC43jeExZf8brb6PiYAa+U60=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=DseyAsOZDqPiIhFqcqVORUW2TZEb9uQpkKOqFFSU2XGAinBSCIIj6PQ2vA7KjLwxe6zO6y+1zFl1YdH0RjuRvIe5LBSy0n+rYNiSxmEoNYUA+p6Szo2pR61a0un77PXbw0mBycztC5RD4c0o0i+QfRJfD2/0G2Q1pSEGuqMqrW4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=SXH2uh4G; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="SXH2uh4G" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 89B551F000E9; Thu, 9 Jul 2026 10:31:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1783593114; bh=Jnd0dDhsHRaigqmdT5qRVE+w4dLZnMejtZ+vWRcs3uY=; h=Date:Subject:To:Cc:References:From:In-Reply-To; b=SXH2uh4Ge1oNU8mwewe+yq1dyeBeAAEIBJBAzC8NLER5JpNHwEeVT70mNhvUViJU0 7xE8C2QUwH+TJmmL3T4n864wDoBd6BcflN7yCKVQ7zm8eyVmvx56dBVszuV5DbKglW nOOgPQ9ruw+wB3pCytWqpBki+Rh8ocsDKMDejVVNV6TShCqwR065w85g/l9trPnmfA PpdHHXcA2aWJUDan0DZRHEo6s5ZhLDG2eLzExn1pxUHTkKMokI+ei5tY+vTtiCAnc5 m7IMeKTUCN5Wfs9uLalta7dlcgH0iVCc9X81u4atqGeNtFj+AvHxt2Nod5SYfQe/7a FKTAuIsqi2IgQ== Message-ID: <6f38e218-d3eb-41a6-a89b-7a48fb6fc663@kernel.org> Date: Thu, 9 Jul 2026 11:31:39 +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 v4 0/5] Add Qualcomm JPEG V4L2 encoder for SM8250 To: Dmitry Baryshkov Cc: Konrad Dybcio , Vladimir Zapolskiy , Atanas Filipov , linux-media@vger.kernel.org, loic.poulain@oss.qualcomm.com, mchehab@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, andersson@kernel.org, konradybcio@kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org References: <437d1eb0-4495-406a-9501-b0515cfa3151@linaro.org> <8116c54e-4050-4e9d-b236-59d74846bd57@linaro.org> <8a09875d-af5f-438b-b947-bc2b61219b70@oss.qualcomm.com> <8fc1ddfd-0f77-4b67-b9bd-33fbd60e2046@oss.qualcomm.com> <54ad9b8a-7596-4bc7-a1c3-7230cca21360@kernel.org> From: Bryan O'Donoghue Content-Language: en-US Autocrypt: addr=bod@kernel.org; keydata= xsFNBGRJNSgBEADD7Vm2ZFa+v+JGJ2QYTJqQAkqis/uOHkhdFNXqpBarVBd47QU/DMNU5Rxg jedMQEmHoeDbJ6UOpjbrUQ63c5sgG1JbroHJJctwsEI75OOlekMuebEbjIJBLfgENGwPBMHv piv5TgCWr0VgYaXfp2eh2LINFywzqj823HiDPibQAXDrjzvF1ogksi/6cQZs8d4if8YQkLOr YISFouG+eR0nN1I7mUfIddXOWu6lJeTyqbWVurv58k2ekIXKaOC9ixLHFbcfYV0hOgRaTwQC B8CYF9nfqZla19iItfsN9QxN+ZdQjcRoYipp6HPCMfJlKH7GfaFcW93LKc4DKJ2lVL+pg/OQ lythZbjRPY492NG9kZ65aYstCs90uhMUEVVPuGUw7wBEku+6IEwZfrbMVKeWzLlPyM4Hv9hM 8ktxSmxWsPTPqpBC8eyeAQLalMELAyVcZlkaCtEcbj7w4l/JkYz+4l37obG8ZD+B34udBUUz MsAJ8foDFrBh2MOFA3hxD6G90D23mmWsri7pnKA2tZs92aQX7Ee+FbCyg6g5ln62Sq83ZDbf 53DdBs55EVpBadeInWmXhzCHPQx06H+CwTEjShTYIaMmBfrewvYUDKvFTC5iKQhAEUgt6i94 JsbG7NoeqcxkUMcBOEUQ3uCQG1D70ugspgXc0wd3Rimiq6535wARAQABzSFCcnlhbiBPJ0Rv bm9naHVlIDxib2RAa2VybmVsLm9yZz7CwZEEEwEIADsWIQTmk/sqq6Nt4Rerb7QicTuzoY3I OgUCZ+R+mwIbAwULCQgHAgIiAgYVCgkICwIEFgIDAQIeBwIXgAAKCRAicTuzoY3IOimUD/94 BwVEJX31JRe2sxbB/e1w2p8x1bxvTw5AeIzpV3ox7coJg1bSU2mnGuj1V4o0Yxf/3zmcJzCN VfVjwRF8Ii3GnC7uUXk2t+87piQfKTyJAYQABhZUKgoVJbjJq/S+C3XCKIyBA+EiezoUsgsA jTzwU+FzV7zVWIXFPJNtBERLwboE9w9U3KjAExOa1kSY8eLrsg6kOwlOHWy5UsQqYOjrS96M mzm2xuc1+RCjrndAyYhCnrOKvJ67HsPnBeJCjw7ImGD/U1GchwYbX8o3DO3JNHm3qfC86ZqX 2sCouENg4OzgPTtLKUrueM6xsu6KMM7gj17vxsiR3KQEoJnnMB8D1xtBofN3mFZE0wD9M24m 8yGunZbtntMCUHzIrlJgAPwKWKuGOYtA8UgMTFkccnUJtQrg9KotKtEF/FuftG9zLG9XEkt4 5ZdNgbSoLWgelu3T47mbOJ8LHhiLaCWP7yrovtVAvLUQ1BsiA42u8ECrFCFvQj9nrejE/ICv kP+uqcKtdDvP9HrIGycF1WZyfZLp0RvopKW92FLvI4I1QFWJ+wenk6+LGyJ5bzlrWzevjxmf nHcXE6sJBHrE7eijlbbImDAi3uLYN8Nd9Dm11IDAy4GAIQxSiQn0yblDhPiyGtchy80EVkCm g9k17Wol+2E2mC4DKgVdCkyUtTRSLgsJCs7BTQRkSTUoARAAuTnmWHBS6izRcEE93ajpzI7h dgQO4U3IRvOEsvIKR5NGcNEs0ngGebwsZ/lVULjN4vYU0LleqVhPBidNXUoZCN3A0F0Z2Ov8 NZdef+2EhQPBVWxFO7JBzhe8Z3ALj+wFtlg8akJjBzU56azW/iJzAobqHVrudzKoO2b1/CMg VbiAQ+RXjgfN5kY/HqYDU7mw+hXuUV9PbtX1L8xqQQac95oM9rHzKHHpiVwxTeJnGQsa+THi Kze+YET3rCoGHMvOQEJhdrucTv5FpAakKdkOFNel9FFckLRKEuWgCzhpFsjQ7xbirQgFUxG9 vlk1+q4hMRGNyEqoD6svYEeqbiUSd0oPUJeioiC3rNMRCNHLVrfZ2J6SCPkxfda08uzSdDQU 1/YPjOh8ZtQDMu7WctZ3XO288Z1gyBR49V7fbFs2w4sQxG+h/enlxqP7fdw1mjUlZjU5huCJ ielS0oEaIpmUpkugli7x4WhwLnhK2EbSoz7nLBC0y+ALUOdMlz/Y1l9xRt+bkDhpmf4O4IcI MxgZ0QMLq8rHDkGaEbsgZZHQPS58T0XE3IP30Q9SNxsruCMXtd2hYtBssf/wohc6JVsTtMg2 VYTPDPIFNZFSXupEJB7jlqpDWJ8ooJfJRLBatbjT5+mVQaMYB7Hs/t+zWYWaJKHyc8O6WLEC NUV5Tdt5EkkAEQEAAcLBdgQYAQoAIBYhBOaT+yqro23hF6tvtCJxO7Ohjcg6BQJkSTUoAhsM AAoJECJxO7Ohjcg6LuIQALnXt36OUuK43wqw6UYt0cnN6EbUqJHApAF5eNFn0jCCB2XELjSz JKJwuNAweowBdabiBniJ+501WIW+ewEsz1uby5fUQjZuCEsIkuaIluyfUFPb73qrQyAGuusd 7teA4WT+/jUku9g7lX5sVoRCrKQPkd16f6Bzfztyqyjcn43/X5yQI+wlboQ6HuKe/3I3yiOx OgmCHzOawpC9PvhEcKj79RLM3Zz5Ts5AuHpRX70Jz8Be76LwVFLp5Msx3S24ZTU1lBo2uiJ3 xSkay2lTpyVWRPx9vgcwzxGguOPJQJwsQeLb7wpoJMPpD3ERoaRii7Q7hvmxklpZjhKYWB3d t6nQ497Ek9loCrp3MIjRCSDN5xEGffiHks9yTeGMUQwO4tX8RE04uOJPkUY7uCFzFqN6/qey X3oFfPgkULMdiHofPAL1OskZSTzGPSfTYRE46NCJw8yoZBQ/oOyWeqaUQbK0wmW/g81wm8p7 LKSGEglMpiX07M1AotgvylN5C8fjbouoK+/RAMsXkk8jba6rPfuuXPaDjCyyKn6zSVHETnHW 3AJbgVY50T8STpnxayBQvWbCvu+6NOEjXCbyaOJig+5l0zlGN9XHjdANXC5HnwmyaGRL9YDq Jh2nVXVJDincOdQRdKcJjYLqaOAoWrYWSDi1iZGspHBTDrnOvfMQzzHY In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 09/07/2026 08:33, Dmitry Baryshkov wrote: >> Lets drop the bus idea then. I'm happy to close the conversation as >> peer-nodes. > It think, this would close the path for upgrading existing platforms to > split the driver / bindings. If all IFE / PHY / etc. are subnodes of the > CAMSS, it is easy to play backwards compatibility tricks and > reinstantiate devices on the fly, pick up only certain resources, etc. > > If they are sibling nodes, it becomes much more complicated. > > I might be wrong here. Maybe we really should try a simple (heh) > experiment. We have your series which adds CSI PHY as a first class > citizen. And we have existing platforms where CSI PHYs were packed into > the camss node (and camss driver FWIW). > > Which way of representing device nodes makes it easier to migrate > existing platform to the CSI PHY driver, while keeping compatibility > with old DTs (yeah, ABI, old DTs must continue to work)? I agree with your concerns about old platform migration. I will run the peer-node through the DT checker and if it spits out a problem we can re-evaluate. x1e will test both the old way and the new way - I'm prepared to take the pain of resolving this change on x1e, so that both dtb bindings work or rather continue to do so.. Either way we will resolve the debate here. I'd put Kona/sm8250 as "next up" for migration - I have one on the desk in front of me, so its easy to work with this. camss { reg = <1234>; reg-names = "csiphy0"; } csiphy0@1234{} Is a legal DT declaration - if the csiphy@1234{} is missing use the old way, if not use the new way. if the phys = <> is found, then the driver will not request the csiphy regs - this means a new kernel will work with an old dtb. I don't have that fallback logic in the CSIPHY series right now so I will add it in. >> camss@{ >> existing CSID/IFE >> IFE SID stuff goes here >> power-domains = TITAN_TOP_GDSC, IFE_GDSC >> } >> >> csiphy @ { >> } >> >> jpeg@ { >> JPEG SID stuff goes here >> power-domains = TITAN_TOP_GDSC, JPEG_GDSC >> } >> >> camnoc@ { >> modelled as an ICC provider and consumed by > I think, it's a part of camss. > >> }