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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS 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 03CBEC07E96 for ; Tue, 13 Jul 2021 08:58:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id DDE8361362 for ; Tue, 13 Jul 2021 08:58:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234752AbhGMJBN (ORCPT ); Tue, 13 Jul 2021 05:01:13 -0400 Received: from mga14.intel.com ([192.55.52.115]:52516 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234121AbhGMJBM (ORCPT ); Tue, 13 Jul 2021 05:01:12 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10043"; a="209942193" X-IronPort-AV: E=Sophos;i="5.84,236,1620716400"; d="scan'208";a="209942193" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Jul 2021 01:58:16 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,236,1620716400"; d="scan'208";a="570222387" Received: from kuha.fi.intel.com ([10.237.72.162]) by fmsmga001.fm.intel.com with SMTP; 13 Jul 2021 01:58:12 -0700 Received: by kuha.fi.intel.com (sSMTP sendmail emulation); Tue, 13 Jul 2021 11:58:11 +0300 Date: Tue, 13 Jul 2021 11:58:11 +0300 From: Heikki Krogerus To: Martin Kepplinger , "Rafael J. Wysocki" , Saravana Kannan Cc: gregkh@linuxfoundation.org, grandmaster@al2klimov.de, kernel@puri.sm, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] usb: typec: tipd: Don't block probing of consumer of "connector" nodes Message-ID: References: <20210713073946.102501-1-martin.kepplinger@puri.sm> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210713073946.102501-1-martin.kepplinger@puri.sm> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org +Rafael, Saravana On Tue, Jul 13, 2021 at 09:39:46AM +0200, Martin Kepplinger wrote: > Similar as with tcpm this patch lets fw_devlink know not to wait on the > fwnode to be populated as a struct device. > > Without this patch, USB functionality can be broken on some previously > supported boards. > > Fixes: 28ec344bb891 ("usb: typec: tcpm: Don't block probing of consumers of "connector" nodes") That patch has gone under my radar. > Signed-off-by: Martin Kepplinger > --- > hi, > > at least the boards based on imx8mq-librem5.dtsi suffer from this, possibly > more. > > thanks, > martin > > > drivers/usb/typec/tipd/core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/usb/typec/tipd/core.c b/drivers/usb/typec/tipd/core.c > index 938219bc1b4b..b31aaf57eb3b 100644 > --- a/drivers/usb/typec/tipd/core.c > +++ b/drivers/usb/typec/tipd/core.c > @@ -629,6 +629,8 @@ static int tps6598x_probe(struct i2c_client *client) > if (!fwnode) > return -ENODEV; > > + fw_devlink_purge_absent_suppliers(fwnode); Why do we have to care about this kind of stuff in the drivers? It sounds like something that only affects DT platforms, so why isn't this being fixed in the DT core code? I didn't have time to study this fw_deflink thing yet, it is completely new to me and unfortunately I'm going on vacation now, but it looks like something that has been added to the driver core in a haste without enough thinking, and which is now causing problems. Not cool. thanks, -- heikki