All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry
@ 2016-01-07  9:08 Souvik Kumar Chakravarty
  0 siblings, 0 replies; 5+ messages in thread
From: Souvik Kumar Chakravarty @ 2016-01-07  9:08 UTC (permalink / raw)
  To: platform-driver-x86
  Cc: dvhart, rjw, linux-kernel, srinidhi.kasagar, qipeng.zha,
	rajeev.d.muralidhar, sukumar.ghorai, ong.hock.yu, aubrey.li,
	david.ginat, Souvik Kumar Chakravarty

Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
It depends on PUNIT and PMC IPC drivers.

Signed-off-by: Souvik Kumar Chakravarty <souvik.k.chakravarty@intel.com>
---
 MAINTAINERS                   |    9 +++++++++
 drivers/platform/x86/Kconfig  |   11 +++++++++++
 drivers/platform/x86/Makefile |    3 +++
 3 files changed, 23 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 46c1393..72d8245 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5693,6 +5693,15 @@ F:	drivers/platform/x86/intel_punit_ipc.c
 F:	arch/x86/include/asm/intel_pmc_ipc.h
 F:	arch/x86/include/asm/intel_punit_ipc.h
 
+INTEL TELEMETRY DRIVER
+M:	Souvik Kumar Chakravarty <souvik.k.chakravarty@intel.com>
+L:	platform-driver-x86@vger.kernel.org
+S:	Maintained
+F:	drivers/platform/x86/intel_telemetry_core.c
+F:	drivers/platform/x86/intel_telemetry_pltdrv.c
+F:	drivers/platform/x86/intel_telemetry_debugfs.c
+F:	arch/x86/include/asm/intel_telemetry.h
+
 IOC3 ETHERNET DRIVER
 M:	Ralf Baechle <ralf@linux-mips.org>
 L:	linux-mips@linux-mips.org
diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index 148ff88..4c1d55d 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -950,4 +950,15 @@ config INTEL_PUNIT_IPC
 	---help---
 	  This driver provides support for Intel P-Unit Mailbox IPC mechanism,
 	  which is used to bridge the communications between kernel and P-Unit.
+
+config INTEL_TELEMETRY
+	tristate "Intel SoC Telemetry Driver"
+	default n
+	depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64
+	---help---
+	  This driver provides interfaces to configure and use
+	  telemetry for INTEL SoC from APL onwards. It is also
+	  used to get various SoC events and parameters
+	  directly via debugfs files. Various tools may use
+	  this interface for SoC state monitoring.
 endif # X86_PLATFORM_DEVICES
diff --git a/drivers/platform/x86/Makefile b/drivers/platform/x86/Makefile
index 5ee5425..17538b0 100644
--- a/drivers/platform/x86/Makefile
+++ b/drivers/platform/x86/Makefile
@@ -63,3 +63,6 @@ obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
 obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.o
 obj-$(CONFIG_SURFACE_PRO3_BUTTON)	+= surfacepro3_button.o
 obj-$(CONFIG_INTEL_PUNIT_IPC)  += intel_punit_ipc.o
+obj-$(CONFIG_INTEL_TELEMETRY)	+= intel_telemetry_core.o \
+				   intel_telemetry_pltdrv.o \
+				   intel_telemetry_debugfs.o
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry
  2015-12-30  1:59   ` Rafael J. Wysocki
@ 2015-12-30  7:32     ` Darren Hart
  0 siblings, 0 replies; 5+ messages in thread
From: Darren Hart @ 2015-12-30  7:32 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Souvik Kumar Chakravarty, platform-driver-x86, srinidhi.kasagar,
	qipeng.zha, rajeev.d.muralidhar, sukumar.ghorai, ong.hock.yu,
	aubrey.li

On Wed, Dec 30, 2015 at 02:59:09AM +0100, Rafael Wysocki wrote:
> On Tuesday, December 29, 2015 04:59:10 PM Darren Hart wrote:
> > On Wed, Dec 23, 2015 at 04:14:41PM +0530, Souvik Kumar Chakravarty wrote:
> > > Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
> > > It depends on PUNIT and PMC IPC drivers.
> > 
> > ...
> > 
> > > +config INTEL_TELEMETRY
> > > +	tristate "Intel SoC Telemetry Driver"
> > > +	default n
> > > +	depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64
> > > +	---help---
> > > +	  This driver provides interfaces to configure and use
> > > +	  telemetry for INTEL SoC from APL onwards. It is also
> > > +	  used to get various SoC events and parameters
> > > +	  directly via debugfs files. Various tools may use
> > 
> > If tools are going to be relying on it, debugfs doesn't seem like the right
> > place for it to me. Would /sys/power be more apt?
> > 
> > Rafael?
> 
> Using debugfs for things like these is not unheard of.
> 
> It all depends on whether or not the tools in question are intended to work
> on systems where debugfs is not enabled.

It was my understanding that debugfs was intended to make it clear that it may
change at any time and shouldn't be relied upon. Strictly for debug.

But, I'll defer to Rafael on this point.

-- 
Darren Hart
Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry
  2015-12-30  0:59 ` Darren Hart
@ 2015-12-30  1:59   ` Rafael J. Wysocki
  2015-12-30  7:32     ` Darren Hart
  0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2015-12-30  1:59 UTC (permalink / raw)
  To: Darren Hart
  Cc: Souvik Kumar Chakravarty, platform-driver-x86, srinidhi.kasagar,
	qipeng.zha, rajeev.d.muralidhar, sukumar.ghorai, ong.hock.yu,
	aubrey.li

On Tuesday, December 29, 2015 04:59:10 PM Darren Hart wrote:
> On Wed, Dec 23, 2015 at 04:14:41PM +0530, Souvik Kumar Chakravarty wrote:
> > Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
> > It depends on PUNIT and PMC IPC drivers.
> 
> ...
> 
> > +config INTEL_TELEMETRY
> > +	tristate "Intel SoC Telemetry Driver"
> > +	default n
> > +	depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64
> > +	---help---
> > +	  This driver provides interfaces to configure and use
> > +	  telemetry for INTEL SoC from APL onwards. It is also
> > +	  used to get various SoC events and parameters
> > +	  directly via debugfs files. Various tools may use
> 
> If tools are going to be relying on it, debugfs doesn't seem like the right
> place for it to me. Would /sys/power be more apt?
> 
> Rafael?

Using debugfs for things like these is not unheard of.

It all depends on whether or not the tools in question are intended to work
on systems where debugfs is not enabled.

Thanks,
Rafael

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry
  2015-12-23 10:44 Souvik Kumar Chakravarty
@ 2015-12-30  0:59 ` Darren Hart
  2015-12-30  1:59   ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: Darren Hart @ 2015-12-30  0:59 UTC (permalink / raw)
  To: Souvik Kumar Chakravarty, Rafael Wysocki
  Cc: platform-driver-x86, srinidhi.kasagar, qipeng.zha,
	rajeev.d.muralidhar, sukumar.ghorai, ong.hock.yu, aubrey.li

On Wed, Dec 23, 2015 at 04:14:41PM +0530, Souvik Kumar Chakravarty wrote:
> Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
> It depends on PUNIT and PMC IPC drivers.

...

> +config INTEL_TELEMETRY
> +	tristate "Intel SoC Telemetry Driver"
> +	default n
> +	depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64
> +	---help---
> +	  This driver provides interfaces to configure and use
> +	  telemetry for INTEL SoC from APL onwards. It is also
> +	  used to get various SoC events and parameters
> +	  directly via debugfs files. Various tools may use

If tools are going to be relying on it, debugfs doesn't seem like the right
place for it to me. Would /sys/power be more apt?

Rafael?

-- 
Darren Hart
Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry
@ 2015-12-23 10:44 Souvik Kumar Chakravarty
  2015-12-30  0:59 ` Darren Hart
  0 siblings, 1 reply; 5+ messages in thread
From: Souvik Kumar Chakravarty @ 2015-12-23 10:44 UTC (permalink / raw)
  To: platform-driver-x86
  Cc: dvhart, srinidhi.kasagar, qipeng.zha, rajeev.d.muralidhar,
	sukumar.ghorai, ong.hock.yu, aubrey.li, Souvik Kumar Chakravarty

Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
It depends on PUNIT and PMC IPC drivers.

Signed-off-by: Souvik Kumar Chakravarty <souvik.k.chakravarty@intel.com>

---
Changes in v3:
* Rebased on top of latest files.
---
Changes in v2:
* Change bool to tristate in Kconfig to enable support as module
* Remove PUNIT_IPC specific additions. These will be merged separately
  by Qipeng's PUNIT_IPC patch.
---
 MAINTAINERS                   |    9 +++++++++
 drivers/platform/x86/Kconfig  |   11 +++++++++++
 drivers/platform/x86/Makefile |    3 +++
 3 files changed, 23 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 0484f70..a05c728 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5693,6 +5693,15 @@ F:	drivers/platform/x86/intel_punit_ipc.c
 F:	arch/x86/include/asm/intel_pmc_ipc.h
 F:	arch/x86/include/asm/intel_punit_ipc.h
 
+INTEL TELEMETRY DRIVER
+M:	Souvik Kumar Chakravarty <souvik.k.chakravarty@intel.com>
+L:	platform-driver-x86@vger.kernel.org
+S:	Maintained
+F:	drivers/platform/x86/intel_telemetry_core.c
+F:	drivers/platform/x86/intel_telemetry_pltdrv.c
+F:	drivers/platform/x86/intel_telemetry_debugfs.c
+F:	arch/x86/include/asm/intel_telemetry.h
+
 IOC3 ETHERNET DRIVER
 M:	Ralf Baechle <ralf@linux-mips.org>
 L:	linux-mips@linux-mips.org
diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index 148ff88..4c1d55d 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -950,4 +950,15 @@ config INTEL_PUNIT_IPC
 	---help---
 	  This driver provides support for Intel P-Unit Mailbox IPC mechanism,
 	  which is used to bridge the communications between kernel and P-Unit.
+
+config INTEL_TELEMETRY
+	tristate "Intel SoC Telemetry Driver"
+	default n
+	depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64
+	---help---
+	  This driver provides interfaces to configure and use
+	  telemetry for INTEL SoC from APL onwards. It is also
+	  used to get various SoC events and parameters
+	  directly via debugfs files. Various tools may use
+	  this interface for SoC state monitoring.
 endif # X86_PLATFORM_DEVICES
diff --git a/drivers/platform/x86/Makefile b/drivers/platform/x86/Makefile
index 5ee5425..17538b0 100644
--- a/drivers/platform/x86/Makefile
+++ b/drivers/platform/x86/Makefile
@@ -63,3 +63,6 @@ obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
 obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.o
 obj-$(CONFIG_SURFACE_PRO3_BUTTON)	+= surfacepro3_button.o
 obj-$(CONFIG_INTEL_PUNIT_IPC)  += intel_punit_ipc.o
+obj-$(CONFIG_INTEL_TELEMETRY)	+= intel_telemetry_core.o \
+				   intel_telemetry_pltdrv.o \
+				   intel_telemetry_debugfs.o
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-01-07  9:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-07  9:08 [PATCH v3 5/5] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry Souvik Kumar Chakravarty
  -- strict thread matches above, loose matches on Subject: below --
2015-12-23 10:44 Souvik Kumar Chakravarty
2015-12-30  0:59 ` Darren Hart
2015-12-30  1:59   ` Rafael J. Wysocki
2015-12-30  7:32     ` Darren Hart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.