powercap/intel_rapl: Add support for AMD family 1Ah

[PATCH] powercap/intel_rapl: Add support for AMD family 1Ah

Posted by Dhananjay Ugwekar 1 year, 5 months ago

AMD Family 1Ah's RAPL MSRs are identical to Family 19h's,
extend Family 19h's support to Family 1Ah.

Signed-off-by: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
---
 drivers/powercap/intel_rapl_common.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/powercap/intel_rapl_common.c b/drivers/powercap/intel_rapl_common.c
index 2f24ca764408..1622f1d6aed0 100644
--- a/drivers/powercap/intel_rapl_common.c
+++ b/drivers/powercap/intel_rapl_common.c
@@ -1285,6 +1285,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
 
 	X86_MATCH_VENDOR_FAM(AMD, 0x17, &rapl_defaults_amd),
 	X86_MATCH_VENDOR_FAM(AMD, 0x19, &rapl_defaults_amd),
+	X86_MATCH_VENDOR_FAM(AMD, 0x1A, &rapl_defaults_amd),
 	X86_MATCH_VENDOR_FAM(HYGON, 0x18, &rapl_defaults_amd),
 	{}
 };
-- 
2.34.1

Re: [PATCH] powercap/intel_rapl: Add support for AMD family 1Ah

Posted by Gautham R.Shenoy 1 year, 4 months ago

Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com> writes:

> AMD Family 1Ah's RAPL MSRs are identical to Family 19h's,
> extend Family 19h's support to Family 1Ah.
>
> Signed-off-by: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>

Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>

> ---
>  drivers/powercap/intel_rapl_common.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/powercap/intel_rapl_common.c b/drivers/powercap/intel_rapl_common.c
> index 2f24ca764408..1622f1d6aed0 100644
> --- a/drivers/powercap/intel_rapl_common.c
> +++ b/drivers/powercap/intel_rapl_common.c
> @@ -1285,6 +1285,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
>  
>  	X86_MATCH_VENDOR_FAM(AMD, 0x17, &rapl_defaults_amd),
>  	X86_MATCH_VENDOR_FAM(AMD, 0x19, &rapl_defaults_amd),
> +	X86_MATCH_VENDOR_FAM(AMD, 0x1A, &rapl_defaults_amd),
>  	X86_MATCH_VENDOR_FAM(HYGON, 0x18, &rapl_defaults_amd),
>  	{}
>  };
> -- 
> 2.34.1

Re: [PATCH] powercap/intel_rapl: Add support for AMD family 1Ah

Posted by Gautham R. Shenoy 1 year, 4 months ago

Hello Rafael,

On Mon, Jul 29, 2024 at 11:36:15AM +0530, Gautham R.Shenoy wrote:
> Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com> writes:
> 
> > AMD Family 1Ah's RAPL MSRs are identical to Family 19h's,
> > extend Family 19h's support to Family 1Ah.
> >
> > Signed-off-by: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
> 
> Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>

Could you please consider this for 6.11 ?

--
Thanks and Regards
gautham.


> 
> > ---
> >  drivers/powercap/intel_rapl_common.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/powercap/intel_rapl_common.c b/drivers/powercap/intel_rapl_common.c
> > index 2f24ca764408..1622f1d6aed0 100644
> > --- a/drivers/powercap/intel_rapl_common.c
> > +++ b/drivers/powercap/intel_rapl_common.c
> > @@ -1285,6 +1285,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
> >  
> >  	X86_MATCH_VENDOR_FAM(AMD, 0x17, &rapl_defaults_amd),
> >  	X86_MATCH_VENDOR_FAM(AMD, 0x19, &rapl_defaults_amd),
> > +	X86_MATCH_VENDOR_FAM(AMD, 0x1A, &rapl_defaults_amd),
> >  	X86_MATCH_VENDOR_FAM(HYGON, 0x18, &rapl_defaults_amd),
> >  	{}
> >  };
> > -- 
> > 2.34.1