Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp107889imu; Thu, 15 Nov 2018 23:04:17 -0800 (PST) X-Google-Smtp-Source: AJdET5f6xGih+SGpZgVWna8cq20eFMpthuHgujo1UOEC0JhBCbMwfe9EW+IalJzQ3Hy3iUX3fiVG X-Received: by 2002:a63:4c6:: with SMTP id 189mr8930521pge.391.1542351857345; Thu, 15 Nov 2018 23:04:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542351857; cv=none; d=google.com; s=arc-20160816; b=vKKz2hQb4uj2aj4kFb1sOLqCYtcbVRKx2g/3XSbChmRUZh2JnRDkxDCSgo/jrnzHwo cPx9Il4RqIH5AQcPETMjh12xS1xmeVSw15Zy7UGEWl0esMmksRP+DEWbmNBSjcX8xXzO gnRN0/FdtEe0+F8RG6VE7YXuhpXWz0irMdvvbL7dNLbyruFsWYYcnBOJ73YVD2VUmTp/ 4YtfpY56k1ukwM2YSmRbMtd/A2woOeD6KeZ6pM8gC1bvN4f2o5FoTg7JB0Bk1gaz6V/W GG3TybnLByTuFY43qw0BN7zR8Sd2PLtaictvmAnhR1MT3TKn7JtezPmD4XkdA4/3HwVr kN7Q== 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=oF9GsIPx0Ix/ezma87UsDDXfEICIzx0Tt7c7C6hKBhs=; b=vN0peydCJWEqhMQDDu6RA8qD1Nb8l+aQiumdZKVc767E3sU+Z/6Je2Ns1pkjahosl8 LCKrV00UdettykVc4H43ot02VQZWGYpT3tFqSSDEIiUwSj+WNLz19fT+kqwS6AYFdmNm lRa5/KrwFVshJa+xY11G5eaVcRjVtDZGLoVMMar7WY4hfPZ30POXw2pTOt2PPCbgj+X4 ixEo1uPdfWwMsyHb6XdTjjmXXeY7Bt/1WUeHGVueZrONQXwyX7wYCztzSIRWD0MfWCxa xaHnQEBzqVTTWK+qkq3F2uTSqmquj0EWDcn2+dn7wF5ogg4gtwRVo5/geK+VJPInB1Sy m3mA== 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 q5si22400082pgk.46.2018.11.15.23.04.01; Thu, 15 Nov 2018 23:04:17 -0800 (PST) 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 S2389397AbeKPRMp (ORCPT + 99 others); Fri, 16 Nov 2018 12:12:45 -0500 Received: from mga06.intel.com ([134.134.136.31]:17778 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727380AbeKPRMp (ORCPT ); Fri, 16 Nov 2018 12:12:45 -0500 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 15 Nov 2018 23:01:31 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,239,1539673200"; d="scan'208";a="280455897" Received: from lahna.fi.intel.com (HELO lahna) ([10.237.72.157]) by fmsmga005.fm.intel.com with SMTP; 15 Nov 2018 23:01:25 -0800 Received: by lahna (sSMTP sendmail emulation); Fri, 16 Nov 2018 09:01:25 +0200 Date: Fri, 16 Nov 2018 09:01:25 +0200 From: Mika Westerberg To: Lorenzo Pieralisi Cc: Lukas Wunner , iommu@lists.linux-foundation.org, Joerg Roedel , David Woodhouse , Lu Baolu , Ashok Raj , Bjorn Helgaas , "Rafael J. Wysocki" , Jacob jun Pan , Andreas Noever , Michael Jamet , Yehezkel Bernat , Christian Kellner , Mario.Limonciello@dell.com, Anthony Wong , linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/4] PCI / ACPI: Identify external PCI devices Message-ID: <20181116070125.GG4179@lahna.fi.intel.com> References: <20181113105636.GB11202@e107981-ln.cambridge.arm.com> <20181113112700.GT2500@lahna.fi.intel.com> <20181113114527.GA12821@e107981-ln.cambridge.arm.com> <20181115102239.GU2500@lahna.fi.intel.com> <20181115111356.GA599@e107981-ln.cambridge.arm.com> <20181115113737.GW2500@lahna.fi.intel.com> <20181115120736.pscly6zwd3k2tvd2@wunner.de> <20181115121627.GA2500@lahna.fi.intel.com> <20181115174608.GA17691@e107981-ln.cambridge.arm.com> <20181115190054.GD4179@lahna.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181115190054.GD4179@lahna.fi.intel.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 15, 2018 at 09:00:54PM +0200, Mika Westerberg wrote: > On Thu, Nov 15, 2018 at 05:46:08PM +0000, Lorenzo Pieralisi wrote: > > Do you really need to parse it if the dev->is_thunderbolt check is enough ? > > Yes, we need to parse it one way or another. dev->is_thunderbolt is > based on heuristics which do not apply anymore when the thing gets > integrated in the SoC. > > The _DSD is there already (on existing systems) and is being used by > Windows so I don't understand why we cannot take advantage of it? Every > new system with Thunderbolt ports will have it. Just to clarify a bit. We can use is_thunderbolt in place of is_external if we don't want to deal with other possible "external" devices right now. However, we still need to parse the _DSD and based on that fill the is_thunderbolt for these devices (same way we do for is_external in this series). So basically we just get rid of the is_external flag and use is_thunderbolt instead.