* [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
@ 2024-08-21 15:04 Shyam Sundar S K
2024-08-21 16:48 ` Mario Limonciello
0 siblings, 1 reply; 6+ messages in thread
From: Shyam Sundar S K @ 2024-08-21 15:04 UTC (permalink / raw)
To: hdegoede, ilpo.jarvinen
Cc: Sanket.Goswami, platform-driver-x86, Shyam Sundar S K
PMC driver has capability to get the IP information, idle mask values and
STB data from the PMFW. Extend this support to the platforms that belong
to family 1Ah model 60h series.
Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
---
drivers/platform/x86/amd/pmc/pmc.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/platform/x86/amd/pmc/pmc.c b/drivers/platform/x86/amd/pmc/pmc.c
index c3e51f0a5c33..bbb8edb62e00 100644
--- a/drivers/platform/x86/amd/pmc/pmc.c
+++ b/drivers/platform/x86/amd/pmc/pmc.c
@@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct amd_pmc_dev *dev)
dev->smu_msg = 0x538;
break;
case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
+ case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
dev->num_ips = 22;
dev->s2d_msg_id = 0xDE;
dev->smu_msg = 0x938;
@@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct amd_pmc_dev *pdev, struct device *dev,
val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
break;
case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
+ case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
break;
default:
@@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct amd_pmc_dev *dev)
case AMD_CPU_ID_CB:
case AMD_CPU_ID_PS:
case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
+ case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
return true;
default:
return false;
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
2024-08-21 15:04 [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform Shyam Sundar S K
@ 2024-08-21 16:48 ` Mario Limonciello
2024-08-21 17:40 ` Shyam Sundar S K
2024-08-21 17:54 ` Ilpo Järvinen
0 siblings, 2 replies; 6+ messages in thread
From: Mario Limonciello @ 2024-08-21 16:48 UTC (permalink / raw)
To: Shyam Sundar S K, hdegoede, ilpo.jarvinen
Cc: Sanket.Goswami, platform-driver-x86
On 8/21/2024 10:04, Shyam Sundar S K wrote:
> PMC driver has capability to get the IP information, idle mask values and
> STB data from the PMFW. Extend this support to the platforms that belong
> to family 1Ah model 60h series.
>
> Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
> Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
This is actually a fix for another commit in 6.11. Hans, Ilpo, can you
please take it in fixes?
Fixes: 426463d94d45d ("platform/x86/amd/pmc: Send OS_HINT command for
new AMD platform")
> ---
> drivers/platform/x86/amd/pmc/pmc.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/platform/x86/amd/pmc/pmc.c b/drivers/platform/x86/amd/pmc/pmc.c
> index c3e51f0a5c33..bbb8edb62e00 100644
> --- a/drivers/platform/x86/amd/pmc/pmc.c
> +++ b/drivers/platform/x86/amd/pmc/pmc.c
> @@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct amd_pmc_dev *dev)
> dev->smu_msg = 0x538;
> break;
> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> dev->num_ips = 22;
> dev->s2d_msg_id = 0xDE;
> dev->smu_msg = 0x938;
> @@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct amd_pmc_dev *pdev, struct device *dev,
> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
> break;
> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
> break;
> default:
> @@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct amd_pmc_dev *dev)
> case AMD_CPU_ID_CB:
> case AMD_CPU_ID_PS:
> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> return true;
> default:
> return false;
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
2024-08-21 16:48 ` Mario Limonciello
@ 2024-08-21 17:40 ` Shyam Sundar S K
2024-08-21 17:42 ` Mario Limonciello
2024-08-21 17:54 ` Ilpo Järvinen
1 sibling, 1 reply; 6+ messages in thread
From: Shyam Sundar S K @ 2024-08-21 17:40 UTC (permalink / raw)
To: Mario Limonciello, hdegoede, ilpo.jarvinen
Cc: Sanket.Goswami, platform-driver-x86
On 8/21/2024 22:18, Mario Limonciello wrote:
> On 8/21/2024 10:04, Shyam Sundar S K wrote:
>> PMC driver has capability to get the IP information, idle mask
>> values and
>> STB data from the PMFW. Extend this support to the platforms that
>> belong
>> to family 1Ah model 60h series.
>>
>> Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
>> Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
>
> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
>
> This is actually a fix for another commit in 6.11. Hans, Ilpo, can
> you please take it in fixes?
>
> Fixes: 426463d94d45d ("platform/x86/amd/pmc: Send OS_HINT command for
> new AMD platform")
>
I'm not certain how this commit addresses the existing issue. Last
week, when I sent out commit 426463d94d45d, I tested the changes on my
test bed, and everything seemed to work fine.
If this new commit is indeed resolving an existing problem, I shall
amend the commit message to make it more meaningful.
Thanks,
Shyam
>> ---
>> drivers/platform/x86/amd/pmc/pmc.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/platform/x86/amd/pmc/pmc.c
>> b/drivers/platform/x86/amd/pmc/pmc.c
>> index c3e51f0a5c33..bbb8edb62e00 100644
>> --- a/drivers/platform/x86/amd/pmc/pmc.c
>> +++ b/drivers/platform/x86/amd/pmc/pmc.c
>> @@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct
>> amd_pmc_dev *dev)
>> dev->smu_msg = 0x538;
>> break;
>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>> dev->num_ips = 22;
>> dev->s2d_msg_id = 0xDE;
>> dev->smu_msg = 0x938;
>> @@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct
>> amd_pmc_dev *pdev, struct device *dev,
>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
>> break;
>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
>> break;
>> default:
>> @@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct
>> amd_pmc_dev *dev)
>> case AMD_CPU_ID_CB:
>> case AMD_CPU_ID_PS:
>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>> return true;
>> default:
>> return false;
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
2024-08-21 17:40 ` Shyam Sundar S K
@ 2024-08-21 17:42 ` Mario Limonciello
2024-08-21 17:44 ` Shyam Sundar S K
0 siblings, 1 reply; 6+ messages in thread
From: Mario Limonciello @ 2024-08-21 17:42 UTC (permalink / raw)
To: Shyam Sundar S K, hdegoede, ilpo.jarvinen
Cc: Sanket.Goswami, platform-driver-x86
On 8/21/2024 12:40, Shyam Sundar S K wrote:
>
>
> On 8/21/2024 22:18, Mario Limonciello wrote:
>> On 8/21/2024 10:04, Shyam Sundar S K wrote:
>>> PMC driver has capability to get the IP information, idle mask
>>> values and
>>> STB data from the PMFW. Extend this support to the platforms that
>>> belong
>>> to family 1Ah model 60h series.
>>>
>>> Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
>>> Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
>>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
>>
>> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
>>
>> This is actually a fix for another commit in 6.11. Hans, Ilpo, can
>> you please take it in fixes?
>>
>> Fixes: 426463d94d45d ("platform/x86/amd/pmc: Send OS_HINT command for
>> new AMD platform")
>>
>
> I'm not certain how this commit addresses the existing issue. Last
> week, when I sent out commit 426463d94d45d, I tested the changes on my
> test bed, and everything seemed to work fine.
>
> If this new commit is indeed resolving an existing problem, I shall
> amend the commit message to make it more meaningful.
Hmm when I was talking to a colleague about it this morning s0i3 didn't
start working on this platform until this change, presumably because the
message IDs weren't properly configured without it.
>
> Thanks,
> Shyam
>
>>> ---
>>> drivers/platform/x86/amd/pmc/pmc.c | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/drivers/platform/x86/amd/pmc/pmc.c
>>> b/drivers/platform/x86/amd/pmc/pmc.c
>>> index c3e51f0a5c33..bbb8edb62e00 100644
>>> --- a/drivers/platform/x86/amd/pmc/pmc.c
>>> +++ b/drivers/platform/x86/amd/pmc/pmc.c
>>> @@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct
>>> amd_pmc_dev *dev)
>>> dev->smu_msg = 0x538;
>>> break;
>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>> dev->num_ips = 22;
>>> dev->s2d_msg_id = 0xDE;
>>> dev->smu_msg = 0x938;
>>> @@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct
>>> amd_pmc_dev *pdev, struct device *dev,
>>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
>>> break;
>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
>>> break;
>>> default:
>>> @@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct
>>> amd_pmc_dev *dev)
>>> case AMD_CPU_ID_CB:
>>> case AMD_CPU_ID_PS:
>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>> return true;
>>> default:
>>> return false;
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
2024-08-21 17:42 ` Mario Limonciello
@ 2024-08-21 17:44 ` Shyam Sundar S K
0 siblings, 0 replies; 6+ messages in thread
From: Shyam Sundar S K @ 2024-08-21 17:44 UTC (permalink / raw)
To: Mario Limonciello, hdegoede, ilpo.jarvinen
Cc: Sanket.Goswami, platform-driver-x86
On 8/21/2024 23:12, Mario Limonciello wrote:
> On 8/21/2024 12:40, Shyam Sundar S K wrote:
>>
>>
>> On 8/21/2024 22:18, Mario Limonciello wrote:
>>> On 8/21/2024 10:04, Shyam Sundar S K wrote:
>>>> PMC driver has capability to get the IP information, idle mask
>>>> values and
>>>> STB data from the PMFW. Extend this support to the platforms that
>>>> belong
>>>> to family 1Ah model 60h series.
>>>>
>>>> Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
>>>> Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
>>>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
>>>
>>> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
>>>
>>> This is actually a fix for another commit in 6.11. Hans, Ilpo, can
>>> you please take it in fixes?
>>>
>>> Fixes: 426463d94d45d ("platform/x86/amd/pmc: Send OS_HINT command for
>>> new AMD platform")
>>>
>>
>> I'm not certain how this commit addresses the existing issue. Last
>> week, when I sent out commit 426463d94d45d, I tested the changes on my
>> test bed, and everything seemed to work fine.
>>
>> If this new commit is indeed resolving an existing problem, I shall
>> amend the commit message to make it more meaningful.
>
> Hmm when I was talking to a colleague about it this morning s0i3
> didn't start working on this platform until this change, presumably
> because the message IDs weren't properly configured without it.
Okay thanks! Let me respin a new version with a updated commit.
>
>>
>> Thanks,
>> Shyam
>>
>>>> ---
>>>> drivers/platform/x86/amd/pmc/pmc.c | 3 +++
>>>> 1 file changed, 3 insertions(+)
>>>>
>>>> diff --git a/drivers/platform/x86/amd/pmc/pmc.c
>>>> b/drivers/platform/x86/amd/pmc/pmc.c
>>>> index c3e51f0a5c33..bbb8edb62e00 100644
>>>> --- a/drivers/platform/x86/amd/pmc/pmc.c
>>>> +++ b/drivers/platform/x86/amd/pmc/pmc.c
>>>> @@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct
>>>> amd_pmc_dev *dev)
>>>> dev->smu_msg = 0x538;
>>>> break;
>>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>>> dev->num_ips = 22;
>>>> dev->s2d_msg_id = 0xDE;
>>>> dev->smu_msg = 0x938;
>>>> @@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct
>>>> amd_pmc_dev *pdev, struct device *dev,
>>>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
>>>> break;
>>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>>> val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
>>>> break;
>>>> default:
>>>> @@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct
>>>> amd_pmc_dev *dev)
>>>> case AMD_CPU_ID_CB:
>>>> case AMD_CPU_ID_PS:
>>>> case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
>>>> + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
>>>> return true;
>>>> default:
>>>> return false;
>>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform
2024-08-21 16:48 ` Mario Limonciello
2024-08-21 17:40 ` Shyam Sundar S K
@ 2024-08-21 17:54 ` Ilpo Järvinen
1 sibling, 0 replies; 6+ messages in thread
From: Ilpo Järvinen @ 2024-08-21 17:54 UTC (permalink / raw)
To: Mario Limonciello
Cc: Shyam Sundar S K, Hans de Goede, Sanket.Goswami,
platform-driver-x86
On Wed, 21 Aug 2024, Mario Limonciello wrote:
> On 8/21/2024 10:04, Shyam Sundar S K wrote:
> > PMC driver has capability to get the IP information, idle mask values and
> > STB data from the PMFW. Extend this support to the platforms that belong
> > to family 1Ah model 60h series.
> >
> > Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com>
> > Signed-off-by: Sanket Goswami <Sanket.Goswami@amd.com>
> > Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
>
> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
>
> This is actually a fix for another commit in 6.11. Hans, Ilpo, can you please
> take it in fixes?
>
> Fixes: 426463d94d45d ("platform/x86/amd/pmc: Send OS_HINT command for new AMD
> platform")
For that, a proper justification is necessary in the commit message (but
I guess Shyam was going to improve that too while respinning given the
other replies).
--
i.
> > ---
> > drivers/platform/x86/amd/pmc/pmc.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/platform/x86/amd/pmc/pmc.c
> > b/drivers/platform/x86/amd/pmc/pmc.c
> > index c3e51f0a5c33..bbb8edb62e00 100644
> > --- a/drivers/platform/x86/amd/pmc/pmc.c
> > +++ b/drivers/platform/x86/amd/pmc/pmc.c
> > @@ -359,6 +359,7 @@ static void amd_pmc_get_ip_info(struct amd_pmc_dev *dev)
> > dev->smu_msg = 0x538;
> > break;
> > case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> > + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> > dev->num_ips = 22;
> > dev->s2d_msg_id = 0xDE;
> > dev->smu_msg = 0x938;
> > @@ -597,6 +598,7 @@ static int amd_pmc_idlemask_read(struct amd_pmc_dev
> > *pdev, struct device *dev,
> > val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_YC);
> > break;
> > case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> > + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> > val = amd_pmc_reg_read(pdev, AMD_PMC_SCRATCH_REG_1AH);
> > break;
> > default:
> > @@ -630,6 +632,7 @@ static bool amd_pmc_is_stb_supported(struct amd_pmc_dev
> > *dev)
> > case AMD_CPU_ID_CB:
> > case AMD_CPU_ID_PS:
> > case PCI_DEVICE_ID_AMD_1AH_M20H_ROOT:
> > + case PCI_DEVICE_ID_AMD_1AH_M60H_ROOT:
> > return true;
> > default:
> > return false;
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-08-21 17:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-21 15:04 [PATCH] platform/x86/amd/pmc: Extend support for PMC features on new AMD platform Shyam Sundar S K
2024-08-21 16:48 ` Mario Limonciello
2024-08-21 17:40 ` Shyam Sundar S K
2024-08-21 17:42 ` Mario Limonciello
2024-08-21 17:44 ` Shyam Sundar S K
2024-08-21 17:54 ` Ilpo Järvinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox