From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f48.google.com (mail-ej1-f48.google.com [209.85.218.48]) (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 CE4BC1A3160 for ; Sat, 9 May 2026 12:00:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778328041; cv=none; b=Uv6XmE0yZMGBc9mbbgsBuoCYEKEikyfto1EIjVtjHJaQPqJhMau5NdSQUsqW+aXKQXAqQEdiikmXDdEBAQhZiyv8vVl0TIBXXZn7K2VV4K4DrjUxnknqnr1cRgq3AzdjGcIf7hL1nVfEg06ggCPifQQjrXoyUhj+oCOjwhPRm0c= 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.48 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-f48.google.com with SMTP id a640c23a62f3a-bcc1c602a87so54346966b.0 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=MjN/r3nuy9o6pwfTKEhebPtEnj/7TJzshv9GtIur/KvO3fLstAAqZBU04vCDY+bhoU kq+n91cb8e9GhCfonoVsHTdWnSfR1LslvG+BGMjbMCJQBUfevHXqleKKVdnQ6DNZoet4 938Row7aGJwKxM/55IqyDBDXUBADdA8ihMtier48zhDbg/bb+bO6V+LGMy6oFBCvufmr YDGsp9pfLLMypFlAU+vitq3tMx/oYKvD6siPDUWjuEF4CaEo7AJloCSYT4V819bpByec ax7fPGqWtSYf+W4HCpfXir4gnPYcAEqCb5AS0bZTpTnA+sSX2dI2w/5Zo67I+xSEj9fG Crgg== X-Forwarded-Encrypted: i=1; AFNElJ+nQXUz9sSr8fdFl0KNSC0Ys8arox8fF5ffKxJTVjc3BTjSSuF17S1OEpgpvZrlY0xB7QW9q4s6w0L4IFU=@vger.kernel.org X-Gm-Message-State: AOJu0YwbcMte04DD1pm7rzptESzGEqcl6lfoavHLa3S87LKDcbMPAzPv w0YBbZJJNevgyCKMy3C/NYRF+gZMID5nnm6KWx8H5txycAsvJYC0FrXj X-Gm-Gg: Acq92OGMzYVS4B95KOTTBshRkO0pvPRCBd0TX8oCLjUzEDUS+vp2rkHMdzkGbibbZei jYgk3R7DJwbcyix6B0jDll9U2Lgg0UfhLdxmEXe7VElNClidsmHONmWyuupQn8+G4NsnMWyUWZH vAKrDAFDk6PgLs3BzCOjlhe7V5rct5Us06d+bWj2vcHwcCRFlsV//9LQEw4Jz02cLwfJTs0PxlP lH5614I1DoWPHXwZFHV58W/CUr8TwipXwqQfLvIyUNwZbBmc7ai4Pj+KZLnxoRmu0RjaS0MiU0U Cna3CjIz1cD8Te84HiCfkQkc0NymUlsyX0l9VJlnDgghEIWx0JC65+7qYPzCxBZZrZEtqJlBYxq qVf+lAJLdmPlwEfKuixF0Ct3NSKja5OPQMr6L/FRYtgm+8rPS1ePaYTERwXSw1LRcRRISct/HEQ TnPhY4tGNiVbImq0Rz5llIkyedz2EiPymHJeVET6vc/SvTARNhXIWyNtp1+KgZH9YNGQh7 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-kernel@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.