Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp1059406pxb; Thu, 28 Jan 2021 07:10:48 -0800 (PST) X-Google-Smtp-Source: ABdhPJyvibTZTL4Uup85uVzk6VZvYReSSuWxvuajWIjlKcjry17UlcBD9iIVhpHMJvEqWBadJbNi X-Received: by 2002:a19:8156:: with SMTP id c83mr557106lfd.546.1611846647745; Thu, 28 Jan 2021 07:10:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1611846647; cv=none; d=google.com; s=arc-20160816; b=NLhlWZDNqRM4kFH2cT43royeU/SpbT/8ly4dMpXgdRAIEw7eu8OAn4w98FnyyjzsoM WORy4UnT64etOtGP9xVTxtSHOEL+OGnHKzOhqzyG/I+cY5v4Aw+XOG+ypvEaP+rmFAlt It0/2xTm/er65NMDm9d/CHzRzar2TNxxGZRRJN29BhOceffLdM+Q/WnQ28879GD5cyIu EpLHSd5VrBSX09u15IMDoajsDy7gHNVpInRuaSp6xGjz48O4Nhwcf2g6TS+29lsYdu4S nvyA89F+Zc6gDo2NtzYkMvUtar66dN1SlWGGz1mlKI3yjvbVjDd+dgy7+qcGerA+6cxm 8EUw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:dkim-signature:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:references:cc:to:from:subject; bh=xyVB/nXmGqUdKAurlytAs4Ivga7O+JRNlwLBfje94Ik=; b=IPYqdFwzEnLIDt/eaIixjPOJb9xoaH5DdAK71h3QgF3DF7gJIVJfzcTSrjCfnT/Yx4 3Ij7cQb7kppckmX5Hs+5b3lnxpr8AnjoQaAmwlQPGwc6c63dxNJ/E2jmxiu9Z04Nt4bP u5HXRsRxheh814ijeN4c/7k1OmQr65qs36XgR723nmHOZVRzIHz/EwN1Kjb2sZbiTQno FtYa2HQTPzubTksaNZIoVYBufCxtCWbar9r4YPLUYLBXS0AqNVs+JQP7M2gkAjfjT7np 6BTRVVvbCuhsJKFYEH5+U6rPpt5R4LtmIbNl7CFMMkTjA4UtBwVIZ7Qfx4xJuM15QF/C J9Eg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=JCa4Snoj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p26si2812739eji.9.2021.01.28.07.10.22; Thu, 28 Jan 2021 07:10:47 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=JCa4Snoj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231487AbhA1PIm (ORCPT + 99 others); Thu, 28 Jan 2021 10:08:42 -0500 Received: from hqnvemgate24.nvidia.com ([216.228.121.143]:11666 "EHLO hqnvemgate24.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232269AbhA1PEI (ORCPT ); Thu, 28 Jan 2021 10:04:08 -0500 Received: from hqmail.nvidia.com (Not Verified[216.228.121.13]) by hqnvemgate24.nvidia.com (using TLS: TLSv1.2, AES256-SHA) id ; Thu, 28 Jan 2021 07:03:28 -0800 Received: from [10.26.73.116] (172.20.145.6) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 28 Jan 2021 15:03:24 +0000 Subject: Re: [PATCH v1 0/5] Enable fw_devlink=on by default From: Jon Hunter To: Saravana Kannan CC: Greg Kroah-Hartman , "Rafael J. Wysocki" , Android Kernel Team , LKML , Jisheng Zhang , Kevin Hilman , John Stultz , Nicolas Saenz Julienne , Marc Zyngier , linux-tegra References: <20201218031703.3053753-1-saravanak@google.com> <56f7d032-ba5a-a8c7-23de-2969d98c527e@nvidia.com> <17939709-f6f4-fa9c-836f-9779081c4087@nvidia.com>

Message-ID: <6a43e209-1d2d-b10a-4564-0289d54135d3@nvidia.com> Date: Thu, 28 Jan 2021 15:03:21 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [172.20.145.6] X-ClientProxiedBy: HQMAIL105.nvidia.com (172.20.187.12) To HQMAIL107.nvidia.com (172.20.187.13) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1611846208; bh=xyVB/nXmGqUdKAurlytAs4Ivga7O+JRNlwLBfje94Ik=; h=Subject:From:To:CC:References:Message-ID:Date:User-Agent: MIME-Version:In-Reply-To:Content-Type:Content-Language: Content-Transfer-Encoding:X-Originating-IP:X-ClientProxiedBy; b=JCa4Snojg2M2xuT8hrJ5OITwTtTwVwvCbBTiUPNEHk06ADPCS3Y6ymFxjU5pUqZmr qqvUbPaq98TqvG59ORl1ejijao6OywKJjENLEEC+/o1x7MKDNtHpEIpjyCeeq6/Gu1 doz4wGc+hQQR48Lz7E96oJ/ftj7i1xinxIe5++TUiJu7iH/v+j6dcCehe1apNiN2Bx ldumRsA5OuYiA2GJkYkaim2Rr4Kbz8NZDsv+HyJy0CsLiLb2HVtmhotEQ9+g5HH/P+ 2qjKYk+lezLKvBPaTBymUccWzypSh+nCYS5RJatDDGb4PAwutGr2U+NmB9c/pTfRFi cPaDSCiB3Yqsw== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14/01/2021 16:56, Jon Hunter wrote: > > On 14/01/2021 16:47, Saravana Kannan wrote: > > ... > >>> Yes this is the warning shown here [0] and this is coming from >>> the 'Generic PHY stmmac-0:00' device. >> >> Can you print the supplier and consumer device when this warning is >> happening and let me know? That'd help too. I'm guessing the phy is >> the consumer. > > > Sorry I should have included that. I added a print to dump this on > another build but failed to include here. > > WARNING KERN Generic PHY stmmac-0:00: supplier 2200000.gpio (status 1) > > The status is the link->status and looks like the supplier is the > gpio controller. I have verified that the gpio controller is probed > before this successfully. > >> So the warning itself isn't a problem -- it's not breaking anything or >> leaking memory or anything like that. But the device link is jumping >> states in an incorrect manner. With enough context of this code (why >> the device_bind_driver() is being called directly instead of going >> through the normal probe path), it should be easy to fix (I'll just >> need to fix up the device link state). > > Correct, the board seems to boot fine, we just get this warning. Have you had chance to look at this further? The following does appear to avoid the warning, but I am not sure if this is the correct thing to do ... index 9179825ff646..095aba84f7c2 100644 --- a/drivers/base/dd.c +++ b/drivers/base/dd.c @@ -456,6 +456,10 @@ int device_bind_driver(struct device *dev) { int ret; + ret = device_links_check_suppliers(dev); + if (ret) + return ret; + ret = driver_sysfs_add(dev); if (!ret) driver_bound(dev); Cheers Jon -- nvpublic