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 27302C32774 for ; Tue, 23 Aug 2022 19:50:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231845AbiHWTu6 (ORCPT ); Tue, 23 Aug 2022 15:50:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51312 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233776AbiHWTum (ORCPT ); Tue, 23 Aug 2022 15:50:42 -0400 Received: from mail-pj1-x1029.google.com (mail-pj1-x1029.google.com [IPv6:2607:f8b0:4864:20::1029]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 188E328E1C for ; Tue, 23 Aug 2022 11:54:26 -0700 (PDT) Received: by mail-pj1-x1029.google.com with SMTP id o14-20020a17090a0a0e00b001fabfd3369cso15603045pjo.5 for ; Tue, 23 Aug 2022 11:54:26 -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=+hvao7pptwOr3qbqsTMhQzQ4Xux7plVKmyAIQ9qQXrw=; b=P31Z96otplQWfiXMsyQ/5+9zDtfWGPOoxwYQVgAmjdOPKi5+N3Yl6yiYRuVALCIZTS U3narRlWKJICIoz4l+QO5SqXhRr6DJVwLiiyKvFvM15aEePqCKgDqYICNl3RokUQS7yI Wvtb8JJcUIiMHh4Noulw2zl0jHMTB+zv6NthU= 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=+hvao7pptwOr3qbqsTMhQzQ4Xux7plVKmyAIQ9qQXrw=; b=upHfiIPF4QKg0P/oCGmw3taTW8Q5WtmVnI32kvYGY/t6VTOWsHaQIIh3GcBz9FLIk0 iCSN66Fyk7/ODhHxRVOFed5hvfeuWxkY4rT7OW/Ju5+JLfFd5cXUC9AMWHIsAG7gWrLH ETw4OdorSpQB8YAwgB4dTCQmkcVY1kqaIwLaSPHaREZ3BiFJdeZms+0pSZ8Y+rNqAn4K DrGKql3O1bmn/OrXzcrD1HN0M3TkcxfVGqkZpOkQMzdZVwxo2n4RyOEZjfyT3naaWaKq R+fmjtYO4pCZxnt4focgB8KtqIdkSsCv5Hr4IVGnmKdzQZPAwQk1aPCJZC4W7/6RjdFo 8q/g== X-Gm-Message-State: ACgBeo1aFxeoPhhHk33fzQx1P+DJ0VdWBZSXqVJAJ8XQDAEHuRYZ81nA YOkcD4F2CAfcfjCrPzKx774fRA== X-Google-Smtp-Source: AA6agR5vpr/23o9DFw09xOhWtsgRNBmIf+1QydM/1X3j+XxPOUinOav23kUN0DUJEFa2Gtv9sVKKvA== X-Received: by 2002:a17:90b:1b42:b0:1fb:7a59:8965 with SMTP id nv2-20020a17090b1b4200b001fb7a598965mr682315pjb.140.1661280865580; Tue, 23 Aug 2022 11:54:25 -0700 (PDT) Received: from localhost ([2620:15c:11a:202:fee6:a961:5d1d:553]) by smtp.gmail.com with UTF8SMTPSA id e14-20020a17090301ce00b00171311470eesm10943906plh.8.2022.08.23.11.54.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 23 Aug 2022 11:54:25 -0700 (PDT) Date: Tue, 23 Aug 2022 11:54:23 -0700 From: Matthias Kaehlcke To: Johan Hovold Cc: Johan Hovold , andersson@kernel.org, Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Andy Gross , Manivannan Sadhasivam , Konrad Dybcio , Sandeep Maheswaram , Krishna Kurapati , Pavankumar Kondeti , quic_ppratap@quicinc.com, quic_vpulyala@quicinc.com, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: dts: qcom: sc7280: move USB wakeup-source property Message-ID: References: <20220802152642.2516-1-johan+linaro@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Tue, Aug 23, 2022 at 01:35:57PM +0200, Johan Hovold wrote: > On Mon, Aug 22, 2022 at 03:31:21PM -0700, Matthias Kaehlcke wrote: > > On Mon, Aug 22, 2022 at 03:13:54PM +0200, Johan Hovold wrote: > > > On Tue, Aug 02, 2022 at 05:26:42PM +0200, Johan Hovold wrote: > > > > Move the USB-controller wakeup-source property to the dwc3 glue node to > > > > match the updated binding. > > > > > > > > Signed-off-by: Johan Hovold > > > > --- > > > > > > > > This one can be applied once the following series has been merged: > > > > > > > > https://lore.kernel.org/all/20220802151404.1797-1-johan+linaro@kernel.org > > > > > > The above series has now been merged (for 6.0): > > > > > > https://lore.kernel.org/all/Yv56fFpuUsxCSxJ8@kroah.com/ > > > > > > so that this patch can be applied. > > > > Please apply it together with "clk: qcom: gcc-sc7280: Keep USB GDSC power > > domains on when USB wakeup is enabled" [1], otherwise USB wakeup won't work, > > and worse, USB would be broken after returning from system suspend. > > If you really need [1] for wakeup to work then it's already broken as > the hack added to 6.0-rc1 that kept the power domain on in suspend has > been reverted. Yep, in v6.0-rc1 it is already broken, it should still work in the current qcom tree. In any case wakeup isn't the primary concern, wakeup support for sc7x80 just landed in a not-so-great shape, and your patches in v6.0-rc generally improve it. > If [1] is also needed for USB to work after resume, we either have a > bigger problem as I alluded to in my comment to [1] (and the PD needs to > be always on) or this is due to the PHY no longer being powered down in > suspend. Yes, it is apparently related with the PHYs no longer being powered down in suspend. With your patch that reverts that [1] wakeup still works (as long as wakeup for the dwc3 core remains enabled) and USB is operational after resume. So as long as the PHYs are powered down in suspend it shouldn't be necessary to keep the GDSC PDs on. For Chrome OS we'll have to evaluate whether that is an option (we observe high power consumption of an onboard USB hub during system suspend when the PHYs are off) or whether sone mechanism (quirk, kconfig option, ...) is needed to keep the PHYs on. [1] https://patchwork.kernel.org/project/linux-usb/patch/20220823124047.14634-1-johan+linaro@kernel.org/ > Only in the latter case, does this patch need to be held of until [1] > has been merged AFAICT. > > Johan > > > [1] https://patchwork.kernel.org/project/linux-arm-msm/patch/20220822115246.2.If09027f73daa6e1ed95f5eab02326b543c67132e@changeid/