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 CE59E23EAB0 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-bcb69c8b30bso159214966b.1 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=Nf0MAD3hhzBFcxaZr38gRK9VyTGdpalw3fNmXzQTw6ix2tuGW/0NRf6ZiueSqVz6bm X07iAO0947sEWR02HA9yBWgKH4fQPws+jTYv5EfAyFA8s1DPOaHWsgYTCC1Y2RBgEuVo HU//OSke9xuHcf02jxRBLzIzVl5Y5MUyrzzURBTgfaVRgdI4ezAMY3TJ+Ehel5UdFSP2 YydTjoKvzXDfEqh3QEjOqZj+d2w3R/UUJ2twd7uTN+GkbD4zlXE1Ver32J7ANzAFWo1S lSa21XPvx8qjJb4L1vGhzyefVZhp8/KnuKuUbq4waASYzYXfKdcMEljVSv0jAVIlHKgQ KczA== X-Forwarded-Encrypted: i=1; AFNElJ/7vVDaOytiby4Z0qjuEyMjQDOQ2XFTwOBEbFXDvkACzXvxCI88m2iMBVLyYo/XDLd1sJwymI9JazuW@vger.kernel.org X-Gm-Message-State: AOJu0YzYye+4pZogcFg4oZ3WarUm19FMdEG+RC7gWP9QR4DiociUJLtO /0CBfMYU0tF6s+ZkdtjgwWK79/GodvA9fokQdoBS1DVZQ1vv912R+lEo X-Gm-Gg: Acq92OEC+4ZU93hLdtufYuD6Dv33sGCGw5OoyPuBJgWX8ge1J4eszJEd829zJZwkgWp 437SSKaT0x2RpxX9CuiHahVw5LQWM6STRel2+BuwGrUUuj7lHBelBtxpIP2sDEKEB2HEfeMeuHH KH3F7NB00Jwv54IJ/ik537u61Anoa9XH+zp9HVR8aP6pg7xkJGWyja0CjSs8Qo6xykmGym51hQw ojcYLoFVxQYFKM6Us0amH+y3aDy5t9Ode5yEk9D5oTi0aGVW1jDQuCH+FrUGZf6TK3Qi0VWkqOk b45bBkUl4kTiCbZKD6RTc/fLNuYVI2Qw/xi/YImfeP7LNzZ+2XqCWZpUw4A/UvrFmmqa4dYg9Ah K4nuqxrr1zKYHX7ysmGMBlJXvZ8qqQwaaJdsNgbONUU9EvHPzdDHZAbyXoZicP0RaMwlGiNXG/z jlO2Li4r3GsnwJ1pdmYxHQDAmKsTtIZOWOw2OCCqitHje2LZDf02ByNnZ3zw1oISVwC0MT 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: devicetree@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.