[PATCH] mfd: intel_pmt: Only compile on x86

[PATCH] mfd: intel_pmt: Only compile on x86

[Prarit Bhargava]

The intel_pmt driver shows up as a compile option for all arches but is
32-bit and 64-bit x86 specific.

Add a CONFIG dependency on X86 for intel_pmt.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: David E. Box <david.e.box@linux.intel.com>
Cc: Hans de Goede <hdegoede@redhat.com>
---
 drivers/mfd/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
index 6a3fd2d75f96..0f12b00955b4 100644
--- a/drivers/mfd/Kconfig
+++ b/drivers/mfd/Kconfig
@@ -676,7 +676,7 @@ config MFD_INTEL_PMC_BXT
 
 config MFD_INTEL_PMT
 	tristate "Intel Platform Monitoring Technology (PMT) support"
-	depends on PCI
+	depends on X86 && PCI
 	select MFD_CORE
 	help
 	  The Intel Platform Monitoring Technology (PMT) is an interface that
-- 
2.31.1

Re: [PATCH] mfd: intel_pmt: Only compile on x86

[Hans de Goede]

Hi,

On 8/25/21 2:57 PM, Prarit Bhargava wrote:
> The intel_pmt driver shows up as a compile option for all arches but is
> 32-bit and 64-bit x86 specific.
> 
> Add a CONFIG dependency on X86 for intel_pmt.
> 
> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: David E. Box <david.e.box@linux.intel.com>
> Cc: Hans de Goede <hdegoede@redhat.com>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede@redhat.com>

Regards,

Hans



> ---
>  drivers/mfd/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index 6a3fd2d75f96..0f12b00955b4 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -676,7 +676,7 @@ config MFD_INTEL_PMC_BXT
>  
>  config MFD_INTEL_PMT
>  	tristate "Intel Platform Monitoring Technology (PMT) support"
> -	depends on PCI
> +	depends on X86 && PCI
>  	select MFD_CORE
>  	help
>  	  The Intel Platform Monitoring Technology (PMT) is an interface that
> 

Re: [PATCH] mfd: intel_pmt: Only compile on x86

[David E. Box]

On Thu, 2021-08-26 at 15:19 +0200, Hans de Goede wrote:
> Hi,
> 
> On 8/25/21 2:57 PM, Prarit Bhargava wrote:
> > The intel_pmt driver shows up as a compile option for all arches
> > but is
> > 32-bit and 64-bit x86 specific.
> > 
> > Add a CONFIG dependency on X86 for intel_pmt.
> > 
> > Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: David E. Box <david.e.box@linux.intel.com>
> > Cc: Hans de Goede <hdegoede@redhat.com>
> 
> Thanks, patch looks good to me:
> 
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> 
> Regards,
> 
> Hans

Okay by me.

Reviewed-by: David E. Box <david.e.box@linux.intel.com>

> 
> 
> 
> > ---
> >  drivers/mfd/Kconfig | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> > index 6a3fd2d75f96..0f12b00955b4 100644
> > --- a/drivers/mfd/Kconfig
> > +++ b/drivers/mfd/Kconfig
> > @@ -676,7 +676,7 @@ config MFD_INTEL_PMC_BXT
> >  
> >  config MFD_INTEL_PMT
> >         tristate "Intel Platform Monitoring Technology (PMT)
> > support"
> > -       depends on PCI
> > +       depends on X86 && PCI
> >         select MFD_CORE
> >         help
> >           The Intel Platform Monitoring Technology (PMT) is an
> > interface that
> > 
> 

Re: [PATCH] mfd: intel_pmt: Only compile on x86

[Lee Jones]

On Wed, 25 Aug 2021, Prarit Bhargava wrote:

> The intel_pmt driver shows up as a compile option for all arches but is
> 32-bit and 64-bit x86 specific.
> 
> Add a CONFIG dependency on X86 for intel_pmt.
> 
> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: David E. Box <david.e.box@linux.intel.com>
> Cc: Hans de Goede <hdegoede@redhat.com>
> ---
>  drivers/mfd/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog