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=-15.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, MISSING_HEADERS,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 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 21E22C433B4 for ; Thu, 6 May 2021 15:15:31 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (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 7C68D61176 for ; Thu, 6 May 2021 15:15:30 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7C68D61176 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=desiato.20200630; 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: Subject:Cc:From:Date:Reply-To:To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=G4KsKVfBPE+AQL+qDkjaMsy4ZUcyA9GaO4uZ395IeBk=; b=gfXDdv0uY0nLpDw5TMhUdCtr3 o1jbQqdHRRwu4WB64SUcxYbTgzCqbEN9g4v3Z2J21lW3voqY5Sx6xYJDfWqRBtCb5cdrXDOQv39aO ULPsIX7S9ehuZinFYiVC9sauWhwYFAaYcKkKW7Nqn6Zi5sNtWdXyBMopa/1XsEeYOJk1kHz7MKU8e AuRZS1b++u27K8M0aSKQzR/yisz8OMQF4Nvp9DvdyoIJB27i6vJ+7AbghPJbDATBATCcf1QQ9hutf cxj3TD1sb8GTAywbHphE/EVBJcgTEx8JKMdx0nOYtzoLLQXpQa2tpOK9B0lQqE6zopLKK6+1Sa0bh Q3NyNexYg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1leffl-004Vk0-1k; Thu, 06 May 2021 15:12:17 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1leffh-004Vjj-FM; Thu, 06 May 2021 15:12:13 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:From: Date:Sender:Reply-To:To:Content-ID:Content-Description; bh=LJa6RoX0kZaKw+NdKPeuIif2bLhnpCMHq2KRdjL1rtY=; b=Fz2ujfWapiIY0JzrOU93lm2KmT 3fZbLXkdIh0KQpnbc6ii1zlrYsmpA+CHHlahfSi//kEkQvPJEHPt7XPN18iQygvcBharLuZ1I8LiO 5z+9quz/WYgD/iIuvwe2O2jIwmuUt3sh1hGLo/X5JwRE76uXfBCOdPizdLe6Oj8Huev4zu6jvh7mL yGLNsFDv1VSruWSxgh60KlSBDEjaL9DgsfOwxykkEYwsrqJUwyD3mT5JqA+ix7mAoTIRBvqfmPtMB qs76DA0cI1G2kwG2NwxrDM03DP47Pa0/+FwNvlT+8QZFqFLfH7zr1oPvLaNS/0yV5NVvO6yFp2BzU +3l12Pig==; Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1leffe-0067o4-BO; Thu, 06 May 2021 15:12:12 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7452D61041; Thu, 6 May 2021 15:12:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1620313928; bh=5pyBiTEHvMxAj8e/yO3yGgrWonLAEYO6vTvkoiWBA6I=; h=Date:From:Cc:Subject:In-Reply-To:References:From; b=gCHwiMQxRg/34kpnWeUtVHGRkYuqPEOIQkZjVTbPZZPFy2jqZ6oslJA4TOQa+gcHd Z0NptbjOm1CDOQB3ZQKMyntN6xQ5y7grAgI5pG3HJhrWpTilFGoxYfvmb6EoYj445K CQ0uc3Z4NuK9pz7n4SC/EZcPlSxDibZYfzeHq83K3HcpNS0SlwJvX6wThXiW6AEk2u hMwzEy9DylBCX+jg+CvlvqsbQ2ABQrFVf8X1Vi88Uh4ZWYq9UGScRPv8ih+Dd5UE7z mmJ11YeiM3YEzKEgUBlxxi6W4vsOo94Lj61ii0dn8Sbwu0SCHHobKO2iHWqCOgTTiV lZpsKjD8tnxqw== Date: Thu, 6 May 2021 17:11:57 +0200 From: Mauro Carvalho Chehab Cc: linuxarm@huawei.com, mauro.chehab@huawei.com, "Lad, Prabhakar" , "Paul J. Murphy" , Andrzej Pietrasiewicz , Andy Gross , Bjorn Andersson , Chen-Yu Tsai , Daniele Alessandrelli , Ezequiel Garcia , Fabio Estevam , Jacek Anaszewski , Jernej Skrabec , Krzysztof Kozlowski , Marek Szyprowski , Matthias Brugger , Mauro Carvalho Chehab , Maxime Ripard , NXP Linux Team , Pengutronix Kernel Team , Philipp Zabel , Rui Miguel Silva , Sakari Ailus , Sascha Hauer , Shawn Guo , Stanimir Varbanov , Steve Longerbeam , Sylwester Nawrocki , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-staging@lists.linux.dev, linux-sunxi@lists.linux.dev Subject: Re: [PATCH 00/25] Fix some PM runtime issues at the media subsystem Message-ID: <20210506171157.678f5c15@coco.lan> In-Reply-To: References: X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-redhat-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210506_081210_490732_D46C5608 X-CRM114-Status: GOOD ( 20.20 ) 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 Em Wed, 5 May 2021 11:41:50 +0200 Mauro Carvalho Chehab escreveu: > As part of an effort to cleanup pm_runtime*get* calls, I detected a number > of issues at the media subsystem. > > Most of the patches here were submitted previously at: > > https://lore.kernel.org/linux-media/cover.1619621413.git.mchehab+huawei@kernel.org/ > > This series contain just the bug fixes and other related issues that are > present with the current code on media. Series merged on my stage tree, at: https://git.linuxtv.org/media_stage.git/log/ I'll be merging it at media_tree (either for 5.13 or 5.14) after the end of the merge window (likely next week). Please let me know if you find any problems on it. PS.: please notice that my stage tree can be rebased. Regards, Mauro > > It address the points from the existing reviews. I also did my own > set of reviews, in order to avoid regressions. > > Changes from v4 of the previous changeset: > > - reworked i2c/css RPM get logic; > - dropped two patches that could cause regressions; > - am437x: keep using pm_runtime_get_sync on suspend/resume; > - atmel: fix the returned code and add a print on failures at start streaming; > - simplify some checks for return code > 0; > - mdk-vcodec: properly handle RPM errors at device on logic; > - venus: rework venus_sys_error_handler() logic; > - sti/delta: fix an issue at the error checking logic. > > Mauro Carvalho Chehab (25): > staging: media: rkvdec: fix pm_runtime_get_sync() usage count > staging: media: imx7-mipi-csis: fix pm_runtime_get_sync() usage count > media: venus: Rework error fail recover logic > media: s5p_cec: decrement usage count if disabled > media: i2c: ccs-core: return the right error code at suspend > media: i2c: imx334: fix the pm runtime get logic > media: exynos-gsc: don't resume at remove time > media: atmel: properly get pm_runtime > media: hantro: do a PM resume earlier > media: marvel-ccic: fix some issues when getting pm_runtime > media: mdk-mdp: fix pm_runtime_get_sync() usage count > media: rcar_fdp1: simplify error check logic at fdp_open() > media: rcar_fdp1: fix pm_runtime_get_sync() usage count > media: renesas-ceu: Properly check for PM errors > media: s5p: fix pm_runtime_get_sync() usage count > media: am437x: fix pm_runtime_get_sync() usage count > media: sh_vou: fix pm_runtime_get_sync() usage count > media: mtk-vcodec: fix PM runtime get logic > media: s5p-jpeg: fix pm_runtime_get_sync() usage count > media: sti/delta: use pm_runtime_resume_and_get() > media: sunxi: fix pm_runtime_get_sync() usage count > media: sti/bdisp: fix pm_runtime_get_sync() usage count > media: exynos4-is: fix pm_runtime_get_sync() usage count > media: exynos-gsc: fix pm_runtime_get_sync() usage count > media: i2c: ccs-core: fix pm_runtime_get_sync() usage count > > drivers/media/cec/platform/s5p/s5p_cec.c | 7 ++- > drivers/media/i2c/ccs/ccs-core.c | 41 ++++++++----- > drivers/media/i2c/imx334.c | 7 ++- > drivers/media/platform/am437x/am437x-vpfe.c | 15 ++++- > drivers/media/platform/atmel/atmel-isc-base.c | 30 +++++++--- > drivers/media/platform/atmel/atmel-isi.c | 19 ++++-- > drivers/media/platform/exynos-gsc/gsc-core.c | 11 ++-- > drivers/media/platform/exynos-gsc/gsc-m2m.c | 4 +- > .../media/platform/exynos4-is/fimc-capture.c | 6 +- > drivers/media/platform/exynos4-is/fimc-is.c | 4 +- > .../platform/exynos4-is/fimc-isp-video.c | 3 +- > drivers/media/platform/exynos4-is/fimc-isp.c | 7 +-- > drivers/media/platform/exynos4-is/fimc-lite.c | 5 +- > drivers/media/platform/exynos4-is/fimc-m2m.c | 5 +- > drivers/media/platform/exynos4-is/media-dev.c | 9 +-- > drivers/media/platform/exynos4-is/mipi-csis.c | 10 ++-- > .../media/platform/marvell-ccic/mcam-core.c | 9 ++- > drivers/media/platform/mtk-mdp/mtk_mdp_m2m.c | 6 +- > .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 4 +- > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.c | 8 ++- > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.h | 2 +- > drivers/media/platform/qcom/venus/core.c | 59 +++++++++++++++---- > drivers/media/platform/rcar_fdp1.c | 28 ++++++--- > drivers/media/platform/renesas-ceu.c | 4 +- > drivers/media/platform/s5p-jpeg/jpeg-core.c | 5 +- > drivers/media/platform/sh_vou.c | 6 +- > drivers/media/platform/sti/bdisp/bdisp-v4l2.c | 7 ++- > drivers/media/platform/sti/delta/delta-v4l2.c | 8 +-- > .../sunxi/sun8i-rotate/sun8i_rotate.c | 2 +- > drivers/staging/media/hantro/hantro_drv.c | 7 ++- > drivers/staging/media/imx/imx7-mipi-csis.c | 7 +-- > drivers/staging/media/rkvdec/rkvdec.c | 2 +- > 32 files changed, 220 insertions(+), 127 deletions(-) > Thanks, Mauro _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel