diff mbox series

PCI: Add quirk to clear AMD strap_15B8 NO_SOFT_RESET dev2 f0

Message ID 20230329172859.699743-1-Basavaraj.Natikar@amd.com (mailing list archive)
State Accepted
Delegated to: Bjorn Helgaas
Headers show
Series PCI: Add quirk to clear AMD strap_15B8 NO_SOFT_RESET dev2 f0 | expand

Commit Message

Basavaraj Natikar March 29, 2023, 5:28 p.m. UTC 
The AMD [1022:15b8] USB controller loses some internal functional
MSI-X context when transitioning from D0 to D3hot. BIOS normally
traps D0->D3hot and D3hot->D0 transitions so it can save and restore
that internal context, but some firmware in the field lacks due to
AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit is set.

Hence add quirk to clear AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET
bit before USB controller initialization during boot.

Reported-by: Thomas Glanzmann <thomas@glanzmann.de>
Link: https://lore.kernel.org/linux-usb/Y%2Fz9GdHjPyF2rNG3@glanzmann.de/T/#u
Signed-off-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
---
 arch/x86/pci/fixup.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

Mario Limonciello March 29, 2023, 6:56 p.m. UTC | #1 
+Yazen

On 3/29/2023 12:28, Basavaraj Natikar wrote:
> The AMD [1022:15b8] USB controller loses some internal functional
> MSI-X context when transitioning from D0 to D3hot. BIOS normally
> traps D0->D3hot and D3hot->D0 transitions so it can save and restore
> that internal context, but some firmware in the field lacks due to
> AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit is set.
> 
> Hence add quirk to clear AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET
> bit before USB controller initialization during boot.
> 
> Reported-by: Thomas Glanzmann <thomas@glanzmann.de>

Thomas Glanzmann,

Can you please test this on your side?

Explicitly test it with amdgpu blacklisted or iGPU disabled in BIOS 
setup so that we can confirm that it is working and the amdgpu version 
of it isn't being used.

Thanks,

> Link: https://lore.kernel.org/linux-usb/Y%2Fz9GdHjPyF2rNG3@glanzmann.de/T/#u
> Signed-off-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>
> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
> ---
>   arch/x86/pci/fixup.c | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)
> 
> diff --git a/arch/x86/pci/fixup.c b/arch/x86/pci/fixup.c
> index 615a76d70019..bf5161dcf89e 100644
> --- a/arch/x86/pci/fixup.c
> +++ b/arch/x86/pci/fixup.c
> @@ -7,6 +7,7 @@
>   #include <linux/dmi.h>
>   #include <linux/pci.h>
>   #include <linux/vgaarb.h>
> +#include <asm/amd_nb.h>
>   #include <asm/hpet.h>
>   #include <asm/pci_x86.h>
>   
> @@ -824,3 +825,23 @@ static void rs690_fix_64bit_dma(struct pci_dev *pdev)
>   DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, 0x7910, rs690_fix_64bit_dma);
>   
>   #endif
> +
> +#ifdef CONFIG_AMD_NB
> +
> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2                                  0x10136008
> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK       0x00000080L
> +
> +static void quirk_clear_strap_no_soft_reset_dev2_f0(struct pci_dev *dev)
> +{
> +	u32 data;
> +
> +	if (!amd_smn_read(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, &data)) {
> +		data &= ~AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK;
> +		if (amd_smn_write(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, data))
> +			pci_err(dev, "Failed to write data 0x%x\n", data);
> +	} else {
> +		pci_err(dev, "Failed to read data\n");
> +	}
> +}
> +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AMD, 0x15b8, quirk_clear_strap_no_soft_reset_dev2_f0);
> +#endif
Thomas Glanzmann March 29, 2023, 7:39 p.m. UTC | #2 
Hello Mario,

* Limonciello, Mario <mario.limonciello@amd.com> [2023-03-29 20:56]:
> Can you please test this on your side?

> Explicitly test it with amdgpu blacklisted or iGPU disabled in BIOS setup so
> that we can confirm that it is working and the amdgpu version of it isn't
> being used.

I tested the same on top of v6.2.8 with amdgpu blacklisted. It works

Tested-by: Thomas Glanzmann <thomas@glanzmann.de>

Find here some additional information:

https://tg.st/u/dabb0ed77c14a8ec6e76d85f9f936977dca04c899d1773603c58350c9ce31768.txt

I tested by replugging mouse and keyboard in opposite ports which did
not work before.

Cheers,
        Thomas
Bjorn Helgaas April 6, 2023, 9:30 p.m. UTC | #3 
On Wed, Mar 29, 2023 at 10:58:59PM +0530, Basavaraj Natikar wrote:
> The AMD [1022:15b8] USB controller loses some internal functional
> MSI-X context when transitioning from D0 to D3hot. BIOS normally
> traps D0->D3hot and D3hot->D0 transitions so it can save and restore
> that internal context,

> but some firmware in the field lacks due to
> AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit is set.

This part doesn't have quite enough words in it.  Does the following
sound right?

  ... but some firmware in the field can't do this because it fails to
  clear the AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit.

If not, let me know and I can update it.

> Hence add quirk to clear AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET
> bit before USB controller initialization during boot.
> 
> Reported-by: Thomas Glanzmann <thomas@glanzmann.de>
> Link: https://lore.kernel.org/linux-usb/Y%2Fz9GdHjPyF2rNG3@glanzmann.de/T/#u
> Signed-off-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>
> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>

Applied to for-linus for v6.3, thank you for all your work on this!

I updated the subject to:

  x86/PCI: Add quirk for AMD XHCI controller that loses MSI-X state in D3hot

so it has a little bit of context.

I also added a stable tag since I assume the same problem will occur
on older kernels.  Let me know if that's not appropriate.

Bjorn

> ---
>  arch/x86/pci/fixup.c | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/arch/x86/pci/fixup.c b/arch/x86/pci/fixup.c
> index 615a76d70019..bf5161dcf89e 100644
> --- a/arch/x86/pci/fixup.c
> +++ b/arch/x86/pci/fixup.c
> @@ -7,6 +7,7 @@
>  #include <linux/dmi.h>
>  #include <linux/pci.h>
>  #include <linux/vgaarb.h>
> +#include <asm/amd_nb.h>
>  #include <asm/hpet.h>
>  #include <asm/pci_x86.h>
>  
> @@ -824,3 +825,23 @@ static void rs690_fix_64bit_dma(struct pci_dev *pdev)
>  DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, 0x7910, rs690_fix_64bit_dma);
>  
>  #endif
> +
> +#ifdef CONFIG_AMD_NB
> +
> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2                                  0x10136008
> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK       0x00000080L
> +
> +static void quirk_clear_strap_no_soft_reset_dev2_f0(struct pci_dev *dev)
> +{
> +	u32 data;
> +
> +	if (!amd_smn_read(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, &data)) {
> +		data &= ~AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK;
> +		if (amd_smn_write(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, data))
> +			pci_err(dev, "Failed to write data 0x%x\n", data);
> +	} else {
> +		pci_err(dev, "Failed to read data\n");
> +	}
> +}
> +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AMD, 0x15b8, quirk_clear_strap_no_soft_reset_dev2_f0);
> +#endif
> -- 
> 2.25.1
>
Basavaraj Natikar April 7, 2023, 4:09 a.m. UTC | #4 
On 4/7/2023 3:00 AM, Bjorn Helgaas wrote:
> On Wed, Mar 29, 2023 at 10:58:59PM +0530, Basavaraj Natikar wrote:
>> The AMD [1022:15b8] USB controller loses some internal functional
>> MSI-X context when transitioning from D0 to D3hot. BIOS normally
>> traps D0->D3hot and D3hot->D0 transitions so it can save and restore
>> that internal context,
>> but some firmware in the field lacks due to
>> AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit is set.
> This part doesn't have quite enough words in it.  Does the following
> sound right?
>
>   ... but some firmware in the field can't do this because it fails to
>   clear the AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET bit.
>
> If not, let me know and I can update it.

Thank You for updating. Looks good to me

>
>> Hence add quirk to clear AMD_15B8_RCC_DEV2_EPF0_STRAP2 NO_SOFT_RESET
>> bit before USB controller initialization during boot.
>>
>> Reported-by: Thomas Glanzmann <thomas@glanzmann.de>
>> Link: https://lore.kernel.org/linux-usb/Y%2Fz9GdHjPyF2rNG3@glanzmann.de/T/#u
>> Signed-off-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>
>> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
> Applied to for-linus for v6.3, thank you for all your work on this!
>
> I updated the subject to:
>
>   x86/PCI: Add quirk for AMD XHCI controller that loses MSI-X state in D3hot
>
> so it has a little bit of context.
>
> I also added a stable tag since I assume the same problem will occur
> on older kernels.  Let me know if that's not appropriate.

Looks great. Thanks a lot Bjron for updating subject and adding to stable tag.

>
> Bjorn
>
>> ---
>>  arch/x86/pci/fixup.c | 21 +++++++++++++++++++++
>>  1 file changed, 21 insertions(+)
>>
>> diff --git a/arch/x86/pci/fixup.c b/arch/x86/pci/fixup.c
>> index 615a76d70019..bf5161dcf89e 100644
>> --- a/arch/x86/pci/fixup.c
>> +++ b/arch/x86/pci/fixup.c
>> @@ -7,6 +7,7 @@
>>  #include <linux/dmi.h>
>>  #include <linux/pci.h>
>>  #include <linux/vgaarb.h>
>> +#include <asm/amd_nb.h>
>>  #include <asm/hpet.h>
>>  #include <asm/pci_x86.h>
>>  
>> @@ -824,3 +825,23 @@ static void rs690_fix_64bit_dma(struct pci_dev *pdev)
>>  DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, 0x7910, rs690_fix_64bit_dma);
>>  
>>  #endif
>> +
>> +#ifdef CONFIG_AMD_NB
>> +
>> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2                                  0x10136008
>> +#define AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK       0x00000080L
>> +
>> +static void quirk_clear_strap_no_soft_reset_dev2_f0(struct pci_dev *dev)
>> +{
>> +	u32 data;
>> +
>> +	if (!amd_smn_read(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, &data)) {
>> +		data &= ~AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK;
>> +		if (amd_smn_write(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, data))
>> +			pci_err(dev, "Failed to write data 0x%x\n", data);
>> +	} else {
>> +		pci_err(dev, "Failed to read data\n");
>> +	}
>> +}
>> +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AMD, 0x15b8, quirk_clear_strap_no_soft_reset_dev2_f0);
>> +#endif
>> -- 
>> 2.25.1
>>
diff mbox series

Patch

diff --git a/arch/x86/pci/fixup.c b/arch/x86/pci/fixup.c
index 615a76d70019..bf5161dcf89e 100644
--- a/arch/x86/pci/fixup.c
+++ b/arch/x86/pci/fixup.c
@@ -7,6 +7,7 @@ 
 #include <linux/dmi.h>
 #include <linux/pci.h>
 #include <linux/vgaarb.h>
+#include <asm/amd_nb.h>
 #include <asm/hpet.h>
 #include <asm/pci_x86.h>
 
@@ -824,3 +825,23 @@  static void rs690_fix_64bit_dma(struct pci_dev *pdev)
 DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, 0x7910, rs690_fix_64bit_dma);
 
 #endif
+
+#ifdef CONFIG_AMD_NB
+
+#define AMD_15B8_RCC_DEV2_EPF0_STRAP2                                  0x10136008
+#define AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK       0x00000080L
+
+static void quirk_clear_strap_no_soft_reset_dev2_f0(struct pci_dev *dev)
+{
+	u32 data;
+
+	if (!amd_smn_read(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, &data)) {
+		data &= ~AMD_15B8_RCC_DEV2_EPF0_STRAP2_NO_SOFT_RESET_DEV2_F0_MASK;
+		if (amd_smn_write(0, AMD_15B8_RCC_DEV2_EPF0_STRAP2, data))
+			pci_err(dev, "Failed to write data 0x%x\n", data);
+	} else {
+		pci_err(dev, "Failed to read data\n");
+	}
+}
+DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AMD, 0x15b8, quirk_clear_strap_no_soft_reset_dev2_f0);
+#endif