From nobody Sun Apr 19 04:03:43 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 493C9C43334 for ; Wed, 6 Jul 2022 12:53:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233741AbiGFMxI (ORCPT ); Wed, 6 Jul 2022 08:53:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54218 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231280AbiGFMxG (ORCPT ); Wed, 6 Jul 2022 08:53:06 -0400 Received: from mail-pj1-x102b.google.com (mail-pj1-x102b.google.com [IPv6:2607:f8b0:4864:20::102b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 037F222BF2 for ; Wed, 6 Jul 2022 05:53:05 -0700 (PDT) Received: by mail-pj1-x102b.google.com with SMTP id z12-20020a17090a7b8c00b001ef84000b8bso9932342pjc.1 for ; Wed, 06 Jul 2022 05:53:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=dwMB+k9IGlD1nBJyNrEkLFj2csDbu5Qc4SyaeENoXs4=; b=VLocBcR67BH55CFePg7A7u3NRm1n4ardKuomKvCqMEuZ1qS29FEGTPbwhupBegRGGV pEFJtFTMkb/b2XpBQPLTY7Oe7jtGOopiYkTJgulNCbmKKEnHW5FeZDiPqZmAKO0i8WqZ Lm9yqq7WX9rKKbs4KFq6mTwidKV/MGZxvUBHk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=dwMB+k9IGlD1nBJyNrEkLFj2csDbu5Qc4SyaeENoXs4=; b=8DN111YYF+J7DiJhMss06d3PPSXS84PQIPHTTzP6jMVGKbVEoV9a6t43W/svm8ZBEI VMZ6GhVJl23ZydOgjoQLQL07fS4zRm2VoR4BL4HyPGnvj9EX/HX8cQP2AZ4a5NM2dHj7 rK0mnVMnClqkek2Sb56mMUF3oG++NBarwvzP5oCkAZ+rGBMZVw1sbHCbUKNcsluhHY9r DkM0EbZdQUi2dTLSJIp6AbCX7H1gvYJb0MMLOJTHMBD9BfPZhElvIQ+UesbHnMXwAINd f7dzPFhFO0KgWDEWErTEWfdl90xSClARZzvsIfej2u4i+GIuNVwqKXfyOtN5PyEQqfoK NPKg== X-Gm-Message-State: AJIora943Hp6TBFnR7Z5ybEuEHHVoy/9iW80YAx2bNqbBNNCE1kG3P6u 5d7jq3ZCb14/ekbTwBPL8An0+A== X-Google-Smtp-Source: AGRyM1vRXlvrOK3qxEFCRY5ntUjBQ+I/TKLHMNestZ/71lmAhunY5e/klvvWxIOHDet94pd164yMmQ== X-Received: by 2002:a17:90a:178b:b0:1ef:7c85:8070 with SMTP id q11-20020a17090a178b00b001ef7c858070mr25348020pja.233.1657111984407; Wed, 06 Jul 2022 05:53:04 -0700 (PDT) Received: from hsinyi-z840.tpe.corp.google.com ([2401:fa00:1:10:5300:b974:1680:1bd]) by smtp.gmail.com with ESMTPSA id u12-20020a17090341cc00b0016a6cd546d6sm25640127ple.251.2022.07.06.05.53.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Jul 2022 05:53:04 -0700 (PDT) From: Hsin-Yi Wang To: Robert Foss , Xin Ji Cc: Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Sam Ravnborg , Thomas Zimmermann , Maxime Ripard , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 1/4] drm/bridge: anx7625: Convert to devm_i2c_new_dummy_device() Date: Wed, 6 Jul 2022 20:52:51 +0800 Message-Id: <20220706125254.2474095-2-hsinyi@chromium.org> X-Mailer: git-send-email 2.37.0.rc0.161.g10f37bed90-goog In-Reply-To: <20220706125254.2474095-1-hsinyi@chromium.org> References: <20220706125254.2474095-1-hsinyi@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Simplify the resource management. Signed-off-by: Hsin-Yi Wang Reviewed-by: Xin Ji --- v1->v2: Fix indent. --- drivers/gpu/drm/bridge/analogix/anx7625.c | 96 +++++++---------------- 1 file changed, 27 insertions(+), 69 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/br= idge/analogix/anx7625.c index 3710fa9ee0acd..ab346d32d8735 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -2436,82 +2436,44 @@ static const struct drm_bridge_funcs anx7625_bridge= _funcs =3D { static int anx7625_register_i2c_dummy_clients(struct anx7625_data *ctx, struct i2c_client *client) { - int err =3D 0; + struct device *dev =3D &ctx->client->dev; =20 - ctx->i2c.tx_p0_client =3D i2c_new_dummy_device(client->adapter, - TX_P0_ADDR >> 1); + ctx->i2c.tx_p0_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + TX_P0_ADDR >> 1); if (IS_ERR(ctx->i2c.tx_p0_client)) return PTR_ERR(ctx->i2c.tx_p0_client); =20 - ctx->i2c.tx_p1_client =3D i2c_new_dummy_device(client->adapter, - TX_P1_ADDR >> 1); - if (IS_ERR(ctx->i2c.tx_p1_client)) { - err =3D PTR_ERR(ctx->i2c.tx_p1_client); - goto free_tx_p0; - } + ctx->i2c.tx_p1_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + TX_P1_ADDR >> 1); + if (IS_ERR(ctx->i2c.tx_p1_client)) + return PTR_ERR(ctx->i2c.tx_p1_client); =20 - ctx->i2c.tx_p2_client =3D i2c_new_dummy_device(client->adapter, - TX_P2_ADDR >> 1); - if (IS_ERR(ctx->i2c.tx_p2_client)) { - err =3D PTR_ERR(ctx->i2c.tx_p2_client); - goto free_tx_p1; - } + ctx->i2c.tx_p2_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + TX_P2_ADDR >> 1); + if (IS_ERR(ctx->i2c.tx_p2_client)) + return PTR_ERR(ctx->i2c.tx_p2_client); =20 - ctx->i2c.rx_p0_client =3D i2c_new_dummy_device(client->adapter, - RX_P0_ADDR >> 1); - if (IS_ERR(ctx->i2c.rx_p0_client)) { - err =3D PTR_ERR(ctx->i2c.rx_p0_client); - goto free_tx_p2; - } + ctx->i2c.rx_p0_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + RX_P0_ADDR >> 1); + if (IS_ERR(ctx->i2c.rx_p0_client)) + return PTR_ERR(ctx->i2c.rx_p0_client); =20 - ctx->i2c.rx_p1_client =3D i2c_new_dummy_device(client->adapter, - RX_P1_ADDR >> 1); - if (IS_ERR(ctx->i2c.rx_p1_client)) { - err =3D PTR_ERR(ctx->i2c.rx_p1_client); - goto free_rx_p0; - } + ctx->i2c.rx_p1_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + RX_P1_ADDR >> 1); + if (IS_ERR(ctx->i2c.rx_p1_client)) + return PTR_ERR(ctx->i2c.rx_p1_client); =20 - ctx->i2c.rx_p2_client =3D i2c_new_dummy_device(client->adapter, - RX_P2_ADDR >> 1); - if (IS_ERR(ctx->i2c.rx_p2_client)) { - err =3D PTR_ERR(ctx->i2c.rx_p2_client); - goto free_rx_p1; - } + ctx->i2c.rx_p2_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + RX_P2_ADDR >> 1); + if (IS_ERR(ctx->i2c.rx_p2_client)) + return PTR_ERR(ctx->i2c.rx_p2_client); =20 - ctx->i2c.tcpc_client =3D i2c_new_dummy_device(client->adapter, - TCPC_INTERFACE_ADDR >> 1); - if (IS_ERR(ctx->i2c.tcpc_client)) { - err =3D PTR_ERR(ctx->i2c.tcpc_client); - goto free_rx_p2; - } + ctx->i2c.tcpc_client =3D devm_i2c_new_dummy_device(dev, client->adapter, + TCPC_INTERFACE_ADDR >> 1); + if (IS_ERR(ctx->i2c.tcpc_client)) + return PTR_ERR(ctx->i2c.tcpc_client); =20 return 0; - -free_rx_p2: - i2c_unregister_device(ctx->i2c.rx_p2_client); -free_rx_p1: - i2c_unregister_device(ctx->i2c.rx_p1_client); -free_rx_p0: - i2c_unregister_device(ctx->i2c.rx_p0_client); -free_tx_p2: - i2c_unregister_device(ctx->i2c.tx_p2_client); -free_tx_p1: - i2c_unregister_device(ctx->i2c.tx_p1_client); -free_tx_p0: - i2c_unregister_device(ctx->i2c.tx_p0_client); - - return err; -} - -static void anx7625_unregister_i2c_dummy_clients(struct anx7625_data *ctx) -{ - i2c_unregister_device(ctx->i2c.tx_p0_client); - i2c_unregister_device(ctx->i2c.tx_p1_client); - i2c_unregister_device(ctx->i2c.tx_p2_client); - i2c_unregister_device(ctx->i2c.rx_p0_client); - i2c_unregister_device(ctx->i2c.rx_p1_client); - i2c_unregister_device(ctx->i2c.rx_p2_client); - i2c_unregister_device(ctx->i2c.tcpc_client); } =20 static int __maybe_unused anx7625_runtime_pm_suspend(struct device *dev) @@ -2723,8 +2685,6 @@ static int anx7625_i2c_probe(struct i2c_client *clien= t, if (!platform->pdata.low_power_mode) pm_runtime_put_sync_suspend(&client->dev); =20 - anx7625_unregister_i2c_dummy_clients(platform); - free_wq: if (platform->workqueue) destroy_workqueue(platform->workqueue); @@ -2754,8 +2714,6 @@ static int anx7625_i2c_remove(struct i2c_client *clie= nt) if (!platform->pdata.low_power_mode) pm_runtime_put_sync_suspend(&client->dev); =20 - anx7625_unregister_i2c_dummy_clients(platform); - if (platform->pdata.audio_en) anx7625_unregister_audio(platform); =20 --=20 2.37.0.rc0.161.g10f37bed90-goog From nobody Sun Apr 19 04:03:43 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B0DEBC43334 for ; Wed, 6 Jul 2022 12:53:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233777AbiGFMxT (ORCPT ); Wed, 6 Jul 2022 08:53:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54294 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233743AbiGFMxI (ORCPT ); Wed, 6 Jul 2022 08:53:08 -0400 Received: from mail-pj1-x1032.google.com (mail-pj1-x1032.google.com [IPv6:2607:f8b0:4864:20::1032]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 816F4237EF for ; Wed, 6 Jul 2022 05:53:07 -0700 (PDT) Received: by mail-pj1-x1032.google.com with SMTP id y14-20020a17090a644e00b001ef775f7118so11903476pjm.2 for ; Wed, 06 Jul 2022 05:53:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=02sx8dmnjBV/TuA75YAbyzwEMk2jxyWLuGZcJUXA1mA=; b=X9gPzsTtEqyMRnO0Ze3pIsRss45disf2+3R39oFMVTCGIQYT5ISLaMeO4Gh8LRckRx OERTPNsDduJXlF7SJsWM6NOCPOLaRNPDFhuL1h/nJ9YD5qa9n21qbyoY8vZic5K1hDcl VxrOjj8xpT3Tn2zhuDO+XCcn7MNtsKGW5+rZg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=02sx8dmnjBV/TuA75YAbyzwEMk2jxyWLuGZcJUXA1mA=; b=IYTgASuo7obDesS6IrvGs05CXIvp65G3ZxhAqbkDuqtIZV95rjlcqX4Z032aDOeMdd qsPM+grFP2smXQ/4uckAQciDqmPnMMighkUna5NsVIarWjRVfgBm2SYqdlShS+4P4g1L fohW5mVsvQ5yBiyjU4HQBH/I3hGE4RkNPCGq5EUkTB/hvlzwT+izQ04aitCdtsPev8O4 ZvBVuNg1ruYENh4jusuBXyE8JmxAmhphFrSoryCD+hehX8/MT4GhrncZ1JU6AqIur9UV 9+Ep6uD40db4LO65URwEEBInrSOn+zbLD0v/8VPLx+hlM68PaaTgH9AzRhVAwLo7LZC8 i4YA== X-Gm-Message-State: AJIora/rpDnVesJCd7ysuqI6DS/h7cjjUeU0wP52Z+plCWeJ/mAjY8yV V03U1cU4Ya/208xLRvTif6ulvQ== X-Google-Smtp-Source: AGRyM1tVtIQMvEuRrEN1V6YdOkUQGwBxQrSJYSlc5cAvGJXjazJWEXQKaWJoonujkv303bFau4bonQ== X-Received: by 2002:a17:90a:1a42:b0:1ef:680b:8ecd with SMTP id 2-20020a17090a1a4200b001ef680b8ecdmr30544374pjl.88.1657111986992; Wed, 06 Jul 2022 05:53:06 -0700 (PDT) Received: from hsinyi-z840.tpe.corp.google.com ([2401:fa00:1:10:5300:b974:1680:1bd]) by smtp.gmail.com with ESMTPSA id u12-20020a17090341cc00b0016a6cd546d6sm25640127ple.251.2022.07.06.05.53.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Jul 2022 05:53:06 -0700 (PDT) From: Hsin-Yi Wang To: Robert Foss , Xin Ji Cc: Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Sam Ravnborg , Thomas Zimmermann , Maxime Ripard , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 2/4] drm/bridge: anx7625: use pm_runtime_force_suspend(resume) Date: Wed, 6 Jul 2022 20:52:52 +0800 Message-Id: <20220706125254.2474095-3-hsinyi@chromium.org> X-Mailer: git-send-email 2.37.0.rc0.161.g10f37bed90-goog In-Reply-To: <20220706125254.2474095-1-hsinyi@chromium.org> References: <20220706125254.2474095-1-hsinyi@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" There's no need to check for IRQ or disable it in suspend. Use pm_runtime_force_suspend(resume) to make sure anx7625 is powered off correctly. Make the system suspend/resume and pm runtime suspend/resume more consistent. Signed-off-by: Hsin-Yi Wang Reviewed-by: Xin Ji --- v1->v2: Fix typo. --- drivers/gpu/drm/bridge/analogix/anx7625.c | 33 ++--------------------- 1 file changed, 2 insertions(+), 31 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/br= idge/analogix/anx7625.c index ab346d32d8735..fcf07a037c2c8 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -2504,38 +2504,9 @@ static int __maybe_unused anx7625_runtime_pm_resume(= struct device *dev) return 0; } =20 -static int __maybe_unused anx7625_resume(struct device *dev) -{ - struct anx7625_data *ctx =3D dev_get_drvdata(dev); - - if (!ctx->pdata.intp_irq) - return 0; - - if (!pm_runtime_enabled(dev) || !pm_runtime_suspended(dev)) { - enable_irq(ctx->pdata.intp_irq); - anx7625_runtime_pm_resume(dev); - } - - return 0; -} - -static int __maybe_unused anx7625_suspend(struct device *dev) -{ - struct anx7625_data *ctx =3D dev_get_drvdata(dev); - - if (!ctx->pdata.intp_irq) - return 0; - - if (!pm_runtime_enabled(dev) || !pm_runtime_suspended(dev)) { - anx7625_runtime_pm_suspend(dev); - disable_irq(ctx->pdata.intp_irq); - } - - return 0; -} - static const struct dev_pm_ops anx7625_pm_ops =3D { - SET_SYSTEM_SLEEP_PM_OPS(anx7625_suspend, anx7625_resume) + SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, + pm_runtime_force_resume) SET_RUNTIME_PM_OPS(anx7625_runtime_pm_suspend, anx7625_runtime_pm_resume, NULL) }; --=20 2.37.0.rc0.161.g10f37bed90-goog From nobody Sun Apr 19 04:03:43 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DAFFCC43334 for ; Wed, 6 Jul 2022 12:53:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233795AbiGFMxX (ORCPT ); Wed, 6 Jul 2022 08:53:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54432 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233755AbiGFMxN (ORCPT ); Wed, 6 Jul 2022 08:53:13 -0400 Received: from mail-pl1-x62b.google.com (mail-pl1-x62b.google.com [IPv6:2607:f8b0:4864:20::62b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1B95824F26 for ; Wed, 6 Jul 2022 05:53:10 -0700 (PDT) Received: by mail-pl1-x62b.google.com with SMTP id y18so5565575plb.2 for ; Wed, 06 Jul 2022 05:53:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=jts1Y9JMpVL0If8P8yimWr5loe4E0VIqYItcqhdJT04=; b=J5h7sC+wpA1ej0AY7GGFDGcLzlBfbS2jtjQAtEj3EooYLMSkvj+E/iH7Ihp5W/wj/U 5av2KGOQdc8rZFtYkRx2Ww0Ty5D2u9A2G7dZfNfbS7wy/za51jTXhMuKYfj+XqabLgs/ EtJmN4yrr1ag0rJ9mDGzrgjDPQDiRCkguqDAE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=jts1Y9JMpVL0If8P8yimWr5loe4E0VIqYItcqhdJT04=; b=m9sycG7hxzb4F5OzyOS6kgSK/lyt+PVnKSVOwsSF51lhPbP915C1bQDvA45isVsXjV W7eisjFOg/WLvMG7h4XyV+wro2GUL3lwegd6+bUaBCTOkuZr0C8Cj8pRY6C1Tfze4dg2 wgzmVyfZl4P5DciTu3Vp+NC44klJtuzAOiFmT7VeZ/AZsVhWrCJzK7hmJdAancYg8QtF NRPp+TAlzIYJBw8ifB8VQCXnFe7ZXPlL1fMKnh8spADyJiW8c7Ex6z2a7IebQOWnchQi fHMxY8ko/zbT6noKEWDGx31p64V4DqhlH6pCqRBFf4fz6Lqd1RTtfI2tTvcL4FmI78bl FvYQ== X-Gm-Message-State: AJIora+nK/wd1mMbAcI+LvaMZlna+XyuxallvcdqZVm/Z4h5PPbLRZjt 94gfLIP8vNLLmTCoj2+0v8Tshw== X-Google-Smtp-Source: AGRyM1t3rkNRA6Pzf7Rhejl5i6Yk/NJFrroHb1CZ1jTGwinOP7wTDoxpkRm6jtCxnL+OTuJWRwSxKw== X-Received: by 2002:a17:903:41d0:b0:16a:55e0:6c3d with SMTP id u16-20020a17090341d000b0016a55e06c3dmr47794830ple.21.1657111989582; Wed, 06 Jul 2022 05:53:09 -0700 (PDT) Received: from hsinyi-z840.tpe.corp.google.com ([2401:fa00:1:10:5300:b974:1680:1bd]) by smtp.gmail.com with ESMTPSA id u12-20020a17090341cc00b0016a6cd546d6sm25640127ple.251.2022.07.06.05.53.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Jul 2022 05:53:09 -0700 (PDT) From: Hsin-Yi Wang To: Robert Foss , Xin Ji Cc: Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Sam Ravnborg , Thomas Zimmermann , Maxime Ripard , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 3/4] drm/bridge: anx7625: Fix NULL pointer crash when using edp-panel Date: Wed, 6 Jul 2022 20:52:53 +0800 Message-Id: <20220706125254.2474095-4-hsinyi@chromium.org> X-Mailer: git-send-email 2.37.0.rc0.161.g10f37bed90-goog In-Reply-To: <20220706125254.2474095-1-hsinyi@chromium.org> References: <20220706125254.2474095-1-hsinyi@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Move devm_of_dp_aux_populate_ep_devices() after pm runtime and i2c setup to avoid NULL pointer crash. edp-panel probe (generic_edp_panel_probe) calls pm_runtime_get_sync() to read EDID. At this time, bridge should have pm runtime enabled and i2c clients ready. Fixes: adca62ec370c ("drm/bridge: anx7625: Support reading edid through aux= channel") Signed-off-by: Hsin-Yi Wang Reviewed-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/br= idge/analogix/anx7625.c index fcf07a037c2c8..aded20b9e25b1 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -2590,14 +2590,6 @@ static int anx7625_i2c_probe(struct i2c_client *clie= nt, platform->aux.dev =3D dev; platform->aux.transfer =3D anx7625_aux_transfer; drm_dp_aux_init(&platform->aux); - devm_of_dp_aux_populate_ep_devices(&platform->aux); - - ret =3D anx7625_parse_dt(dev, pdata); - if (ret) { - if (ret !=3D -EPROBE_DEFER) - DRM_DEV_ERROR(dev, "fail to parse DT : %d\n", ret); - goto free_wq; - } =20 if (anx7625_register_i2c_dummy_clients(platform, client) !=3D 0) { ret =3D -ENOMEM; @@ -2613,6 +2605,15 @@ static int anx7625_i2c_probe(struct i2c_client *clie= nt, if (ret) goto free_wq; =20 + devm_of_dp_aux_populate_ep_devices(&platform->aux); + + ret =3D anx7625_parse_dt(dev, pdata); + if (ret) { + if (ret !=3D -EPROBE_DEFER) + DRM_DEV_ERROR(dev, "fail to parse DT : %d\n", ret); + goto free_wq; + } + if (!platform->pdata.low_power_mode) { anx7625_disable_pd_protocol(platform); pm_runtime_get_sync(dev); --=20 2.37.0.rc0.161.g10f37bed90-goog From nobody Sun Apr 19 04:03:43 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2974C43334 for ; Wed, 6 Jul 2022 12:53:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233819AbiGFMx1 (ORCPT ); Wed, 6 Jul 2022 08:53:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54410 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233768AbiGFMxO (ORCPT ); Wed, 6 Jul 2022 08:53:14 -0400 Received: from mail-pl1-x635.google.com (mail-pl1-x635.google.com [IPv6:2607:f8b0:4864:20::635]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E9C79255A9 for ; Wed, 6 Jul 2022 05:53:12 -0700 (PDT) Received: by mail-pl1-x635.google.com with SMTP id 5so7695505plk.9 for ; Wed, 06 Jul 2022 05:53:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=vpU/47CIGfcMKZ4E2aqoCQ1BOqNCcamkTX7gRfUbreo=; b=N4/8r99A/+2PtNfA6zKC7zYa6daai8aymv0oo3OLKvNL1LFMgLq9MqhuNCRPZ3Orxk 9zvYKc6A5GGxS5wljKrCbsmwjwHl6kAnoYdQMXqFdMF7SXJW4lvoOmGOpHB6cWTmBgbP vttKZCao/XCM9jfG6jGi8vO8rWAAtlO5bajew= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=vpU/47CIGfcMKZ4E2aqoCQ1BOqNCcamkTX7gRfUbreo=; b=EmFY6ABkNsHpq67KkEGiiJ/E1FTlz9EVeEwm1qYmytSoqcDkM0RO0Z6TVFoCYaobRe IWpeokP6yCTUXJ0ZiM/8Lwow2S93oZG/cnsj0DirBQwXunkmTbDsczX2zbcuf9CA1s3Z GNeLtdfXwWQ95nbYerSp/ia830eu2arxlwFnjA/crkjTkadcIzZP/k9HENfiflMlSLAc UaCjSW95HwVvAkVGpCx8N38kuyR3tle6xQa/OQq28/BX+NoxFsMOV7+edbCb457uyB6J mWw3JKfeNS3OEiTwQR6Cr7NaoLmAdxPNgE0QoAsOy4AlhtXxC5oqzdBRaB9Lkiayllyi ju3Q== X-Gm-Message-State: AJIora+THBupUrg5MkSLmmdHbD2AzU94u5kH5zqZbvd27XfmLQkBe7dy ixGGg2j+Lva2tWJ/aNOFTfJsBw== X-Google-Smtp-Source: AGRyM1vPZy1bE7pCVFrwvF6JhtLpJ3e5q30yrQdoFZdI6S+SjkzAjaWd6jRhqaEDx5rYAAusjkJdSg== X-Received: by 2002:a17:902:d50d:b0:16b:ff69:35 with SMTP id b13-20020a170902d50d00b0016bff690035mr4257876plg.160.1657111992179; Wed, 06 Jul 2022 05:53:12 -0700 (PDT) Received: from hsinyi-z840.tpe.corp.google.com ([2401:fa00:1:10:5300:b974:1680:1bd]) by smtp.gmail.com with ESMTPSA id u12-20020a17090341cc00b0016a6cd546d6sm25640127ple.251.2022.07.06.05.53.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Jul 2022 05:53:11 -0700 (PDT) From: Hsin-Yi Wang To: Robert Foss , Xin Ji Cc: Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Sam Ravnborg , Thomas Zimmermann , Maxime Ripard , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 4/4] drm/bridge: anx7625: Add wait_hpd_asserted() callback Date: Wed, 6 Jul 2022 20:52:54 +0800 Message-Id: <20220706125254.2474095-5-hsinyi@chromium.org> X-Mailer: git-send-email 2.37.0.rc0.161.g10f37bed90-goog In-Reply-To: <20220706125254.2474095-1-hsinyi@chromium.org> References: <20220706125254.2474095-1-hsinyi@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Move hpd polling check into wait_hpd_asserted() callback. For the cases that aux transfer function wasn't used, do hpd polling check after pm runtime resume, which will power on the bridge. Signed-off-by: Hsin-Yi Wang Reviewed-by: Xin Ji --- v1->v2: fix indent. --- drivers/gpu/drm/bridge/analogix/anx7625.c | 33 ++++++++++++++++++----- 1 file changed, 27 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/br= idge/analogix/anx7625.c index aded20b9e25b1..d1f1d525aeb6d 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -1443,23 +1443,24 @@ static int anx7625_read_hpd_status_p0(struct anx762= 5_data *ctx) return anx7625_reg_read(ctx, ctx->i2c.rx_p0_client, SYSTEM_STSTUS); } =20 -static void anx7625_hpd_polling(struct anx7625_data *ctx) +static int _anx7625_hpd_polling(struct anx7625_data *ctx, + unsigned long wait_us) { int ret, val; struct device *dev =3D &ctx->client->dev; =20 /* Interrupt mode, no need poll HPD status, just return */ if (ctx->pdata.intp_irq) - return; + return 0; =20 ret =3D readx_poll_timeout(anx7625_read_hpd_status_p0, ctx, val, ((val & HPD_STATUS) || (val < 0)), - 5000, - 5000 * 100); + wait_us / 100, + wait_us); if (ret) { DRM_DEV_ERROR(dev, "no hpd.\n"); - return; + return ret; } =20 DRM_DEV_DEBUG_DRIVER(dev, "system status: 0x%x. HPD raise up.\n", val); @@ -1472,6 +1473,23 @@ static void anx7625_hpd_polling(struct anx7625_data = *ctx) =20 if (!ctx->pdata.panel_bridge && ctx->bridge_attached) drm_helper_hpd_irq_event(ctx->bridge.dev); + + return 0; +} + +static int anx7625_wait_hpd_asserted(struct drm_dp_aux *aux, + unsigned long wait_us) +{ + struct anx7625_data *ctx =3D container_of(aux, struct anx7625_data, aux); + struct device *dev =3D &ctx->client->dev; + int ret; + + pm_runtime_get_sync(dev); + ret =3D _anx7625_hpd_polling(ctx, wait_us); + pm_runtime_mark_last_busy(dev); + pm_runtime_put_autosuspend(dev); + + return ret; } =20 static void anx7625_remove_edid(struct anx7625_data *ctx) @@ -1741,6 +1759,7 @@ static struct edid *anx7625_get_edid(struct anx7625_d= ata *ctx) } =20 pm_runtime_get_sync(dev); + _anx7625_hpd_polling(ctx, 5000 * 100); edid_num =3D sp_tx_edid_read(ctx, p_edid->edid_raw_data); pm_runtime_put_sync(dev); =20 @@ -2378,6 +2397,7 @@ static void anx7625_bridge_atomic_enable(struct drm_b= ridge *bridge, ctx->connector =3D connector; =20 pm_runtime_get_sync(dev); + _anx7625_hpd_polling(ctx, 5000 * 100); =20 anx7625_dp_start(ctx); } @@ -2497,7 +2517,6 @@ static int __maybe_unused anx7625_runtime_pm_resume(s= truct device *dev) mutex_lock(&ctx->lock); =20 anx7625_power_on_init(ctx); - anx7625_hpd_polling(ctx); =20 mutex_unlock(&ctx->lock); =20 @@ -2589,6 +2608,7 @@ static int anx7625_i2c_probe(struct i2c_client *clien= t, platform->aux.name =3D "anx7625-aux"; platform->aux.dev =3D dev; platform->aux.transfer =3D anx7625_aux_transfer; + platform->aux.wait_hpd_asserted =3D anx7625_wait_hpd_asserted; drm_dp_aux_init(&platform->aux); =20 if (anx7625_register_i2c_dummy_clients(platform, client) !=3D 0) { @@ -2617,6 +2637,7 @@ static int anx7625_i2c_probe(struct i2c_client *clien= t, if (!platform->pdata.low_power_mode) { anx7625_disable_pd_protocol(platform); pm_runtime_get_sync(dev); + _anx7625_hpd_polling(platform, 5000 * 100); } =20 /* Add work function */ --=20 2.37.0.rc0.161.g10f37bed90-goog