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 991B43BC668 for ; Fri, 24 Apr 2026 11:36:29 +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=1777030591; cv=none; b=cdECfGWu7KX9x9EfErnGIcQyKHI7Spd3aFhrptmr22zAFtGJr+6F05K/Lhv0aV34HI4BIdjTJj8VH9evO29tP6CrDWxIxSMptu+GijCNeeV92wzKp3rvicX2+A80Wq4x5bNWjhN3m2GUt6m1Zlc7fJiqoM/dKjkScE15Bk9ATmE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777030591; c=relaxed/simple; bh=kPMGUNNSmroYSsiqLoYTt4gz7zVsODQoZrsEYXDMkX0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=mAAAYkXsMpnQMSKPNhdR1LIRkKwTkJyJhp7QuvpNhqaDUR9JNRB4W+i2NBMmxN84GvEBg5rY0pK3rmsR+VlVdPfbPs8SQHntlG+eWiDXFcBJ5MS+kxBbS/ILUPtGL2B31Ya73tmZ4DT6AmUVOCvOPtKq+wYtAm5Su26vL2Dwp2I= 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=OLx4MBQV; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=T/Ngsdjp; 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="OLx4MBQV"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="T/Ngsdjp" Received: from pps.filterd (m0279866.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 63O9XP0q518502 for ; Fri, 24 Apr 2026 11:36:29 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= szb/C/NuDvGqRRu1u78P1sc9wQPhVPl5cH0BZflLHds=; b=OLx4MBQVZcIfvtqp EyXnMunn34PmO6FdZqwrOQ5eJ7VdohfMBqrt1DIAgMr2uE8R3t9hM7eaLq2z72eo YipG+oORIKgtnonaVXYLPwHnQX3/gaAImzkd8WBfO4S9Oq0M1/zeMi6pBgzWRqvY wj0oPjcg3Xle7KGsaJt+w+smHfOrNCWSKTMu1i4CA51wtbDO6x7DWl9jBeW3+hF8 mcrSBlcOojvz1YkxZZWCh6e4sMZxg6xXe6z3XnF6M7i0Il2qdAw9u2SOT2dt6KRT hIEgI4TMaxO5QOruwTXU4tLt/KEddekZOcaugBsK/PDF2nTLE7mkOXePdNEpfPvW lV83gA== 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 4dr0fnhv3g-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 24 Apr 2026 11:36:28 +0000 (GMT) Received: by mail-qt1-f197.google.com with SMTP id d75a77b69052e-5073ed1ec6fso22463171cf.1 for ; Fri, 24 Apr 2026 04:36:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1777030588; x=1777635388; 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=szb/C/NuDvGqRRu1u78P1sc9wQPhVPl5cH0BZflLHds=; b=T/NgsdjppkTMSp11a9U6JVNm4ynwClbtt+qmBTanaFnXqQizUrI7EaBBfkCZWhM2yC 1rRH5fqrDEKMZ8qp77aCRIGt2HJ+BeY2I8flCVgAJPgObjsW6TymW+Qdsx/NPJo6Yde3 pC/ZeGg/pqA5c5yhF5vCjjeKIzq9PjlsIC////+hKtTHQBE6b6NKYX9PCJH4+TQoxEMd 9AmOctokE4pIF56on5KqBcWeOxOfJCiuB9TIAvr+2cHcBaM9SPtuRdxQW5PyNJtqG90N QxQWTNSI3kWaSsh3wgGSHsIqscdTFLViQYBnJ+wMCMiPNBzxT2OfciWMUQuuwa8ZNM0w /J9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777030588; x=1777635388; 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=szb/C/NuDvGqRRu1u78P1sc9wQPhVPl5cH0BZflLHds=; b=MUBZZ/qqn30d8wEpIytSdhSmzYELXwIK6wxxwjUmcbBucg6peOhKvhzT0q6WDkFe1D GGqBVJ/B5f6gl1Nq6RsmTmDPbCC5cN/VCvPDBLCdYibAMkN+WccAunoByW+0DfBwvA0v +xJsDTqWAdE3zBvcRWlqcdR22lb5c/EbNCgrcg2oUcBig7cvkZZvWjpQomo5A4aIjcwl pvje2fGf/eISpVueTf/AyBT+B/Ix+hk3Dq9tQHizK+mODdzbq9VO0TmvSfv/iPfxlEXv 7fWnQk+CADC6wSSqdEuCiEHuuvkWVbrR6KNgvbNmsHEsRD6vxlbJR/u6ZY8BRQZmIehY 245A== X-Forwarded-Encrypted: i=1; AFNElJ/35tA60VTvdyA9RplLlK0Mw9Wkm3w/u7jDFDvUduyIA0f+FCLXTlv7frUbYMTyad7ex+oJq4WMFAGs@vger.kernel.org X-Gm-Message-State: AOJu0YxY1HnudyiyU9BfSv9ZCEqxMWlhMj1bp3q2yEUtfxE64W7nX31M t7Qm8bekUZhY4LWblhtH6UE/jrWE/3+BK+kkfrgypti4qoKRtZTYsshRrQ4kax/n5YgJLnDKGBZ 8AjpS1UvNyA6De2CpKawAREeQxAm61UB0aVAkh38D3MhvO2dIBwwbatfJ1EWk+zmz X-Gm-Gg: AeBDiesMX75Bm8asRrTu763QwcHXsrjQ1LogMvadot0jkE3qfrlagVJi+nOwj9hT4/t pICEdF+LpyZ+LYCDXZEfBHHgokAzjJJZ06JTVbDdWLO9WOQZfGliI6QDzSE/PCbwDQLelGwqegI rCM9LXQ7AEORTYm6wmj2EusaLHhmqIhlwQTrZGTaX/heHGFom5XguSsBa/1nHEyZBXnt5xc1yuX I3OpipD4lusIGme+Y3zTAPsg8X6ZmaiH0yrzGpqD8H6wbNLYKdl5bDySxznm/LkL0XBG50pKpFT wLvJdCuD+F4ijWi2GhNdXl5U6mPvxfNfvUkfOP3vQzTTzm1LSso8VQvIz7jRUwM91uRKIvW8H6J fNa7LArs/CEQ0UO3mCEzVw4e7YXVvUeGhtD/5q7sdLMCdQ7XBvV2R11X8vQIxNh//PncsJZhf6d TXMBodq7j3LtI02w== X-Received: by 2002:a05:622a:1f88:b0:50f:bfe3:a04e with SMTP id d75a77b69052e-50fbfe3a765mr119513761cf.7.1777030587702; Fri, 24 Apr 2026 04:36:27 -0700 (PDT) X-Received: by 2002:a05:622a:1f88:b0:50f:bfe3:a04e with SMTP id d75a77b69052e-50fbfe3a765mr119513231cf.7.1777030587197; Fri, 24 Apr 2026 04:36:27 -0700 (PDT) Received: from [192.168.119.254] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-672c47fe836sm4924815a12.3.2026.04.24.04.36.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 24 Apr 2026 04:36:25 -0700 (PDT) Message-ID: Date: Fri, 24 Apr 2026 13:36:20 +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 RFC 04/10] arm64: dts: qcom: msm8939: Add venus node To: Erikas Bitovtas , Bryan O'Donoghue , Vikash Garodia , Dikshita Agarwal , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , =?UTF-8?Q?Andr=C3=A9_Apitzsch?= , Bjorn Andersson , Konrad Dybcio , Michael Turquette , Stephen Boyd , Stephan Gerhold Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org References: <20260416-msm8939-venus-rfc-v1-0-a09fcf2c23df@gmail.com> <20260416-msm8939-venus-rfc-v1-4-a09fcf2c23df@gmail.com> <717459db-479a-4a68-84d1-a718c8ba5a54@gmail.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <717459db-479a-4a68-84d1-a718c8ba5a54@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Authority-Analysis: v=2.4 cv=LfYMLDfi c=1 sm=1 tr=0 ts=69eb55bc cx=c_pps a=EVbN6Ke/fEF3bsl7X48z0g==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=YMgV9FUhrdKAYTUUvYB2:22 a=bAtGagR5AAAA:20 a=KePjsaevwCBX4e1EH2wA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=a_PwQJl-kcHnX1M80qC6:22 a=bA3UWDv6hWIuX7UZL3qL:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDI0MDExMSBTYWx0ZWRfX8/yDS7OCHHe6 CzakECjQ/TmJg21qQM/gPHXxyeP0M7DZQSn+oHTkhwGEDQRVOZyWCI4J9McjEuo3aiPsIjWjDAU r/rbyXTWKj0EqhtNxRIMRxA0+EU2hfdnj5IKInN4VQw7+DLyocCBkLf7jAShSx/4Wr3HRORuN7Z Jgt3ixNtd/VzSSG4Y+84M/46rn3pXcx5q++mFyPRYMePbikIUQ6xtmGtVVY9H6BusazSjKB3fAF Bh1uGcZ5RrTldiZ6pv9DYktXIEjlJl6xroEg70Q10Hgzfj6WMHMGeRkUH8np0r4xw29j5WhPWVQ bvCHAt0yv9sDAXup7wjNE2+5mPj3e3V5elKO9T27iSo9MyAytqzITXsGUdyG2PyaIETc77bEv+o xYWazW1F3XJ5/ce2LYtDjz0zqcWFkzE84VUx92zYKI17cPsAKT/o92tAD0Pk1a82g4qNv1f2xzQ HZL4AQktRq/xrRxUxTQ== X-Proofpoint-ORIG-GUID: RbyOoudI0WLz3Apo7uD7ieOMH4j8MgIA X-Proofpoint-GUID: RbyOoudI0WLz3Apo7uD7ieOMH4j8MgIA X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-04-23_03,2026-04-21_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 lowpriorityscore=0 adultscore=0 priorityscore=1501 malwarescore=0 clxscore=1015 spamscore=0 impostorscore=0 bulkscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2604200000 definitions=main-2604240111 On 4/18/26 5:34 PM, Erikas Bitovtas wrote: > > > On 4/16/26 5:31 PM, Bryan O'Donoghue wrote: >> On 16/04/2026 14:43, Erikas Bitovtas wrote: >>> +            video-decoder { >>> +                compatible = "venus-decoder"; >>> +                clocks = <&gcc GCC_VENUS0_CORE0_VCODEC0_CLK>, >>> +                     <&gcc GCC_VENUS0_CORE1_VCODEC0_CLK>; >>> +                clock-names = "core0", "core1"; >>> +                power-domains = <&gcc VENUS_CORE0_GDSC>, >>> +                        <&gcc VENUS_CORE1_GDSC>; >> >> This doesn't make sense. >> >> You have two cores => assign one to encoder and the other to decoder. >> > > This way during decode only one of the cores gets powered up instead of > both, resulting in power collapse fails. > Core clocks and power domains can be moved into Venus node instead of > sub-nodes, like this: > venus: video-codec@1d00000 { > compatible = "qcom,msm8939-venus"; > reg = <0x01d00000 0xff000>; > interrupts = ; > clocks = <&gcc GCC_VENUS0_VCODEC0_CLK>, > <&gcc GCC_VENUS0_AHB_CLK>, > <&gcc GCC_VENUS0_AXI_CLK>, > <&gcc GCC_VENUS0_CORE0_VCODEC0_CLK>, > <&gcc GCC_VENUS0_CORE1_VCODEC0_CLK>; > clock-names = "core", > "iface", > "bus", > "core0", > "core1"; > power-domains = <&gcc VENUS_GDSC>, > <&gcc VENUS_C0RE0_GDSC>, > <&gcc VENUS_CORE1_GDSC>; > power-domain-names = "venus", "core0", "core1"; > }; > > And then they can be powered up regardless if the session is for > encoding or decoding. > My first question was actually about this - whether these cores should > be powered up only decoding or for encoding as well. Bus configs > downstream signify they are only for decoding: > https://github.com/msm8916-mainline/linux-downstream/blob/b20608408caff817ec874f325127b07609fbaeb8/arch/arm/boot/dts/qcom/msm8939-common.dtsi#L1589 > https://github.com/msm8916-mainline/linux-downstream/blob/b20608408caff817ec874f325127b07609fbaeb8/Documentation/devicetree/bindings/media/video/msm-vidc.txt#L35 > Unfortunately, I couldn't test encoding on my device. It appears to be > broken. Do we know whether it works on msm8916? (+Stephan) Konrad