Received: by 2002:a25:824b:0:0:0:0:0 with SMTP id d11csp8474117ybn; Tue, 1 Oct 2019 08:36:10 -0700 (PDT) X-Google-Smtp-Source: APXvYqwnglyG1/z+3SSYlmC2O3FKatJ+3ZvjrPdjuN57jLwEOPNBKJZ8dsnP/sp0/EP7U1hw16pl X-Received: by 2002:aa7:d692:: with SMTP id d18mr25715708edr.95.1569944170746; Tue, 01 Oct 2019 08:36:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1569944170; cv=none; d=google.com; s=arc-20160816; b=vkJurfsdnuyxxM/YUG9BncFH7tEm9+yM1pWFKUB2kAOErEiwLT2wGqLDLQkionMMRG zbpk9NiT/sEJSdlW3bPUT3XVtCZiFYzUuJc0FMNiqJ9zlHqraUBGLcl4e6q90KkTDKSR 6kO/IPrzHQP3zruWkkuP0wO+1hbIMQm5i1+MD/tpK2PmGsOAT+4C9BUqfj09pY1gtky8 fK8Pyb/PQt9mIUpK0DuucFabx5gqaSc8rPD0wFxq7wgJkQhYFzQEkFcd0xnTQAoUSyYe 4/545UhoRaGvBhQRM6iwpPMt7TAsscvT+qkApNagnlxLUOaBqkWL2oTaOEUVZSzMusB1 5ioQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:organization:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=uJRbYRaZXfGMYM/Ch9gMEULseBpOkUMS33UXbCHgspA=; b=vffei72rigw+Lqzqwgv4vIldhG9MQj681N+RR2VFEhTkFCe1TQiX4AwNZ8eKh7Wevt 76eBLDdy3BHuKij8DoW4DK9lxp70dfIispzhW4kKVBoFpDlK6PLKtc2OpbUXimKZn2X/ zh1CTsHKZzQyb9pez4gbMDVeVQvb9XT9ALGkVFdKzrKtELkKO0TxejEvx7HmdW2SeWsw FJLiQT7vPSKoPyByhyA60yhnn4zp5WGeO1+ZWRdivU6S7rl75zSSe7s2ysM5Ssn/nqrf Xzg2sPtqgIb2eoqdlxtq8uafzOYQDMeG7zCz9L5OUrMyBBt4f1OW6cqEIE6391ZD/M8g FlHA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id bt13si9038072edb.245.2019.10.01.08.35.46; Tue, 01 Oct 2019 08:36:10 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389601AbfJAPe2 (ORCPT + 99 others); Tue, 1 Oct 2019 11:34:28 -0400 Received: from mga02.intel.com ([134.134.136.20]:62567 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727236AbfJAPe1 (ORCPT ); Tue, 1 Oct 2019 11:34:27 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Oct 2019 08:32:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,571,1559545200"; d="scan'208";a="205087932" Received: from lahna.fi.intel.com (HELO lahna) ([10.237.72.157]) by fmsmga001.fm.intel.com with SMTP; 01 Oct 2019 08:32:55 -0700 Received: by lahna (sSMTP sendmail emulation); Tue, 01 Oct 2019 18:32:54 +0300 Date: Tue, 1 Oct 2019 18:32:54 +0300 From: Mika Westerberg To: Greg KH Cc: Mario.Limonciello@dell.com, linux-usb@vger.kernel.org, andreas.noever@gmail.com, michael.jamet@intel.com, YehezkelShB@gmail.com, rajmohan.mani@intel.com, nicholas.johnson-opensource@outlook.com.au, lukas@wunner.de, stern@rowland.harvard.edu, anthony.wong@canonical.com, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 17/22] thunderbolt: Add initial support for USB4 Message-ID: <20191001153254.GD2714@lahna.fi.intel.com> References: <20191001113830.13028-1-mika.westerberg@linux.intel.com> <20191001113830.13028-18-mika.westerberg@linux.intel.com> <20191001124748.GH2954373@kroah.com> <20191001130905.GO2714@lahna.fi.intel.com> <20191001145354.GA3366714@kroah.com> <924ce4d5862c4d859e238c0e706a3d5b@AUSX13MPC105.AMER.DELL.COM> <20191001152213.GB3450435@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191001152213.GB3450435@kroah.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.12.1 (2019-06-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 01, 2019 at 05:22:13PM +0200, Greg KH wrote: > On Tue, Oct 01, 2019 at 02:59:06PM +0000, Mario.Limonciello@dell.com wrote: > > > > > > > -----Original Message----- > > > From: Greg Kroah-Hartman > > > Sent: Tuesday, October 1, 2019 9:54 AM > > > To: Mika Westerberg > > > Cc: linux-usb@vger.kernel.org; Andreas Noever; Michael Jamet; Yehezkel Bernat; > > > Rajmohan Mani; Nicholas Johnson; Lukas Wunner; Alan Stern; Limonciello, Mario; > > > Anthony Wong; linux-kernel@vger.kernel.org > > > Subject: Re: [RFC PATCH 17/22] thunderbolt: Add initial support for USB4 > > > > > > > > > [EXTERNAL EMAIL] > > > > > > On Tue, Oct 01, 2019 at 04:09:05PM +0300, Mika Westerberg wrote: > > > > On Tue, Oct 01, 2019 at 02:47:48PM +0200, Greg Kroah-Hartman wrote: > > > > > > - Thunderbolt Controller driver. This driver is required if you > > > > > > - want to hotplug Thunderbolt devices on Apple hardware or on PCs > > > > > > - with Intel Falcon Ridge or newer. > > > > > > + USB4 (Thunderbolt) driver. USB4 is the public spec based on > > > > > > + Thunderbolt 3 protocol. This driver is required if you want to > > > > > > + hotplug Thunderbolt and USB4 compliant devices on Apple > > > > > > + hardware or on PCs with Intel Falcon Ridge or newer. > > > > > > > > > > Wait, did "old" thunderbolt just get re-branded as USB4? > > > > > > > > Not but the driver started supporting USB4 as well :) > > > > > > > > USB4 is pretty much public spec of Thunderbolt 3 but with some > > > > differences in register layouts (this is because Thunderbolt uses some > > > > vendor specific capabilities which are now moved to more "standard" > > > > places). > > > > > > Ok, then we need to rename the Kconfig option as well, otherwise no one > > > will "know" that this changed, so they will not be prompted for it. > > > > > > > > Because if I have an "old" laptop that needs Thunderbolt support, how am > > > > > I going to know it is now called USB4 instead? > > > > > > > > Well the Kconfig option tries to have both names there: > > > > > > > > tristate "USB4 (Thunderbolt) support" > > > > > > > > and then > > > > > > > > USB4 (Thunderbolt) driver. USB4 is the public spec based on > > > > Thunderbolt 3 protocol. This driver is required if you want to hotplug > > > > Thunderbolt and USB4 compliant devices on Apple hardware or on PCs > > > > with Intel Falcon Ridge or newer. > > > > > > > > and the Kconfig option is still CONFIG_THUNDERBOLT. I know this is > > > > confusing but I don't have better ideas how we can advertise both. I > > > > borrowed this "format" from firewire. > > > > > > CONFIG_USB4 instead? > > > > How about CONFIG_USB4_PCIE? > > > > I think that will help align that certain aspects of USB4 can be built optionally. > > What aspects? We don't have that here at all. > > I guess the parts of USB4 that are not just this "hook up the PCIe > lane" that need to still be developed? Actually PCIe tunneling is already there in the driver. USB4 has one bit that is needed to be set before PCIe tunneling can happen (we do that in patch 17/22) but other than that the existing PCIe tunneling code in the driver works as is.