From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f54.google.com (mail-ej1-f54.google.com [209.85.218.54]) (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 D670830E821 for ; Sat, 9 May 2026 12:00:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778328041; cv=none; b=Q3waSnui0CqOTGC0Mrqx1bG66v2FVqf+/0fC/S/di/lsEs2e29/RaD8H8uzUs+wg2y9SCCNAEux/4vYxh74d1dY3+VVgu6dFjgpG3Hn7SoGi9k4i6KU2pdrZM7VsIPlWIzwJ2FwLfLTLj2UJht2+UX8UTAJNBFri/mMsyBwZ/0A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778328041; c=relaxed/simple; bh=iemCui2l8qqvzux3lam3RV5hSucVjjAACXX7LzCZHcA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=K7lbEElEqM/2zOL7c+RjE5AEqxV0y5UBDdrkJCpSWU8Ged5L6FH9stnIlIZ5thBKfqAWp53dHNgoX1gXR+kSOs9bpqef055K5I0hzpZV+vA/gqRQ/o1lJ+xklfFvZtuTTPx5qUzSNIs+GwTlRpDW0ISG74uVRueEdm7dMfjUpFI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=A7TN+NgJ; arc=none smtp.client-ip=209.85.218.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="A7TN+NgJ" Received: by mail-ej1-f54.google.com with SMTP id a640c23a62f3a-b9d9971d059so433895166b.2 for ; Sat, 09 May 2026 05:00:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778328038; x=1778932838; 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=zmUf6SNpF4clO18ErlUoo6BNS+Vk6nKV8PcksNLHfkQ=; b=A7TN+NgJDOiROdSbAmP7uh/zrwBSV+nGEFjf6F2Gp9Wbx1FCzxQxNvTT9cfw7JLzmd gq+isEPxiAl7kKjsjKvZEYOxPBv/446cLEKUun52UKzI4zCbw/0tw8UmDSWANXcgP4y3 Cf8UcV3gdKmhKzUp9TTd4ZbngpmyZo8eDl7s3tQ+xGlbyf6qxMCBsSxkGH23W3oIHAcX nP/fcModN7+s0dVCP4b9YtD4khIqFG5uoZDQa9ofKOTpmFHNyJ/ijVodp1PeU9K+6y9e C4ZQECaBbPzW/XubM7Ijk4gG0JG5cfyg85ToHzxwqdi4i3UM01T4gaStKNbEzQYNc5Uw yGlA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778328038; x=1778932838; 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=zmUf6SNpF4clO18ErlUoo6BNS+Vk6nKV8PcksNLHfkQ=; b=SaSE0S8xfdz2uKtS7JkvSD7wbFkyrJ7PhV8/5gUiKWrPFXHI/MDb2M9m9k1Mi/jQKI 4RB06UVy9Hfa6f4Oi8kXYuIbf341AWUlIDZ4PRzo1y2yTIa2JCRxw36ZhKa9JY5/Auaj qPbwOeU6VJn8H2yb2jyPeiEuxBlmKaXjt6L+Vry6rP+YenjUc8wc5zWTcbYWVZPUm2Dh vJC2iyRLQ3B71s00dKGQCfIdXrClE3jrA2L9RLZ1tZM02FUX98NF4Y1Hrzthjfnp4fHm dSLUvZhxZHo4/cCUmViyd6DHJFVxicHYNa1TcpR6C8wG/qXsH2pfJXKyRT9AtvoDmBg+ suaA== X-Forwarded-Encrypted: i=1; AFNElJ80+fYWCshcnoM4NE9XLB9El81Y1GQRWMsJ7vljTfGF2w9a/n6JZapmk5pZpdqc/KvhDvnmOJbJUF1HzLTu@vger.kernel.org X-Gm-Message-State: AOJu0YwBm/KmtaOMUYmGm8W4Cjt7TqKOXzP2Mx/kMHKV2zb8E2tpPx/V giUJD6xUF1XuiR13EZvdUI0eBZ9Ub9wVqx2fOYxhy6d71MYy2qae+Pxs X-Gm-Gg: Acq92OGgX6plx9aP6wFQROdxlFFFo7s2/usU6ZEtNzXj/ahi2tGD7vgVyDtbUssGIoo PfScuCiJF4br0GQOm8f70vGieCKZGt8i/P5ffli9KOuq03CIgaeF4AObIUOIorAKuFxmBLOD4zq fwymh6GWQNWghYTIIPlTakOgK+PIRdr9dNa8nrqtOeUKqjr3X5QQgELwk7iLG4NuPUlkIawmoDc ZCSorx+VELh8GGcUuoduIVjTR6Rj2r1LoivgOyeo74w4a+FA+taSvgIv1pqw2BFLc05FeFrUAE9 0ZNQCea0pulMrnOfPXfDC7waci2xueWAWgNnjtkqU9aPLkcIoTc5GiYO4DhMceM/iTPGAt6Q2Qn KIDX78T3BdI5iUg6D2iLtkiaIqgGxZ6QRt3L8/A9ZQbeMb6mPXdaFHsqjBB827tPd88dNlcUbn+ LKoN53+HQ1HAOd+XoE8d4ixW8f0/AelDhJN6qYS6954zvoziJD4kYzIM7DtaycsGk9Ivku X-Received: by 2002:a17:907:c817:b0:bae:456f:fbb2 with SMTP id a640c23a62f3a-bc56cd3b1ccmr1054228966b.23.1778328037744; Sat, 09 May 2026 05:00:37 -0700 (PDT) Received: from ?IPV6:2a00:f502:160:4e2:a099:cad9:2ecd:93de? ([2a00:f502:160:4e2:a099:cad9:2ecd:93de]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-bcac4359ea4sm219872166b.50.2026.05.09.05.00.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 09 May 2026 05:00:37 -0700 (PDT) Message-ID: <69f9a6b7-7431-442a-82a2-6305635a508a@gmail.com> Date: Sat, 9 May 2026 15:00:34 +0300 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 5/8] clk: qcom: gcc-msm8939: mark Venus core GDSCs as hardware controlled To: 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 , Brian Masney 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: <20260507-msm8939-venus-rfc-v5-0-d7b5ea2ce591@gmail.com> <20260507-msm8939-venus-rfc-v5-5-d7b5ea2ce591@gmail.com> <17c4626e-8926-4cad-842d-a1b171d1e962@kernel.org> <4f1ab5b8-3bfe-43a6-8b91-317456188544@gmail.com> <7df3bff7-b872-4b20-8e7c-698157a41f33@kernel.org> <18998766-62fe-4a5e-95d8-1bfae712d8fb@gmail.com> <614a087d-d219-49cb-85ee-772044558649@kernel.org> <0cee27b6-f566-4958-8634-2c43e099281e@gmail.com> Content-Language: en-US From: Erikas Bitovtas In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 5/8/26 6:55 PM, Bryan O'Donoghue wrote: > On 07/05/2026 16:51, Erikas Bitovtas wrote: >> >> >> On 5/7/26 4:37 PM, Bryan O'Donoghue wrote: >>> v4l2-ctl --verbose --set-fmt-video- >>> out=width=1280,height=720,pixelformat=NV12 --set-selection-output >>> target=crop,top=0,left=0,width=1280,height=720 --set-fmt- >>> video=pixelformat=H264 --stream-mmap --stream-out-mmap --stream- >>> from=cyclists_1280x720_92frames.yuv --stream-to=/tmp/ >>> cyclists_1280x720_92frames.h264 -d /dev/video1 >> >> With the patch kept: >> v4l2-ctl --verbose >> --set-fmt-video-out=width=1280,height=720,pixelformat=NV12 >> --set-selection-output target=crop,top=0,left=0,width=1280,height=720 >> --set-fmt-video=pixelformat=H26 >> 4 --stream-mmap --stream-out-mmap >> --stream-from=cyclists_1280x720_92frames.yuv >> --stream-to=/tmp/cyclists_1280x720_92frames.h264 -d /dev/video1 >> VIDIOC_QUERYCAP: ok >> VIDIOC_G_FMT: ok >> VIDIOC_S_FMT: ok >> Format Video Capture Multiplanar: >>     Width/Height      : 1280/736 >>     Pixel Format      : 'H264' (H.264) >>     Field             : None >>     Number of planes  : 1 >>     Flags             : >>     Colorspace        : Default >>     Transfer Function : Default >>     YCbCr/HSV Encoding: Default >>     Quantization      : Default >>     Plane 0           : >>        Bytes per Line : 0 >>        Size Image     : 708608 >> VIDIOC_G_FMT: ok >> VIDIOC_S_FMT: ok >> Format Video Output Multiplanar: >>     Width/Height      : 1280/736 >>     Pixel Format      : 'NV12' (Y/UV 4:2:0) >>     Field             : None >>     Number of planes  : 1 >>     Flags             : >>     Colorspace        : Default >>     Transfer Function : Default >>     YCbCr/HSV Encoding: Default >>     Quantization      : Default >>     Plane 0           : >>        Bytes per Line : 1280 >>        Size Image     : 1425408 >> VIDIOC_G_SELECTION: ok >> VIDIOC_S_SELECTION: ok >>         VIDIOC_G_FMT returned 0 (No error information) >>         VIDIOC_G_FMT returned 0 (No error information) >>         VIDIOC_G_FMT returned 0 (No error information) >>         VIDIOC_REQBUFS returned -1 (Invalid argument) >> >> With the patch reverted: >> v4l2-ctl --verbose >> --set-fmt-video-out=width=1280,height=720,pixelformat=NV12 >> --set-selection-output target=crop,top=0,left=0,width=1280,height=720 >> --set-fmt-video=pixelformat=H26 >> 4 --stream-mmap --stream-out-mmap >> --stream-from=cyclists_1280x720_92frames.yuv >> --stream-to=/tmp/cyclists_1280x720_92frames.h264 -d /dev/video1 >> VIDIOC_QUERYCAP: ok >> VIDIOC_G_FMT: ok >> The pixelformat 'H264' is invalid >> VIDIOC_G_FMT: ok >> The pixelformat 'NV12' is invalid >> >> Decoding breaks if I revert the patch as well, it fails with a power >> collapse fail. > > If you declare both as decoders can you successfully decode @ video0 and > video1 ? > Apparently device nodes for encoder and decoder are not persistent across reboots, so when I reverted the patch, I was actually testing against decoder, not encoder. So, the result is the same when the patch is applied, when it is reverted, and when testing against v1 where the cores are enabled only for decoding.