[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <DC0GRKB9V014.1J7A2PO1B7U8H@cknow.org>
Date: Tue, 12 Aug 2025 14:55:49 +0200
From: "Diederik de Haas" <didi.debian@...ow.org>
To: "Diederik de Haas" <didi.debian@...ow.org>, "Jonas Karlman"
<jonas@...boo.se>, "Ezequiel Garcia" <ezequiel@...guardiasur.com.ar>,
"Detlev Casanova" <detlev.casanova@...labora.com>, "Mauro Carvalho Chehab"
<mchehab@...nel.org>
Cc: "Alex Bee" <knaerzche@...il.com>, "Nicolas Dufresne"
<nicolas.dufresne@...labora.com>, <linux-media@...r.kernel.org>,
<linux-rockchip@...ts.infradead.org>, <devicetree@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/7] media: rkvdec: Add HEVC backend
Hi again,
On Tue Aug 12, 2025 at 2:11 PM CEST, Diederik de Haas wrote:
> On Sun Aug 10, 2025 at 11:24 PM CEST, Jonas Karlman wrote:
>> This series add a HEVC backend to the Rockchip Video Decoder driver.
>>
>> Patch 1 add the new HEVC backend.
>> Patch 2-3 add variants support to the driver.
>> Patch 4 add support for a rk3288 variant.
>> Patch 5 add a rk3328 variant to work around hw quirks.
>> Patch 6-7 add device tree node for rk3288.
>
> It looks like I had a previous version of linuxtv-rkvdec-hevc-v2 branch
> locally and that also had this commit:
> - media: rkvdec: Keep decoder clocks gated
>
> Is that one no longer needed/useful/etc ?
>
> And 'chewitt' also had a commit to fix 8/10-bit selection:
> https://github.com/chewitt/linux/commit/4b93b05d2ca608bc23f1d52bcc32df926d435c7c
> "WIP: media: rkvdec: fix 8-bit/10-bit format selection"
>
> I haven't tried that one (yet), but did try an other variant with
> changing the ordering in rkvdec_hevc_decoded_fmts but that didn't work
> in my tests. (Can ofc be PEBKAC)
>
> Would that be useful? I do/did have consistent problems with playing
> 10-bit encoded video files.
nvm about the 10-bit problem. It exists, but it's not restricted to HEVC
as it also exists with with H.264 files.
Cheers,
Diederik
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists