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 58C9FC433F5 for ; Fri, 24 Sep 2021 16:25:47 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 2445060E76 for ; Fri, 24 Sep 2021 16:25:47 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 2445060E76 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=vanguardiasur.com.ar Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=EHS+uvAytvbBOdDte8GUYgua4/sojkpHZZvukgHrp9I=; b=L8JwRCst5dIlQg V0GyxTOh8PKp63q545VQnMnHqDEbBIPYzj9RTdTQ2Z4KgFY/0r4G74omSgkEjzBvMQWaEOaoNwDu8 QRZErDeEau21u2uE1eZBv3cQtq2NgXD3ca39HMF6+X2txFtEC6xh2/iUKom0lCd0XoYS1Jz62Yk9r CAqi9o0b2c//Ym6k3a5yVAsDQ+IRYmxZU5v2JLkqIhLtbCTck86aNyWp6BRVjkg8sQuq0RPRfKIzs XcXi0+f8nkxFczbGGJxt1Oud0DWE+2QTMAnIV+wKFoEcykwabftNMy/uLt8aK7k1Qsfci34zX5nkF 93Ctjt3eqNkbZ8YVkD2w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTnz6-00F38w-P1; Fri, 24 Sep 2021 16:23:36 +0000 Received: from mail-qk1-x72d.google.com ([2607:f8b0:4864:20::72d]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTnz2-00F380-KD for linux-arm-kernel@lists.infradead.org; Fri, 24 Sep 2021 16:23:34 +0000 Received: by mail-qk1-x72d.google.com with SMTP id c7so28874910qka.2 for ; Fri, 24 Sep 2021 09:23:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vanguardiasur-com-ar.20210112.gappssmtp.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=8RykFUS+zIQyPPrYlPKKvEG+geEoI9vSseHxlpcXrKQ=; b=h989I/HASgZ7AIF7P/YOMqX4kXGgqJNzuOzQ0waHkcENUVdEHqlLKXODhzloVPZyi9 4yi/EHzKbdhBlwbwiQMwvvfG/crbxB9VqCyfzmdA6LCF/qtofzrlvsiFneZTNytlGw64 28AXtt3FZGWF9nkU8gCoDB1+iSRlNh/u7sjLteAlpCYVw7YNrFv8DCW51PeI4HhbLxUy eazO1xqYZWg8/7Ya6YO3jiEpycDfVY/jd9VD+cvtqoPPqY7GsAA/+TCPmhc2CzQsdaPm 7Yz/lOrDVk1yqzdfNfqK74DbsztAbMasJW4726PtaVVC2Yay7X2CIS22QXGHfxCRcE0U AjYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=8RykFUS+zIQyPPrYlPKKvEG+geEoI9vSseHxlpcXrKQ=; b=Jr379A9XNEYCGzgHdVd/RwSQZHKG7yTK7sp3f3MPhA+Vev4/2mlfReaSwZ1pabI0yi uhcMQRjjSKIz0Tf5tIPrcfM/BDu1IQzUTaKhWzR8dPJPCHrua7gdgkgcETEnZMcJcQLl +zLp5z0jlKJgDtdyRpIlB3mhL1I7c1Aw+V1q8bNuPy01NeSHxkx5CwBgnC72abIN28gR 3Hpl//c5am/+iAKs9fpRx8erjpfC+Sxo/hCo/OQeD+1XZOaA6bzfzW/Fgf/UGcA3qC2W maWPfGeURD8ej1FJOWmi5ycFtxO9ktWQf0W4Ybv6gg1x7QOUD50YsYHWZqDck4YEcWFl qqKg== X-Gm-Message-State: AOAM533apmnIUB8J9UPgjz2M68392d8RXzqlN+FRrW6MVgpsg1n2a/Ce 6ofNqlEsuwi/Pq114nxFlPvA5A== X-Google-Smtp-Source: ABdhPJw1kkgVb6LnRxSEtDqPeAhQjar+Ov9byHFml/Xm3S236dE4glk6vx5O+A7pJWzwQivL/2QSSw== X-Received: by 2002:ae9:eb84:: with SMTP id b126mr11137072qkg.67.1632500611095; Fri, 24 Sep 2021 09:23:31 -0700 (PDT) Received: from fedora ([196.32.91.248]) by smtp.gmail.com with ESMTPSA id g13sm615144qkk.22.2021.09.24.09.23.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 24 Sep 2021 09:23:29 -0700 (PDT) Date: Fri, 24 Sep 2021 13:23:23 -0300 From: Ezequiel Garcia To: Andrzej Pietrasiewicz Cc: linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, Benjamin Gaignard , Boris Brezillon , Fabio Estevam , Greg Kroah-Hartman , Hans Verkuil , Heiko Stuebner , Jernej Skrabec , Mauro Carvalho Chehab , Nicolas Dufresne , NXP Linux Team , Pengutronix Kernel Team , Philipp Zabel , Sascha Hauer , Shawn Guo , kernel@collabora.com, Ezequiel Garcia , Adrian Ratiu , Daniel Almeida Subject: Re: [PATCH v5 05/10] media: uapi: Add VP9 stateless decoder controls Message-ID: References: <20210922101146.13762-1-andrzej.p@collabora.com> <20210922101146.13762-6-andrzej.p@collabora.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210922101146.13762-6-andrzej.p@collabora.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210924_092332_864243_A129BF1D X-CRM114-Status: GOOD ( 25.41 ) 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 Andrzej, On Wed, Sep 22, 2021 at 12:11:41PM +0200, Andrzej Pietrasiewicz wrote: > Add the VP9 stateless decoder controls plus the documentation that goes > with it. > > Signed-off-by: Boris Brezillon > Co-developed-by: Ezequiel Garcia > Signed-off-by: Ezequiel Garcia > Signed-off-by: Adrian Ratiu > Signed-off-by: Andrzej Pietrasiewicz > Co-developed-by: Daniel Almeida > Signed-off-by: Daniel Almeida > --- [snip] > + > +#define V4L2_VP9_FRAME_FLAG_KEY_FRAME 0x001 > +#define V4L2_VP9_FRAME_FLAG_SHOW_FRAME 0x002 > +#define V4L2_VP9_FRAME_FLAG_ERROR_RESILIENT 0x004 > +#define V4L2_VP9_FRAME_FLAG_INTRA_ONLY 0x008 > +#define V4L2_VP9_FRAME_FLAG_ALLOW_HIGH_PREC_MV 0x010 > +#define V4L2_VP9_FRAME_FLAG_REFRESH_FRAME_CTX 0x020 > +#define V4L2_VP9_FRAME_FLAG_PARALLEL_DEC_MODE 0x040 > +#define V4L2_VP9_FRAME_FLAG_X_SUBSAMPLING 0x080 > +#define V4L2_VP9_FRAME_FLAG_Y_SUBSAMPLING 0x100 > +#define V4L2_VP9_FRAME_FLAG_COLOR_RANGE_FULL_SWING 0x200 > + > +#define V4L2_VP9_SIGN_BIAS_LAST 0x1 > +#define V4L2_VP9_SIGN_BIAS_GOLDEN 0x2 > +#define V4L2_VP9_SIGN_BIAS_ALT 0x4 > + > +#define V4L2_VP9_RESET_FRAME_CTX_NONE 0 > +#define V4L2_VP9_RESET_FRAME_CTX_SPEC 1 > +#define V4L2_VP9_RESET_FRAME_CTX_ALL 2 > + > +#define V4L2_VP9_INTERP_FILTER_EIGHTTAP 0 > +#define V4L2_VP9_INTERP_FILTER_EIGHTTAP_SMOOTH 1 > +#define V4L2_VP9_INTERP_FILTER_EIGHTTAP_SHARP 2 > +#define V4L2_VP9_INTERP_FILTER_BILINEAR 3 > +#define V4L2_VP9_INTERP_FILTER_SWITCHABLE 4 > + > +#define V4L2_VP9_REFERENCE_MODE_SINGLE_REFERENCE 0 > +#define V4L2_VP9_REFERENCE_MODE_COMPOUND_REFERENCE 1 > +#define V4L2_VP9_REFERENCE_MODE_SELECT 2 > + > +#define V4L2_VP9_PROFILE_MAX 3 > + > +#define V4L2_CID_STATELESS_VP9_FRAME (V4L2_CID_CODEC_STATELESS_BASE + 300) > +/** > + * struct v4l2_ctrl_vp9_frame - VP9 frame decoding control > + * > + * @lf: loop filter parameters. See &v4l2_vp9_loop_filter for more details Seems these documentation is missing an ending period for many fields. > + * @quant: quantization parameters. See &v4l2_vp9_quantization for more details > + * @seg: segmentation parameters. See &v4l2_vp9_segmentation for more details > + * @flags: combination of V4L2_VP9_FRAME_FLAG_* flags > + * @compressed_header_size: compressed header size in bytes > + * @uncompressed_header_size: uncompressed header size in bytes > + * @frame_width_minus_1: add 1 to it and you'll get the frame width expressed in pixels > + * @frame_height_minus_1: add 1 to it and you'll get the frame height expressed in pixels > + * @render_width_minus_1: add 1 to it and you'll get the expected render width expressed in > + * pixels. This is not used during the decoding process but might be used by HW scalers > + * to prepare a frame that's ready for scanout > + * @render_height_minus_1: add 1 to it and you'll get the expected render height expressed in > + * pixels. This is not used during the decoding process but might be used by HW scalers > + * to prepare a frame that's ready for scanout > + * @last_frame_ts: "last" reference buffer timestamp. > + * The timestamp refers to the timestamp field in struct v4l2_buffer. > + * Use v4l2_timeval_to_ns() to convert the struct timeval to a __u64. > + * @golden_frame_ts: "golden" reference buffer timestamp. > + * The timestamp refers to the timestamp field in struct v4l2_buffer. > + * Use v4l2_timeval_to_ns() to convert the struct timeval to a __u64. > + * @alt_frame_ts: "alt" reference buffer timestamp. > + * The timestamp refers to the timestamp field in struct v4l2_buffer. > + * Use v4l2_timeval_to_ns() to convert the struct timeval to a __u64. > + * @ref_frame_sign_bias: a bitfield specifying whether the sign bias is set for a given > + * reference frame. Either of V4L2_VP9_SIGN_BIAS_*. > + * @reset_frame_context: specifies whether the frame context should be reset to default values. > + * Either of V4L2_VP9_RESET_FRAME_CTX_*. > + * @frame_context_idx: frame context that should be used/updated > + * @profile: VP9 profile. Can be 0, 1, 2 or 3 > + * @bit_depth: bits per components. Can be 8, 10 or 12. Note that not all profiles support > + * 10 and/or 12 bits depths > + * @interpolation_filter: specifies the filter selection used for performing inter prediction. > + * Either of V4L2_VP9_INTERP_FILTER_* > + * @tile_cols_log2: specifies the base 2 logarithm of the width of each tile (where the width > + * is measured in units of 8x8 blocks). Shall be less than or equal to 6 > + * @tile_rows_log2: specifies the base 2 logarithm of the height of each tile (where the height > + * is measured in units of 8x8 blocks) > + * @reference_mode: specifies the type of inter prediction to be used. See See what? :-) > + * Either of V4L2_VP9_REFERENCE_MODE_* Other controls use V4L2_VP9_REFERENCE_MODE_{}, {} instead of *. The same applies to all the documentation. > + * @reserved: padding field. Should be zeroed by applications. > + */ > +struct v4l2_ctrl_vp9_frame { > + struct v4l2_vp9_loop_filter lf; > + struct v4l2_vp9_quantization quant; > + struct v4l2_vp9_segmentation seg; > + __u32 flags; > + __u16 compressed_header_size; > + __u16 uncompressed_header_size; > + __u16 frame_width_minus_1; > + __u16 frame_height_minus_1; > + __u16 render_width_minus_1; > + __u16 render_height_minus_1; > + __u64 last_frame_ts; > + __u64 golden_frame_ts; > + __u64 alt_frame_ts; > + __u8 ref_frame_sign_bias; > + __u8 reset_frame_context; > + __u8 frame_context_idx; > + __u8 profile; > + __u8 bit_depth; > + __u8 interpolation_filter; > + __u8 tile_cols_log2; > + __u8 tile_rows_log2; > + __u8 reference_mode; > + __u8 reserved[7]; > +}; > + Also, have you checked html and pdf docs and make sure it looks as you expect? Thanks, Ezequiel _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel