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 04741C19F2D for ; Wed, 10 Aug 2022 00:25:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229904AbiHJAZU (ORCPT ); Tue, 9 Aug 2022 20:25:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36406 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229476AbiHJAZS (ORCPT ); Tue, 9 Aug 2022 20:25:18 -0400 Received: from mail-yw1-x112d.google.com (mail-yw1-x112d.google.com [IPv6:2607:f8b0:4864:20::112d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 21F16625F for ; Tue, 9 Aug 2022 17:25:14 -0700 (PDT) Received: by mail-yw1-x112d.google.com with SMTP id 00721157ae682-32194238c77so127658297b3.4 for ; Tue, 09 Aug 2022 17:25:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc; bh=1VUI2HFiheDo58arrnc79/Qbxj5WTqZQ2QhX4iv40Nk=; b=ABPWt08krXKfSXeDcebcG/m72KIW6vO/4ih2HdtXW6Xcflaa31kKaLJUzYJyA+hjN5 /hos42tL2v9m84JqszthxP3dn6c/aGKTK66PuFX4Z0fjxJzOK5uctw+SStmmWvgWafd7 MKbDZ8Sg0XjW/oLYmkjK93RHOUprdKYO8gm7b5y2cWYfPwkQcxfkl1aJQz7u3MOHnZCH rdFJVgtMvEFznU1m+kpmWML4RW6xlptVIsorWBXGkpapFj7ucqQosGltArtrkMjUMvr8 bzpYAIDT8iGLrKJTU7i0DiQ4ZrsFOu903DZ2B9J+RQpVsmlU48hAaJ4q+isVtpHsUKog 1ADA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc; bh=1VUI2HFiheDo58arrnc79/Qbxj5WTqZQ2QhX4iv40Nk=; b=a7yn+Bm0bh0ayVOz62JG7pT1dlH/89u6CQb243I2DP6AdJ+mckNBwMF6b5v7hxZkc1 D1MwNSjIO75a/ruzk34moKEdypnWSHeOuAjYLbje/O9DKZgWHl6lCqlMIO4I67rzDprS TStfLEsbz2njI6lEjLvtoXCSDoaDaieV9K3N0nAMR/s9RBKQTLVD+NIc9ahgbVV6qpiC /zhRh6vDK9ZqmdANQhvjj+jqztmknkaZ6GZ+X2Nax/r64YPmzmzs1iHF54J/qZy06HlB clfE0ul/cKtELrZZ/GjxoNSSc+SM1o+fQ23jGPtyQEYvvAxXcyINgjRXZSGboxX5Sz+3 HNvQ== X-Gm-Message-State: ACgBeo0o/Bw7KgeanwjiQ6ImItB7aAF88PhrwW15qqsSOI+AIw5R7+dY Cje6bf9H63/K67qg++op9i2T39lKXLVh0vxjGFxkA6iTnmQ= X-Google-Smtp-Source: AA6agR6ldAkTAJtS3AzePk06qtNOc1jE0hT59EJbzuTuMPqiGKi5H6FdfLEZjbfEkE01KyVeWy2EIeHuNtHBK05yhwk= X-Received: by 2002:a0d:fd05:0:b0:329:3836:53ac with SMTP id n5-20020a0dfd05000000b00329383653acmr23670331ywf.455.1660091113143; Tue, 09 Aug 2022 17:25:13 -0700 (PDT) MIME-Version: 1.0 References: <20220727185012.3255200-1-saravanak@google.com> In-Reply-To: From: Saravana Kannan Date: Tue, 9 Aug 2022 17:24:37 -0700 Message-ID: Subject: Re: [PATCH v1 0/3] Bring back driver_deferred_probe_check_state() for now To: Doug Anderson Cc: Greg Kroah-Hartman , "Rafael J. Wysocki" , Kevin Hilman , Ulf Hansson , Pavel Machek , Len Brown , Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Naresh Kamboju , kernel-team@android.com, LKML , Linux PM , netdev , Stephen Boyd Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 9, 2022 at 4:47 PM Doug Anderson wrote: > > Hi, > > On Thu, Jul 28, 2022 at 12:29 AM Greg Kroah-Hartman > wrote: > > > > On Wed, Jul 27, 2022 at 11:50:08AM -0700, Saravana Kannan wrote: > > > More fixes/changes are needed before driver_deferred_probe_check_state() > > > can be deleted. So, bring it back for now. > > > > > > Greg, > > > > > > Can we get this into 5.19? If not, it might not be worth picking up this > > > series. I could just do the other/more fixes in time for 5.20. > > > > Wow, no, it is _WAY_ too late for 5.19 to make a change like this, > > sorry. > > > > What is so broken that we need to revert these now? I could do so for > > 5.20-rc1, and then backport to 5.19.y if that release is really broken, > > but this feels odd so late in the cycle. Greg, I didn't realize the patches I'm trying to revert never landed on 5.19. So you can ignore this thread. > > I spent a bunch of time bisecting mainline today on my > sc7180-trogdor-lazor board. When building the top of Linus's tree > today the display doesn't come up. I can make it come up by turning > fw_devlink off (after fixing a regulator bug that I just posted a fix > for). > > I found that the first bad commit was commit 5a46079a9645 ("PM: > domains: Delete usage of driver_deferred_probe_check_state()") > > ...but only when applied to mainline. When I cherry-pick that back to > v5.19-rc1 (and pick another bugfix needed to boot my board against > v5.19-rc1) then it works OK. After yet more bisecting, I found that on > trogdor there's a bad interaction with the commit e511a760 ("arm64: > dts: qcom: sm7180: remove assigned-clock-rate property for mdp clk"). > That commit is perfectly legit but I guess it somehow changed how > fw_devlink was interpreting things? > > Sure enough, picking this revert series fixes things on Linus's tree. > Any chance we can still get the revert in for v5.20-rc1? ;-) I guess it's 6.0 now. But I'm almost done with my actual fixes that rewrite some parts of fw_devlink to make it a lot more robust. So, I'm not sure if all these reverts need to land anymore. I'm hoping to send out the proper fixes by the end of this week. Maybe you can try that out and let me know if it solves your issues (I expect it to). I'm surprised that specific clock patch has an impact though. It's just touching properties that fw_devlink doesn't parse. -Saravana