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 C1482C46CD4 for ; Wed, 27 Dec 2023 23:09:11 +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=/xFqvkStKzlW7ymKwo35XTCnuKM0q6Rp3T0N/LWbS0U=; b=SDaQA/0wRRKw/G b54+ARxVDkU0f+W9OyfrymdYm86gUSPtVU8Y6M4pYg4Z2Q4Eyeqd7o/moxLyRUT0YQSrz0w0LEDZ4 +moVcCl1RNePpxAeoZvLNMpiOQKngh3oFtmONOYZhE8rZGzSb52x9K6xdh/MBD0g4fjW22nD9ruuv 8rgAGXcy06Iy7BiMW7x7a45IINgoWYQoleW/cdhDoNw+eBV4OqaWsI3VZ/vbokmaL/jPj+Lq/hEZa 40jyRbCUlw0sX1NomEuF7lH88N9Wj8ChK0Et/UvcLbdjNUXBvZ7ComJBNE9GBkhYmeqe7ib6nYVsA 0WOgdzqY5pGbn6q+qjxw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rId1H-00Fedp-2a; Wed, 27 Dec 2023 23:08:59 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rId1E-00FedM-2J for linux-rockchip@lists.infradead.org; Wed, 27 Dec 2023 23:08:58 +0000 Received: from i53875a56.versanet.de ([83.135.90.86] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rId1A-0000uL-L2; Thu, 28 Dec 2023 00:08:52 +0100 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org Cc: liujianfeng1994@gmail.com, Jianfeng Liu Subject: Re: [PATCH 1/3] media: verisilicon: Add support for Hantro G1 on RK3588 Date: Thu, 28 Dec 2023 00:08:51 +0100 Message-ID: <2608913.C4sosBPzcN@diego> In-Reply-To: <20231227173911.3295410-2-liujianfeng1994@gmail.com> References: <20231227173911.3295410-1-liujianfeng1994@gmail.com> <20231227173911.3295410-2-liujianfeng1994@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231227_150856_781849_8ECB91BF X-CRM114-Status: GOOD ( 23.64 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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, Am Mittwoch, 27. Dezember 2023, 18:39:09 CET schrieb Jianfeng Liu: > This patch adds support for RK3588 in existing Hantro > video decoder kernel driver. > > Tested with FFmpeg v4l2_request code taken from [1] > with MPEG2, H.264 and VP8 samples. > > [1] https://github.com/LibreELEC/LibreELEC.tv/blob/master/packages/multimedia/ffmpeg/patches/v4l2-request/ffmpeg-001-v4l2-request.patch > thanks so much for looking into this :-) . But please use scripts/get_maintainers.pl when submitting patches, because not everyone reads mailing lists all the time. Most of the time people enjoy being able to read the whole series, so also please don't do per-patch recipients. Especially when adding devicetree bindings you need to add the devicetree maintainers and lists. So running for example # scripts/get_maintainer.pl -f Documentation/devicetree/bindings/media/rockchip-vpu.yaml will get you this pretty correct list of recipients for your series: Ezequiel Garcia Philipp Zabel Mauro Carvalho Chehab Rob Herring Krzysztof Kozlowski Conor Dooley Heiko Stuebner linux-media@vger.kernel.org linux-rockchip@lists.infradead.org devicetree@vger.kernel.org linux-arm-kernel@lists.infradead.org linux-kernel@vger.kernel.org > Signed-off-by: Jianfeng Liu > --- > drivers/media/platform/verisilicon/hantro_drv.c | 1 + > drivers/media/platform/verisilicon/hantro_hw.h | 1 + > .../media/platform/verisilicon/rockchip_vpu_hw.c | 14 ++++++++++++++ > 3 files changed, 16 insertions(+) > > diff --git a/drivers/media/platform/verisilicon/hantro_drv.c b/drivers/media/platform/verisilicon/hantro_drv.c > index db3df6cc4..4c988f272 100644 > --- a/drivers/media/platform/verisilicon/hantro_drv.c > +++ b/drivers/media/platform/verisilicon/hantro_drv.c > @@ -722,6 +722,7 @@ static const struct of_device_id of_hantro_match[] = { > { .compatible = "rockchip,rk3399-vpu", .data = &rk3399_vpu_variant, }, > { .compatible = "rockchip,rk3568-vepu", .data = &rk3568_vepu_variant, }, > { .compatible = "rockchip,rk3568-vpu", .data = &rk3568_vpu_variant, }, > + { .compatible = "rockchip,rk3588-vpu", .data = &rk3588_vpu_variant, }, > { .compatible = "rockchip,rk3588-av1-vpu", .data = &rk3588_vpu981_variant, }, I do think this list is sorted alphabetically, so rk3588-vpu would be _after_ rk3588-a... Thanks Heiko > #endif > #ifdef CONFIG_VIDEO_HANTRO_IMX8M > diff --git a/drivers/media/platform/verisilicon/hantro_hw.h b/drivers/media/platform/verisilicon/hantro_hw.h > index 9aec8a79a..276f09a7a 100644 > --- a/drivers/media/platform/verisilicon/hantro_hw.h > +++ b/drivers/media/platform/verisilicon/hantro_hw.h > @@ -405,6 +405,7 @@ extern const struct hantro_variant rk3328_vpu_variant; > extern const struct hantro_variant rk3399_vpu_variant; > extern const struct hantro_variant rk3568_vepu_variant; > extern const struct hantro_variant rk3568_vpu_variant; > +extern const struct hantro_variant rk3588_vpu_variant; > extern const struct hantro_variant rk3588_vpu981_variant; > extern const struct hantro_variant sama5d4_vdec_variant; > extern const struct hantro_variant sunxi_vpu_variant; > diff --git a/drivers/media/platform/verisilicon/rockchip_vpu_hw.c b/drivers/media/platform/verisilicon/rockchip_vpu_hw.c > index f97527670..4ee8d64eb 100644 > --- a/drivers/media/platform/verisilicon/rockchip_vpu_hw.c > +++ b/drivers/media/platform/verisilicon/rockchip_vpu_hw.c > @@ -797,6 +797,20 @@ const struct hantro_variant px30_vpu_variant = { > .num_clocks = ARRAY_SIZE(rockchip_vpu_clk_names) > }; > > +const struct hantro_variant rk3588_vpu_variant = { > + .dec_offset = 0x400, > + .dec_fmts = rockchip_vdpu2_dec_fmts, > + .num_dec_fmts = ARRAY_SIZE(rockchip_vdpu2_dec_fmts), > + .codec = HANTRO_MPEG2_DECODER | > + HANTRO_VP8_DECODER | HANTRO_H264_DECODER, > + .codec_ops = rk3399_vpu_codec_ops, > + .irqs = rockchip_vdpu2_irqs, > + .num_irqs = ARRAY_SIZE(rockchip_vdpu2_irqs), > + .init = rockchip_vpu_hw_init, > + .clk_names = rockchip_vpu_clk_names, > + .num_clocks = ARRAY_SIZE(rockchip_vpu_clk_names) > +}; > + > const struct hantro_variant rk3588_vpu981_variant = { > .dec_offset = 0x0, > .dec_fmts = rockchip_vpu981_dec_fmts, > -- > 2.34.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip