ACK: [PATCH 1/1][SRU][J/OEM-5.14] UBUNTU: SAUCE: Revert "drm/amd/pm: enable ASPM by default"

[Stefan Bader]

On 28.03.22 15:13, Koba Ko wrote:
> BugLink: https://bugs.launchpad.net/bugs/1966680
> 
> This reverts commit 0064b0ce85bb86d8a6fa066323f6318956c2dd59.
> 
> since the multiple adl SKUs with amdgpu polaris/gcn series have
> suspend/resum issue, disable amdgpu's aspm before the real fix is landed.
> 
> Signed-off-by: Koba Ko <koba.ko at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/nv.c                         | 2 +-
>   drivers/gpu/drm/amd/amdgpu/soc15.c                      | 2 +-
>   drivers/gpu/drm/amd/amdgpu/vi.c                         | 2 +-
>   drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 2 +-
>   4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/nv.c b/drivers/gpu/drm/amd/amdgpu/nv.c
> index fbcee5d7d6a11..3215c8f254a3a 100644
> --- a/drivers/gpu/drm/amd/amdgpu/nv.c
> +++ b/drivers/gpu/drm/amd/amdgpu/nv.c
> @@ -583,7 +583,7 @@ static void nv_pcie_gen3_enable(struct amdgpu_device *adev)
>   
>   static void nv_program_aspm(struct amdgpu_device *adev)
>   {
> -	if (!amdgpu_aspm)
> +	if (amdgpu_aspm != 1)
>   		return;
>   
>   	if (!(adev->flags & AMD_IS_APU) &&
> diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c b/drivers/gpu/drm/amd/amdgpu/soc15.c
> index b7d350be80502..38bf8077a90dc 100644
> --- a/drivers/gpu/drm/amd/amdgpu/soc15.c
> +++ b/drivers/gpu/drm/amd/amdgpu/soc15.c
> @@ -689,7 +689,7 @@ static void soc15_pcie_gen3_enable(struct amdgpu_device *adev)
>   
>   static void soc15_program_aspm(struct amdgpu_device *adev)
>   {
> -	if (!amdgpu_aspm)
> +	if (amdgpu_aspm != 1)
>   		return;
>   
>   	if (!(adev->flags & AMD_IS_APU) &&
> diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
> index 748654bf5a085..5ed01b1aa7e85 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vi.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vi.c
> @@ -1140,7 +1140,7 @@ static void vi_program_aspm(struct amdgpu_device *adev)
>   	bool bL1SS = false;
>   	bool bClkReqSupport = true;
>   
> -	if (!amdgpu_aspm)
> +	if (amdgpu_aspm != 1)
>   		return;
>   
>   	if (adev->flags & AMD_IS_APU ||
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> index d285f3f644512..fce498625ee40 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> @@ -338,7 +338,7 @@ sienna_cichlid_get_allowed_feature_mask(struct smu_context *smu,
>   	if (smu->dc_controlled_by_gpio)
>          *(uint64_t *)feature_mask |= FEATURE_MASK(FEATURE_ACDC_BIT);
>   
> -	if (amdgpu_aspm)
> +	if (amdgpu_aspm == 1)
>   		*(uint64_t *)feature_mask |= FEATURE_MASK(FEATURE_DS_LCLK_BIT);
>   
>   	return 0;

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20220426/9a7296e9/attachment-0001.sig>