Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp4197156pxb; Tue, 10 Nov 2020 10:08:44 -0800 (PST) X-Google-Smtp-Source: ABdhPJw2nG11I9wEjHfuiAYB8p6EWE3KD27knv4aOk+zJz0jr/G14lEKpQaXHtuZH9pBR4IsleM0 X-Received: by 2002:a17:906:134e:: with SMTP id x14mr22048233ejb.173.1605031724359; Tue, 10 Nov 2020 10:08:44 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1605031724; cv=none; d=google.com; s=arc-20160816; b=V+qNHVZmSbiu0HCr50k93g4EpeRRybyi/cGJ3p/6aFiEpCEDsdB02qOFw0oDHIGno+ UTmBBU4Kb8rvcI40mYl48LhaG4nzXcL99m5ocjIs4B3YFTzbEqoRsHyxMBdNnMvLtCaS +q6hs182rU1Yn8AyK8rZYVax2EzuIXdx2aV2UFP6O0q/ng+jQjPt6ZQJuPTnUo3/B/gx 65x4aGFoHn3sQTtEntNM9Z2Jyl4wPai8k2Zp+xXit7vetRXop5upGg0OG4/2wsBhy5mf 6H0aNIQ63uk5opzKi8zc3N9OcEYOoMZ5pD4yC/OQFpvCpalzbExwYjZpCQag6ZVKqtRE cv3A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:organization:references:in-reply-to:date:cc:to:reply-to :from:subject:message-id:ironport-sdr:ironport-sdr; bh=Kv+HQY73tFffWyrDFWE6qIQnFneRBu//ITGwe7DWyAY=; b=XislpqdiumvsIG83EcDtdhidHtActC2k05Wto3BzjvvpBzcX7SNAmei1iZKoTfiv+I A97I9yVXfjOuNr1xt/kMZWywjOTLRHzTp5nbeKQeuufQB3+BTENWntv2ZfutaPCw0vfm sA+boTilDoYUeHKsIp5VMHSoH2No0oW56tjxXFT2jO5TN2cm7OdHM+CXDfgiq5gtpDK1 1uQZHNd2TmXLtLXyrseet20nVU2TEHIe5RSEaoJbZh7fsvlxxmS7SBZFOuYjaM+wlgfs 1G3HOrn2iG10PcoV4ZDUsTZPN7u7qX7DNR0DLFYv9AwJL6cZypAPKuIepYFbn0Ahbc0h tWOw== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a14si10087168ejj.581.2020.11.10.10.08.21; Tue, 10 Nov 2020 10:08:44 -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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730786AbgKJSHA (ORCPT + 99 others); Tue, 10 Nov 2020 13:07:00 -0500 Received: from mga07.intel.com ([134.134.136.100]:56468 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726428AbgKJSHA (ORCPT ); Tue, 10 Nov 2020 13:07:00 -0500 IronPort-SDR: 0zA6zNcxjRvIP9M/4krLs9m7neNw04/+fTgIKdBKSaXViF9b+dMJqsxOPiGTbERfAf0vWlOzEc 6lR0o25MobfQ== X-IronPort-AV: E=McAfee;i="6000,8403,9801"; a="234189632" X-IronPort-AV: E=Sophos;i="5.77,467,1596524400"; d="scan'208";a="234189632" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Nov 2020 10:06:53 -0800 IronPort-SDR: /nfePAnqjQMI6cF0FnQ+JBblmL1nP/m91B+IUFee3M202qcuN6NkaC3M+Qu5UeYCMuh2ZRuiBn qeJw7PPWPwCQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,467,1596524400"; d="scan'208";a="356252584" Received: from linux.intel.com ([10.54.29.200]) by fmsmga004.fm.intel.com with ESMTP; 10 Nov 2020 10:06:53 -0800 Received: from debox1-desk1.jf.intel.com (debox1-desk1.jf.intel.com [10.7.201.137]) by linux.intel.com (Postfix) with ESMTP id 86B9F58088D; Tue, 10 Nov 2020 10:06:53 -0800 (PST) Message-ID: Subject: Re: [PATCH V8 2/5] mfd: Intel Platform Monitoring Technology support From: "David E. Box" Reply-To: david.e.box@linux.intel.com To: Geert Uytterhoeven Cc: Lee Jones , Darren Hart , andy@infradead.org, Bjorn Helgaas , Hans de Goede , alexey.budankov@linux.intel.com, Linux Kernel Mailing List , platform-driver-x86@vger.kernel.org, linux-pci , Andy Shevchenko Date: Tue, 10 Nov 2020 10:06:53 -0800 In-Reply-To: References: <20201003013123.20269-1-david.e.box@linux.intel.com> <20201003013123.20269-3-david.e.box@linux.intel.com> Organization: David E. Box Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Geert, On Tue, 2020-11-10 at 11:39 +0100, Geert Uytterhoeven wrote: > Hi David, > > On Sat, Oct 3, 2020 at 3:32 AM David E. Box < > david.e.box@linux.intel.com> wrote: > > Intel Platform Monitoring Technology (PMT) is an architecture for > > enumerating and accessing hardware monitoring facilities. PMT > > supports > > multiple types of monitoring capabilities. This driver creates > > platform > > devices for each type so that they may be managed by capability > > specific > > drivers (to be introduced). Capabilities are discovered using PCIe > > DVSEC > > ids. Support is included for the 3 current capability types, > > Telemetry, > > Watcher, and Crashlog. The features are available on new Intel > > platforms > > starting from Tiger Lake for which support is added. This patch > > adds > > support for Tiger Lake (TGL), Alder Lake (ADL), and Out-of-Band > > Management > > Services Module (OOBMSM). > > > > Also add a quirk mechanism for several early hardware differences > > and bugs. > > For Tiger Lake and Alder Lake, do not support Watcher and Crashlog > > capabilities since they will not be compatible with future product. > > Also, > > fix use a quirk to fix the discovery table offset. > > > > Co-developed-by: Alexander Duyck > > > > Signed-off-by: Alexander Duyck > > Signed-off-by: David E. Box > > Reviewed-by: Andy Shevchenko > > Thanks for your patch, which is now commit 4f8217d5b0ca8ace ("mfd: > Intel > Platform Monitoring Technology support") in the mfd/for-mfd-next. > > > --- a/drivers/mfd/Kconfig > > +++ b/drivers/mfd/Kconfig > > @@ -670,6 +670,16 @@ config MFD_INTEL_PMC_BXT > > Register and P-unit access. In addition this creates > > devices > > for iTCO watchdog and telemetry that are part of the PMC. > > > > +config MFD_INTEL_PMT > > + tristate "Intel Platform Monitoring Technology (PMT) > > support" > > + depends on PCI > > Does this need a "depend on X86 || COMPILE_TEST", to prevent the > question from showing up on platforms where the PMT cannot be > present? Though not currently available on non X86 hardware it is not restricted. The use of PCIE Designated Vendor Specific Capability (DVSEC) was to specifically allow use of this IP by other vendors. > > I see the TGL and ADL PCI IDs are also referenced from > drivers/platform/x86/intel_pmt_telemetry.c, which suggests this is > X86-only. > Perhaps the OOBMSM is a PCI device that can be used on non-X86 > platforms? TGL and AGL are only referenced in this driver because they require quirks. Thanks David