From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5F81EC433FE for ; Tue, 12 Oct 2021 15:57:58 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 433BC610C7 for ; Tue, 12 Oct 2021 15:57:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237538AbhJLP76 (ORCPT ); Tue, 12 Oct 2021 11:59:58 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:54990 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234892AbhJLP75 (ORCPT ); Tue, 12 Oct 2021 11:59:57 -0400 Received: from [IPv6:2a01:e0a:4cb:a870:dcd8:9f87:c3be:dc06] (unknown [IPv6:2a01:e0a:4cb:a870:dcd8:9f87:c3be:dc06]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: benjamin.gaignard) by bhuna.collabora.co.uk (Postfix) with ESMTPSA id AF3521F43876; Tue, 12 Oct 2021 16:57:53 +0100 (BST) Subject: Re: [PATCH v2 0/4] media: HEVC: RPS clean up To: =?UTF-8?Q?Jernej_=c5=a0krabec?= , mchehab@kernel.org, p.zabel@pengutronix.de, gregkh@linuxfoundation.org, mripard@kernel.org, paul.kocialkowski@bootlin.com, wens@csie.org, hverkuil-cisco@xs4all.nl, jc@kynesim.co.uk, ezequiel@vanguardiasur.com.ar Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev References: <20211012143552.661751-1-benjamin.gaignard@collabora.com> <21222555.EfDdHjke4D@kista> From: Benjamin Gaignard Message-ID: <3c5851ac-3b8a-decc-93c1-01a65b1f8611@collabora.com> Date: Tue, 12 Oct 2021 17:57:50 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <21222555.EfDdHjke4D@kista> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 12/10/2021 à 17:34, Jernej Škrabec a écrit : > Hi Benjamin! > > Dne torek, 12. oktober 2021 ob 16:35:48 CEST je Benjamin Gaignard napisal(a): >> This series aims to clean up Reference Picture Set usage and flags. >> >> Long term flag was named with RPS prefix while it is not used for RPS >> but for mark long term references in DBP. Remane it and remove the two >> other useless RPS flags. >> >> Clarify documentation about RPS lists content and make sure that Hantro >> driver use them correctly (i.e without look up in DBP). >> >> These patches are the last in my backlog impacting HEVC uAPI. >> From my point of view, once they get merged, you could start talking >> about how move HEVC uAPI to stable. > With your changes, HEVC uAPI controls still won't be complete. Cedrus needs > entry point control, which in turn needs dynamic array support. I'm a bit lazy > implementing that control, but I guess I can take a look in a month or so. > rkvdec also needs more fields for HEVC. With patches collected here: > https://github.com/LibreELEC/LibreELEC.tv/blob/master/projects/Rockchip/ > patches/linux/default/linux-2001-v4l-wip-rkvdec-hevc.patch > fluster HEVC test score is reportedly 121/135 (8-bit tests only). Hi Jernej, Thanks for your feedback, getting a list of missing items in HEVC uAPI will definitively help to fill the hope. The patch you mention for rkvdec are already merged in mainline kernel (at least for uAPI part). Cedrus needs are about num_entry_point_offsets, offset_len_minus1 and entry_point_offset_minus1[ i ] in HEVC specifications ? Regards, Benjamin > > I would certainly wait with moving HEVC uAPI to stable. > > Best regards, > Jernej > >> version 2: >> - change DPB field name from rps to flags >> >> Please note that the only purpose of commits 3 and 4 is to allow to test >> G2 hardware block for IMX8MQ until a proper solution isuing power domain >> can be found. Do not merge them. >> >> GStreamer HEVC plugin merge request can be found here: >> https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1079 >> >> With those piece of code fluster score is 77/147. >> >> Benjamin >> >> Benjamin Gaignard (4): >> media: hevc: Remove RPS named flags >> media: hevc: Embedded indexes in RPS >> media: hantro: Use syscon instead of 'ctrl' register >> arm64: dts: imx8mq: Add node to G2 hardware >> >> .../media/v4l/ext-ctrls-codec.rst | 14 +++--- >> arch/arm64/boot/dts/freescale/imx8mq.dtsi | 43 +++++++++++++---- >> drivers/staging/media/hantro/hantro.h | 5 +- >> .../staging/media/hantro/hantro_g2_hevc_dec.c | 27 +++-------- >> drivers/staging/media/hantro/imx8m_vpu_hw.c | 48 ++++++++++++------- >> .../staging/media/sunxi/cedrus/cedrus_h265.c | 2 +- >> include/media/hevc-ctrls.h | 6 +-- >> 7 files changed, 84 insertions(+), 61 deletions(-) >> >> -- >> 2.30.2 >> >> >