Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d9fa1648e63542dc4cc1ff6f56d8cd019a0d3e67.camel@collabora.com>
Subject: Re: [PATCH] hantro: Remove dedicated control documentation
From:   Nicolas Dufresne <nicolas.dufresne@collabora.com>
To:     Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>,
        linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        Benjamin Gaignard <benjamin.gaignard@collabora.com>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Tomasz Figa <tfiga@chromium.org>,
        Chris Healy <cphealy@gmail.com>
Date:   Tue, 19 Jul 2022 15:34:24 -0400
In-Reply-To: <20220718214123.73275-2-ezequiel@vanguardiasur.com.ar>
References: <20220718214123.73275-1-ezequiel@vanguardiasur.com.ar>
         <20220718214123.73275-2-ezequiel@vanguardiasur.com.ar>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.44.2 (3.44.2-1.fc36) 
MIME-Version: 1.0
Precedence: bulk

Le lundi 18 juillet 2022 =C3=A0 18:41 -0300, Ezequiel Garcia a =C3=A9crit=
=C2=A0:
> The dedicated control required by the HEVC support
> was removed, and the driver now calculates the value
> internally. Remove the ad-hoc documentation as well.
>=20
> Fixes: 3360755ef89ab ("media: hantro: Stop using Hantro dedicated control=
")
> Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>

Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>

> ---
>  .../userspace-api/media/drivers/hantro.rst    | 19 -------------------
>  1 file changed, 19 deletions(-)
>  delete mode 100644 Documentation/userspace-api/media/drivers/hantro.rst
>=20
> diff --git a/Documentation/userspace-api/media/drivers/hantro.rst b/Docum=
entation/userspace-api/media/drivers/hantro.rst
> deleted file mode 100644
> index cd9754b4e005..000000000000
> --- a/Documentation/userspace-api/media/drivers/hantro.rst
> +++ /dev/null
> @@ -1,19 +0,0 @@
> -.. SPDX-License-Identifier: GPL-2.0
> -
> -Hantro video decoder driver
> -=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D
> -
> -The Hantro video decoder driver implements the following driver-specific=
 controls:
> -
> -``V4L2_CID_HANTRO_HEVC_SLICE_HEADER_SKIP (integer)``
> -    Specifies to Hantro HEVC video decoder driver the number of data (in=
 bits) to
> -    skip in the slice segment header.
> -    If non-IDR, the bits to be skipped go from syntax element "pic_outpu=
t_flag"
> -    to before syntax element "slice_temporal_mvp_enabled_flag".
> -    If IDR, the skipped bits are just "pic_output_flag"
> -    (separate_colour_plane_flag is not supported).
> -
> -.. note::
> -
> -        This control is not yet part of the public kernel API and
> -        it is expected to change.