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 X-Spam-Level: X-Spam-Status: No, score=-10.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 60F23C433DF for ; Mon, 12 Oct 2020 23:40:09 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 6AF622076D for ; Mon, 12 Oct 2020 23:40:08 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="L8tJd5jg"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kwiboo.se header.i=@kwiboo.se header.b="hWlEplUm" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6AF622076D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kwiboo.se Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:To:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=E4QPq6yYu8t5fDNK/DWA73jgE2e8SyEEwUx+r+B+qg8=; b=L8tJd5jgjEAh71C5zFVgyDXX6 hmlC2vSoiDcm+YGiBUgGg7lpNvWACCO8LSueCtPmnJAuxrXIaoEogRMkYw5aeZjFv4wSGN9EpEQDO DXb46+W3JEbqeGlN0DSQUS9/naAz5Pl5vEAP3uYYdTL6yjGUxKFpWAbq5pim98ePYAki7z+NQPAbj 4MkQtOK9MrLGTaV1+EmShJHHIeBX67iKzLgAWqhDcCs4sSUqB1A7NdRzo615KgMT4JE8eXx/xBKfd odCYy4AO2ZN2KgyyJaMtu1j8yAG7WeaG6rmiNm7s3UnRbA32iquO9jDAzuYFo2486qUPGkVS+d6YL yCymU4R6w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kS7Q7-0002Zm-Qw; Mon, 12 Oct 2020 23:39:59 +0000 Received: from o1.b.az.sendgrid.net ([208.117.55.133]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kS7Q4-0002Z6-BW for linux-rockchip@lists.infradead.org; Mon, 12 Oct 2020 23:39:57 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kwiboo.se; h=subject:references:from:mime-version:in-reply-to:to:cc:content-type: content-transfer-encoding; s=001; bh=FWd7GLu/tKLDYZWyqY42ek5eJQUDEXNUAlc9iLUoOSQ=; b=hWlEplUmTpnYzRFWyIf4FCE6wfVXkJ1rfr7fkYfsKO8P15QdaCtSTB2hHa/RrVx5a0AQ HhHIHIYOf3857gyEAWb5XbG8TsAazMN7Rgfn1etr5zH+N0waxqett+LYdKlRqwtw8/P8mv eyofcmrs3avYSGmUfi6j2nYUg/5YY9hdE= Received: by filterdrecv-p3las1-dcbfbb89c-tcntw with SMTP id filterdrecv-p3las1-dcbfbb89c-tcntw-18-5F84E946-82 2020-10-12 23:39:51.179332393 +0000 UTC m=+435832.437857433 Received: from [192.168.1.14] (unknown) by ismtpd0006p1lon1.sendgrid.net (SG) with ESMTP id C8HTQRspSkaVK3O5MSCmWA Mon, 12 Oct 2020 23:39:50.729 +0000 (UTC) Subject: Re: [PATCH 00/18] Add Hantro regmap and VC8000 h264 decode support References: <20201012205957.889185-1-adrian.ratiu@collabora.com> From: Jonas Karlman Message-ID: <97e84bb5-972a-091d-a159-6ab1151f17ab@kwiboo.se> Date: Mon, 12 Oct 2020 23:39:51 +0000 (UTC) User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.12.1 MIME-Version: 1.0 In-Reply-To: <20201012205957.889185-1-adrian.ratiu@collabora.com> X-SG-EID: =?us-ascii?Q?TdbjyGynYnRZWhH+7lKUQJL+ZxmxpowvO2O9SQF5CwCVrYgcwUXgU5DKUU3QxA?= =?us-ascii?Q?fZekEeQsTe+RrMu3cja6a0h4m=2F0JLyOkHdDHSTb?= =?us-ascii?Q?GzI4=2FudO3qzCik37O+eo=2FnD6F3CkCKMBwB34tRk?= =?us-ascii?Q?aKxv5b6rTNjK84lRypk2iCtJ7H31qwCPs=2F7acy1?= =?us-ascii?Q?x5eo0LJ7XJhU+cGjCH5zRDr+hRMcAj5R8HdlJdO?= =?us-ascii?Q?WUflq43eaMvMtQeqzv1hvlO0IXZUBggCP2Wv+O+?= =?us-ascii?Q?1JQObbH=2FqfQ8bJDfRTxvA=3D=3D?= To: Adrian Ratiu , Ezequiel Garcia , Philipp Zabel X-Entity-ID: wSPGWgGSXUap++qShBI+ag== Content-Language: sv X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201012_193956_654744_9CC582F0 X-CRM114-Status: GOOD ( 26.65 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Fruehberger Peter , Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, Mark Brown , kuhanh.murugasen.krishnan@intel.com, Daniel Vetter , kernel@collabora.com, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Hi, On 2020-10-12 22:59, Adrian Ratiu wrote: > Dear all, > > This series introduces a regmap infrastructure for the Hantro driver > which is used to compensate for different HW-revision register layouts. > To justify it h264 decoding capability is added for newer VC8000 chips. > > This is a gradual conversion to the new infra - a complete conversion > would have been very big and I do not have all the HW yet to test (I'm > expecting a RK3399 shipment next week though ;). I think converting the > h264 decoder provides a nice blueprint for how the other codecs can be > converted and enabled for different HW revisions. > > The end goal of this is to make the driver more generic and eliminate > entirely custom boilerplate like `struct hantro_reg` or headers with > core-specific bit manipulations like `hantro_g1_regs.h` and instead rely > on the well-tested albeit more verbose regmap subsytem. > > To give just two examples of bugs which are easily discovered by using > more verbose regmap fields (very easy to compare with the datasheets) > instead of relying on bit-magic tricks: G1_REG_DEC_CTRL3_INIT_QP(x) was > off-by-1 and the wrong .clk_gate bit was set in hantro_postproc.c. > > Anyway, this series also extends the MMIO regmap API to allow relaxed > writes for the theoretical reason that avoiding unnecessary membarriers > leads to less CPU usage and small improvements to battery life. However, > in practice I could not measure differences between relaxed/non-relaxed > IO, so I'm on the fence whether to keep or remove the relaxed calls. > > What I could masure is the performance impact of adding more sub-reg > field acesses: a constant ~ 20 microsecond bump per G1 h264 frame. This > is acceptable considering the total time to decode a frame takes three > orders of magnitude longer, i.e. miliseconds ranges, depending on the > frame size and bitstream params, so it is an acceptable trade-off to > have a more generic driver. In the RK3399 variant all fields use completely different positions so in order to make the driver fully generic all around 145 sub-reg fields used for h264 needs to be converted, see [1] for a quick generation of field mappings used for h264 decoding. Any indication on how the performance will be impacted with 145 fields compared to around 20 fields used in this series? Another issue with RK3399 variant is that some fields use different position depending on the codec used, e.g. two dec_ref_frames in [2]. Should we use codec specific field maps? or any other suggestion on how we can handle such case? [1] https://github.com/Kwiboo/rockchip-vpu-regtool/commit/8b88d94d2ed966c7d88d9a735c0c97368eb6c92d [2] https://github.com/Kwiboo/rockchip-vpu-regtool/blob/master/rk3399_dec_regs.c#L1065 [3] https://github.com/Kwiboo/rockchip-vpu-regtool/commit/9498326296445a9ce153b585cc48e0cea05d3c93 Best regards, Jonas > > This has been tested on next-20201009 with imx8mq for G1 and an SoC with > VC8000 which has not yet been added (hopefuly support lands soon). > > Kind regards, > Adrian > > Adrian Ratiu (18): > media: hantro: document all int reg bits up to vc8000 > media: hantro: make consistent use of decimal register notation > media: hantro: make G1_REG_SOFT_RESET Rockchip specific > media: hantro: add reset controller support > media: hantro: prepare clocks before variant inits are run > media: hantro: imx8mq: simplify ctrlblk reset logic > regmap: mmio: add config option to allow relaxed MMIO accesses > media: hantro: add initial MMIO regmap infrastructure > media: hantro: default regmap to relaxed MMIO > media: hantro: convert G1 h264 decoder to regmap fields > media: hantro: convert G1 postproc to regmap > media: hantro: add VC8000D h264 decoding > media: hantro: add VC8000D postproc support > media: hantro: make PP enablement logic a bit smarter > media: hantro: add user-selectable, platform-selectable H264 High10 > media: hantro: rename h264_dec as it's not G1 specific anymore > media: hantro: add dump registers debug option before decode start > media: hantro: document encoder reg fields > > drivers/base/regmap/regmap-mmio.c | 34 +- > drivers/staging/media/hantro/Makefile | 3 +- > drivers/staging/media/hantro/hantro.h | 79 +- > drivers/staging/media/hantro/hantro_drv.c | 41 +- > drivers/staging/media/hantro/hantro_g1_regs.h | 92 +- > ...hantro_g1_h264_dec.c => hantro_h264_dec.c} | 237 +++- > drivers/staging/media/hantro/hantro_hw.h | 23 +- > .../staging/media/hantro/hantro_postproc.c | 144 ++- > drivers/staging/media/hantro/hantro_regmap.c | 1015 +++++++++++++++++ > drivers/staging/media/hantro/hantro_regmap.h | 295 +++++ > drivers/staging/media/hantro/hantro_v4l2.c | 3 +- > drivers/staging/media/hantro/imx8m_vpu_hw.c | 75 +- > drivers/staging/media/hantro/rk3288_vpu_hw.c | 5 +- > include/linux/regmap.h | 5 + > 14 files changed, 1795 insertions(+), 256 deletions(-) > rename drivers/staging/media/hantro/{hantro_g1_h264_dec.c => hantro_h264_dec.c} (58%) > create mode 100644 drivers/staging/media/hantro/hantro_regmap.c > create mode 100644 drivers/staging/media/hantro/hantro_regmap.h > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip