From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.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 BA933282F2E for ; Fri, 19 Jun 2026 14:38:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781879929; cv=none; b=nsj4qt4TxiOY6Sj3bOGvxIqJGq/Zi766Gr3gMSTD7UZK4S/2X3K+4JBzycTEkSzUv7w73ns9qAipLCF2j6EXgHWsbyERAfsg87Llor4DT2FpRSn0NVctmkYW+MV38Jva2s5VDhPXDQmpavGv0adINoafo2gNV3NmzEwohk+7G1I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781879929; c=relaxed/simple; bh=7WNkuLU2jAqmKjI9Dp9aEobZPigX424u//KaDG2RG0s=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=AsZ9i44BfO7MEOLCF5DjxZmKJE3jooyXhLOXzpLPdZYn7lA/TMzf3gbWdQHbnbXnSqvBYab5yr8h9uH+AD1sHYMqQ8WO58vkujMLgFiBnHZzLrmjuzMgzPtz/YAVyhDXiXT2KwlEAwBAf4DuYZE2qV7IPTrHDlD0303V3H65Y9k= 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=V9hjm0+N; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=AeUiTn4i; arc=none smtp.client-ip=205.220.180.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="V9hjm0+N"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="AeUiTn4i" Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 65JDC7gq833889 for ; Fri, 19 Jun 2026 14:38:46 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= FE0PbmaaIwD3McTQ/ANcym99Ski3EPUCXszv8MIpDmQ=; b=V9hjm0+N784VlmU2 Bzu28+2dDrzslJsaIB2tgkDZcUVhAcOdegsJuyir1JI+9oyAvD1fk3DA5eOC05Kf 9XjKiDS4MCICv967gBJ58jt8OFxA05wVXAsQNymjs5RCkt6pKTgz4EWwjJfZcUR/ XVBJdGHDvsVUY6wSipqBEFJjTKA/rjKh0HKVMPnb5sN+DQU7UlLcldxc6mx5OIYm uvA89xYT2P40JbKaxvbKw3Av8CR6FVXSPzRITwKNaWb0ZKL6RZ0jSMw+UcKCA3tH 3hO3+dlhpW0jP//ZfTxPDadsG0g8AdAo0aDRVHFdyBpW0/lNZxmCQ2O3YZMeRUMW yAkXfQ== Received: from mail-qt1-f200.google.com (mail-qt1-f200.google.com [209.85.160.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4evmtjby59-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 19 Jun 2026 14:38:46 +0000 (GMT) Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-517647fbff1so5630911cf.2 for ; Fri, 19 Jun 2026 07:38:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1781879926; x=1782484726; darn=vger.kernel.org; h=content-transfer-encoding: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; bh=FE0PbmaaIwD3McTQ/ANcym99Ski3EPUCXszv8MIpDmQ=; b=AeUiTn4ilKSveHOfy3rIX5w4pUKDWyOC6FudChxFMmsSRYaO6sO185dcUJWj8yegef UmLhSx7UfaSmG6NV0ZA2POVbGx8E0YahH1s2PSXlT++7bN5D2CJOdig6iaQP+DioWApA NXtcWYcGlZRwZuMkc9sgSDbm3hD5SjWlj/rPYvBn8IRU3bmUBvxDIuGLqRb0ZWOW+TBh UyrZs46GenuOwa53Sb0cBcg249ZTgnX2Yk8R3SA7HZW2vrKX6z1RSlLI2PjSqGnV6DIB +IV8f87eK+zy5nE+ZI1c2/F29mVmotV/Rg/4u6XfVztPUr2PE8KQ4rlsp3iMiFYbG+p5 tEeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781879926; x=1782484726; h=content-transfer-encoding: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; bh=FE0PbmaaIwD3McTQ/ANcym99Ski3EPUCXszv8MIpDmQ=; b=aMYtT+lTkPqPL74/PMY6hLqeE+gnuGJ+bDM+FouJUkIFzXSyPf1pMTdbvkGHeyagGU pc8rCxNqc48SZimVnMHrypADY3OsbszlFOrVyMYTpl+w0XabvaK3WSef7e8aBkl2EyBO 9N8Rav8YXVNe5b4QAOZWGOv2+A4uUoOxWWTzb3KFnPxzbV0J8ItScTg+qufi6cUSRsA7 BE6q2mnQFm4PxYaJ1JZX7BSAA/CAmU4atY/NrTerucz8ZNHUBbuRd/lkq34pJahVemTN uujGRmNMr5nzFUTlclbeU0uupRQYH4DP+NLVN7jmp6SI4br6AAF6haXin88K2QgMKbHK RIhg== X-Forwarded-Encrypted: i=1; AFNElJ+oPevhh9xjiJgzdlJljPTzOFj6D6abvG5IuZt5eNGBsDkxUwM/WkF+2vKGBVibeJo1jd9w8fbio4H9@vger.kernel.org X-Gm-Message-State: AOJu0Yw67VChlXaAJlVOI2NmdMAqQZprDMrjCthgrREJw9hdtDtibJyG 8IzciB2ENmsyZetrLlGf8ZwrnKpCMqH1PL7HJUq6I1oQbFhVNVCZfocKiWU+ywzLtASkPjLHzLv /35TEqtqN2n+4pURp3rQ5aV5vMxmMO7deZPnyPv8LAU2dLWihhKBm8qhbWXbqBpm+ X-Gm-Gg: AfdE7cmFTj2BTtq9VlN8K1MY9JNfbS50FM7tVa3HaAdIMsWEo93XmgvhhkzzZokPB2+ ZNfKdOeXBLluMRJaTLbXJpaeo235mddd4xctmGapfWVN0T19Bfi37hAqwEEigM461weox9eYVDS 74vxiH+qZAiXQHhnq6Jxm+q45HbPoXpEA8o8VeJ8ASNOyUOn5p/2Vctb9YTlNKSU2Id5IOX3yZ4 KiQVPVFvMGhkgA+tZR9z07p3D70O4PB0sKaikqemi5VwpmN4sTuKi3Vebwqb4R8/VgPUjGW3Ng3 f6Dov4K4ecr/epQymnUcEOCWdRkH4sM53mMun60e1NBsFcQA8vJusRcf2YMz7CUkzUqr+cgHdKo LurqouJOMLyG6Po2rfAac2rtWZS1ad6o3QxE= X-Received: by 2002:ac8:5f11:0:b0:517:6d82:9d7b with SMTP id d75a77b69052e-519e4c06f53mr34688691cf.4.1781879925868; Fri, 19 Jun 2026 07:38:45 -0700 (PDT) X-Received: by 2002:ac8:5f11:0:b0:517:6d82:9d7b with SMTP id d75a77b69052e-519e4c06f53mr34688221cf.4.1781879925265; Fri, 19 Jun 2026 07:38:45 -0700 (PDT) Received: from [192.168.120.170] ([178.235.128.140]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c0a9aa27319sm89814766b.59.2026.06.19.07.38.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 19 Jun 2026 07:38:44 -0700 (PDT) Message-ID: Date: Fri, 19 Jun 2026 16:38:41 +0200 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 v1 2/4] arm64: dts: qcom: sm8550: Add JPEG encoder node To: Bryan O'Donoghue , Atanas Filipov , linux-media@vger.kernel.org Cc: 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, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260612194417.1737009-1-atanas.filipov@oss.qualcomm.com> <20260612194417.1737009-3-atanas.filipov@oss.qualcomm.com> <8d230cca-2023-4a13-876f-d5db8eb200a1@kernel.org> <3d4e0147-8e62-4872-b881-1452f5e09e85@oss.qualcomm.com> <9fab1877-976b-4495-86de-a8c853b9ba24@oss.qualcomm.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-Spam-Info: AW1haW4tMjYwNjE5MDEzOSBTYWx0ZWRfX4yfI2Ov+xRbB S1VB7RP4trxrF684550jugI3IJwvWZWoWKulaw02OIlqW+POijHqkEZrf3YJL4JdlG1/qd16j2e giFJf4kgl43XBmDRQe6bg/kx7k/rIJ0= X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNjE5MDEzOSBTYWx0ZWRfXwDmHNVOLvQJy nkv5zAxtOxwK35gAV3Y8itt5XuC2mJA+dCywWNZYgeuyOldhTOOVoH5Jz7tzjdBkoJwtdOQEm/i OSERmtudpvvDXKHcKFn9XIZ6pBDRb16AY4gW94cxqggvgbvMt0EIwnyzni/7yUuXTn+DBHrUR6y cvStJWLQT2vs7c7au11BuSvO/PeF3+t03ax2l5Ao0R7AzeZ85pOTqzx31/cnK5z1q6ah3u2yU9Z PoYTsDgrOD7UDrr1XzOwE41zUQhYZr/E6Vpp/e1aiQnLd4i7crzB9CQV1IkHhYM8nRBf5nuuHxY JKcpJneciYtKI868CYdOVwc1uGTAhvjVADU0BZpKQ4/tYyT3JJK9tbcUioPU99adbzcB08Mmp/2 KKUnEkprr/JWA4S4lzHVjPwYVJjQpmayQC+nEDImRVbalqtTy9vNrKCQHhs/LETwVFY02tq4Z/t XIgPLqKLtoOBC2SsFtg== X-Proofpoint-GUID: KumzVNmwaQRFV4woogC7yBqzGc0mec9n X-Proofpoint-ORIG-GUID: KumzVNmwaQRFV4woogC7yBqzGc0mec9n X-Authority-Analysis: v=2.4 cv=OM8XGyaB c=1 sm=1 tr=0 ts=6a355476 cx=c_pps a=JbAStetqSzwMeJznSMzCyw==:117 a=PRfkaYvzSr8QmIIGAkY2Sg==:17 a=IkcTkHD0fZMA:10 a=FelO9ux0wxsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=gowsoOTTUOVcmtlkKump:22 a=5YxtafxWi65fj7d2G88A:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=uxP6HrT_eTzRwkO_Te1X:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.125,FMLib:17.12.100.49 definitions=2026-06-19_03,2026-06-18_03,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 malwarescore=0 adultscore=0 clxscore=1015 priorityscore=1501 lowpriorityscore=0 impostorscore=0 bulkscore=0 suspectscore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2606190139 On 6/14/26 3:13 AM, Bryan O'Donoghue wrote: > On 13/06/2026 12:16, Atanas Filipov wrote: >> Thank you for the detailed explanation. Let me share my understanding of >> the shared upper-level blocks. They are exactly the reason we have >> frameworks like ICC with aggregate bandwidth voting, reference counting >> in the clock framework, and so on — the same applies to power domains. I >> do not think using shared resources is a problem when the drivers are >> correctly designed. >> >> We have actually validated this: we got CAMSS working alongside the >> Qualcomm downstream camera stack after fixing the shared resource >> management — something everyone considered nearly impossible at the time. >> >> On the CAMNOC and CPAS concern: if that coordination becomes necessary, >> the right fix is to address the resource management in both drivers >> independently, using the aggregate capabilities of the existing >> frameworks — not to introduce a >> hierarchical dependency between them. Moving JPEG under CAMSS does not >> solve the CAMNOC, clock and power domain coordination problems, it just >> papers over them. >> >> IMO the problem you are pointing at is more general than just CAMNOC — I >> would add priorities, QoS and other shared resources to the list as >> well. The answer to all of them is the same: correct use of the existing >> frameworks, not driver >> merging. >> >> On the idea of putting JPEG inside CAMSS with an external API: > > I haven't remotely suggested that. > >> no engine or pipeline that produces YUV output, which is what the JPEG >> encoder needs as input. If JPEG moves into CAMSS without an external >> API, it becomes >> inaccessible to userspace. If it does expose one, we end up with a >> standalone interface anyway, just with an extra layer of indirection on top. > > This is a very long winded way of saying no without acknowledging the core point that the DT should scribe the hardware the way it really is, as opposed to following software architecture preference. > > It is the case JPEG lives inside of CAMSS. This is a fact of the hardware, the DT should express those facts not software preferences. That's also precisely what the "Tree" part is about - CAMSS is essentially a bus (as evidenced by the existence of a set of resources, like the AHB/CPAS clocks, the TITAN_TOP GDSC and the interconnect paths that gate access to everything on it), just like MDSS essentially is a bus. The JPEG encoder, just like all the other blocks are then devices on that bus, logically belonging to the CAMSS node Konrad