From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (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 82E1F3FA5FD for ; Tue, 7 Jul 2026 12:17:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783426675; cv=none; b=M90mi0vrex2Eon0EeG2/CJbQclnMyqAC/ldMiS6Sz4eXEUyaCMbWsmrAlZkD8gZgaYuy3iKWbG58p6h/WRiMdyHCGl/KN3762fh7+KnxElFdqs/MCVi1AVXriJ+kg8VyRh+m0C19ro95FThc+bHyz0TNs5812CrdedrXgwD6Dgc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783426675; c=relaxed/simple; bh=Ta/bE9egbWl7Xktartl8cK27C2xuvBOh7ciGfNUeAeo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=u22YVR8PEZhMR5b+uEpQ4mkB4q8PpXqU02gPM3NSQrRn8MYxApsbNwKmMm7lxqBddB201oDXToopfhYHrJR6AJQSaOcYQs3P8VPYeOPKgnoPe5ZkNW+dsUQI3msILu3gfkLqgelrd9zkll2Rzhr1QHH6srTIlVEEuFfhqB0IDO4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=UzIJtxOz; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=gvmRMiT3; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="UzIJtxOz"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="gvmRMiT3" Received: from pps.filterd (m0279867.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 667C8VwK3741874 for ; Tue, 7 Jul 2026 12:17:53 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= 2SiZ+GGo/Uh7dQhyhwAq6wJ90giuKEqOR8zd+Na239c=; b=UzIJtxOzV1cRutQd LSIBy1pYgk5U6t0gbqXaVaPELgnzQ7Fgt7J0491uxgy4H8Det52h/mOZIsq8l5OX VRG2x+1+D6XaVC1Ji7WT4pONK6cM0my+ta274ZayqF7h/fLoQAXC8IoRXFucj80s GEFO/+tCb2On0YkvEtbKrBwY+1UMdOV/DMGAWtG/usI2gXg1HWaQxwUF4Mn8lWBQ tAa4Vi+Lv6EJLukmd0UK8UfIe3rdtOwysmQ4YdpGIDlFndlMSdqE/CzlFaE6LgAd 4EMVHX72KPC+RuMzuo/tSwT+ff2qK5yFnuPcfs+NxfZkG/L2i7vIbgUnXNJVuBMg EpVH9Q== Received: from mail-qt1-f197.google.com (mail-qt1-f197.google.com [209.85.160.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4f8ye0gff9-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 07 Jul 2026 12:17:53 +0000 (GMT) Received: by mail-qt1-f197.google.com with SMTP id d75a77b69052e-51c1e6f602cso71711851cf.3 for ; Tue, 07 Jul 2026 05:17:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1783426673; x=1784031473; darn=vger.kernel.org; h=content-transfer-encoding:content-type: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 :content-type; bh=2SiZ+GGo/Uh7dQhyhwAq6wJ90giuKEqOR8zd+Na239c=; b=gvmRMiT3WL4SnnE1vtXtuMrIC0z/mT98qoaU7cK4/ThdVWeOlIIOy+ZP6S+mnuEJ3o 3DylnsrKgAqF+s2KI3wK+duzLOpqtZ1Xp5gXnvXyFHRuZx2mz/wr2cCf1Dy/LFVCnRIE AJfdZeN80DUsNoBIJVLE0okD/5H4qaVLEsQR6giAdVGoHHup6XUQHsdeFgOZWT7WgxDU ghWR8NDb3yreCHADtMuXLvJ4sZnyw2tMRz10Xd5MprlTGMOhNRdXKDfI73JJae7BbUDT 7VfqZUe18SPgHRAgp0H05Fsjj9woFgZWXCXdvmsI3YeBZKgs8F+AkJyCvYz+fgZc7apa yiUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783426673; x=1784031473; h=content-transfer-encoding:content-type:in-reply-to:from :content-language: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=2SiZ+GGo/Uh7dQhyhwAq6wJ90giuKEqOR8zd+Na239c=; b=l2LNhy5KfaeZEl47/ZH9nWQKt1J3t3sKJvZKVMUnEJIkrlWyChMQ5sPvAa1pHSdpm4 EY8NyuK3A9HKI+5todmrHKeLUAZfJD0SQsLP+afKbuTJOcpva+EXV7die+qBY7O5mABo W95i/WwRsevLv9OXlyKW5kqKe1MH3H4wxaPa50PHzwavGQ3MRjIzlttCgnCQidFMsAsk I5wN9ayVyMq8Gq1/zCmTRllviWmQauHZ+UvqejuVD+WX6Ki2mQhHkVVDtbrmS9wirmPr wQ2doLsxhvLGdKWGekPMv9cl9nXFznUgt80boA5S0y4f97HXubF3CNpp+92MiKWRvw0h Vx/A== X-Forwarded-Encrypted: i=1; AHgh+RpCw1f7rrVtLtvjNM64R6izPRPh9dWbyZCxXDBbNa4Y8V13W2rZZoBp/jt4zCwrVJopd+YYzW9DqCKRzFo=@vger.kernel.org X-Gm-Message-State: AOJu0YydfYLk+pgXVMh2rRzlE1Vm9Pso15iFUMr98AokQ7/jtjLRdk6V 4L0fhEIJebgt+cCN73JTaVJsQqs+mfk6lAL7sYP0ZRQ/rSjYPxFWoe2oNGtTPcTv6ybLWdsfGkQ pHVcdTYJ7oNKPzcHhP3xpxfYsf2+BoVWSsJfSgx76TxG3X4Q+2W62Ply11rK1R4BQsKI= X-Gm-Gg: AfdE7cknClZd3zQObFOuK7dd3xa3PC+QEPiXY7lS0HP6/YaSkIPi3dF76D90X7G9Jwa i0osYvVNdVn9VSvTF/cSz4KYqerY0efvXj9i+Eazils43mktqm78XL7YIvIYOFLPqDGjkbtddDz eJ2gFHsCzxKp85BTj3PRLRCS9OvYa1Nx8mrt0xu7ZSPS4kfdph8bGnaXH+IIl+4dZDtE7oKfNpX 5i9Xn3Q7BBnvZBIA1gl+l9MDeHdeb1BA2tt2zq5plRY/G2qoOajzbjIx0ghYKmgbvtv7xTJ0n7q 1Qzw39JhTsGgjDLlNte2BngETcwt85OoV9FRecBkgiaRer0ZNLX/DV0W6XcPFZ7ep7H+/rfat9k CiLsfEr+ygqBfjneAx68Rr1QK07EbnPTDWejdWZLjr3UkrYWUKK9gAuQjWybuiTJ5jq9+dbucLK ZkAgDs6TE= X-Received: by 2002:a05:622a:286:b0:51c:7b12:120e with SMTP id d75a77b69052e-51c7b12172fmr21403991cf.76.1783426672499; Tue, 07 Jul 2026 05:17:52 -0700 (PDT) X-Received: by 2002:a05:622a:286:b0:51c:7b12:120e with SMTP id d75a77b69052e-51c7b12172fmr21403491cf.76.1783426671913; Tue, 07 Jul 2026 05:17:51 -0700 (PDT) Received: from [192.168.1.31] ([85.196.172.179]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c15ad9bc655sm118332266b.41.2026.07.07.05.17.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Jul 2026 05:17:50 -0700 (PDT) Message-ID: Date: Tue, 7 Jul 2026 15:17:48 +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 v4 0/5] Add Qualcomm JPEG V4L2 encoder for SM8250 To: Bryan O'Donoghue , Konrad Dybcio , Vladimir Zapolskiy , Atanas Filipov , linux-media@vger.kernel.org Cc: 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, Dmitry Baryshkov References: <20260706071113.383215-1-atanas.filipov@oss.qualcomm.com> <68e59764-5a10-4abc-849c-14ea2d9ef76e@linaro.org> <437d1eb0-4495-406a-9501-b0515cfa3151@linaro.org> <8116c54e-4050-4e9d-b236-59d74846bd57@linaro.org> <8a09875d-af5f-438b-b947-bc2b61219b70@oss.qualcomm.com> Content-Language: en-US From: "Gjorgji Rosikopulos (Consultant)" In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Proofpoint-ORIG-GUID: JZw3dLeLxfOpj4X0JZ07dZvFeZDzu77w X-Proofpoint-Spam-Info: AW1haW4tMjYwNzA3MDExOSBTYWx0ZWRfX/RYW8Zc2y4os v0dWNLs3XHVmFGi6z8jsVg/kfUPGEKXEpU8JrQX3isjw9aZ7Hom6fmmwT7lN6SUrKzEYV6ShUdO Gtuo3IU0+K9+ISGNxVsmHHbczeEoyig= X-Authority-Analysis: v=2.4 cv=SZfHsPRu c=1 sm=1 tr=0 ts=6a4cee71 cx=c_pps a=EVbN6Ke/fEF3bsl7X48z0g==:117 a=Q/e3f29T3Hw2hnAEzBPF7w==:17 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=eoimf2acIAo5FJnRuUoq:22 a=VwQbUJbxAAAA:8 a=uTxpKD40luCw_XNDSPAA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=a_PwQJl-kcHnX1M80qC6:22 X-Proofpoint-GUID: JZw3dLeLxfOpj4X0JZ07dZvFeZDzu77w X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzA3MDExOSBTYWx0ZWRfX/8dy2r63/whs ABJemNOKJR2SEqETKfBiNrGjGSaQkALVbkupTQepg0pdhZ5xG51EeOhu6D2dOWFG5ZKz/a19bCe vANlBqpGK46Ebkyu5byBSL+gMy2l7UfJp4VQ8eS7NqgyOwuVx3NC1LTf+l0dQW87Kkn9xmeGtVS gLqAEIeiAAcFKL0tShKVFLbIGKltzE783zXeKToq+HQ8M86KR5vP7L3kNhfdj75fpDJMwaCmGLl mLn93dH/xGu3NvwToXRJWSNZT3foY9Ne9SZVZjha1wMpf+zRA9h6Aykj8QlYhmNdaKNu355xhDw i1H7F1ZKcz/iFDJVGBRHIIAYc+ZVKZEy/SlttvZo2q1Qduznuq+rpV+fB3e8O49mB/l5PaoHmgb cS7JdPKlGjj8USUv6uAxCf5xyTGqMS5jwotadV6BdOH8hNgHfokwLNK7bGIzf4ZXOFUfJEvT3eT F22vbW2SBk6mIxqgfKA== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-07_03,2026-07-06_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 impostorscore=0 malwarescore=0 bulkscore=0 spamscore=0 adultscore=0 lowpriorityscore=0 phishscore=0 clxscore=1015 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607070119 Hi Bryan, On 7/7/2026 1:00 PM, Bryan O'Donoghue wrote: > On 07/07/2026 10:24, Konrad Dybcio wrote: >> On 7/6/26 3:37 PM, Bryan O'Donoghue wrote: >>> On 06/07/2026 14:02, Vladimir Zapolskiy wrote: > >>>>> >>>>> The idea is to make all of these peers, eventually. Not particular >>>>> block >>>>> owns the TITAN_TOP_GDSC - the entire CAMSS block is a collapsible >>>>> block. >>>>> >>>> >>>> The statement provides the same information as before, e.g. it's >>>> found here: >>>> >>>> https://lore.kernel.org/linux-media/fbc018f5- >>>> c025-4747-85f2-53b45b0f0496@nxsw.ie/ >>>> >>>>       There's no functional linkage between CAMSS/IFE and JPEG - >>>> they are >>>>       peers within the CAMSS power-island. >>>> >>>> Thus "the CAMSS power-island" exists, and it got its hardware >>>> description >>>> as the CAMSS top device tree node, and TITAN_TOP_GDSC power domain is >>>> a natural resource of all devices on "CAMSS power-island", >>> >>> Vlad honestly, TITAN_TOP_GDSC is provided to the CAMSS island - all >>> of the nodes within it depend on that and it is _external_ to the >>> block. >>> >>> There is a CSID MUX on some SOC other than that I'll restate it >>> again, I'm not sure why you won't accept it. >>> >>> There is no dependency between JPEG and IFE, one is _not_ >>> subordinate to the other and trying to find ways to invalidate that >>> statement runs counter to the facts. >> >> They are all subordinate to CAMSS_TOP - Vlad indeed said they are >> peers (i.e. there's no co-dependence between them specifically, but >> their parent is shared). >> >> You can't use any of these IP blocks if you don't first power up >> TOP because of the physical layout. > > Yes and I'm asking for TOP to be listed in each of the subordinate > nodes for reasons given below. > > But unlike MDSS - OPE for example doesn't require IFE or CSID to do > its job. > > Over time we can move the bindings to a model where the only thing in > CAMSS is a top-level bus like declaration but _right_now_ what you > would do is create an artificial dependency on say CSIPHY for JPEG. > >> This is exactly identical to the MDSS case. >> I was hoping this was clear after N other similar threads. > When we split IFE, CSID etc up into their own individual nodes it > would be possible to model as for argument sake > > compat = "camss-bus"; > power-domains = ; > > jpeg@x { >     power-domains = , , ; > }; > > What I'm saying here is do this > > compat = "camss"; > regs = ife, csid; // exisiting bindings for the last ~ 11 years > power-domains = ; > > jpeg@x { >     power-domains = , , , ; > }; > > Otherwise you create entirely artificial dependencies between IFE, > CSIPHY and JPEG, OPE et al. > > And BTW if/when we get to the camss-bus stage of things having > TITAN_TOP_GDSC listed in OPE and JPEG is a very small price to pay. > > We've already agreed to gradually transition the bindings instead of > go for a big-band integration several times. Just to clarify, from the jpeg driver perspective does not make any difference whether is child node of camss or other node. For sure it will be explained in binding documentation, but just in theory we may have jpeg encoder in future platforms without camera sub-system. So where jpeg is placed is strictly platform depended. Do you agree to have series just for the jpeg driver as it should be, the changes in camss and device tree changes for this platform will be not part of this patch series but send as additional one. Or you prefer everything to have in this patch series with different order of the changes. ~Gjorgji > > --- > bod >