Received: by 2002:a05:6358:c692:b0:131:369:b2a3 with SMTP id fe18csp5537474rwb; Tue, 1 Aug 2023 04:25:18 -0700 (PDT) X-Google-Smtp-Source: APBJJlEVKugyHgQWp5aKMWF4XeKS4E/7U+V+TBC/xnd3nxXwzroLJPnhZvJe+JWCrfC54JrjTVDs X-Received: by 2002:a19:ca0c:0:b0:4fe:2d93:2b50 with SMTP id a12-20020a19ca0c000000b004fe2d932b50mr1667336lfg.31.1690889118353; Tue, 01 Aug 2023 04:25:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690889118; cv=none; d=google.com; s=arc-20160816; b=mu5Y7RVvL9iPrhUaB7NR411jbUErNcdRaK2Wzcy2ozlynuX2dU9MzewbLliv0sleHN k0mP0NKz13Wl3ABsF9/KxOHG4Ibmz1I9wZiZBzM7XERO9aZo+nX2MV7uie+omE62J2pw 8SIQigFExFV3oPh/Tbepb7l9HctkSDtklBP1xC2yzlnjDknmWZH3xlElGr0+7vlD831K SbqMgAGWpwDxk1HYp4Pz80oYyyN/e3EI0Cat1H/Fbe59LeZ1VeS0bwukX2x8fIKb/AOt ycAuey2MfaXy+wJYAfVLc/f1qP7k7ra71f5dsXXEtKBznv8vFP2tj58jeAsDEB6MuyiL ZCEQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:references:cc:to:content-language:subject:reply-to :from:user-agent:mime-version:date:message-id:dkim-signature; bh=KPQo2Sh4IXaPrcdWyouCsvWXkycrqlPg24jNznzZZU8=; fh=+z5MvsYY7qJHrcwvc3XOdqChOie9P+PXtoLEP36bZDI=; b=Q3H6FMdBgdJBlvv/oIJboDj5v+pcUw2BlSVWlQ/wEWsn5lnmp4C2gKXQdNzEbJHJ66 +VLSyus5M31K9xAcxPQ9qHTa95tmE3dRL/xVVUhgMiNfzsqSxU9CMkp4L50oxgE7nvjD WgN+C2p3TQu3GMHagpdgf1rBNsKYTPDaydENgHI3gdNgF0nYtSAlKov50CwJtH0wSdYn BCtiz5Py6+Zcg63pVlQyjHgjAR4evsmlQqR2ioaBp6wAQqSududYQcXPWFWO18ITxM8f DBG3xlhXdOoSqDEiU7OKIl2CUAMdM4VSnRQoJviJ+c5uAG7zPBIo5NiqcwQvaSkhxeXt VksA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xPrZ0QsT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u12-20020aa7d98c000000b0051df544225csi3878689eds.540.2023.08.01.04.24.53; Tue, 01 Aug 2023 04:25:18 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xPrZ0QsT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233219AbjHAJkW (ORCPT + 99 others); Tue, 1 Aug 2023 05:40:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233303AbjHAJjy (ORCPT ); Tue, 1 Aug 2023 05:39:54 -0400 Received: from mail-wr1-x42b.google.com (mail-wr1-x42b.google.com [IPv6:2a00:1450:4864:20::42b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 264021996 for ; Tue, 1 Aug 2023 02:37:41 -0700 (PDT) Received: by mail-wr1-x42b.google.com with SMTP id ffacd0b85a97d-31765792c7cso5679085f8f.0 for ; Tue, 01 Aug 2023 02:37:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1690882659; x=1691487459; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=KPQo2Sh4IXaPrcdWyouCsvWXkycrqlPg24jNznzZZU8=; b=xPrZ0QsT+UCwbOrRmQ1iTIgW5yLAxZGGQqiow0SZZkONajlepMLeQGKf+pUxiUVpni KPaXzaJpC85Lp8y3d1XsgoRj9+8WuvMQV9q1yMgrqGsFjlg8ivD+Pl3Q97MZQKakSd23 swWMMlKChNDlsbz8BTs96LibCclG20GI8kLt793fFNTbVeHQKp+Y6vq/rigZ92gZwktV jdnOJU7RmWicqqlovnGi2z5Nvcom6MPsZrFYRGgsKfFFrEWVkHuvm9veLdSZaJqvyWzG LnP3Yyc8mzEObLF17JiS/jyqK+RwPdm4zVmxzvoZ5ws04ay7y9PN0H38KB3dNKkoQfLF vt6w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690882659; x=1691487459; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=KPQo2Sh4IXaPrcdWyouCsvWXkycrqlPg24jNznzZZU8=; b=L+Y5b04yJOkSTXB72UkMU6Co1o24jbwYmbZtiJI/Is/FL5BJdpPgnRM5w392kc61xI Ek7IwVrTUa6aw3Sk82MI7xnHrrgRre5JyopBSB+PpVsLNQQt8DvIeUC7hr+XqqrAswih C8CyaFDOyyY3D3olFEG+qSwTOkMAqesv+cNdlNGkHXixBdrKgs54WVSVI6KMMnS1L95T f8eFelmYqUE+s4CmOFeFhVdc7CWPaS8P8A/XkEeiorxARZh+2DVfizlMXDfJ5B3ivued kyBw77f2eDz92FvGR3d2SN1lEifyU7ZgL2Erxcf9Kb4dXSHjQON+032Xhprq/fgTeKJ8 YJBw== X-Gm-Message-State: ABy/qLb4BGtsO7HD/8svunCR1JiF4qj/bUOsh3p4r8UJAd976GH1jF/P 3ZT9ZHf4GP8v0B8Ry7mLBgyz/w== X-Received: by 2002:adf:ef8c:0:b0:317:62c3:13a7 with SMTP id d12-20020adfef8c000000b0031762c313a7mr1916263wro.19.1690882659482; Tue, 01 Aug 2023 02:37:39 -0700 (PDT) Received: from ?IPV6:2a01:e0a:982:cbb0:b92a:81a9:df6e:1e3? ([2a01:e0a:982:cbb0:b92a:81a9:df6e:1e3]) by smtp.gmail.com with ESMTPSA id n5-20020a5d6b85000000b0031455482d1fsm1771236wrx.47.2023.08.01.02.37.38 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 01 Aug 2023 02:37:39 -0700 (PDT) Message-ID: Date: Tue, 1 Aug 2023 11:37:38 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 From: Neil Armstrong Reply-To: neil.armstrong@linaro.org Subject: Re: [PATCH] drm/bridge: it6505: Check power state with it6505->powered in IRQ handler Content-Language: en-US To: Pin-yen Lin , Andrzej Hajda , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org References: <20230727100131.2338127-1-treapking@chromium.org> Organization: Linaro Developer Services In-Reply-To: <20230727100131.2338127-1-treapking@chromium.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 27/07/2023 12:01, Pin-yen Lin wrote: > On system resume, the driver might call it6505_poweron directly if the > runtime PM hasn't been enabled. In such case, pm_runtime_get_if_in_use > will always return 0 because dev->power.runtime_status stays at > RPM_SUSPENDED, and the IRQ will never be handled. > > Use it6505->powered from the driver struct fixes this because it always > gets updated when it6505_poweron is called. > > Fixes: 5eb9a4314053 ("drm/bridge: it6505: Guard bridge power in IRQ handler") > Signed-off-by: Pin-yen Lin > > --- > > drivers/gpu/drm/bridge/ite-it6505.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c > index 6c2fcd8b8780..2f300f5ca051 100644 > --- a/drivers/gpu/drm/bridge/ite-it6505.c > +++ b/drivers/gpu/drm/bridge/ite-it6505.c > @@ -2517,9 +2517,11 @@ static irqreturn_t it6505_int_threaded_handler(int unused, void *data) > }; > int int_status[3], i; > > - if (it6505->enable_drv_hold || pm_runtime_get_if_in_use(dev) <= 0) > + if (it6505->enable_drv_hold || !it6505->powered) > return IRQ_HANDLED; > > + pm_runtime_get_sync(dev); > + > int_status[0] = it6505_read(it6505, INT_STATUS_01); > int_status[1] = it6505_read(it6505, INT_STATUS_02); > int_status[2] = it6505_read(it6505, INT_STATUS_03); Reviewed-by: Neil Armstrong