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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 73D23C433EF for ; Sun, 13 Feb 2022 11:35:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=A3b4Tbqjs1EHRa0BLXy81Wd+hTUCZ1ZEeRp1WJC7DdA=; b=t7WtFAxcJIUzJ8 11tgIS2VsztTeZctVfolUzn6ePxQ9S3AmbPc9wut8w+gAZGokVFVnlqInaowqa62Il2HkomMcmj3J ImYbJt7Qm2bxIP8LU1AHA9/iO68tS/utx+Z+KsH7xgOCLu9LADKRyIiqRlHqXh9wT8cIniQfez9Wc 73cf3PeHLDQi2nr4gQKcsyaWzM/ajpX2dkuRNL5SjuV7Hg+8NPW6niDR9rAEG0Vu6/+vufsx7umRL zh4nv37jGM7MEi6aNu/w8a7bgDZHtGM+RqJzNe/7e6gQilGUQv3nQRSHn+j/x4rj3PWaCDNrvnNsX 4Y6fRxAEQxJp/cS4DHEw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nJD8j-00BZZa-5J; Sun, 13 Feb 2022 11:34:01 +0000 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nJD8f-00BZYF-Ij for linux-arm-kernel@lists.infradead.org; Sun, 13 Feb 2022 11:33:59 +0000 Received: by mail-wr1-x436.google.com with SMTP id h6so22487961wrb.9 for ; Sun, 13 Feb 2022 03:33:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=02vp1JcshzGAmd266InAE4p0SMLcVM41XUbfgVtkk8w=; b=OK0IROEtmVHSd6bnjgTZAXeNZrVBxdV791WH8w6dkmD8N9Rc4BOr1zMpPhM0oAZXUu F5+8tdpFeHqnUYZviWCBWzlZ2ikr2llcEWe2hHPNQg6igNlyv09Bv68r6NlXQUNEZpz6 YcMc3pe37dtNU0ALFTxUqgVf86I1ffFURsf3MdcQndVytg90HmTZCJb756kytkkqTQtI dcPT6EL5NDuYJd8o9tSAWNY3Li7cEkY/rZW0c0BS0vTv8hfMvTEwzXxpaDF3dnt7JW3f UPvTSd+ff+mcbppFm0AnkM2BufIEg5BbEtHreOY8Qynr2HRYSjgO3THIXWe4xdJDVtF8 adUg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=02vp1JcshzGAmd266InAE4p0SMLcVM41XUbfgVtkk8w=; b=3YtKH2xX2qDSD5gPoiWtBllek2zeARFWruCwNK0v5zYIN/ug2GWt2M0EcKtDucHYwd k9FBBsXU8tFXY6TyuzIqNQvxQbk3h9LLzAkFA9D+urKHbVDl99NsvGUAmKC1mJ5U2uIS nQI6prJZfPHH7woz5Br/v/VdYIxxAso04j2ta8HLS/6mt3ZFVD2Fx/M9YtFtF9Q5ssvr 7C2YE/34uCVm3+1FyyY5kznfJo+tiuH//kTwoHCBRZyvEtfUIqa+VP7ZDgWF6eOG6+5G ntuQo79vn+wpnZ7LF6A9byM/NUHDBDtT575h8XXCrVU73R8wt8rg43PJxTPXRAyRkJU5 zoWg== X-Gm-Message-State: AOAM530w5SNMCDSGME1l/oXnAQrSdexIIdYgTU5nXVNYFzPB1Ggd3Hjw DzuwZtNlCISYge8fHvRkMFs= X-Google-Smtp-Source: ABdhPJyTwqnXzmlapAqyauG6oyW/56U04p6xjFiix7W8KKn7b5P3uTYi4MmaqWyzQZ3BYqmtZlTA1A== X-Received: by 2002:a5d:4fc7:: with SMTP id h7mr7611085wrw.673.1644752034944; Sun, 13 Feb 2022 03:33:54 -0800 (PST) Received: from kista.localnet (cpe-86-58-32-107.static.triera.net. [86.58.32.107]) by smtp.gmail.com with ESMTPSA id p27sm9091564wms.39.2022.02.13.03.33.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 13 Feb 2022 03:33:54 -0800 (PST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: mchehab@kernel.org, ezequiel@vanguardiasur.com.ar, p.zabel@pengutronix.de, gregkh@linuxfoundation.org, mripard@kernel.org, paul.kocialkowski@bootlin.com, wens@csie.org, hverkuil-cisco@xs4all.nl, jonas@kwiboo.se, nicolas@ndufresne.ca, Benjamin Gaignard Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, kernel@collabora.com, Benjamin Gaignard , Alex Bee , jc@kynesim.co.uk Subject: Re: [RFC] media: uapi: Move HEVC stateless controls out of staging Date: Sun, 13 Feb 2022 12:33:53 +0100 Message-ID: <8038233.T7Z3S40VBb@kista> In-Reply-To: <20220201123439.353854-1-benjamin.gaignard@collabora.com> References: <20220201123439.353854-1-benjamin.gaignard@collabora.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220213_033357_671336_78F069D4 X-CRM114-Status: GOOD ( 16.75 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Benjamin, CC: Alex, John Sorry for late response, but I've been very busy last week. First of all, thank you for doing this! It's about time that HEVC moves forward. Dne torek, 01. februar 2022 ob 13:34:39 CET je Benjamin Gaignard napisal(a): > The HEVC stateless 'uAPI' was staging and marked explicitly in the > V4L2 specification that it will change and is unstable. > > Note that these control IDs were never exported as a public API, > they were only defined in kernel-local headers (hevc-ctrls.h). > > While moving the controls out of staging they are renamed and > control IDs get new numbers. > Drivers (Hantro, Cedrus) and Documentation are updated accordaly. accordaly -> accordingly > > Additional structures fields has been added for RKVDEC driver usage. You should do separate patch for that, preceding this one. One patch should only do one thing. I also suggest that you add additional patch for removing bit_size field in struct v4l2_ctrl_hevc_slice_params. Similar fields were already removed from MPEG2 and H264 structures. Bit size can be deduced from output buffer size and it doesn't hurt if bit size in Cedrus is set to bigger value than actual slice bit size. > Hantro dedicated control is moving to hantro-media.h > Since hevc-ctrls.h content has been dispatched in others file, remove it. > > fluster tests results on IMX8MQ is 77/147 for HEVC codec. > > Signed-off-by: Benjamin Gaignard Note that Cedrus still needs additional information in order to decode some HEVC videos. Missing info is num_entry_point_offsets and list of all entry_point_offset_minus1 (obviously, num_entry_point_offsets in size). I suggest that this is represented in a new control, which would use dynamic array feature, written by Hans. While Cedrus supports max. 256 entries, it can be much bigger in theory, but in reality, it's much smaller (like 4-8 entries). Last but not least, data_bit_offset should be better defined. Currently it points right after last header bit, just like Cedrus needs it. However, there is padding after that, at least 1 bit and 8 bits at most, so slice data always starts from byte aligned address. It probably make sense to rework that field to be byte offset, not bit, just like in VA-API. Note that RPi HEVC driver also uses byte aligned address directly. Cedrus would need some kind of workaround and only one that works is this one: https://github.com/bootlin/libva-v4l2-request/blob/master/src/h265.c#L191-L209 Best regards, Jernej _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel