lkml.org 
[lkml]   [2022]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 085/135] drm/amdgpu: Ensure HDA function is suspended before ASIC reset
    Date
    From: Kai-Heng Feng <kai.heng.feng@canonical.com>

    commit 887f75cfd0da44c19dda93b2ff9e70ca8792cdc1 upstream.

    DP/HDMI audio on AMD PRO VII stops working after S3:
    [ 149.450391] amdgpu 0000:63:00.0: amdgpu: MODE1 reset
    [ 149.450395] amdgpu 0000:63:00.0: amdgpu: GPU mode1 reset
    [ 149.450494] amdgpu 0000:63:00.0: amdgpu: GPU psp mode1 reset
    [ 149.983693] snd_hda_intel 0000:63:00.1: refused to change power state from D0 to D3hot
    [ 150.003439] amdgpu 0000:63:00.0: refused to change power state from D0 to D3hot
    ...
    [ 155.432975] snd_hda_intel 0000:63:00.1: CORB reset timeout#2, CORBRP = 65535

    The offending commit is daf8de0874ab5b ("drm/amdgpu: always reset the asic in
    suspend (v2)"). Commit 34452ac3038a7 ("drm/amdgpu: don't use BACO for
    reset in S3 ") doesn't help, so the issue is something different.

    Assuming that to make HDA resume to D0 fully realized, it needs to be
    successfully put to D3 first. And this guesswork proves working, by
    moving amdgpu_asic_reset() to noirq callback, so it's called after HDA
    function is in D3.

    Fixes: daf8de0874ab5b ("drm/amdgpu: always reset the asic in suspend (v2)")
    Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
    Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
    Cc: stable@vger.kernel.org
    Cc: "Limonciello, Mario" <Mario.Limonciello@amd.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 18 ++++++++++++------
    1 file changed, 12 insertions(+), 6 deletions(-)

    --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
    +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
    @@ -2246,18 +2246,23 @@ static int amdgpu_pmops_suspend(struct d
    {
    struct drm_device *drm_dev = dev_get_drvdata(dev);
    struct amdgpu_device *adev = drm_to_adev(drm_dev);
    - int r;

    if (amdgpu_acpi_is_s0ix_active(adev))
    adev->in_s0ix = true;
    else
    adev->in_s3 = true;
    - r = amdgpu_device_suspend(drm_dev, true);
    - if (r)
    - return r;
    + return amdgpu_device_suspend(drm_dev, true);
    +}
    +
    +static int amdgpu_pmops_suspend_noirq(struct device *dev)
    +{
    + struct drm_device *drm_dev = dev_get_drvdata(dev);
    + struct amdgpu_device *adev = drm_to_adev(drm_dev);
    +
    if (!adev->in_s0ix)
    - r = amdgpu_asic_reset(adev);
    - return r;
    + return amdgpu_asic_reset(adev);
    +
    + return 0;
    }

    static int amdgpu_pmops_resume(struct device *dev)
    @@ -2494,6 +2499,7 @@ static const struct dev_pm_ops amdgpu_pm
    .prepare = amdgpu_pmops_prepare,
    .complete = amdgpu_pmops_complete,
    .suspend = amdgpu_pmops_suspend,
    + .suspend_noirq = amdgpu_pmops_suspend_noirq,
    .resume = amdgpu_pmops_resume,
    .freeze = amdgpu_pmops_freeze,
    .thaw = amdgpu_pmops_thaw,

    \
     
     \ /
      Last update: 2022-05-10 16:10    [W:3.909 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site