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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E1B03ECAAD4 for ; Fri, 26 Aug 2022 13:05:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232994AbiHZNFc (ORCPT ); Fri, 26 Aug 2022 09:05:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36132 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344022AbiHZNFY (ORCPT ); Fri, 26 Aug 2022 09:05:24 -0400 Received: from mail-pf1-x42e.google.com (mail-pf1-x42e.google.com [IPv6:2607:f8b0:4864:20::42e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 36B5220193 for ; Fri, 26 Aug 2022 06:05:22 -0700 (PDT) Received: by mail-pf1-x42e.google.com with SMTP id g19so1530252pfb.0 for ; Fri, 26 Aug 2022 06:05:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc; bh=CHTCunlpvy9js5sdVWAAlZTqUYUZ7Iaa0wqOedrNwHI=; b=h3HcFtjP7/AnVHUZVv1r9YFQtSPge8mn4LlAXrSafWS5Ym1Y0N0FWtnK715/Ty5FNp pQquKsGavtHhFQQEOpmxzZHkFlDUxkZBI63wz71Nb/uGvSzDZk/fGsPSv7krJWyXzV27 fpUOmVpwZTUZayOZ1dez3L0at8LJzcHKOPyd4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=CHTCunlpvy9js5sdVWAAlZTqUYUZ7Iaa0wqOedrNwHI=; b=bQVQ/K8WTVc3QMK664bYUcufbm68BIrn0U9KZCsmKE7fvkKt4zoR4G1QFYwEQCmzqp Kqll47P9WUwKwaZD8o6os5W5rlNDbpoCqIBFxor1RWE5v/Lchj+x2TZKA7lC3xhpJTp+ mM6lCMVUHvNpwxMsiLTihvLDIjIqFkun5GeURVSUr5LBrDoZUtQx4fh6142UcZeKoaEO Xx0EUTJnuMmwJLky8ldT57A2UN0RNh7fNm2ygjc1j7SpshRysK0nMCI/dw2AkgWbX/Hh ELhkl5yK/q6+5fqK5OFgAIYebMMOzA61jhhvuexQ20OYOrqUA1XEKzl3KNK8HjCFj1GY EViA== X-Gm-Message-State: ACgBeo2i2+ENwWV1K6bOpZw78pubhY4Z1efH4KCkne9uulPdu4xSkK5Q sq7h14KuVJNZoLByuOShHCL/hw== X-Google-Smtp-Source: AA6agR6QMXVT+Is/zY+vj5eb09KzV5ldD7r7frhK93V8Wk7JvJLzRo44KiYFalLZ5j50lg3iDeTnEw== X-Received: by 2002:a63:6b05:0:b0:41c:2216:d2b2 with SMTP id g5-20020a636b05000000b0041c2216d2b2mr3300925pgc.118.1661519121548; Fri, 26 Aug 2022 06:05:21 -0700 (PDT) Received: from localhost ([2620:15c:11a:202:2a3c:773f:ac25:a127]) by smtp.gmail.com with UTF8SMTPSA id o9-20020a17090a4b4900b001fbb0f0b00fsm1570275pjl.35.2022.08.26.06.05.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 26 Aug 2022 06:05:21 -0700 (PDT) Date: Fri, 26 Aug 2022 06:05:19 -0700 From: Matthias Kaehlcke To: Johan Hovold Cc: Andy Gross , Bjorn Andersson , Konrad Dybcio , Michael Turquette , Stephen Boyd , linux-arm-msm@vger.kernel.org, Johan Hovold , Douglas Anderson , Krishna Kurapati , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, Bjorn Andersson Subject: Re: [PATCH 2/2] clk: qcom: gcc-sc7280: Keep USB GDSC power domains on when USB wakeup is enabled Message-ID: References: <20220822115246.1.I45235b7c40997bc2abf813e4722b4dcdd6aecf6b@changeid> <20220822115246.2.If09027f73daa6e1ed95f5eab02326b543c67132e@changeid> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Thu, Aug 25, 2022 at 03:16:58PM -0700, Matthias Kaehlcke wrote: > On Wed, Aug 24, 2022 at 10:47:44AM +0200, Johan Hovold wrote: > > On Tue, Aug 23, 2022 at 09:42:53AM -0700, Matthias Kaehlcke wrote: > > > On Tue, Aug 23, 2022 at 01:16:36PM +0200, Johan Hovold wrote: > > > > On Mon, Aug 22, 2022 at 11:53:11AM -0700, Matthias Kaehlcke wrote: > > > > > Set GENPD_FLAG_ACTIVE_WAKEUP for the USB GDSC power domains of SC7280. > > > > > > > > > > Suggested-by: Johan Hovold > > > > > Signed-off-by: Matthias Kaehlcke > > > > > --- > > > > > > > > > > drivers/clk/qcom/gcc-sc7280.c | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/drivers/clk/qcom/gcc-sc7280.c b/drivers/clk/qcom/gcc-sc7280.c > > > > > index 7ff64d4d5920..4ff855269467 100644 > > > > > --- a/drivers/clk/qcom/gcc-sc7280.c > > > > > +++ b/drivers/clk/qcom/gcc-sc7280.c > > > > > @@ -3125,6 +3125,7 @@ static struct gdsc gcc_usb30_prim_gdsc = { > > > > > .gdscr = 0xf004, > > > > > .pd = { > > > > > .name = "gcc_usb30_prim_gdsc", > > > > > + .flags = GENPD_FLAG_ACTIVE_WAKEUP, > > > > > > > > Have you verified that the power-domain doesn't need to remain on also > > > > when USB isn't used for wakeup? > > > > > > So far I haven't observed issues with this on sc7180 and sc7280 when USB > > > wakeup is disabled. > > > > Ok, good. > > > > I would have assumed that it needed to stay always-on before the > > s/device_can_wakeup/device_may_wakeup/ change as before that the PHYs > > would be left on regardless of the (sysfs) wakeup setting. > > That make sense, not sure why GENPD_FLAG_ACTIVE_WAKEUP works with wakeup > disabled. > > In any case setting the genpd flags directly in the gdsc descriptor doesn't > seem to be the right thing to do. With GENPD_FLAG_ALWAYS_ON my sc7280 system > stalls at boot. It boots when ALWAYS_ON is set in the gdsc flags, which ends > up setting GENPD_FLAG_ALWAYS_ON of the genpd. I'll send a new version of this > series which sets the correct flag. > > > > > This is the case for sc8280xp and indicates that there are further > > > > missing pieces here (at least for that platform). > > > > > > What are you observing on sc8280xp when wakeup is disabled? > > > > The wakeup setting doesn't seem to have anything to do with the genpd > > issues on sc8280xp and the controller doesn't resume properly regardless > > of whether the PHYs have been disabled or not during suspend unless the > > PD is left on. > > I'm essentially seeing the same. USB is hosed after resume unless the PD > is left on. > > On Chrome OS we currently work around that with a version of commit > d9be8d5c5b03 ("usb: dwc3: qcom: Keep power domain on to retain controller > status") which was reverted upstream. I'm not sure whether USB worked after > resume before we enabled wakeup support. I would have sworn it did, but we > landed an old version of the wakeup patches a long time ago, so my > memory might be failing me. I need to remind myself that keeping the GDSC on is only needed when the PHYs are kept on. The PHYs were always off before wakeup support was added, which is why USB wasn't broken after suspend in the pre-wakeup days. > I wonder what the status on other QC platforms is, from a quick grep it > seems only msm8953 sets the USB gdsc to ALWAYS_ON.