diff mbox

[4/4] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry

Message ID 1442551533-24915-1-git-send-email-souvik.k.chakravarty@intel.com (mailing list archive)
State Changes Requested, archived
Headers show

Commit Message

Chakravarty, Souvik K Sept. 18, 2015, 4:45 a.m. UTC
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  |   16 ++++++++++++++++
 drivers/platform/x86/Makefile |    2 ++
 3 files changed, 27 insertions(+)

Comments

Darren Hart Sept. 18, 2015, 6:25 p.m. UTC | #1
On Fri, Sep 18, 2015 at 10:15:33AM +0530, Souvik Kumar Chakravarty wrote:
> 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  |   16 ++++++++++++++++
>  drivers/platform/x86/Makefile |    2 ++
>  3 files changed, 27 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4d8c8e1..a7f3c1f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5548,6 +5548,15 @@ S:	Maintained
>  F:	drivers/platform/x86/intel_pmc_ipc.c
>  F:	arch/x86/include/asm/intel_pmc_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 6dc13e4..7daf453 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -919,4 +919,20 @@ config INTEL_PMC_IPC
>  	The PMC is an ARC processor which defines IPC commands for communication
>  	with other entities in the CPU.
>  
> +config INTEL_PUNIT_IPC
> +        bool "Intel P-UNIT IPC Driver"
> +        ---help---
> +          IPC is used to bridge the communications between kernel and PUNIT

This is being added separately by Qipeng, please don't include it here. Just
state that this driver is dependent on his patches.

Are you testing this driver in conjunction with his?

> +
> +config INTEL_TELEMETRY
> +	bool "Intel SoC Telemetry Driver"
> +	def_bool 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 dda95a9..0f99a85 100644
> --- a/drivers/platform/x86/Makefile
> +++ b/drivers/platform/x86/Makefile
> @@ -60,3 +60,5 @@ obj-$(CONFIG_INTEL_SMARTCONNECT)	+= intel-smartconnect.o
>  obj-$(CONFIG_PVPANIC)           += pvpanic.o
>  obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
>  obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.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
> 
>
Darren Hart Sept. 18, 2015, 7:20 p.m. UTC | #2
On Fri, Sep 18, 2015 at 10:15:33AM +0530, Souvik Kumar Chakravarty wrote:
> 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  |   16 ++++++++++++++++
>  drivers/platform/x86/Makefile |    2 ++
>  3 files changed, 27 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4d8c8e1..a7f3c1f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5548,6 +5548,15 @@ S:	Maintained
>  F:	drivers/platform/x86/intel_pmc_ipc.c
>  F:	arch/x86/include/asm/intel_pmc_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 6dc13e4..7daf453 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -919,4 +919,20 @@ config INTEL_PMC_IPC
>  	The PMC is an ARC processor which defines IPC commands for communication
>  	with other entities in the CPU.
>  
> +config INTEL_PUNIT_IPC
> +        bool "Intel P-UNIT IPC Driver"
> +        ---help---
> +          IPC is used to bridge the communications between kernel and PUNIT
> +
> +config INTEL_TELEMETRY
> +	bool "Intel SoC Telemetry Driver"
> +	def_bool n
> +        depends on INTEL_PMC_IPC && INTEL_PUNIT_IPC && X86_64

Note that this fails to compile with INTL_PMC_IPC is built as a module and this
as a built-in - the result of "make allmodconfig".

> +	---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 dda95a9..0f99a85 100644
> --- a/drivers/platform/x86/Makefile
> +++ b/drivers/platform/x86/Makefile
> @@ -60,3 +60,5 @@ obj-$(CONFIG_INTEL_SMARTCONNECT)	+= intel-smartconnect.o
>  obj-$(CONFIG_PVPANIC)           += pvpanic.o
>  obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
>  obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.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
> 
>
Chakravarty, Souvik K Sept. 21, 2015, 4:05 a.m. UTC | #3
> -----Original Message-----
> From: Darren Hart [mailto:dvhart@infradead.org]
> Sent: Friday, September 18, 2015 11:55 PM
> To: Chakravarty, Souvik K <souvik.k.chakravarty@intel.com>
> Cc: platform-driver-x86@vger.kernel.org; Kasagar, Srinidhi
> <srinidhi.kasagar@intel.com>; Zha, Qipeng <qipeng.zha@intel.com>
> Subject: Re: [PATCH 4/4] platform:x86: Makefile/Kconfig/MAINTAINERS
> changes for Intel Telemetry
> 
> On Fri, Sep 18, 2015 at 10:15:33AM +0530, Souvik Kumar Chakravarty wrote:
> > 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  |   16 ++++++++++++++++
> >  drivers/platform/x86/Makefile |    2 ++
> >  3 files changed, 27 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS index 4d8c8e1..a7f3c1f 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -5548,6 +5548,15 @@ S:	Maintained
> >  F:	drivers/platform/x86/intel_pmc_ipc.c
> >  F:	arch/x86/include/asm/intel_pmc_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 6dc13e4..7daf453 100644
> > --- a/drivers/platform/x86/Kconfig
> > +++ b/drivers/platform/x86/Kconfig
> > @@ -919,4 +919,20 @@ config INTEL_PMC_IPC
> >  	The PMC is an ARC processor which defines IPC commands for
> communication
> >  	with other entities in the CPU.
> >
> > +config INTEL_PUNIT_IPC
> > +        bool "Intel P-UNIT IPC Driver"
> > +        ---help---
> > +          IPC is used to bridge the communications between kernel and
> > +PUNIT
> 
> This is being added separately by Qipeng, please don't include it here. Just state
> that this driver is dependent on his patches.

OK.
> 
> Are you testing this driver in conjunction with his?

Yes this driver will depend on PUNIT_IPC for PUNIT specific configurations.
> 
> > +
> > +config INTEL_TELEMETRY
> > +	bool "Intel SoC Telemetry Driver"
> > +	def_bool 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 dda95a9..0f99a85 100644
> > --- a/drivers/platform/x86/Makefile
> > +++ b/drivers/platform/x86/Makefile
> > @@ -60,3 +60,5 @@ obj-$(CONFIG_INTEL_SMARTCONNECT)	+= intel-
> smartconnect.o
> >  obj-$(CONFIG_PVPANIC)           += pvpanic.o
> >  obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
> >  obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.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
> >
> >
> 
> --
> Darren Hart
> Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 4d8c8e1..a7f3c1f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5548,6 +5548,15 @@  S:	Maintained
 F:	drivers/platform/x86/intel_pmc_ipc.c
 F:	arch/x86/include/asm/intel_pmc_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 6dc13e4..7daf453 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -919,4 +919,20 @@  config INTEL_PMC_IPC
 	The PMC is an ARC processor which defines IPC commands for communication
 	with other entities in the CPU.
 
+config INTEL_PUNIT_IPC
+        bool "Intel P-UNIT IPC Driver"
+        ---help---
+          IPC is used to bridge the communications between kernel and PUNIT
+
+config INTEL_TELEMETRY
+	bool "Intel SoC Telemetry Driver"
+	def_bool 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 dda95a9..0f99a85 100644
--- a/drivers/platform/x86/Makefile
+++ b/drivers/platform/x86/Makefile
@@ -60,3 +60,5 @@  obj-$(CONFIG_INTEL_SMARTCONNECT)	+= intel-smartconnect.o
 obj-$(CONFIG_PVPANIC)           += pvpanic.o
 obj-$(CONFIG_ALIENWARE_WMI)	+= alienware-wmi.o
 obj-$(CONFIG_INTEL_PMC_IPC)	+= intel_pmc_ipc.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