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=-0.8 required=3.0 tests=FROM_EXCESS_BASE64, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 07698C433FF for ; Mon, 12 Aug 2019 12:21:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D53F720820 for ; Mon, 12 Aug 2019 12:21:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728455AbfHLMVR (ORCPT ); Mon, 12 Aug 2019 08:21:17 -0400 Received: from mailoutvs53.siol.net ([185.57.226.244]:51170 "EHLO mail.siol.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727525AbfHLMVR (ORCPT ); Mon, 12 Aug 2019 08:21:17 -0400 Received: from localhost (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTP id 791135206FB; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) X-Virus-Scanned: amavisd-new at psrvmta10.zcs-production.pri Received: from mail.siol.net ([127.0.0.1]) by localhost (psrvmta10.zcs-production.pri [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id gEiOe5RdhaUL; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) Received: from mail.siol.net (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTPS id 1E9B452379C; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) Received: from jernej-laptop.localnet (89-212-178-211.dynamic.t-2.net [89.212.178.211]) (Authenticated sender: jernej.skrabec@siol.net) by mail.siol.net (Postfix) with ESMTPA id 7FD855206FB; Mon, 12 Aug 2019 14:21:12 +0200 (CEST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Hans Verkuil Cc: paul.kocialkowski@bootlin.com, maxime.ripard@bootlin.com, wens@csie.org, mchehab@kernel.org, gregkh@linuxfoundation.org, linux-media@vger.kernel.org, devel@driverdev.osuosl.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/7] media: cedrus: Improvements/cleanup Date: Mon, 12 Aug 2019 14:21:11 +0200 Message-ID: <4636127.h7jaTrrqaR@jernej-laptop> In-Reply-To: <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> References: <20190530211516.1891-1-jernej.skrabec@siol.net> <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dne ponedeljek, 12. avgust 2019 ob 14:12:21 CEST je Hans Verkuil napisal(a): > On 5/30/19 11:15 PM, Jernej Skrabec wrote: > > Here is first batch of random Cedrus improvements/cleanups. Only patch 2 > > has a change which raises a question about H264 controls. > > > > Changes were tested on H3 SoC using modified ffmpeg and Kodi. > > > > Please take a look. > > This has been sitting in patchwork for quite some time. I've updated the > status of the various patches and most needed extra work. > > It seems that patches 4/7 and 5/7 are OK. Maxime, can you please confirm > that these two are still valid? They apply cleanly on the latest master > at least, but since they are a bit old I prefer to have confirmation that > it's OK to merge them. I'm not sure about patch 4, IIRC Boris Brezillon also wants to improve that area in separate series, but patch 5 should be safe to merge. Anyway, I didn't post new version because I'm waiting on close-to-be-merged H264 and HEVC patch series to be actually merged. Best regards, Jernej > > Regards, > > Hans > > > Best regards, > > Jernej > > > > Jernej Skrabec (7): > > media: cedrus: Disable engine after each slice decoding > > media: cedrus: Fix H264 default reference index count > > media: cedrus: Fix decoding for some H264 videos > > media: cedrus: Remove dst_bufs from context > > media: cedrus: Don't set chroma size for scale & rotation > > media: cedrus: Add infra for extra buffers connected to capture > > > > buffers > > > > media: cedrus: Improve H264 memory efficiency > > > > drivers/staging/media/sunxi/cedrus/cedrus.h | 12 +- > > .../staging/media/sunxi/cedrus/cedrus_h264.c | 115 ++++++++---------- > > .../staging/media/sunxi/cedrus/cedrus_hw.c | 4 +- > > .../staging/media/sunxi/cedrus/cedrus_video.c | 25 ++-- > > 4 files changed, 68 insertions(+), 88 deletions(-) From mboxrd@z Thu Jan 1 00:00:00 1970 From: jernej.skrabec@siol.net (Jernej =?utf-8?B?xaBrcmFiZWM=?=) Date: Mon, 12 Aug 2019 14:21:11 +0200 Subject: [PATCH 0/7] media: cedrus: Improvements/cleanup In-Reply-To: <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> References: <20190530211516.1891-1-jernej.skrabec@siol.net> <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> Message-ID: <4636127.h7jaTrrqaR@jernej-laptop> List-Id: Linux Driver Project Developer List Dne ponedeljek, 12. avgust 2019 ob 14:12:21 CEST je Hans Verkuil napisal(a): > On 5/30/19 11:15 PM, Jernej Skrabec wrote: > > Here is first batch of random Cedrus improvements/cleanups. Only patch 2 > > has a change which raises a question about H264 controls. > > > > Changes were tested on H3 SoC using modified ffmpeg and Kodi. > > > > Please take a look. > > This has been sitting in patchwork for quite some time. I've updated the > status of the various patches and most needed extra work. > > It seems that patches 4/7 and 5/7 are OK. Maxime, can you please confirm > that these two are still valid? They apply cleanly on the latest master > at least, but since they are a bit old I prefer to have confirmation that > it's OK to merge them. I'm not sure about patch 4, IIRC Boris Brezillon also wants to improve that area in separate series, but patch 5 should be safe to merge. Anyway, I didn't post new version because I'm waiting on close-to-be-merged H264 and HEVC patch series to be actually merged. Best regards, Jernej > > Regards, > > Hans > > > Best regards, > > Jernej > > > > Jernej Skrabec (7): > > media: cedrus: Disable engine after each slice decoding > > media: cedrus: Fix H264 default reference index count > > media: cedrus: Fix decoding for some H264 videos > > media: cedrus: Remove dst_bufs from context > > media: cedrus: Don't set chroma size for scale & rotation > > media: cedrus: Add infra for extra buffers connected to capture > > > > buffers > > > > media: cedrus: Improve H264 memory efficiency > > > > drivers/staging/media/sunxi/cedrus/cedrus.h | 12 +- > > .../staging/media/sunxi/cedrus/cedrus_h264.c | 115 ++++++++---------- > > .../staging/media/sunxi/cedrus/cedrus_hw.c | 4 +- > > .../staging/media/sunxi/cedrus/cedrus_video.c | 25 ++-- > > 4 files changed, 68 insertions(+), 88 deletions(-) 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=-0.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,FROM_EXCESS_BASE64,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 7A8AAC31E40 for ; Mon, 12 Aug 2019 12:21:31 +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 4B97A20820 for ; Mon, 12 Aug 2019 12:21:31 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Dlgb1vT8" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4B97A20820 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=siol.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=4CKPWT+QhoNP6O4H5apv3HIj3I/el3owXE0k6xpys3Y=; b=Dlgb1vT8U8RVtc 4CMEu5YKR3KTyw6dqUhjLwisZzq/99kXJ7u44GlZnkjIVbgXzmRDZOvyAB1OjEVtYgUOgHyb8AtTq lANanbO4Vn/nswtyt3TzXf8hZhqgraJFZTCoER3OwTj47nQfi366CMC7JXBasd+M/4RVN2bKfhQyw thKtHQT5FYYGDbIW/MXCluAmHTzet3aPtUZVeYvL28BRsD6unUbnXs/zZx/bRh3tWybBA0d7haqUC jua4Np7386DqZ9SX9bFfRPiDIJAtKERt+PovGqoG2U5muT7lucV+KvVENtpYrxotyzij+4hz1fXBe sfc7E+sQWYlmkV6MP72Q==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hx9KF-0007Hc-3L; Mon, 12 Aug 2019 12:21:23 +0000 Received: from mailoutvs25.siol.net ([185.57.226.216] helo=mail.siol.net) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hx9KB-0007Gu-Hp for linux-arm-kernel@lists.infradead.org; Mon, 12 Aug 2019 12:21:21 +0000 Received: from localhost (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTP id 791135206FB; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) X-Virus-Scanned: amavisd-new at psrvmta10.zcs-production.pri Received: from mail.siol.net ([127.0.0.1]) by localhost (psrvmta10.zcs-production.pri [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id gEiOe5RdhaUL; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) Received: from mail.siol.net (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTPS id 1E9B452379C; Mon, 12 Aug 2019 14:21:13 +0200 (CEST) Received: from jernej-laptop.localnet (89-212-178-211.dynamic.t-2.net [89.212.178.211]) (Authenticated sender: jernej.skrabec@siol.net) by mail.siol.net (Postfix) with ESMTPA id 7FD855206FB; Mon, 12 Aug 2019 14:21:12 +0200 (CEST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Hans Verkuil Subject: Re: [PATCH 0/7] media: cedrus: Improvements/cleanup Date: Mon, 12 Aug 2019 14:21:11 +0200 Message-ID: <4636127.h7jaTrrqaR@jernej-laptop> In-Reply-To: <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> References: <20190530211516.1891-1-jernej.skrabec@siol.net> <274221f1-b2d2-83aa-d84b-e1c572a1b832@xs4all.nl> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190812_052119_747285_0BB7284E X-CRM114-Status: GOOD ( 16.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devel@driverdev.osuosl.org, maxime.ripard@bootlin.com, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, wens@csie.org, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Dne ponedeljek, 12. avgust 2019 ob 14:12:21 CEST je Hans Verkuil napisal(a): > On 5/30/19 11:15 PM, Jernej Skrabec wrote: > > Here is first batch of random Cedrus improvements/cleanups. Only patch 2 > > has a change which raises a question about H264 controls. > > > > Changes were tested on H3 SoC using modified ffmpeg and Kodi. > > > > Please take a look. > > This has been sitting in patchwork for quite some time. I've updated the > status of the various patches and most needed extra work. > > It seems that patches 4/7 and 5/7 are OK. Maxime, can you please confirm > that these two are still valid? They apply cleanly on the latest master > at least, but since they are a bit old I prefer to have confirmation that > it's OK to merge them. I'm not sure about patch 4, IIRC Boris Brezillon also wants to improve that area in separate series, but patch 5 should be safe to merge. Anyway, I didn't post new version because I'm waiting on close-to-be-merged H264 and HEVC patch series to be actually merged. Best regards, Jernej > > Regards, > > Hans > > > Best regards, > > Jernej > > > > Jernej Skrabec (7): > > media: cedrus: Disable engine after each slice decoding > > media: cedrus: Fix H264 default reference index count > > media: cedrus: Fix decoding for some H264 videos > > media: cedrus: Remove dst_bufs from context > > media: cedrus: Don't set chroma size for scale & rotation > > media: cedrus: Add infra for extra buffers connected to capture > > > > buffers > > > > media: cedrus: Improve H264 memory efficiency > > > > drivers/staging/media/sunxi/cedrus/cedrus.h | 12 +- > > .../staging/media/sunxi/cedrus/cedrus_h264.c | 115 ++++++++---------- > > .../staging/media/sunxi/cedrus/cedrus_hw.c | 4 +- > > .../staging/media/sunxi/cedrus/cedrus_video.c | 25 ++-- > > 4 files changed, 68 insertions(+), 88 deletions(-) _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel