diff mbox series

powercap/intel_rapl: Add support for AMD family 1Ah

Message ID 20240719101234.50827-1-Dhananjay.Ugwekar@amd.com
State Accepted
Commit 166df51097a258a14fe9e946e2157f3b75eeb3c2
Headers show
Series powercap/intel_rapl: Add support for AMD family 1Ah | expand

Commit Message

Dhananjay Ugwekar July 19, 2024, 10:12 a.m. UTC
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(+)

Comments

Gautham R. Shenoy July 29, 2024, 6:06 a.m. UTC | #1
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
Gautham R. Shenoy Aug. 21, 2024, 5:15 a.m. UTC | #2
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
diff mbox series

Patch

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),
 	{}
 };