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 X-Spam-Level: X-Spam-Status: No, score=-18.3 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS, USER_IN_DEF_DKIM_WL autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AE794C433E9 for ; Thu, 14 Jan 2021 16:42:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 84E6E23B1C for ; Thu, 14 Jan 2021 16:42:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726602AbhANQlx (ORCPT ); Thu, 14 Jan 2021 11:41:53 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37052 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726558AbhANQlx (ORCPT ); Thu, 14 Jan 2021 11:41:53 -0500 Received: from mail-yb1-xb2d.google.com (mail-yb1-xb2d.google.com [IPv6:2607:f8b0:4864:20::b2d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3F76DC0613C1 for ; Thu, 14 Jan 2021 08:41:13 -0800 (PST) Received: by mail-yb1-xb2d.google.com with SMTP id w127so2347061ybw.8 for ; Thu, 14 Jan 2021 08:41:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=D+BmvCFWRLkA7Rq+8+HAffylejgMTO0t+Xl+DVgoXxA=; b=SuvR1Xho7USYBOUKOMwurqVjLwKuMIaUDHpECDeE7kkjOAAbEr8VbtJ90iTnt5fM/5 X8P6lvkesGPjFdWNPstWUpRkiNcqXNFz4JbSVFlTnJz/A9JHLYLgaZw3mpV81Yrl6L9Y +frW1Zt19QP0uhpPthKlLqefHvP2MZd5lMJI8T8T9R7gxf3C5xpaL96Ii+MNISpX9BLe XoZrm8Rv3BeNOHHmzam7W/pi9IteR0Wi5oOIXtcBH5u+Udy+0jQmtZduddyWef3VIX+S o0v+WVgwawULGlJNr+Vp4L9BqpGkVFQbVvKZueA9p8nIhcQ/qS7piHOe7/OCH9Y7r3wS v86Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=D+BmvCFWRLkA7Rq+8+HAffylejgMTO0t+Xl+DVgoXxA=; b=JjWBgNMVueWB6bO6iJ9Cr7aS/KJuikKEjVVcfyYXuOcBHPj5U5bEb8e4yhK7rmOLQP Z4tFBuWBXi53FhuKBKnuQ9Z/qbckqCEipqdlhPILO3fc2SMNAF1y4XSBsuuJMYDnT2qR IgeXMq9uIH/kUwy+P9lQkJgO6bjqixXy7rjHOCjDQspHsQzFWcofubdKOfgut2BAsbDy yYnBleflUwlyfyhJ/IQ8eid7md3Nwqazywpem0Djij2lLI5gHiqiJdAqMtS/RnP16Gj2 ZZJoFZ2Oq6WnU1Sl44U9K8LwUhA2JOsz4yoxWBk9ZZBupvCuTm13rgH3C5Z7G6ihs0He tyuA== X-Gm-Message-State: AOAM533BCjVSUTbECygxtNSU18KxUxZxf2lQsPFIlxYlk9MiMbMImqnc d4tnLc/8TpUn2H6VytwFam/MRthYoBqqjA8Vd1xruQ== X-Google-Smtp-Source: ABdhPJzcBka6FCI4W4FK+68t+EIujVLMBcaCKK0eaAK11y+/dlf6iD+d3K5NXDM/fGfqkfMGtSHjX/5c2PiEhkZDM4U= X-Received: by 2002:a25:6604:: with SMTP id a4mr12224326ybc.412.1610642472224; Thu, 14 Jan 2021 08:41:12 -0800 (PST) MIME-Version: 1.0 References: <20201218031703.3053753-1-saravanak@google.com> <17703ac8-2238-0b64-3c98-ddadc7ae8a36@nvidia.com> In-Reply-To: <17703ac8-2238-0b64-3c98-ddadc7ae8a36@nvidia.com> From: Saravana Kannan Date: Thu, 14 Jan 2021 08:40:35 -0800 Message-ID: Subject: Re: [PATCH v1 0/5] Enable fw_devlink=on by default To: Jon Hunter Cc: Marc Zyngier , Greg Kroah-Hartman , "Rafael J. Wysocki" , Android Kernel Team , LKML , Jisheng Zhang , Kevin Hilman , John Stultz , Nicolas Saenz Julienne , linux-tegra Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org On Thu, Jan 14, 2021 at 3:35 AM Jon Hunter wrote: > > > On 13/01/2021 21:29, Saravana Kannan wrote: > > ... > > >> I am seeing the same problem on Tegra30 Cardhu A04 where several regulators > >> are continuously deferred and prevents the board from booting ... > >> > >> [ 2.518334] platform panel: probe deferral - supplier regulator@11 not ready > >> > >> [ 2.525503] platform regulator@1: probe deferral - supplier 4-002d not ready > >> > >> [ 2.533141] platform regulator@3: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.540856] platform regulator@5: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.548589] platform regulator@6: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.556316] platform regulator@7: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.564041] platform regulator@8: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.571743] platform regulator@9: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.579463] platform regulator@10: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.587273] platform regulator@11: probe deferral - supplier regulator@101 not ready > >> > >> [ 2.595088] platform regulator@12: probe deferral - supplier regulator@104 not ready > >> > >> [ 2.603837] platform regulator@102: probe deferral - supplier regulator@104 not ready > >> > >> [ 2.611726] platform regulator@103: probe deferral - supplier regulator@104 not ready > >> > >> [ 2.620137] platform 3000.pcie: probe deferral - supplier regulator@5 not ready > > > > Looks like this is not the whole log? Do you see any "wait for > > supplier" logs? That's what all these boot issues should boil down to. > > And as usual, pointer to DT for this board please. > > Ah yes I see ... > > platform regulator@1: probe deferral - wait for supplier tps65911@2d Do you mind sharing the full log please? It's hard to tell you anything useful with bits and pieces of logs. > Yes the device-tree for this board can be found here [0]. Looks like > there is a circular dependency between the vddctrl_reg and vddcore_reg. > This is part of coupled regulators which have a two-way linkage [1]. So > this change appears to conflict with this. fw_devlink doesn't track "regulator-coupled-with". So that's probably not it. Also, this patch series was made to handle simple cycles properly. It'll functionally disable the device links it created when it comes to probe ordering. Only two overlapping cycles might cause issues -- and even that, not all the time. So yeah, full log please. -Saravana > Jon > > [0] > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/tegra30-cardhu-a04.dts > [1] > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/regulator/regulator.yaml#n129 > > -- > nvpublic